From: Benjamin Herrenschmidt <benh@kernel.crashing.org>
To: Aruna Balakrishnaiah <aruna@linux.vnet.ibm.com>
Cc: jkenisto@linux.vnet.ibm.com, tony.luck@intel.com,
mahesh@linux.vnet.ibm.com, cbouatmailru@gmail.com,
linux-kernel@vger.kernel.org, linuxppc-dev@ozlabs.org,
paulus@samba.org, anton@samba.org, ccross@android.com,
keescook@chromium.org
Subject: Re: [PATCH v3 5/8] powerpc/pseries: Read rtas partition via pstore
Date: Sat, 01 Jun 2013 14:49:45 +1000 [thread overview]
Message-ID: <1370062185.3766.27.camel@pasglop> (raw)
In-Reply-To: <20130425101851.21017.90041.stgit@aruna-ThinkPad-T420>
On Thu, 2013-04-25 at 15:48 +0530, Aruna Balakrishnaiah wrote:
> 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].
>
.../...
> diff --git a/include/linux/pstore.h b/include/linux/pstore.h
> index 75d0176..d7a8fe9 100644
> --- a/include/linux/pstore.h
> +++ b/include/linux/pstore.h
> @@ -35,6 +35,8 @@ enum pstore_type_id {
> PSTORE_TYPE_MCE = 1,
> PSTORE_TYPE_CONSOLE = 2,
> PSTORE_TYPE_FTRACE = 3,
> + /* PPC64 partition types */
> + PSTORE_TYPE_PPC_RTAS = 4,
> PSTORE_TYPE_UNKNOWN = 255
> };
>
Not sure about that list...
What do you mean by "RTAS" ? The error logs ? What about our "common"
partition (firmware settings ?). We should probably at least define
a generic PSTORE_TYPE_FIRMWARE for firmware private stuff...
Cheers,
Ben.
WARNING: multiple messages have this Message-ID (diff)
From: Benjamin Herrenschmidt <benh@kernel.crashing.org>
To: Aruna Balakrishnaiah <aruna@linux.vnet.ibm.com>
Cc: linuxppc-dev@ozlabs.org, paulus@samba.org,
linux-kernel@vger.kernel.org, jkenisto@linux.vnet.ibm.com,
tony.luck@intel.com, ananth@in.ibm.com,
mahesh@linux.vnet.ibm.com, ccross@android.com, anton@samba.org,
cbouatmailru@gmail.com, keescook@chromium.org
Subject: Re: [PATCH v3 5/8] powerpc/pseries: Read rtas partition via pstore
Date: Sat, 01 Jun 2013 14:49:45 +1000 [thread overview]
Message-ID: <1370062185.3766.27.camel@pasglop> (raw)
In-Reply-To: <20130425101851.21017.90041.stgit@aruna-ThinkPad-T420>
On Thu, 2013-04-25 at 15:48 +0530, Aruna Balakrishnaiah wrote:
> 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].
>
.../...
> diff --git a/include/linux/pstore.h b/include/linux/pstore.h
> index 75d0176..d7a8fe9 100644
> --- a/include/linux/pstore.h
> +++ b/include/linux/pstore.h
> @@ -35,6 +35,8 @@ enum pstore_type_id {
> PSTORE_TYPE_MCE = 1,
> PSTORE_TYPE_CONSOLE = 2,
> PSTORE_TYPE_FTRACE = 3,
> + /* PPC64 partition types */
> + PSTORE_TYPE_PPC_RTAS = 4,
> PSTORE_TYPE_UNKNOWN = 255
> };
>
Not sure about that list...
What do you mean by "RTAS" ? The error logs ? What about our "common"
partition (firmware settings ?). We should probably at least define
a generic PSTORE_TYPE_FIRMWARE for firmware private stuff...
Cheers,
Ben.
next prev parent reply other threads:[~2013-06-01 4:50 UTC|newest]
Thread overview: 38+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-04-25 10:17 [PATCH v3 0/8] Nvram-to-pstore Aruna Balakrishnaiah
2013-04-25 10:17 ` Aruna Balakrishnaiah
2013-04-25 10:18 ` [PATCH v3 1/8] powerpc/pseries: Remove syslog prefix in uncompressed oops text Aruna Balakrishnaiah
2013-04-25 10:18 ` Aruna Balakrishnaiah
2013-04-25 10:18 ` [PATCH v3 2/8] powerpc/pseries: Add version and timestamp to oops header Aruna Balakrishnaiah
2013-04-25 10:18 ` Aruna Balakrishnaiah
2013-04-25 10:18 ` [PATCH v3 3/8] powerpc/pseries: Introduce generic read function to read nvram-partitions Aruna Balakrishnaiah
2013-04-25 10:18 ` Aruna Balakrishnaiah
2013-04-25 10:18 ` [PATCH v3 4/8] powerpc/pseries: Read/Write oops nvram partition via pstore Aruna Balakrishnaiah
2013-04-25 10:18 ` Aruna Balakrishnaiah
2013-04-25 10:18 ` [PATCH v3 5/8] powerpc/pseries: Read rtas " Aruna Balakrishnaiah
2013-04-25 10:18 ` Aruna Balakrishnaiah
2013-06-01 4:49 ` Benjamin Herrenschmidt [this message]
2013-06-01 4:49 ` Benjamin Herrenschmidt
2013-06-01 4:50 ` Benjamin Herrenschmidt
2013-06-01 4:50 ` Benjamin Herrenschmidt
2013-04-25 10:18 ` [PATCH v3 6/8] powerpc/pseries: Distinguish between a os-partition and non-os partition Aruna Balakrishnaiah
2013-04-25 10:18 ` Aruna Balakrishnaiah
2013-04-25 10:19 ` [PATCH v3 7/8] powerpc/pseries: Read of-config partition via pstore Aruna Balakrishnaiah
2013-04-25 10:19 ` Aruna Balakrishnaiah
2013-04-25 10:19 ` [PATCH v3 8/8] powerpc/pseries: Read common " Aruna Balakrishnaiah
2013-04-25 10:19 ` Aruna Balakrishnaiah
2013-06-01 4:52 ` Benjamin Herrenschmidt
2013-06-01 4:52 ` Benjamin Herrenschmidt
2013-06-04 9:02 ` Aruna Balakrishnaiah
2013-06-04 9:02 ` Aruna Balakrishnaiah
2013-06-01 4:40 ` [PATCH v3 0/8] Nvram-to-pstore Benjamin Herrenschmidt
2013-06-01 4:40 ` Benjamin Herrenschmidt
2013-06-01 4:43 ` Benjamin Herrenschmidt
2013-06-01 4:43 ` Benjamin Herrenschmidt
2013-06-01 5:25 ` Benjamin Herrenschmidt
2013-06-01 5:25 ` Benjamin Herrenschmidt
2013-06-05 9:00 ` Aruna Balakrishnaiah
2013-06-05 9:00 ` Aruna Balakrishnaiah
2013-06-05 9:43 ` Benjamin Herrenschmidt
2013-06-05 9:43 ` Benjamin Herrenschmidt
2013-06-05 9:52 ` Aruna Balakrishnaiah
2013-06-05 9:52 ` Aruna Balakrishnaiah
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1370062185.3766.27.camel@pasglop \
--to=benh@kernel.crashing.org \
--cc=anton@samba.org \
--cc=aruna@linux.vnet.ibm.com \
--cc=cbouatmailru@gmail.com \
--cc=ccross@android.com \
--cc=jkenisto@linux.vnet.ibm.com \
--cc=keescook@chromium.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linuxppc-dev@ozlabs.org \
--cc=mahesh@linux.vnet.ibm.com \
--cc=paulus@samba.org \
--cc=tony.luck@intel.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.