Re: [RFC] persistent store (version 2) (part 2 of 2)

[Huang Ying <ying.huang@intel.com>]

Hi, Tony,

On Wed, 2010-12-01 at 08:20 +0800, Luck, Tony wrote:
> Here are the changes to ERST to use the persistent store.
> 
> -Tony
> 
> ---
> 
> diff --git a/drivers/acpi/apei/Kconfig b/drivers/acpi/apei/Kconfig
> index fca34cc..e91680c 100644
> --- a/drivers/acpi/apei/Kconfig
> +++ b/drivers/acpi/apei/Kconfig
> @@ -1,5 +1,6 @@
>  config ACPI_APEI
>  	bool "ACPI Platform Error Interface (APEI)"
> +	select PSTORE
>  	depends on X86
>  	help
>  	  APEI allows to report errors (for example from the chipset)
> diff --git a/drivers/acpi/apei/erst.c b/drivers/acpi/apei/erst.c
> index 1211c03..2a49015 100644
> --- a/drivers/acpi/apei/erst.c
> +++ b/drivers/acpi/apei/erst.c
> @@ -34,6 +34,7 @@
>  #include <linux/cper.h>
>  #include <linux/nmi.h>
>  #include <linux/hardirq.h>
> +#include <linux/pstore.h>
>  #include <acpi/apei.h>
>  
>  #include "apei-internal.h"
> @@ -781,6 +782,111 @@ static int erst_check_table(struct acpi_table_erst *erst_tab)
>  	return 0;
>  }
>  
> +#define CPER_CREATOR_MCE						\
> +	UUID_LE(0x75a574e3, 0x5052, 0x4b29, 0x8a, 0x8e, 0xbe, 0x2c,	\
> +		0x64, 0x90, 0xb8, 0x9d)

How about rename this to CPER_CREATOR_LINUX or CPER_CREATOR_PSTORE?

> +#define CPER_SECTION_TYPE_DMESG						\
> +	UUID_LE(0xc197e04e, 0xd545, 0x4a70, 0x9c, 0x17, 0xa5, 0x54,	\
> +		0x94, 0x19, 0xeb, 0x12)
> +#define CPER_SECTION_TYPE_MCE						\
> +	UUID_LE(0xfe08ffbe, 0x95e4, 0x4be7, 0xbc, 0x73, 0x40, 0x96,	\
> +		0x04, 0x4a, 0x38, 0xfc)
> +
> +struct cper_mce_record {
> +	struct cper_record_header hdr;
> +	struct cper_section_descriptor sec_hdr;
> +	char data[];
> +} __packed;

Renamed to cper_pstore_record?

[...]

Best Regards,
Huang Ying