Re: [PATCH 4/8] Read/Write oops nvram partition via pstore

2013-04-16 Thread Aruna Balakrishnaiah
Hi Michael, Thanks for reviewing my patches. On Monday 15 April 2013 01:25 PM, Michael Ellerman wrote: On Wed, Apr 10, 2013 at 12:53:03PM +0530, Aruna Balakrishnaiah wrote: This patch exploits pstore infrastructure in power systems. IBM's system p machines provide persistent storage for LPARs

Re: [PATCH 5/8] Read rtas partition via pstore

2013-04-16 Thread Aruna Balakrishnaiah
On Monday 15 April 2013 01:31 PM, Michael Ellerman wrote: On Wed, Apr 10, 2013 at 12:53:27PM +0530, Aruna Balakrishnaiah wrote: This patch exploits pstore infrastructure to read the details from NVRAM's rtas partition. Does that mean it's exposed in the pstore filesystem? Yeah thats right

Re: [PATCH 4/8] Read/Write oops nvram partition via pstore

2013-04-16 Thread Aruna Balakrishnaiah
On Tuesday 16 April 2013 12:44 PM, Benjamin Herrenschmidt wrote: On Tue, 2013-04-16 at 11:50 +0530, Aruna Balakrishnaiah wrote: Sure. I will have one #ifdef for declarations and one for function definitions. Declarations generally don't need #ifdef's Sorry by declarations I meant variable

Re: [PATCH 4/8] Read/Write oops nvram partition via pstore

2013-04-16 Thread Aruna Balakrishnaiah
On Tuesday 16 April 2013 11:50 AM, Aruna Balakrishnaiah wrote: Currently with this patchset, pstore is not supporting compression of oops-messages since it involves some changes in the pstore framework. big_oops_buf will hold the large part of oops data which will be compressed and put

[PATCH 0/8] Nvram-to-pstore

2013-04-10 Thread Aruna Balakrishnaiah
file in /dev/pstore. For instance Oops messages will stored in a file named [dmesg-nvram-2]. --- Aruna Balakrishnaiah (8): Remove syslog prefix in uncompressed oops text Add version and timestamp to oops header Introduce generic read function to read nvram-partitions Read

[PATCH 1/8] Remove syslog prefix in uncompressed oops text

2013-04-10 Thread Aruna Balakrishnaiah
Removal of syslog prefix in the uncompressed oops text will help in capturing more oops data. Signed-off-by: Aruna Balakrishnaiah ar...@linux.vnet.ibm.com Reviewed-by: Jim Keniston jkeni...@us.ibm.com --- arch/powerpc/platforms/pseries/nvram.c |2 +- 1 file changed, 1 insertion(+), 1

[PATCH 2/8] Add version and timestamp to oops header

2013-04-10 Thread Aruna Balakrishnaiah
Introduce version and timestamp information in the oops header. oops_log_info (oops header) holds version (to distinguish between old and new format oops header), length of the oops text (compressed or uncompressed) and timestamp. Signed-off-by: Aruna Balakrishnaiah ar...@linux.vnet.ibm.com

[PATCH 3/8] Introduce generic read function to read nvram-partitions

2013-04-10 Thread Aruna Balakrishnaiah
Introduce generic read function to read nvram partitions other than rtas. nvram_read_error_log will be retained which is used to read rtas partition from rtasd. nvram_read_partition is the generic read function to read from any nvram partition. Signed-off-by: Aruna Balakrishnaiah ar

[PATCH 4/8] Read/Write oops nvram partition via pstore

2013-04-10 Thread Aruna Balakrishnaiah
/write the oops messages from/to this partition via pstore. Signed-off-by: Jim Keniston jkeni...@us.ibm.com Signed-off-by: Aruna Balakrishnaiah ar...@linux.vnet.ibm.com --- arch/powerpc/platforms/pseries/nvram.c | 145 1 file changed, 145 insertions(+) diff --git

[PATCH 6/8] Distinguish between a os-partition and non-os partition

2013-04-10 Thread Aruna Balakrishnaiah
Introduce os_partition member in nvram_os_partition structure to identify if the partition is an os partition or not. This will be useful to handle non-os partitions of-config and common in subsequent patches. Signed-off-by: Aruna Balakrishnaiah ar...@linux.vnet.ibm.com Reviewed-by: Jim Keniston

[PATCH 7/8] Read of-config partition via pstore

2013-04-10 Thread Aruna Balakrishnaiah
This patch exploits pstore infrastructure to read the details from NVRAM's of-config partition. Signed-off-by: Aruna Balakrishnaiah ar...@linux.vnet.ibm.com Reviewed-by: Jim Keniston jkeni...@us.ibm.com --- arch/powerpc/platforms/pseries/nvram.c | 58 ++-- fs/pstore

[PATCH 8/8] Read common partition via pstore

2013-04-10 Thread Aruna Balakrishnaiah
This patch exploits pstore infrastructure to read the details from NVRAM's common partition. Signed-off-by: Aruna Balakrishnaiah ar...@linux.vnet.ibm.com Reviewed-by: Jim Keniston jkeni...@us.ibm.com --- arch/powerpc/platforms/pseries/nvram.c | 17 - fs/pstore/inode.c

[PATCH 5/8] Read rtas partition via pstore

2013-04-10 Thread Aruna Balakrishnaiah
This patch exploits pstore infrastructure to read the details from NVRAM's rtas partition. Signed-off-by: Aruna Balakrishnaiah ar...@linux.vnet.ibm.com Reviewed-by: Jim Keniston jkeni...@us.ibm.com --- arch/powerpc/platforms/pseries/nvram.c | 33 +--- fs/pstore

