[PATCH 1/2] powerpc/powernv: Add OPAL message log interface

2014-03-27 Thread Joel Stanley
OPAL provides an in-memory circular buffer containing a message log populated with various runtime messages produced by the firmware. Provide a sysfs interface /sys/firmware/opal/messages for userspace to view the messages. Signed-off-by: Joel Stanley --- arch/powerpc/include/asm/opal.h

Re: [PATCH 1/2] powerpc/powernv: Add OPAL message log interface

2014-03-30 Thread Stewart Smith
Joel Stanley writes: > OPAL provides an in-memory circular buffer containing a message log > populated with various runtime messages produced by the firmware. > > Provide a sysfs interface /sys/firmware/opal/messages for userspace to > view the messages. Acked-by: Stewart Smith > Signed-off-by:

Re: [PATCH 1/2] powerpc/powernv: Add OPAL message log interface

2014-03-30 Thread Michael Neuling
Joel Stanley wrote: > OPAL provides an in-memory circular buffer containing a message log > populated with various runtime messages produced by the firmware. > > Provide a sysfs interface /sys/firmware/opal/messages for userspace to > view the messages. > > Signed-off-by: Joel Stanley > --- >

Re: [PATCH 1/2] powerpc/powernv: Add OPAL message log interface

2014-03-30 Thread Michael Neuling
Michael Neuling wrote: > Joel Stanley wrote: > > > OPAL provides an in-memory circular buffer containing a message log > > populated with various runtime messages produced by the firmware. > > > > Provide a sysfs interface /sys/firmware/opal/messages for userspace to > > view the messages. > >

Re: [PATCH 1/2] powerpc/powernv: Add OPAL message log interface

2014-03-31 Thread Benjamin Herrenschmidt
On Mon, 2014-03-31 at 15:08 +1100, Michael Neuling wrote: > > +/* OPAL in-memory console. Defined in OPAL source at core/console.c */ > > +struct memcons { > > + __be64 magic; > > +#define MEMCONS_MAGIC 0x6630696567726173L > > 0x6630696567726173 == f0iegras ... Ben!!! :-P Yummy ! :-) > >

Re: [PATCH 1/2] powerpc/powernv: Add OPAL message log interface

2014-03-31 Thread Joel Stanley
On Mon, Mar 31, 2014 at 10:29 PM, Benjamin Herrenschmidt wrote: >> > + conbuf = phys_to_virt(be64_to_cpu(mc->obuf_phys)); >> > + wrapped = be32_to_cpu(mc->out_pos) & MEMCONS_OUT_POS_WRAP; >> > + out_pos = be32_to_cpu(mc->out_pos) & MEMCONS_OUT_POS_MASK; >> > + >> >> Are there ordering issues