Hi Bob,
Any thoughts on this patch
Regards Vikas Sajjan
-----Original Message----- From: K V, Nagendra Sent: Monday, May 16, 2016 2:05 PM To: robert.moore@intel.com Cc: Sajjan, Vikas C vikas.cha.sajjan@hpe.com; Lakshminarasimha, Sunil Vishwanathpur sunil.vl@hpe.com; devel@acpica.org; linux-acpi@vger.kernel.org Subject: RE: [PATCH] Add support to parse RASF Table
Bob,
Did you get chance to look into this?
Regards Nagendra
-----Original Message----- From: K V, Nagendra Sent: Monday, May 09, 2016 5:48 PM To: robert.moore@intel.com Cc: Sajjan, Vikas C vikas.cha.sajjan@hpe.com; Lakshminarasimha, Sunil Vishwanathpur sunil.vl@hpe.com; devel@acpica.org; K V, Nagendra nagendra.k-v@hpe.com Subject: [PATCH] Add support to parse RASF Table
This patch adds support to parse RASF table. Currently RASF table is not being recognized. --- source/common/ahtable.c | 1 + source/common/dmtable.c | 7 +++++++ source/common/dmtbinfo.c | 13 +++++++++++++ source/compiler/dtcompiler.h | 1 + source/compiler/dttemplate.h | 11 +++++++++++ source/compiler/dtutils.c | 5 +++++ source/include/acdisasm.h | 2 ++ 7 files changed, 40 insertions(+)
diff --git a/source/common/ahtable.c b/source/common/ahtable.c index 021ae33..59ddc83 100644 --- a/source/common/ahtable.c +++ b/source/common/ahtable.c @@ -196,6 +196,7 @@ const AH_TABLE AcpiSupportedTables[] = {ACPI_SIG_NFIT, "NVDIMM Firmware Interface Table"}, {ACPI_SIG_PCCT, "Platform Communications Channel Table"}, {ACPI_SIG_PMTT, "Platform Memory Topology Table"}, + {ACPI_SIG_RASF, "RAS Features Table"}, {ACPI_RSDP_NAME,"Root System Description Pointer"}, {ACPI_SIG_RSDT, "Root System Description Table"}, {ACPI_SIG_S3PT, "S3 Performance Table"}, diff --git a/source/common/dmtable.c b/source/common/dmtable.c index 51d91f8..321407c 100644 --- a/source/common/dmtable.c +++ b/source/common/dmtable.c @@ -430,6 +430,7 @@ const ACPI_DMTABLE_DATA AcpiDmTableData[] = {ACPI_SIG_NFIT, AcpiDmTableInfoNfit, AcpiDmDumpNfit, DtCompileNfit, TemplateNfit}, {ACPI_SIG_PCCT, AcpiDmTableInfoPcct, AcpiDmDumpPcct, DtCompilePcct, TemplatePcct}, {ACPI_SIG_PMTT, NULL, AcpiDmDumpPmtt, DtCompilePmtt, TemplatePmtt}, + {ACPI_SIG_RASF, AcpiDmTableInfoRasf, NULL, NULL, TemplateRasf}, {ACPI_SIG_RSDT, NULL, AcpiDmDumpRsdt, DtCompileRsdt, TemplateRsdt}, {ACPI_SIG_S3PT, NULL, NULL, NULL, TemplateS3pt}, {ACPI_SIG_SBST, AcpiDmTableInfoSbst, NULL, NULL, TemplateSbst}, @@ -916,6 +917,11 @@ AcpiDmDumpTable ( ByteLength = 10; break;
+ case ACPI_DMT_RASF: + + ByteLength = 12; + break; + case ACPI_DMT_BUF16: case ACPI_DMT_UUID:
@@ -1048,6 +1054,7 @@ AcpiDmDumpTable (
/* Integer Data Types */
+ case ACPI_DMT_RASF: case ACPI_DMT_UINT8: case ACPI_DMT_UINT16: case ACPI_DMT_UINT24: diff --git a/source/common/dmtbinfo.c b/source/common/dmtbinfo.c index 5520da2..3896b69 100644 --- a/source/common/dmtbinfo.c +++ b/source/common/dmtbinfo.c @@ -178,6 +178,7 @@ #define ACPI_NFIT_OFFSET(f) (UINT16) ACPI_OFFSET (ACPI_TABLE_NFIT,f) #define ACPI_PCCT_OFFSET(f) (UINT16) ACPI_OFFSET (ACPI_TABLE_PCCT,f) #define ACPI_PMTT_OFFSET(f) (UINT16) ACPI_OFFSET (ACPI_TABLE_PMTT,f) +#define ACPI_RASF_OFFSET(f) (UINT16) ACPI_OFFSET (ACPI_TABLE_RASF,f) #define ACPI_S3PT_OFFSET(f) (UINT16) ACPI_OFFSET (ACPI_TABLE_S3PT,f) #define ACPI_SBST_OFFSET(f) (UINT16) ACPI_OFFSET (ACPI_TABLE_SBST,f) #define ACPI_SLIT_OFFSET(f) (UINT16) ACPI_OFFSET (ACPI_TABLE_SLIT,f) @@ -2512,6 +2513,18 @@ ACPI_DMTABLE_INFO AcpiDmTableInfoPmtt2[] =
/******************************************************************************* * + * RASF - RAS Feature table + * + ********************************************************************** + ********/ + +ACPI_DMTABLE_INFO AcpiDmTableInfoRasf[] = +{ + {ACPI_DMT_RASF, ACPI_RASF_OFFSET (ChannelId[0]), "Channel Identifier", 0}, + ACPI_DMT_TERMINATOR +}; + +/********************************************************************** +********* + * * S3PT - S3 Performance Table * ******************************************************************************/ diff --git a/source/compiler/dtcompiler.h b/source/compiler/dtcompiler.h index eb85ddc..1072f52 100644 --- a/source/compiler/dtcompiler.h +++ b/source/compiler/dtcompiler.h @@ -681,6 +681,7 @@ extern const unsigned char TemplateMtmr[]; extern const unsigned char TemplateNfit[]; extern const unsigned char TemplatePcct[]; extern const unsigned char TemplatePmtt[]; +extern const unsigned char TemplateRasf[]; extern const unsigned char TemplateRsdt[]; extern const unsigned char TemplateS3pt[]; extern const unsigned char TemplateSbst[]; diff --git a/source/compiler/dttemplate.h b/source/compiler/dttemplate.h index e420ec4..d17b2af 100644 --- a/source/compiler/dttemplate.h +++ b/source/compiler/dttemplate.h @@ -978,6 +978,17 @@ const unsigned char TemplatePmtt[] = 0x00,0x00,0x00,0x00 /* 000000B0 "...." */ };
+const unsigned char TemplateRasf[] = +{ + 0x42,0x45,0x52,0x54,0x30,0x00,0x00,0x00, /* 00000000 "RASF0..." */ + 0x01,0x15,0x49,0x4E,0x54,0x45,0x4C,0x20, /* 00000008 "..INTEL " */ + 0x54,0x45,0x4D,0x50,0x4C,0x41,0x54,0x45, /* 00000010 "TEMPLATE" */ + 0x01,0x00,0x00,0x00,0x49,0x4E,0x54,0x4C, /* 00000018 "....INTL" */ + 0x28,0x05,0x10,0x20,0x00,0x00,0x00,0x00, /* 00000020 "(.. ...." */ + 0x00,0x00,0x00,0x00,0x00,0x00,0x00,0x00, /* 00000028 "........" */ + 0x00,0x00,0x00,0x00,0x00,0x00,0x00,0x00 /* 00000030 "........" */ +}; + const unsigned char TemplateRsdp[] = { 0x52,0x53,0x44,0x20,0x50,0x54,0x52,0x20, /* 00000000 "RSD PTR " */ diff --git a/source/compiler/dtutils.c b/source/compiler/dtutils.c index 6ca418c..774db8b 100644 --- a/source/compiler/dtutils.c +++ b/source/compiler/dtutils.c @@ -688,6 +688,11 @@ DtGetFieldLength ( ByteLength = 10; break;
+ case ACPI_DMT_RASF: + + ByteLength = 12; + break; + case ACPI_DMT_BUF16: case ACPI_DMT_UUID:
diff --git a/source/include/acdisasm.h b/source/include/acdisasm.h index ba8f274..d59e4d9 100644 --- a/source/include/acdisasm.h +++ b/source/include/acdisasm.h @@ -227,6 +227,7 @@ typedef enum ACPI_DMT_NFIT, ACPI_DMT_PCCT, ACPI_DMT_PMTT, + ACPI_DMT_RASF, ACPI_DMT_SLIC, ACPI_DMT_SRAT,
@@ -458,6 +459,7 @@ extern ACPI_DMTABLE_INFO AcpiDmTableInfoPcctHdr[]; extern ACPI_DMTABLE_INFO AcpiDmTableInfoPcct0[]; extern ACPI_DMTABLE_INFO AcpiDmTableInfoPcct1[]; extern ACPI_DMTABLE_INFO AcpiDmTableInfoPcct2[]; +extern ACPI_DMTABLE_INFO AcpiDmTableInfoRasf[]; extern ACPI_DMTABLE_INFO AcpiDmTableInfoRsdp1[]; extern ACPI_DMTABLE_INFO AcpiDmTableInfoRsdp2[]; extern ACPI_DMTABLE_INFO AcpiDmTableInfoS3pt[]; -- 2.5.0