Re: [PATCH v3 0/8] Nvram-to-pstore

2013-06-05 Thread Aruna Balakrishnaiah
On Wednesday 05 June 2013 03:13 PM, Benjamin Herrenschmidt wrote: On Wed, 2013-06-05 at 14:30 +0530, Aruna Balakrishnaiah wrote: Hi Ben, On Saturday 01 June 2013 10:55 AM, Benjamin Herrenschmidt wrote: Another question... Should the core pstore fail to unlink partitions that don't have an ->e

Re: [PATCH v3 0/8] Nvram-to-pstore

2013-06-05 Thread Benjamin Herrenschmidt
On Wed, 2013-06-05 at 14:30 +0530, Aruna Balakrishnaiah wrote: > Hi Ben, > > On Saturday 01 June 2013 10:55 AM, Benjamin Herrenschmidt wrote: > > Another question... > > > > Should the core pstore fail to unlink partitions that don't have > > an ->erase callback ? IE. Why would you let anyone eras

Re: [PATCH v3 0/8] Nvram-to-pstore

2013-06-05 Thread Aruna Balakrishnaiah
Hi Ben, On Saturday 01 June 2013 10:55 AM, Benjamin Herrenschmidt wrote: Another question... Should the core pstore fail to unlink partitions that don't have an ->erase callback ? IE. Why would you let anyone erase the OFW common partition for example ? That means that userspace tools can no lo

Re: [PATCH v3 0/8] Nvram-to-pstore

2013-05-31 Thread Benjamin Herrenschmidt
Another question... Should the core pstore fail to unlink partitions that don't have an ->erase callback ? IE. Why would you let anyone erase the OFW common partition for example ? That means that userspace tools can no longer manipulate it but we certainly don't want to remove it from the nvram i

Re: [PATCH v3 0/8] Nvram-to-pstore

2013-05-31 Thread Benjamin Herrenschmidt
On Sat, 2013-06-01 at 14:40 +1000, Benjamin Herrenschmidt wrote: .../... > In fact, this applies to at least all the BookS server platforms... > > Things that come to mind: > > - nvram_64.c duplicates generic_nvram.c as far as the user accessors > are concerned, it should be possible to get

Re: [PATCH v3 0/8] Nvram-to-pstore

2013-05-31 Thread Benjamin Herrenschmidt
On Thu, 2013-04-25 at 15:47 +0530, Aruna Balakrishnaiah wrote: > Currently the kernel provides the contents of p-series NVRAM only as a > simple stream of bytes via /dev/nvram, which must be interpreted in user > space by the nvram command in the powerpc-utils package. This patch set > exploits the

[PATCH v3 0/8] Nvram-to-pstore

2013-04-25 Thread Aruna Balakrishnaiah
Currently the kernel provides the contents of p-series NVRAM only as a simple stream of bytes via /dev/nvram, which must be interpreted in user space by the nvram command in the powerpc-utils package. This patch set exploits the pstore subsystem to expose each partition in NVRAM as a separate file