lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <333bf7dc-7a87-438f-a4dc-fc5e254ae049@amd.com>
Date:   Tue, 12 Dec 2023 18:48:25 -0600
From:   Avadhut Naik <avadnaik@....com>
To:     Randy Dunlap <rdunlap@...radead.org>, linux-acpi@...r.kernel.org,
        linux-edac@...r.kernel.org
Cc:     linux-kernel@...r.kernel.org, yazen.ghannam@....com,
        Avadhut Naik <avadhut.naik@....com>
Subject: [PATCH 1/1] parse_phat: Add support for ACPI PHAT Table parsing

Hi,

On 12/12/2023 17:55, Randy Dunlap wrote:
> Hi--
> 
> On 12/12/23 11:52, Avadhut Naik wrote:
>> ACPI Platform Health Assessment Table (PHAT) enables a platform to expose
>> an extensible set of platform health-related telemetry. The telemetry is
>> exposed through Firmware Version and Firmware Health Data Records which
>> provide version data and health-related information of their associated
>> components respectively.
>>
>> Additionally, the platform also provides Reset Reason Health Record
>> in the table highlighting the cause of the last system reset in case
>> of expected and unexpected events. Vendor-specific data capturing the
>> underlying state of the system during reset can also be optionally
>> provided through the record.[1]
>>
>> Introduce support for parsing, decoding, and logging the PHAT table, if
>> supported by the system. By default, the table will be logged to stdout.
>> The same, however, can be changed through the "-o" command line parameter.
>> Additionally, the tool also provides a hexdump of the entire table, if
>> needed, through the "-x" parameter.
>>
>> Signed-off-by: Avadhut Naik <avadhut.naik@....com>
>> ---
>>  Makefile     |  12 ++
>>  parse_phat.c | 516 +++++++++++++++++++++++++++++++++++++++++++++++++++
>>  2 files changed, 528 insertions(+)
>>  create mode 100644 Makefile
>>  create mode 100644 parse_phat.c
>>
> 
> Does acpidump not parse the PHAT table?
> Could it?  Should it?
> 
IIUC, acpidump will only provide a raw hex dump of the entire PHAT table, if present,
much like it does for other ACPI tables. The onus is on the user for deciphering the
hex dump.

I'd think that the parsing functionality can be incorporated in acpidump but not
sure if it should be. Would love to head from its maintainers, if possible.
> 
> thanks.

-- 
Thanks,
Avadhut Naik

Powered by blists - more mailing lists