[v3] Parse SMBIOS additional entries

[PATCH v3 0/5] Parse SMBIOS additional entries

Posted by Mario Limonciello (AMD) 3 weeks, 6 days ago

The SMBIOS additional entries on AMD Zen4+ systems running an AGESA
based BIOS contain information about the AGESA version which can be
useful for matching the software stack when debugging an issue.

Add support for parsing this from SMBIOS tables and output it into
the logs at bootup.  Additionally export the information to debugfs
in case users are interested in any other strings.

v3:
 * Drop / merge patches from feedback
 * Don't save additional strings, only show AGESA string if found

Mario Limonciello (1):
  firmware: dmi: Add pr_fmt() for dmi_scan.c

Mario Limonciello (AMD) (4):
  firmware: dmi: Correct an indexing error in dmi.h
  firmware: dmi: Adjust dmi_decode() to use enums
  firmware: dmi: Add missing DMI entry types
  firmware: dmi: Read additional information when decoding DMI table

 drivers/firmware/dmi_scan.c | 61 +++++++++++++++++++++++++++++--------
 include/linux/dmi.h         | 12 ++++++++
 2 files changed, 60 insertions(+), 13 deletions(-)

-- 
2.43.0

Re: [PATCH v3 0/5] Parse SMBIOS additional entries

Posted by Yazen Ghannam 3 weeks ago

On Sat, Jan 10, 2026 at 07:48:29AM -0600, Mario Limonciello (AMD) wrote:
> The SMBIOS additional entries on AMD Zen4+ systems running an AGESA
> based BIOS contain information about the AGESA version which can be
> useful for matching the software stack when debugging an issue.
> 
> Add support for parsing this from SMBIOS tables and output it into
> the logs at bootup.  Additionally export the information to debugfs
> in case users are interested in any other strings.
> 
> v3:
>  * Drop / merge patches from feedback
>  * Don't save additional strings, only show AGESA string if found
> 
> Mario Limonciello (1):
>   firmware: dmi: Add pr_fmt() for dmi_scan.c
> 
> Mario Limonciello (AMD) (4):
>   firmware: dmi: Correct an indexing error in dmi.h
>   firmware: dmi: Adjust dmi_decode() to use enums
>   firmware: dmi: Add missing DMI entry types
>   firmware: dmi: Read additional information when decoding DMI table
> 

One patch is authored with a different email. Is this intentional?

Not a problem, but wanted to ask just in case.

Thanks,
Yazen

Re: [PATCH v3 0/5] Parse SMBIOS additional entries

Posted by Mario Limonciello (AMD) (kernel.org) 3 weeks ago


On 1/16/2026 9:03 AM, Yazen Ghannam wrote:
> On Sat, Jan 10, 2026 at 07:48:29AM -0600, Mario Limonciello (AMD) wrote:
>> The SMBIOS additional entries on AMD Zen4+ systems running an AGESA
>> based BIOS contain information about the AGESA version which can be
>> useful for matching the software stack when debugging an issue.
>>
>> Add support for parsing this from SMBIOS tables and output it into
>> the logs at bootup.  Additionally export the information to debugfs
>> in case users are interested in any other strings.
>>
>> v3:
>>   * Drop / merge patches from feedback
>>   * Don't save additional strings, only show AGESA string if found
>>
>> Mario Limonciello (1):
>>    firmware: dmi: Add pr_fmt() for dmi_scan.c
>>
>> Mario Limonciello (AMD) (4):
>>    firmware: dmi: Correct an indexing error in dmi.h
>>    firmware: dmi: Adjust dmi_decode() to use enums
>>    firmware: dmi: Add missing DMI entry types
>>    firmware: dmi: Read additional information when decoding DMI table
>>
> 
> One patch is authored with a different email. Is this intentional?
> 
> Not a problem, but wanted to ask just in case.
> 

It was an oversight because I worked on two different machines and 
forgot I had git configured differently when I made the commits.

If there are other changes needed for the series I'll fix it in next 
version, otherwise leaving it as is is totally fine for me.