Re: [PATCH 00/11] Add compression support to pstore

2013-08-05 Thread Aruna Balakrishnaiah
Hi Tony, Thank you very much for testing my patches. On Saturday 03 August 2013 03:42 AM, Tony Luck wrote: A quick experiment to use your patchset - but with compression disabled by tweaking this line in pstore_dump(): zipped_len = -1; //zip_data(dst, hsize + len); turned out well. This

Re: [PATCH 00/11] Add compression support to pstore

2013-08-05 Thread Aruna Balakrishnaiah
same issue on Power too. Please give a final try with the patch I have attached and I will dig into this more tomorrow. Patch to be applied on top of my patch series (without your fix patch). - Aruna -Tony commit 35b489152ae8f673fa79e7eeffc0bc8503c608b6 Author: Aruna Balakrishnaiah ar

Re: [PATCH 00/11] Add compression support to pstore

2013-08-06 Thread Aruna Balakrishnaiah
On Wednesday 07 August 2013 05:06 AM, Tony Luck wrote: On Mon, Aug 5, 2013 at 2:20 PM, Tony Luck tony.l...@gmail.com wrote: Still have problems booting if there are any compressed images in ERST to be inflated. So I took another look at this part of the code ... and saw a couple of issues:

Re: [PATCH 00/11] Add compression support to pstore

2013-08-06 Thread Aruna Balakrishnaiah
Hi Tony, On Wednesday 07 August 2013 08:55 AM, Tony Luck wrote: On Tue, Aug 6, 2013 at 6:58 PM, Aruna Balakrishnaiah ar...@linux.vnet.ibm.com wrote: The patch looks right. I will clean it up. Does the issue still persist after this? Things seem to be working - but testing has hardly been

Re: [RFC PATCH v2 04/11] pstore: Add compression support to pstore

2013-08-26 Thread Aruna Balakrishnaiah
On Friday 23 August 2013 04:47 AM, Luck, Tony wrote: 1[ 383.209057] RIP [813d3946] sysrq_handle_crash+0x16/0x20 4[ 383.209057] RSP 88006f551e80 4[ 383.209057] CR2: 4[ 383.209057] ---[ end trace 04a1cddad37b4b33 ]--- 3[ 383.209057] pstore: compression failed

Re: [RFC PATCH v2 06/11] pstore: Add decompression support to pstore

2013-08-27 Thread Aruna Balakrishnaiah
On Friday 23 August 2013 04:34 AM, Seiji Aguchi wrote: -Original Message- From: linux-kernel-ow...@vger.kernel.org [mailto:linux-kernel-ow...@vger.kernel.org] On Behalf Of Aruna Balakrishnaiah Sent: Friday, August 16, 2013 9:18 AM To: linuxppc-...@ozlabs.org; tony.l...@intel.com

[RFC PATCH v2 00/11] Add (de)compression support to pstore

2013-08-16 Thread Aruna Balakrishnaiah
and persistent ram. --- Aruna Balakrishnaiah (11): powerpc/pseries: Remove (de)compression in nvram with pstore enabled pstore: Add new argument 'compressed' in pstore write callback pstore/Kconfig: Select ZLIB_DEFLATE and ZLIB_INFLATE when PSTORE is selected pstore: Add compression

[RFC PATCH v2 02/11] pstore: Add new argument 'compressed' in pstore write callback

2013-08-16 Thread Aruna Balakrishnaiah
-off-by: Aruna Balakrishnaiah ar...@linux.vnet.ibm.com --- arch/powerpc/platforms/pseries/nvram.c |4 ++-- drivers/acpi/apei/erst.c |4 ++-- drivers/firmware/efi/efi-pstore.c |2 +- fs/pstore/platform.c |7 --- fs/pstore/ram.c

[RFC PATCH v2 10/11] efi-pstore: Read and write to the 'compressed' flag of pstore

2013-08-16 Thread Aruna Balakrishnaiah
In pstore write, Efi will add a character 'C'(compressed) or D'(decompressed) in its header while writing to persistent store. In pstore read, read the header and update the 'compressed' flag accordingly. Signed-off-by: Aruna Balakrishnaiah ar...@linux.vnet.ibm.com --- drivers/firmware/efi/efi

[RFC PATCH v2 07/11] pstore: Add file extension to pstore file if compressed

2013-08-16 Thread Aruna Balakrishnaiah
In case decompression fails, add a .enc.z to indicate the file has compressed data. This will help user space utilities to figure out the file contents. Signed-off-by: Aruna Balakrishnaiah ar...@linux.vnet.ibm.com --- fs/pstore/inode.c|7 --- fs/pstore/internal.h |5 +++-- fs

[RFC PATCH v2 05/11] pstore: Introduce new argument 'compressed' in the read callback

2013-08-16 Thread Aruna Balakrishnaiah
Backends will set the flag 'compressed' after reading the log from persistent store to indicate the data being returned to pstore is compressed or not. Signed-off-by: Aruna Balakrishnaiah ar...@linux.vnet.ibm.com --- arch/powerpc/platforms/pseries/nvram.c |2 +- drivers/acpi/apei/erst.c

[RFC PATCH v2 01/11] powerpc/pseries: Remove (de)compression in nvram with pstore enabled

