mirrors/u-boot
1
0
Fork 0
mirror of https://source.denx.de/u-boot/u-boot.git synced 2025-08-08 16:26:58 +02:00
Code Issues Actions Packages Projects Releases Wiki Activity

smbios: Use SMBIOS 3.0 to support an address above 4GB

Browse Source 
When the SMBIOS table is written to an address above 4GB a 32-bit table
address is not large enough.

Use an SMBIOS3 table in that case.

Note that we cannot use efi_allocate_pages() since this function has
nothing to do with EFI. There is no equivalent function to allocate
memory below 4GB in U-Boot. One solution would be to create a separate
malloc() pool, or just always put the malloc() pool below 4GB.

- Use log_debug() for warning
- Rebase on Heinrich's smbios.h patch
- Set the checksum for SMBIOS3

Signed-off-by: Simon Glass <sjg@chromium.org>
Reviewed-by: Heinrich Schuchardt <xypron.glpk@gmx.de>
This commit is contained in:
Simon Glass 2023-12-31 08:25:47 -07:00
parent de4b91ca68
commit 70924294f3
2 changed files with 30 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
include/smbios.h
View File

@ -12,7 +12,7 @@
/* SMBIOS spec version implemented */ /* SMBIOS spec version implemented */
#define SMBIOS_MAJOR_VER 3 #define SMBIOS_MAJOR_VER 3
#define SMBIOS_MINOR_VER 0 #define SMBIOS_MINOR_VER 7
enum { enum {
SMBIOS_STR_MAX = 64, /* Maximum length allowed for a string */ SMBIOS_STR_MAX = 64, /* Maximum length allowed for a string */
@ -80,6 +80,10 @@ struct __packed smbios3_entry {
u64 struct_table_address; u64 struct_table_address;
}; };
/* These two structures should use the same amount of 16-byte-aligned space */
static_assert(ALIGN(16, sizeof(struct smbios_entry)) ==
ALIGN(16, sizeof(struct smbios3_entry)));
/* BIOS characteristics */ /* BIOS characteristics */
#define BIOS_CHARACTERISTICS_PCI_SUPPORTED (1 << 7) #define BIOS_CHARACTERISTICS_PCI_SUPPORTED (1 << 7)
#define BIOS_CHARACTERISTICS_UPGRADEABLE (1 << 11) #define BIOS_CHARACTERISTICS_UPGRADEABLE (1 << 11)

30
lib/smbios.c
View File

@ -567,7 +567,11 @@ ulong write_smbios_table(ulong addr)
addr = ALIGN(addr, 16); addr = ALIGN(addr, 16);
start_addr = addr; start_addr = addr;
addr += sizeof(struct smbios_entry); /*
* So far we don't know which struct will be used, but they both end
* up using the same amount of 16-bit-aligned space
*/
addr += max(sizeof(struct smbios_entry), sizeof(struct smbios3_entry));
addr = ALIGN(addr, 16); addr = ALIGN(addr, 16);
tables = addr; tables = addr;
@ -590,16 +594,32 @@ ulong write_smbios_table(ulong addr)
* We must use a pointer here so things work correctly on sandbox. The * We must use a pointer here so things work correctly on sandbox. The
* user of this table is not aware of the mapping of addresses to * user of this table is not aware of the mapping of addresses to
* sandbox's DRAM buffer. * sandbox's DRAM buffer.
*
* Check the address of the end of the tables. If it is above 4GB then
* it is sensible to use SMBIOS3 even if the start of the table is below
* 4GB (this case is very unlikely to happen in practice)
*/ */
table_addr = (ulong)map_sysmem(tables, 0); table_addr = (ulong)map_sysmem(tables, 0);
if (sizeof(table_addr) > sizeof(u32) && table_addr > (ulong)UINT_MAX) { if (sizeof(table_addr) > sizeof(u32) && addr >= (ulong)UINT_MAX) {
struct smbios3_entry *se;
/* /*
* We need to put this >32-bit pointer into the table but the * We need to put this >32-bit pointer into the table but the
* field is only 32 bits wide. * field is only 32 bits wide.
*/ */
printf("WARNING: SMBIOS table_address overflow %llx\n", log_debug("WARNING: Using SMBIOS3.0 due to table-address overflow %lx\n",
(unsigned long long)table_addr); table_addr);
addr = 0; se = map_sysmem(start_addr, sizeof(struct smbios_entry));
memset(se, '\0', sizeof(struct smbios_entry));
memcpy(se->anchor, "_SM3_", 5);
se->length = sizeof(struct smbios3_entry);
se->major_ver = SMBIOS_MAJOR_VER;
se->minor_ver = SMBIOS_MINOR_VER;
se->doc_rev = 0;
se->entry_point_rev = 1;
se->max_struct_size = len;
se->struct_table_address = table_addr;
se->checksum = table_compute_checksum(se,
sizeof(struct smbios3_entry));
} else { } else {
struct smbios_entry *se; struct smbios_entry *se;