diff options
author | Rebecca Cran <rebecca@nuviainc.com> | 2021-10-04 10:22:33 -0600 |
---|---|---|
committer | mergify[bot] <37929162+mergify[bot]@users.noreply.github.com> | 2021-10-05 09:25:52 +0000 |
commit | c49cb8f30e6223dc6b55903af178afa1dfde857f (patch) | |
tree | 7a10e87bf6eb250d7051f102e06d4163c1bdcf77 /ArmPkg/Universal/Smbios/SmbiosMiscDxe | |
parent | 4cc1458dbe004b1d70534caa55f475f6d19fe14a (diff) | |
download | edk2-c49cb8f30e6223dc6b55903af178afa1dfde857f.zip edk2-c49cb8f30e6223dc6b55903af178afa1dfde857f.tar.gz edk2-c49cb8f30e6223dc6b55903af178afa1dfde857f.tar.bz2 |
ArmPkg: SmbiosMiscDxe: Don't populate ExtendedBiosSize when size < 16MB
According to the SMBIOS specification, the ExtendedBiosSize field should
be zero when the BIOS size is less than 16MB:
"Size (n) where 64K * (n+1) is the size of the
physical device containing the BIOS, in
bytes.
FFh - size is 16MB or greater, see Extended
BIOS ROM Size for actual size."
Fix the code in MiscBiosVendorFunction.c to only populate the
ExtendedBiosSize field if the BIOS size is greater than 16MB.
Fix the code to correctly populate the ExtendedBiosSize field with the
unit bits set to MB if the size is between 16MB and 16GB.
Signed-off-by: Rebecca Cran <rebecca@nuviainc.com>
Reviewed-by: Sami Mujawar <sami.mujawar@arm.com>
Acked-by: Nhi Pham <nhi@os.amperecomputing.com>
Diffstat (limited to 'ArmPkg/Universal/Smbios/SmbiosMiscDxe')
-rw-r--r-- | ArmPkg/Universal/Smbios/SmbiosMiscDxe/Type00/MiscBiosVendorFunction.c | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/ArmPkg/Universal/Smbios/SmbiosMiscDxe/Type00/MiscBiosVendorFunction.c b/ArmPkg/Universal/Smbios/SmbiosMiscDxe/Type00/MiscBiosVendorFunction.c index 5aea325..5679eba 100644 --- a/ArmPkg/Universal/Smbios/SmbiosMiscDxe/Type00/MiscBiosVendorFunction.c +++ b/ArmPkg/Universal/Smbios/SmbiosMiscDxe/Type00/MiscBiosVendorFunction.c @@ -240,11 +240,12 @@ SMBIOS_MISC_TABLE_FUNCTION (MiscBiosVendor) SmbiosRecord->BiosSegment = (UINT16)(FixedPcdGet32 (PcdFdBaseAddress) / SIZE_64KB);
if (BiosPhysicalSize < SIZE_16MB) {
SmbiosRecord->BiosSize = Base2ToByteWith64KUnit (BiosPhysicalSize) - 1;
- SmbiosRecord->ExtendedBiosSize.Size = BiosPhysicalSize / SIZE_1MB;
- SmbiosRecord->ExtendedBiosSize.Unit = 0; // Size is in MB
} else {
SmbiosRecord->BiosSize = 0xFF;
- if (BiosPhysicalSize > 0x3FFF) {
+ if (BiosPhysicalSize < SIZE_16GB) {
+ SmbiosRecord->ExtendedBiosSize.Size = BiosPhysicalSize / SIZE_1MB;
+ SmbiosRecord->ExtendedBiosSize.Unit = 0; // Size is in MB
+ } else {
SmbiosRecord->ExtendedBiosSize.Size = BiosPhysicalSize / SIZE_1GB;
SmbiosRecord->ExtendedBiosSize.Unit = 1; // Size is in GB
}
|