2013-08-16 Thread Aruna Balakrishnaiah
as 'hsize' will be removed in the subsequent patch. Signed-off-by: Aruna Balakrishnaiah ar...@linux.vnet.ibm.com --- arch/powerpc/platforms/pseries/nvram.c | 102 1 file changed, 12 insertions(+), 90 deletions(-) diff --git a/arch/powerpc/platforms/pseries/nvram.c

[RFC PATCH v2 09/11] erst: Read and write to the 'compressed' flag of pstore

2013-08-16 Thread Aruna Balakrishnaiah
In pstore write, set the section type to CPER_SECTION_TYPE_DMESG_COMPR if the data is compressed. In pstore read, read the section type and update the 'compressed' flag accordingly. Signed-off-by: Aruna Balakrishnaiah ar...@linux.vnet.ibm.com --- drivers/acpi/apei/erst.c | 13 - 1

[RFC PATCH v2 11/11] pstore/ram: Read and write to the 'compressed' flag of pstore

2013-08-16 Thread Aruna Balakrishnaiah
In pstore write, add character 'C'(compressed) or 'D'(decompressed) in the header while writing to Ram persistent buffer. In pstore read, read the header and update the 'compressed' flag accordingly. Signed-off-by: Aruna Balakrishnaiah ar...@linux.vnet.ibm.com --- fs/pstore/ram.c | 36

[RFC PATCH v2 08/11] powerpc/pseries: Read and write to the 'compressed' flag of pstore

2013-08-16 Thread Aruna Balakrishnaiah
compatibilty with old format oops header when reading from pstore. Signed-off-by: Aruna Balakrishnaiah ar...@linux.vnet.ibm.com --- arch/powerpc/platforms/pseries/nvram.c |8 1 file changed, 8 insertions(+) diff --git a/arch/powerpc/platforms/pseries/nvram.c b/arch/powerpc/platforms/pseries

[RFC PATCH v2 03/11] pstore/Kconfig: Select ZLIB_DEFLATE and ZLIB_INFLATE when PSTORE is selected

2013-08-16 Thread Aruna Balakrishnaiah
Pstore will make use of deflate and inflate algorithm to compress and decompress the data. So when Pstore is enabled select zlib_deflate and zlib_inflate. Signed-off-by: Aruna Balakrishnaiah ar...@linux.vnet.ibm.com --- fs/pstore/Kconfig |2 ++ 1 file changed, 2 insertions(+) diff --git

[RFC PATCH v2 04/11] pstore: Add compression support to pstore

2013-08-16 Thread Aruna Balakrishnaiah
will capture the uncompressed data by making a call again to kmsg_dump with registered_buffer of registered size. Pstore will indicate the data is compressed or not with a flag in the write callback. Signed-off-by: Aruna Balakrishnaiah ar...@linux.vnet.ibm.com --- fs/pstore/platform.c | 148

[RFC PATCH v2 06/11] pstore: Add decompression support to pstore

2013-08-16 Thread Aruna Balakrishnaiah
Based on the flag 'compressed' set or not, pstore will decompress the data returning a plain text file. If decompression fails for a particular record it will have the compressed data in the file which can be decompressed with 'openssl' command line tool. Signed-off-by: Aruna Balakrishnaiah ar

[PATCH 03/11] pstore/Kconfig: Select ZLIB_DEFLATE and ZLIB_INFLATE when PSTORE is selected

2013-07-15 Thread Aruna Balakrishnaiah
Pstore will make use of deflate and inflate algorithm to compress and decompress the data. So when Pstore is enabled select zlib_deflate and zlib_inflate. Signed-off-by: Aruna Balakrishnaiah ar...@linux.vnet.ibm.com --- fs/pstore/Kconfig |2 ++ 1 file changed, 2 insertions(+) diff --git

[PATCH 00/11] Add compression support to pstore

