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
--
To unsubscribe from this list: send the line "unsubscribe linux-arch" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
