On Tue, 1 Apr 2014 12:26:32 +0200
Geert Uytterhoeven wrote:
> On Mon, Mar 31, 2014 at 5:02 PM, Nathan Fontenot
> wrote:
> > struct rtas_error_log {
> > - unsigned long version:8;/* Architectural version */
> > - unsigned long severity:3; /* Severity lev
On Mon, Mar 31, 2014 at 5:02 PM, Nathan Fontenot
wrote:
> struct rtas_error_log {
> - unsigned long version:8;/* Architectural version */
> - unsigned long severity:3; /* Severity level of error */
> - unsigned long disposition:2;/* Degr
Benjamin Herrenschmidt writes:
> On Mon, 2014-03-31 at 09:27 +1100, Stewart Smith wrote:
>> Greg Kurz writes:
>> > struct rtas_error_log {
>> > +#ifdef __BIG_ENDIAN__
>> > + /* Byte 0 */
>> >unsigned long version:8;/* Architectural version */
>> > + /* Byte 1 */
>>
>> I th
Greg Kurz writes:
> On Mon, 31 Mar 2014 09:27:16 +1100
> Stewart Smith wrote:
>> Greg Kurz writes:
>> > struct rtas_error_log {
>> > +#ifdef __BIG_ENDIAN__
>> > + /* Byte 0 */
>> >unsigned long version:8;/* Architectural version */
>> > + /* Byte 1 */
>>
>> I think it wou
This is the patch that I worked up at the same time as Greg, the
biggest difference being that I took the approach of doing and's,
and shifting as opposed to re-defining the bit fields for LE.
One other difference is that I left out defines for bits in the
error log structures that we currently do
On Mon, 2014-03-31 at 09:27 +1100, Stewart Smith wrote:
> Greg Kurz writes:
> > struct rtas_error_log {
> > +#ifdef __BIG_ENDIAN__
> > + /* Byte 0 */
> > unsigned long version:8;/* Architectural version */
> > + /* Byte 1 */
>
> I think it would be great if we got rid of
On Mon, 31 Mar 2014 09:27:16 +1100
Stewart Smith wrote:
> Greg Kurz writes:
> > struct rtas_error_log {
> > +#ifdef __BIG_ENDIAN__
> > + /* Byte 0 */
> > unsigned long version:8;/* Architectural version */
> > + /* Byte 1 */
>
> I think it would be great if we got rid of
Greg Kurz writes:
> struct rtas_error_log {
> +#ifdef __BIG_ENDIAN__
> + /* Byte 0 */
> unsigned long version:8;/* Architectural version */
> + /* Byte 1 */
I think it would be great if we got rid of the usage of bitfields. As
soon as the mood of the compiler change
On Fri, 28 Mar 2014 09:57:03 -0500
Nathan Fontenot wrote:
> Greg,
>
> There is one more place that needs fixing up, in mobility_rtas_call(),
> and handle_rtas_event() in arch/powerpc/platforms/pseries/mobility.c.
>
> This relates to rtas event handling for PRRN notifications, we need to
> conver
Greg,
There is one more place that needs fixing up, in mobility_rtas_call(),
and handle_rtas_event() in arch/powerpc/platforms/pseries/mobility.c.
This relates to rtas event handling for PRRN notifications, we need to
convert the scope variable (PRRN notifications re-use the extended log
length f
On 28/03/2014 08:33, Greg Kurz wrote:
> The current kernel code assumes big endian and parses RTAS events all
> wrong. The most visible effect is that we cannot honor EPOW events,
> meaning, for example, we cannot shut down a guest properly from the
> hypervisor.
>
> This patch fixes that.
>
> Si
On Fri, 28 Mar 2014 09:29:22 +0100
Laurent Dufour wrote:
> On 28/03/2014 08:33, Greg Kurz wrote:
> > The current kernel code assumes big endian and parses RTAS events all
> > wrong. The most visible effect is that we cannot honor EPOW events,
> > meaning, for example, we cannot shut down a guest p
The current kernel code assumes big endian and parses RTAS events all
wrong. The most visible effect is that we cannot honor EPOW events,
meaning, for example, we cannot shut down a guest properly from the
hypervisor.
This patch fixes that.
Signed-off-by: Greg Kurz
---
arch/powerpc/include/asm/
13 matches
Mail list logo