2013-07-15 Thread Aruna Balakrishnaiah
call back. Pstore will decompress the data based on the flag and writes decompressed data to the file. Test results: Have tested the patches on powerpc/pseries. On Intel have only tested with erst backend. Efi-pstore and RAM persistent buffer requires testing. --- Aruna Balakrishnaiah (11

[PATCH 02/11] pstore: Add new argument 'compressed' in pstore write callback

2013-07-15 Thread Aruna Balakrishnaiah
-off-by: Aruna Balakrishnaiah ar...@linux.vnet.ibm.com --- arch/powerpc/platforms/pseries/nvram.c |4 ++-- drivers/acpi/apei/erst.c |4 ++-- drivers/firmware/efi/efi-pstore.c |2 +- fs/pstore/platform.c |7 --- fs/pstore/ram.c

[PATCH 01/11] powerpc/pseries: Remove (de)compression in nvram with pstore enabled

2013-07-15 Thread Aruna Balakrishnaiah
as 'hsize' will be removed in the subsequent patch. Signed-off-by: Aruna Balakrishnaiah ar...@linux.vnet.ibm.com --- arch/powerpc/platforms/pseries/nvram.c | 40 1 file changed, 40 deletions(-) diff --git a/arch/powerpc/platforms/pseries/nvram.c b/arch/powerpc

[PATCH 05/11] pstore: Introduce new argument 'compressed' in the read callback

2013-07-15 Thread Aruna Balakrishnaiah
Backends will set the flag 'compressed' after reading the log from persistent store to indicate the data being returned to pstore is compressed or not. Signed-off-by: Aruna Balakrishnaiah ar...@linux.vnet.ibm.com --- arch/powerpc/platforms/pseries/nvram.c |2 +- drivers/acpi/apei/erst.c

[PATCH 07/11] pstore: Add file extension to pstore file if compressed

2013-07-15 Thread Aruna Balakrishnaiah
In case decompression fails, add a .enc.z to indicate the file has compressed data. This will help user space utilities to figure out the file contents. Signed-off-by: Aruna Balakrishnaiah ar...@linux.vnet.ibm.com --- fs/pstore/inode.c|9 + fs/pstore/internal.h |5 +++-- fs

[PATCH 06/11] pstore: Provide decompression support to pstore

2013-07-15 Thread Aruna Balakrishnaiah
Based on the flag 'compressed' set or not, pstore will decompress the data returning a plain text file. If decompression fails for a particular record it will have the compressed data in the file which can be decompressed with 'openssl' command line tool. Signed-off-by: Aruna Balakrishnaiah ar

[PATCH 04/11] pstore: Add compression support to pstore

2013-07-15 Thread Aruna Balakrishnaiah
will capture the uncompressed data by making a call again to kmsg_dump with registered_buffer of registered size. Pstore will indicate the data is compressed or not with a flag in the write callback. Signed-off-by: Aruna Balakrishnaiah ar...@linux.vnet.ibm.com --- fs/pstore/platform.c | 124

[PATCH 08/11] powerpc/pseries: Read and write to the 'compressed' flag of pstore

2013-07-15 Thread Aruna Balakrishnaiah
compatibilty with old format oops header when reading from pstore. Signed-off-by: Aruna Balakrishnaiah ar...@linux.vnet.ibm.com --- arch/powerpc/platforms/pseries/nvram.c | 34 +--- 1 file changed, 27 insertions(+), 7 deletions(-) diff --git a/arch/powerpc/platforms/pseries

[PATCH 10/11] efi-pstore: Read and write to the 'compressed' flag of pstore

2013-07-15 Thread Aruna Balakrishnaiah
In pstore write, Efi will add a character 'C'(compressed) or D'(decompressed) in its header while writing to persistent store. In pstore read, read the header and update the 'compressed' flag accordingly. Signed-off-by: Aruna Balakrishnaiah ar...@linux.vnet.ibm.com --- drivers/firmware/efi/efi

[PATCH 11/11] pstore/ram: Read and write to the 'compressed' flag of pstore

2013-07-15 Thread Aruna Balakrishnaiah
In pstore write, add character 'C'(compressed) or 'D'(decompressed) in the header while writing to Ram persistent buffer. In pstore read, read the header and update the 'compressed' flag accordingly. Signed-off-by: Aruna Balakrishnaiah ar...@linux.vnet.ibm.com --- fs/pstore/ram.c | 36

[PATCH 09/11] erst: Read and write to the 'compressed' flag of pstore

2013-07-15 Thread Aruna Balakrishnaiah
In pstore write, set the section type to CPER_SECTION_TYPE_DMESG_COMPR if the data is compressed. In pstore read, read the section type and update the 'compressed' flag accordingly. Signed-off-by: Aruna Balakrishnaiah ar...@linux.vnet.ibm.com --- drivers/acpi/apei/erst.c | 13 - 1

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

2013-06-19 Thread Aruna Balakrishnaiah
Hi Michael, On Wednesday 19 June 2013 11:45 AM, Michael Neuling wrote: Aruna Balakrishnaiah ar...@linux.vnet.ibm.com 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

[PATCH] powerpc/pseries: Enable PSTORE in pseries_defconfig

2013-06-21 Thread Aruna Balakrishnaiah
Enable PSTORE in pseries_defconfig Signed-off-by: Aruna Balakrishnaiah ar...@linux.vnet.ibm.com --- arch/powerpc/configs/pseries_defconfig |1 + 1 file changed, 1 insertion(+) diff --git a/arch/powerpc/configs/pseries_defconfig b/arch/powerpc/configs/pseries_defconfig index c4dfbaf

[PATCH 1/3] pstore: Adjust buffer size for compression for smaller registered buffers

2013-09-11 Thread Aruna Balakrishnaiah
experiments with plain text for buffers of size 1k - 4k (Smaller the buffer size repeated occurence will be less) and with sample crash log for buffers ranging from 4k - 10k. Reported-by: Seiji Aguchi seiji.agu...@hds.com Signed-off-by: Aruna Balakrishnaiah ar...@linux.vnet.ibm.com --- fs/pstore

[PATCH 3/3] pstore: Remove the messages related to compression failure

2013-09-11 Thread Aruna Balakrishnaiah
Remove the messages indicating compression failure as it will add to the space during panic path. Reported-by: Seiji Aguchi seiji.agu...@hds.com Signed-off-by: Aruna Balakrishnaiah ar...@linux.vnet.ibm.com --- fs/pstore/platform.c |4 1 file changed, 4 deletions(-) diff --git a/fs

[PATCH 2/3] pstore: Use zlib_inflateInit2 instead of zlib_inflateInit

2013-09-11 Thread Aruna Balakrishnaiah
Since zlib_deflateInit2() is used for specifying window bit during compression, zlib_inflateInit2() is appropriate for decompression. Reported-by: Seiji Aguchi seiji.agu...@hds.com Signed-off-by: Aruna Balakrishnaiah ar...@linux.vnet.ibm.com --- fs/pstore/platform.c |2 +- 1 file changed, 1

[PATCH v2] pstore: Adjust buffer size for compression for smaller registered buffers

2013-09-12 Thread Aruna Balakrishnaiah
experiments with plain text for buffers of size 1k - 4k (Smaller the buffer size repeated occurence will be less) and with sample crash log for buffers ranging from 4k - 10k. Reported-by: Seiji Aguchi seiji.agu...@hds.com Signed-off-by: Aruna Balakrishnaiah ar...@linux.vnet.ibm.com --- Changes from

Re: [PATCH v2] pstore: Adjust buffer size for compression for smaller registered buffers

2013-09-12 Thread Aruna Balakrishnaiah
On Thursday 12 September 2013 11:13 PM, Luck, Tony wrote: + default: + cmpr = 60; + break; + } Is this the right default? It may be a good choice for a backend with a really tiny buffer (1 ... 999). But less good for a (theoretical) backend with a

