[PATCH v5 00/12] smbios: Deal with tables beyond 4GB

[Heinrich Schuchardt]

Am 1. Januar 2024 01:37:39 MEZ schrieb Simon Glass <sjg at chromium.org>:
>Hi Heinrich,
>
>On Sun, Dec 31, 2023 at 9:07 AM Heinrich Schuchardt <xypron.glpk at gmx.de> wrote:
>>
>> On 12/31/23 16:56, Heinrich Schuchardt wrote:
>> >
>> >
>> > Am 31. Dezember 2023 16:25:43 MEZ schrieb Simon Glass <sjg at chromium.org>:
>> >> When the malloc() region extends beyond 4GB on ARM we may end up with
>> >> an SMBIOS table in that region.
>> >>
>> >> Add support for writing an SMBIOS3 table, which supports a 64-bit
>> >> address.
>> >>
>> >> Note that this problem does not happen on x86 since it requires the
>> >> tables to be placed just below 1MB in memory, unless
>> >> CONFIG_BLOBLIST_TABLES is enabled.
>> >
>> > The ACPI specification only requires that a low memory RSDP exists on non-UEFI x86 systems.
>> >
>> > Nothing stops us from copying the RSDP only to low memory when not booting via EFI. Maybe that way we could have more common code. But this is beyond the scope of this series.
>> >
>>
>> Same is true for the SMBIOS anchor. You could have all tables in high
>> memory and only copy the anchor to low memory when booting non-UEFI x86.
>
>Yes, that's right...do you think it is worth mentioning that, though?

Reducing the x86 speific code is something we   should investigate in future. The current patch series looks good to me and hopefully will be merged soon.

Regards

Heinrich


>
>[..]
>
>Regards,
>Simon