Re: [PATCH 00/11] Add compression support to pstore

2013-08-01 Thread Aruna Balakrishnaiah
Hi Tony/Kees, Could you please review and let me know your comments!! Regards, Aruna On Monday 15 July 2013 10:25 PM, Aruna Balakrishnaiah wrote: The patchset adds compression support to pstore. As the non-volatile storage space is limited, adding compression support results in capturing

Re: [PATCH v3 8/8] powerpc/pseries: Read common partition via pstore

2013-06-04 Thread Aruna Balakrishnaiah
On Saturday 01 June 2013 10:22 AM, Benjamin Herrenschmidt wrote: On Thu, 2013-04-25 at 15:49 +0530, Aruna Balakrishnaiah wrote: diff --git a/fs/pstore/inode.c b/fs/pstore/inode.c index 8d4fb65..88cc050 100644 --- a/fs/pstore/inode.c +++ b/fs/pstore/inode.c @@ -330,6 +330,9 @@ int pstore_mkfile

Re: [PATCH 3/3] powerpc/pseries: Support compression of oops text via pstore

2013-06-04 Thread Aruna Balakrishnaiah
Hi Ben, On Saturday 01 June 2013 10:24 AM, Benjamin Herrenschmidt wrote: On Fri, 2013-04-26 at 15:26 +0530, Aruna Balakrishnaiah wrote: The patch set supports compression of oops messages while writing to NVRAM, this helps in capturing more of oops data to lnx,oops-log. The pstore file

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

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

[PATCH v4 1/8] powerpc/pseries: Remove syslog prefix in uncompressed oops text

2013-06-05 Thread Aruna Balakrishnaiah
Removal of syslog prefix in the uncompressed oops text will help in capturing more oops data. Signed-off-by: Aruna Balakrishnaiah ar...@linux.vnet.ibm.com Reviewed-by: Jim Keniston jkeni...@us.ibm.com --- arch/powerpc/platforms/pseries/nvram.c |2 +- 1 file changed, 1 insertion(+), 1

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

2013-06-05 Thread Aruna Balakrishnaiah
by and remove forward declarations of pstore callbacks - Handle return value of nvram_write_os_partition - Remove empty pstore callbacks and register pstore only when pstore is configured --- Aruna Balakrishnaiah (8): powerpc/pseries: Remove syslog prefix in uncompressed

[PATCH v4 2/8] powerpc/pseries: Add version and timestamp to oops header

2013-06-05 Thread Aruna Balakrishnaiah
headers. version is assigned 5000 (greater than oops partition size) so that existing tools will refuse to dump new style partitions as the length is too large. The updated tools will work with both old and new format headers. Signed-off-by: Aruna Balakrishnaiah ar...@linux.vnet.ibm.com Reviewed

[PATCH v4 3/8] powerpc/pseries: Introduce generic read function to read nvram-partitions

2013-06-05 Thread Aruna Balakrishnaiah
Introduce generic read function to read nvram partitions other than rtas. nvram_read_error_log will be retained which is used to read rtas partition from rtasd. nvram_read_partition is the generic read function to read from any nvram partition. Signed-off-by: Aruna Balakrishnaiah ar

[PATCH v4 5/8] powerpc/pseries: Read rtas partition via pstore

2013-06-05 Thread Aruna Balakrishnaiah
This patch set exploits the pstore subsystem to read details of rtas partition in NVRAM to a separate file in /dev/pstore. For instance, rtas details will be stored in a file named [rtas-nvram-4]. Signed-off-by: Aruna Balakrishnaiah ar...@linux.vnet.ibm.com Reviewed-by: Jim Keniston jkeni

[PATCH v4 6/8] powerpc/pseries: Distinguish between a os-partition and non-os partition

2013-06-05 Thread Aruna Balakrishnaiah
Introduce os_partition member in nvram_os_partition structure to identify if the partition is an os partition or not. This will be useful to handle non-os partitions of-config and common. Signed-off-by: Aruna Balakrishnaiah ar...@linux.vnet.ibm.com Reviewed-by: Jim Keniston jkeni...@us.ibm.com

[PATCH v4 7/8] powerpc/pseries: Read of-config partition via pstore

2013-06-05 Thread Aruna Balakrishnaiah
This patch set exploits the pstore subsystem to read details of of-config partition in NVRAM to a separate file in /dev/pstore. For instance, of-config partition details will be stored in a file named [of-nvram-5]. Signed-off-by: Aruna Balakrishnaiah ar...@linux.vnet.ibm.com Reviewed-by: Jim

[PATCH v4 4/8] powerpc/pseries: Read/Write oops nvram partition via pstore

2013-06-05 Thread Aruna Balakrishnaiah
mechanism. This patch will read/write the oops messages from/to this partition via pstore. Signed-off-by: Jim Keniston jkeni...@us.ibm.com Signed-off-by: Aruna Balakrishnaiah ar...@linux.vnet.ibm.com --- arch/powerpc/platforms/pseries/nvram.c | 172 +--- 1 file changed

[PATCH v4 8/8] powerpc/pseries: Read common partition via pstore

2013-06-05 Thread Aruna Balakrishnaiah
This patch exploits pstore subsystem to read details of common partition in NVRAM to a separate file in /dev/pstore. For instance, common partition details will be stored in a file named [common-nvram-6]. Signed-off-by: Aruna Balakrishnaiah ar...@linux.vnet.ibm.com Reviewed-by: Jim Keniston jkeni

Re: [RFC PATCH 0/3] Nvram-to-pstore: compression support for oops data

2013-06-05 Thread Aruna Balakrishnaiah
parititons. --- Aruna Balakrishnaiah (3): Retreive header size from pstore. powerpc/pseries: Re-organise the oops compression code powerpc/pseries: Support compression of oops text via pstore arch/powerpc/platforms/pseries/nvram.c | 236 +++- fs

[RESEND PATCH 0/3] Nvram-to-pstore: compression support for oops data

2013-06-24 Thread Aruna Balakrishnaiah
of having the compressed data (junk) in the dmesg file it will skip and continue reading other partitions. This results in absence of dmesg file but will still have files relating to other parititons. --- Aruna Balakrishnaiah (3): Retreive header size from pstore powerpc/pseries: Re-organise

[PATCH 1/3] Retreive header size from pstore

2013-06-24 Thread Aruna Balakrishnaiah
pstore_get_header_size will return the size of the header added by pstore while logging messages to the registered buffer. Signed-off-by: Aruna Balakrishnaiah ar...@linux.vnet.ibm.com --- fs/pstore/platform.c |7 ++- include/linux/pstore.h |6 ++ 2 files changed, 12 insertions

[PATCH 3/3] powerpc/pseries: Support compression of oops text via pstore

2013-06-24 Thread Aruna Balakrishnaiah
by pstore and last oops_data_sz bytes of big_oops_buf to NVRAM so that we have recent oops messages in lnx,oops-log. In case decompression fails, it will result in absence of oops file but still have files (in /dev/pstore) for other partitions. Signed-off-by: Aruna Balakrishnaiah ar

[PATCH 2/3] powerpc/pseries: Re-organise the oops compression code

2013-06-24 Thread Aruna Balakrishnaiah
nvram_compress() and zip_oops() is used by the nvram_pstore_write API to compress oops messages hence re-organise the functions accordingly to avoid forward declarations. Signed-off-by: Aruna Balakrishnaiah ar...@linux.vnet.ibm.com --- arch/powerpc/platforms/pseries/nvram.c | 104

[PATCH v2] powerpc/pseries: Enable PSTORE in pseries_defconfig

2013-06-24 Thread Aruna Balakrishnaiah
Since now we have pstore support for nvram in pseries, enable it in the default config. With this config option enabled, pstore infra-structure will be used to read/write the messages from/to nvram. Signed-off-by: Aruna Balakrishnaiah ar...@linux.vnet.ibm.com --- arch/powerpc/configs

Re: [PATCH] powerpc/pseries: Enable PSTORE in pseries_defconfig

2013-06-24 Thread Aruna Balakrishnaiah
can always determine what is being changed, by looking at the diff. The why will be long forgotten. Signed-off-by: Aruna Balakrishnaiah ar...@linux.vnet.ibm.com --- arch/powerpc/configs/pseries_defconfig |1 + 1 file changed, 1 insertion(+) diff --git a/arch/powerpc/configs

[PATCH v3] powerpc/pseries: Enable PSTORE in pseries_defconfig

2013-06-24 Thread Aruna Balakrishnaiah
Since now we have pstore support for nvram in pseries, enable it in the default config. With this config option enabled, pstore infra-structure will be used to read/write the messages from/to nvram. Signed-off-by: Aruna Balakrishnaiah ar...@linux.vnet.ibm.com --- v3: Move pstore config

[PATCH] pstore: Fail to unlink if a driver has not defined pstore_erase

2013-06-24 Thread Aruna Balakrishnaiah
pstore_erase is used to erase the record from the persistent store. So if a driver has not defined pstore_erase callback return -EINVAL instead of unlinking a file as deleting the file without erasing its record in persistent store will give a wrong impression to customers. Signed-off-by: Aruna

Re: [PATCH] pstore: Fail to unlink if a driver has not defined pstore_erase

2013-06-24 Thread Aruna Balakrishnaiah
Hi Keek, On Monday 24 June 2013 10:33 PM, Kees Cook wrote: On Mon, Jun 24, 2013 at 12:48 AM, Aruna Balakrishnaiah ar...@linux.vnet.ibm.com wrote: pstore_erase is used to erase the record from the persistent store. So if a driver has not defined pstore_erase callback return -EINVAL instead

Re: [PATCH 3/3] powerpc/pseries: Support compression of oops text via pstore

2013-06-25 Thread Aruna Balakrishnaiah
Hi Kees, On Monday 24 June 2013 11:27 PM, Kees Cook wrote: On Sun, Jun 23, 2013 at 11:23 PM, Aruna Balakrishnaiah ar...@linux.vnet.ibm.com wrote: The patch set supports compression of oops messages while writing to NVRAM, this helps in capturing more of oops data to lnx,oops-log. The pstore

[PATCH v2] pstore: Fail to unlink if a driver has not defined pstore_erase

2013-06-25 Thread Aruna Balakrishnaiah
pstore_erase is used to erase the record from the persistent store. So if a driver has not defined pstore_erase callback return -EPERM instead of unlinking a file as deleting the file without erasing its record in persistent store will give a wrong impression to customers. Signed-off-by: Aruna

Re: [PATCH v2] pstore: Fail to unlink if a driver has not defined pstore_erase

2013-06-26 Thread Aruna Balakrishnaiah
On Tuesday 25 June 2013 10:40 PM, Tony Luck wrote: On Tue, Jun 25, 2013 at 9:41 AM, Kees Cook keesc...@chromium.org wrote: On Tue, Jun 25, 2013 at 2:03 AM, Aruna Balakrishnaiah ar...@linux.vnet.ibm.com wrote: pstore_erase is used to erase the record from the persistent store. So if a driver

[PATCH v2 0/8] powerpc/pseries: Nvram-to-pstore

2013-04-24 Thread Aruna Balakrishnaiah
pstore only when pstore is configured --- Aruna Balakrishnaiah (8): powerpc/pseries: Remove syslog prefix in uncompressed oops text powerpc/pseries: Add version and timestamp to oops header powerpc/pseries: Introduce generic read function to read nvram-partitions

[PATCH v2 1/8] powerpc/pseries: Remove syslog prefix in uncompressed oops text

2013-04-24 Thread Aruna Balakrishnaiah
Removal of syslog prefix in the uncompressed oops text will help in capturing more oops data. Signed-off-by: Aruna Balakrishnaiah ar...@linux.vnet.ibm.com Reviewed-by: Jim Keniston jkeni...@us.ibm.com --- arch/powerpc/platforms/pseries/nvram.c |2 +- 1 file changed, 1 insertion(+), 1

[PATCH v2 4/8] powerpc/pseries: Read/Write oops nvram partition via pstore

2013-04-24 Thread Aruna Balakrishnaiah
mechanism. This patch will read/write the oops messages from/to this partition via pstore. Signed-off-by: Jim Keniston jkeni...@us.ibm.com Signed-off-by: Aruna Balakrishnaiah ar...@linux.vnet.ibm.com --- arch/powerpc/platforms/pseries/nvram.c | 172 +--- 1 file changed

[PATCH v2 3/8] powerpc/pseries: Introduce generic read function to read nvram-partitions

2013-04-24 Thread Aruna Balakrishnaiah
Introduce generic read function to read nvram partitions other than rtas. nvram_read_error_log will be retained which is used to read rtas partition from rtasd. nvram_read_partition is the generic read function to read from any nvram partition. Signed-off-by: Aruna Balakrishnaiah ar

[PATCH v2 5/8] powerpc/pseries: Read rtas partition via pstore

2013-04-24 Thread Aruna Balakrishnaiah
This patch set exploits the pstore subsystem to read details of rtas partition in NVRAM to a separate file in /dev/pstore. For instance, rtas details will be stored in a file named [rtas-nvram-4]. Signed-off-by: Aruna Balakrishnaiah ar...@linux.vnet.ibm.com Reviewed-by: Jim Keniston jkeni

[PATCH v2 6/8] powerpc/pseries: Distinguish between a os-partition and non-os partition

2013-04-24 Thread Aruna Balakrishnaiah
Introduce os_partition member in nvram_os_partition structure to identify if the partition is an os partition or not. This will be useful to handle non-os partitions of-config and common. Signed-off-by: Aruna Balakrishnaiah ar...@linux.vnet.ibm.com Reviewed-by: Jim Keniston jkeni...@us.ibm.com

[PATCH v2 7/8] powerpc/pseries: Read of-config partition via pstore

2013-04-24 Thread Aruna Balakrishnaiah
This patch set exploits the pstore subsystem to read details of of-config partition in NVRAM to a separate file in /dev/pstore. For instance, of-config partition details will be stored in a file named [of-nvram-5]. Signed-off-by: Aruna Balakrishnaiah ar...@linux.vnet.ibm.com Reviewed-by: Jim

[PATCH v2 8/8] powerpc/pseries: Read common partition via pstore

2013-04-24 Thread Aruna Balakrishnaiah
This patch exploits pstore subsystem to read details of common partition in NVRAM to a separate file in /dev/pstore. For instance, common partition details will be stored in a file named [common-nvram-6]. Signed-off-by: Aruna Balakrishnaiah ar...@linux.vnet.ibm.com Reviewed-by: Jim Keniston jkeni

[PATCH v2 2/8] powerpc/pseries: Add version and timestamp to oops header

2013-04-24 Thread Aruna Balakrishnaiah
headers. version is assigned 5000 (greater than oops partition size) so that existing tools will refuse to dump new style partitions as the length is too large. The updated tools will work with both old and new format headers. Signed-off-by: Aruna Balakrishnaiah ar...@linux.vnet.ibm.com Reviewed

Re: [PATCH v2 7/8] powerpc/pseries: Read of-config partition via pstore

2013-04-24 Thread Aruna Balakrishnaiah
On Thursday 25 April 2013 02:13 AM, Kees Cook wrote: Hi Kees, On Tue, Apr 23, 2013 at 11:20 PM, Aruna Balakrishnaiah ar...@linux.vnet.ibm.com wrote: This patch set exploits the pstore subsystem to read details of of-config partition in NVRAM to a separate file in /dev/pstore. For instance

Re: [PATCH v2 0/8] powerpc/pseries: Nvram-to-pstore

2013-04-24 Thread Aruna Balakrishnaiah
Hi Kees, On Thursday 25 April 2013 02:15 AM, Kees Cook wrote: On Tue, Apr 23, 2013 at 11:19 PM, Aruna Balakrishnaiah ar...@linux.vnet.ibm.com 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

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

2013-04-25 Thread Aruna Balakrishnaiah
of nvram_write_os_partition - Remove empty pstore callbacks and register pstore only when pstore is configured --- Aruna Balakrishnaiah (8): powerpc/pseries: Remove syslog prefix in uncompressed oops text powerpc/pseries: Add version and timestamp to oops header powerpc

[PATCH v3 1/8] powerpc/pseries: Remove syslog prefix in uncompressed oops text

2013-04-25 Thread Aruna Balakrishnaiah
Removal of syslog prefix in the uncompressed oops text will help in capturing more oops data. Signed-off-by: Aruna Balakrishnaiah ar...@linux.vnet.ibm.com Reviewed-by: Jim Keniston jkeni...@us.ibm.com --- arch/powerpc/platforms/pseries/nvram.c |2 +- 1 file changed, 1 insertion(+), 1

[PATCH v3 2/8] powerpc/pseries: Add version and timestamp to oops header

2013-04-25 Thread Aruna Balakrishnaiah
headers. version is assigned 5000 (greater than oops partition size) so that existing tools will refuse to dump new style partitions as the length is too large. The updated tools will work with both old and new format headers. Signed-off-by: Aruna Balakrishnaiah ar...@linux.vnet.ibm.com Reviewed

[PATCH v3 3/8] powerpc/pseries: Introduce generic read function to read nvram-partitions

2013-04-25 Thread Aruna Balakrishnaiah
Introduce generic read function to read nvram partitions other than rtas. nvram_read_error_log will be retained which is used to read rtas partition from rtasd. nvram_read_partition is the generic read function to read from any nvram partition. Signed-off-by: Aruna Balakrishnaiah ar

[PATCH v3 4/8] powerpc/pseries: Read/Write oops nvram partition via pstore

2013-04-25 Thread Aruna Balakrishnaiah
mechanism. This patch will read/write the oops messages from/to this partition via pstore. Signed-off-by: Jim Keniston jkeni...@us.ibm.com Signed-off-by: Aruna Balakrishnaiah ar...@linux.vnet.ibm.com --- arch/powerpc/platforms/pseries/nvram.c | 172 +--- 1 file changed

[PATCH v3 5/8] powerpc/pseries: Read rtas partition via pstore

2013-04-25 Thread Aruna Balakrishnaiah
This patch set exploits the pstore subsystem to read details of rtas partition in NVRAM to a separate file in /dev/pstore. For instance, rtas details will be stored in a file named [rtas-nvram-4]. Signed-off-by: Aruna Balakrishnaiah ar...@linux.vnet.ibm.com Reviewed-by: Jim Keniston jkeni

[PATCH v3 8/8] powerpc/pseries: Read common partition via pstore

2013-04-25 Thread Aruna Balakrishnaiah
This patch exploits pstore subsystem to read details of common partition in NVRAM to a separate file in /dev/pstore. For instance, common partition details will be stored in a file named [common-nvram-6]. Signed-off-by: Aruna Balakrishnaiah ar...@linux.vnet.ibm.com Reviewed-by: Jim Keniston jkeni

[PATCH v3 7/8] powerpc/pseries: Read of-config partition via pstore

2013-04-25 Thread Aruna Balakrishnaiah
This patch set exploits the pstore subsystem to read details of of-config partition in NVRAM to a separate file in /dev/pstore. For instance, of-config partition details will be stored in a file named [of-nvram-5]. Signed-off-by: Aruna Balakrishnaiah ar...@linux.vnet.ibm.com Reviewed-by: Jim

[PATCH v3 6/8] powerpc/pseries: Distinguish between a os-partition and non-os partition

2013-04-25 Thread Aruna Balakrishnaiah
Introduce os_partition member in nvram_os_partition structure to identify if the partition is an os partition or not. This will be useful to handle non-os partitions of-config and common. Signed-off-by: Aruna Balakrishnaiah ar...@linux.vnet.ibm.com Reviewed-by: Jim Keniston jkeni...@us.ibm.com

[RFC PATCH 0/3] Nvram-to-pstore: compression support for oops data

2013-04-26 Thread Aruna Balakrishnaiah
of having the compressed data (junk) in the dmesg file it will skip and continue reading other partitions. This results in absence of dmesg file but will still have files relating to other parititons. --- Aruna Balakrishnaiah (3): Retreive header size from pstore. powerpc/pseries: Re

[PATCH 1/3] Retreive header size from pstore.

2013-04-26 Thread Aruna Balakrishnaiah
pstore_get_header_size will return the size of the header added by pstore while logging messages to the registered buffer. Signed-off-by: Aruna Balakrishnaiah ar...@linux.vnet.ibm.com --- fs/pstore/platform.c |7 ++- include/linux/pstore.h |6 ++ 2 files changed, 12 insertions

[PATCH 3/3] powerpc/pseries: Support compression of oops text via pstore

2013-04-26 Thread Aruna Balakrishnaiah
by pstore and last oops_data_sz bytes of big_oops_buf to NVRAM so that we have recent oops messages in lnx,oops-log. In case decompression fails, it will result in absence of oops file but still have files (in /dev/pstore) for other partitions. Signed-off-by: Aruna Balakrishnaiah ar

  1   2   3   >