From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mga17.intel.com (mga17.intel.com [192.55.52.151]) by mx.groups.io with SMTP id smtpd.web08.9091.1645271832752102625 for ; Sat, 19 Feb 2022 03:58:12 -0800 Authentication-Results: mx.groups.io; dkim=fail reason="unable to parse pub key" header.i=@intel.com header.s=intel header.b=Unvzfy20; spf=pass (domain: intel.com, ip: 192.55.52.151, mailfrom: min.m.xu@intel.com) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1645271892; x=1676807892; h=from:to:cc:subject:date:message-id:in-reply-to: references:mime-version:content-transfer-encoding; bh=8VPIPn6dARC9d7HcY+TQoJSkWT9d+IDh3ZU9StYaygw=; b=Unvzfy20nSkvmb5P0jwoRKs6NZrFRdd4541zJ0t3TH8+HmLQ1LqE4Syp 2JSjbjDYFl36O32LDux6HfWWR61O7Zz2DvUadItciHgq8WKUSiTY3gRbU UDTa9IBNQ8XcHV9YA+Vztarf4QrThvpGRpMB5TO3q9UIiNbUJxr2h2YZA /yLqx10CPYvdALUwW5GlF59keaIYQ0AW369mG+MqYqMSbBAnybhD+czPW UNPkbbY2mL8qy57C9aSscQjdN8k6PMlUNWMSsXhg1ibZB6erlCD/zftdn LeJOer00Zmg6LJ3teCsGcMjgxTWJo9qiXavFGjABuDx/hrMPqzxa3/jsV w==; X-IronPort-AV: E=McAfee;i="6200,9189,10262"; a="231915439" X-IronPort-AV: E=Sophos;i="5.88,381,1635231600"; d="scan'208";a="231915439" Received: from orsmga008.jf.intel.com ([10.7.209.65]) by fmsmga107.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 19 Feb 2022 03:58:12 -0800 X-IronPort-AV: E=Sophos;i="5.88,381,1635231600"; d="scan'208";a="546691249" Received: from mxu9-mobl1.ccr.corp.intel.com ([10.249.175.253]) by orsmga008-auth.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 19 Feb 2022 03:58:09 -0800 From: "Min Xu" To: devel@edk2.groups.io Cc: Min Xu , Ard Biesheuvel , Jordan Justen , Brijesh Singh , Erdem Aktas , James Bottomley , Jiewen Yao , Tom Lendacky , Gerd Hoffmann Subject: [PATCH V6 21/42] OvmfPkg/PlatformInitLib: Add memory functions Date: Sat, 19 Feb 2022 19:56:34 +0800 Message-Id: <808b28400795301aeae824853fcb033cc39d3873.1645261990.git.min.m.xu@intel.com> X-Mailer: git-send-email 2.29.2.windows.2 In-Reply-To: References: MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Below functions are introduced in PlatformInitLib: - PlatformGetFirstNonAddress - PlatformAddressWidthInitialization - PlatformGetSystemMemorySizeBelow4gb - PlatformQemuUc32BaseInitialization - PlatformInitializeRamRegions They correspond to the below functions in OvmfPkg/PlatformPei: - GetFirstNonAddress - AddressWidthInitialization - GetSystemMemorySizeBelow4gb - QemuUc32BaseInitialization - InitializeRamRegions Note: PlatformInitLib will not determine whether SMM or S3 is supported or not. Instead the caller of these functions should input SMM / S3 support as the IN parameter by themselves. This is to reduce the complexity of PlatformInitLib. Another reason is that some PCDs cannot be declared as FixedAtBuild while PlatformInitLib is designed to be used in both SEC and PEI phase. Cc: Ard Biesheuvel Cc: Jordan Justen Cc: Brijesh Singh Cc: Erdem Aktas Cc: James Bottomley Cc: Jiewen Yao Cc: Tom Lendacky Cc: Gerd Hoffmann Signed-off-by: Min Xu --- OvmfPkg/Include/Library/PlatformInitLib.h | 78 ++ OvmfPkg/Library/PlatformInitLib/MemDetect.c | 694 ++++++++++++++++++ .../PlatformInitLib/PlatformInitLib.inf | 16 + 3 files changed, 788 insertions(+) create mode 100644 OvmfPkg/Library/PlatformInitLib/MemDetect.c diff --git a/OvmfPkg/Include/Library/PlatformInitLib.h b/OvmfPkg/Include/Library/PlatformInitLib.h index bc540f549d60..df2646880909 100644 --- a/OvmfPkg/Include/Library/PlatformInitLib.h +++ b/OvmfPkg/Include/Library/PlatformInitLib.h @@ -63,6 +63,84 @@ PlatformDebugDumpCmos ( VOID ); +/** + * Return the highest address that DXE could possibly use, plus one. + * + * @param Pci64Base The 64-bit PCI host aperture base address. + * @param Pci64Size The 64-bit PCI host aperture size. + * @param DefaultPciMmio64Size The default 64-bit PCI host aperture size. + * + * @return The highest address that DXE could possibly use, plus one. + */ +UINT64 +EFIAPI +PlatformGetFirstNonAddress ( + OUT UINT64 *Pci64Base, + OUT UINT64 *Pci64Size, + IN UINT64 DefaultPciMmio64Size + ); + +/** + * Initialize the PhysMemAddressWidth variable, based on guest RAM size. + * + * @param FirstNonAddress The highest address that DXE could possibly use, plus one. + * + * @return The physical memory address width based on the guest RAM size. + */ +UINT8 +EFIAPI +PlatformAddressWidthInitialization ( + IN UINT64 FirstNonAddress + ); + +/** + * Get the memory size below 4GB. + * + * @return UINT32 The lower memory size. + */ +UINT32 +EFIAPI +PlatformGetSystemMemorySizeBelow4gb ( + VOID + ); + +/** + * Initializatoin of Qemu UC32Base. + * + * @param HostBridgeDevId The host bridge Dev Id. + * @param LowerMemorySize The lower memory size (under 4G). + * @return UINT32 The Qemu UC32 base address. + */ +UINT32 +EFIAPI +PlatformQemuUc32BaseInitialization ( + IN UINT16 HostBridgeDevId, + IN UINT32 LowerMemorySize + ); + +/** + Publish system RAM and reserve memory regions. + + @param Uc32Base + @param HostBridgeDevId + @param SmmSmramRequire + @param BootMode + @param S3Supported + @param LowerMemorySize + @param Q35TsegMbytes +**/ +VOID +EFIAPI +PlatformInitializeRamRegions ( + IN UINT32 Uc32Base, + IN UINT16 HostBridgeDevId, + IN BOOLEAN SmmSmramRequire, + IN EFI_BOOT_MODE BootMode, + IN BOOLEAN S3Supported, + IN UINT32 LowerMemorySize, + IN UINT16 Q35TsegMbytes + ); + VOID EFIAPI PlatformAddIoMemoryBaseSizeHob ( diff --git a/OvmfPkg/Library/PlatformInitLib/MemDetect.c b/OvmfPkg/Library/PlatformInitLib/MemDetect.c new file mode 100644 index 000000000000..5a9cb6e638ed --- /dev/null +++ b/OvmfPkg/Library/PlatformInitLib/MemDetect.c @@ -0,0 +1,694 @@ +/**@file + Memory Detection for Virtual Machines. + + Copyright (c) 2006 - 2016, Intel Corporation. All rights reserved.
+ SPDX-License-Identifier: BSD-2-Clause-Patent + +Module Name: + + MemDetect.c + +**/ + +// +// The package level header files this module uses +// +#include +#include +#include +#include +#include + +// +// The Library classes this module consumes +// +#include +#include +#include +#include +#include +#include +#include +#include +#include +#include +#include +#include + +#include + +/** + * Initializatoin of Qemu UC32Base. + * + * @param HostBridgeDevId The host bridge Dev Id. + * @param LowerMemorySize The lower memory size (under 4G). + * @return UINT32 The Qemu UC32 base address. + */ +UINT32 +EFIAPI +PlatformQemuUc32BaseInitialization ( + IN UINT16 HostBridgeDevId, + IN UINT32 LowerMemorySize + ) +{ + UINT32 Uc32Size; + UINT32 Uc32Base; + + if (HostBridgeDevId == 0xffff /* microvm */) { + return 0; + } + + if (HostBridgeDevId == INTEL_Q35_MCH_DEVICE_ID) { + // + // On q35, the 32-bit area that we'll mark as UC, through variable MTRRs, + // starts at PcdPciExpressBaseAddress. The platform DSC is responsible for + // setting PcdPciExpressBaseAddress such that describing the + // [PcdPciExpressBaseAddress, 4GB) range require a very small number of + // variable MTRRs (preferably 1 or 2). + // + + ASSERT (FixedPcdGet64 (PcdPciExpressBaseAddress) <= MAX_UINT32); + Uc32Base = (UINT32)FixedPcdGet64 (PcdPciExpressBaseAddress); + return Uc32Base; + } + + if (HostBridgeDevId == CLOUDHV_DEVICE_ID) { + Uc32Base = CLOUDHV_MMIO_HOLE_ADDRESS; + return Uc32Base; + } + + ASSERT (HostBridgeDevId == INTEL_82441_DEVICE_ID); + // + // On i440fx, start with the [LowerMemorySize, 4GB) range. Make sure one + // variable MTRR suffices by truncating the size to a whole power of two, + // while keeping the end affixed to 4GB. This will round the base up. + // + Uc32Size = GetPowerOfTwo32 ((UINT32)(SIZE_4GB - LowerMemorySize)); + Uc32Base = (UINT32)(SIZE_4GB - Uc32Size); + // + // Assuming that LowerMemorySize is at least 1 byte, Uc32Size is at most 2GB. + // Therefore mQemuUc32Base is at least 2GB. + // + ASSERT (Uc32Base >= BASE_2GB); + + if (Uc32Base != LowerMemorySize) { + DEBUG (( + DEBUG_VERBOSE, + "%a: rounded UC32 base from 0x%x up to 0x%x, for " + "an UC32 size of 0x%x\n", + __FUNCTION__, + LowerMemorySize, + Uc32Base, + Uc32Size + )); + } + + return Uc32Base; +} + +/** + Iterate over the RAM entries in QEMU's fw_cfg E820 RAM map that start outside + of the 32-bit address range. + + Find the highest exclusive >=4GB RAM address, or produce memory resource + descriptor HOBs for RAM entries that start at or above 4GB. + + @param[out] MaxAddress If MaxAddress is NULL, then ScanOrAdd64BitE820Ram() + produces memory resource descriptor HOBs for RAM + entries that start at or above 4GB. + + Otherwise, MaxAddress holds the highest exclusive + >=4GB RAM address on output. If QEMU's fw_cfg E820 + RAM map contains no RAM entry that starts outside of + the 32-bit address range, then MaxAddress is exactly + 4GB on output. + + @retval EFI_SUCCESS The fw_cfg E820 RAM map was found and processed. + + @retval EFI_PROTOCOL_ERROR The RAM map was found, but its size wasn't a + whole multiple of sizeof(EFI_E820_ENTRY64). No + RAM entry was processed. + + @return Error codes from QemuFwCfgFindFile(). No RAM + entry was processed. +**/ +EFI_STATUS +ScanOrAdd64BitE820Ram ( + IN BOOLEAN AddHighHob, + OUT UINT64 *LowMemory OPTIONAL, + OUT UINT64 *MaxAddress OPTIONAL + ) +{ + EFI_STATUS Status; + FIRMWARE_CONFIG_ITEM FwCfgItem; + UINTN FwCfgSize; + EFI_E820_ENTRY64 E820Entry; + UINTN Processed; + + Status = QemuFwCfgFindFile ("etc/e820", &FwCfgItem, &FwCfgSize); + if (EFI_ERROR (Status)) { + return Status; + } + + if (FwCfgSize % sizeof E820Entry != 0) { + return EFI_PROTOCOL_ERROR; + } + + if (LowMemory != NULL) { + *LowMemory = 0; + } + + if (MaxAddress != NULL) { + *MaxAddress = BASE_4GB; + } + + QemuFwCfgSelectItem (FwCfgItem); + for (Processed = 0; Processed < FwCfgSize; Processed += sizeof E820Entry) { + QemuFwCfgReadBytes (sizeof E820Entry, &E820Entry); + DEBUG (( + DEBUG_VERBOSE, + "%a: Base=0x%Lx Length=0x%Lx Type=%u\n", + __FUNCTION__, + E820Entry.BaseAddr, + E820Entry.Length, + E820Entry.Type + )); + if (E820Entry.Type == EfiAcpiAddressRangeMemory) { + if (AddHighHob && (E820Entry.BaseAddr >= BASE_4GB)) { + UINT64 Base; + UINT64 End; + + // + // Round up the start address, and round down the end address. + // + Base = ALIGN_VALUE (E820Entry.BaseAddr, (UINT64)EFI_PAGE_SIZE); + End = (E820Entry.BaseAddr + E820Entry.Length) & + ~(UINT64)EFI_PAGE_MASK; + if (Base < End) { + PlatformAddMemoryRangeHob (Base, End); + DEBUG (( + DEBUG_VERBOSE, + "%a: PlatformAddMemoryRangeHob [0x%Lx, 0x%Lx)\n", + __FUNCTION__, + Base, + End + )); + } + } + + if (MaxAddress || LowMemory) { + UINT64 Candidate; + + Candidate = E820Entry.BaseAddr + E820Entry.Length; + if (MaxAddress && (Candidate > *MaxAddress)) { + *MaxAddress = Candidate; + DEBUG (( + DEBUG_VERBOSE, + "%a: MaxAddress=0x%Lx\n", + __FUNCTION__, + *MaxAddress + )); + } + + if (LowMemory && (Candidate > *LowMemory) && (Candidate < BASE_4GB)) { + *LowMemory = Candidate; + DEBUG (( + DEBUG_VERBOSE, + "%a: LowMemory=0x%Lx\n", + __FUNCTION__, + *LowMemory + )); + } + } + } + } + + return EFI_SUCCESS; +} + +/** + * Get the memory size below 4GB. + * + * @return UINT32 The lower memory size. + */ +UINT32 +EFIAPI +PlatformGetSystemMemorySizeBelow4gb ( + VOID + ) +{ + EFI_STATUS Status; + UINT64 LowerMemorySize = 0; + UINT8 Cmos0x34; + UINT8 Cmos0x35; + + Status = ScanOrAdd64BitE820Ram (FALSE, &LowerMemorySize, NULL); + if ((Status == EFI_SUCCESS) && (LowerMemorySize > 0)) { + return (UINT32)LowerMemorySize; + } + + // + // CMOS 0x34/0x35 specifies the system memory above 16 MB. + // * CMOS(0x35) is the high byte + // * CMOS(0x34) is the low byte + // * The size is specified in 64kb chunks + // * Since this is memory above 16MB, the 16MB must be added + // into the calculation to get the total memory size. + // + + Cmos0x34 = (UINT8)PlatformCmosRead8 (0x34); + Cmos0x35 = (UINT8)PlatformCmosRead8 (0x35); + + return (UINT32)(((UINTN)((Cmos0x35 << 8) + Cmos0x34) << 16) + SIZE_16MB); +} + +UINT64 +GetSystemMemorySizeAbove4gb ( + ) +{ + UINT32 Size; + UINTN CmosIndex; + + // + // CMOS 0x5b-0x5d specifies the system memory above 4GB MB. + // * CMOS(0x5d) is the most significant size byte + // * CMOS(0x5c) is the middle size byte + // * CMOS(0x5b) is the least significant size byte + // * The size is specified in 64kb chunks + // + + Size = 0; + for (CmosIndex = 0x5d; CmosIndex >= 0x5b; CmosIndex--) { + Size = (UINT32)(Size << 8) + (UINT32)PlatformCmosRead8 (CmosIndex); + } + + return LShiftU64 (Size, 16); +} + +/** + * Return the highest address that DXE could possibly use, plus one. + * + * @param Pci64Base The 64-bit PCI host aperture base address. + * @param Pci64Size The 64-bit PCI host aperture size. + * @param DefaultPciMmio64Size The default 64-bit PCI host aperture size. + * + * @return UINT64 The highest address that DXE could possibly use, plus one. + */ +UINT64 +EFIAPI +PlatformGetFirstNonAddress ( + OUT UINT64 *Pci64Base, + OUT UINT64 *Pci64Size, + IN UINT64 DefaultPciMmio64Size + ) +{ + UINT64 FirstNonAddress; + UINT32 FwCfgPciMmio64Mb; + EFI_STATUS Status; + FIRMWARE_CONFIG_ITEM FwCfgItem; + UINTN FwCfgSize; + UINT64 HotPlugMemoryEnd; + + // + // set FirstNonAddress to suppress incorrect compiler/analyzer warnings + // + FirstNonAddress = 0; + + // + // If QEMU presents an E820 map, then get the highest exclusive >=4GB RAM + // address from it. This can express an address >= 4GB+1TB. + // + // Otherwise, get the flat size of the memory above 4GB from the CMOS (which + // can only express a size smaller than 1TB), and add it to 4GB. + // + Status = ScanOrAdd64BitE820Ram (FALSE, NULL, &FirstNonAddress); + if (EFI_ERROR (Status)) { + FirstNonAddress = BASE_4GB + GetSystemMemorySizeAbove4gb (); + } + + // + // If DXE is 32-bit, then we're done; PciBusDxe will degrade 64-bit MMIO + // resources to 32-bit anyway. See DegradeResource() in + // "PciResourceSupport.c". + // + #ifdef MDE_CPU_IA32 + if (!FeaturePcdGet (PcdDxeIplSwitchToLongMode)) { + return FirstNonAddress; + } + + #endif + + // + // Otherwise, in order to calculate the highest address plus one, we must + // consider the 64-bit PCI host aperture too. Fetch the default size. + // + *Pci64Size = DefaultPciMmio64Size; + + // + // See if the user specified the number of megabytes for the 64-bit PCI host + // aperture. Accept an aperture size up to 16TB. + // + // As signaled by the "X-" prefix, this knob is experimental, and might go + // away at any time. + // + Status = QemuFwCfgParseUint32 ( + "opt/ovmf/X-PciMmio64Mb", + FALSE, + &FwCfgPciMmio64Mb + ); + switch (Status) { + case EFI_UNSUPPORTED: + case EFI_NOT_FOUND: + break; + case EFI_SUCCESS: + if (FwCfgPciMmio64Mb <= 0x1000000) { + *Pci64Size = LShiftU64 (FwCfgPciMmio64Mb, 20); + break; + } + + // + // fall through + // + default: + DEBUG (( + DEBUG_WARN, + "%a: ignoring malformed 64-bit PCI host aperture size from fw_cfg\n", + __FUNCTION__ + )); + break; + } + + if (*Pci64Size == 0) { + DEBUG (( + DEBUG_INFO, + "%a: disabling 64-bit PCI host aperture\n", + __FUNCTION__ + )); + + // + // There's nothing more to do; the amount of memory above 4GB fully + // determines the highest address plus one. The memory hotplug area (see + // below) plays no role for the firmware in this case. + // + return FirstNonAddress; + } + + // + // The "etc/reserved-memory-end" fw_cfg file, when present, contains an + // absolute, exclusive end address for the memory hotplug area. This area + // starts right at the end of the memory above 4GB. The 64-bit PCI host + // aperture must be placed above it. + // + Status = QemuFwCfgFindFile ( + "etc/reserved-memory-end", + &FwCfgItem, + &FwCfgSize + ); + if (!EFI_ERROR (Status) && (FwCfgSize == sizeof HotPlugMemoryEnd)) { + QemuFwCfgSelectItem (FwCfgItem); + QemuFwCfgReadBytes (FwCfgSize, &HotPlugMemoryEnd); + DEBUG (( + DEBUG_VERBOSE, + "%a: HotPlugMemoryEnd=0x%Lx\n", + __FUNCTION__, + HotPlugMemoryEnd + )); + + ASSERT (HotPlugMemoryEnd >= FirstNonAddress); + FirstNonAddress = HotPlugMemoryEnd; + } + + // + // SeaBIOS aligns both boundaries of the 64-bit PCI host aperture to 1GB, so + // that the host can map it with 1GB hugepages. Follow suit. + // + *Pci64Base = ALIGN_VALUE (FirstNonAddress, (UINT64)SIZE_1GB); + *Pci64Size = ALIGN_VALUE (*Pci64Size, (UINT64)SIZE_1GB); + + // + // The 64-bit PCI host aperture should also be "naturally" aligned. The + // alignment is determined by rounding the size of the aperture down to the + // next smaller or equal power of two. That is, align the aperture by the + // largest BAR size that can fit into it. + // + *Pci64Base = ALIGN_VALUE (*Pci64Base, GetPowerOfTwo64 (*Pci64Size)); + + // + // The useful address space ends with the 64-bit PCI host aperture. + // + FirstNonAddress = *Pci64Base + *Pci64Size; + return FirstNonAddress; +} + +/** + * Initialize the PhysMemAddressWidth variable, based on guest RAM size. + * + * @param FirstNonAddress The highest address that DXE could possibly use, plus one. + * + * @return The physical memory address width based on the guest RAM size. + */ +UINT8 +EFIAPI +PlatformAddressWidthInitialization ( + IN UINT64 FirstNonAddress + ) +{ + UINT8 PhysMemAddressWidth; + + // + // As guest-physical memory size grows, the permanent PEI RAM requirements + // are dominated by the identity-mapping page tables built by the DXE IPL. + // The DXL IPL keys off of the physical address bits advertized in the CPU + // HOB. To conserve memory, we calculate the minimum address width here. + // + PhysMemAddressWidth = (UINT8)HighBitSet64 (FirstNonAddress); + + // + // If FirstNonAddress is not an integral power of two, then we need an + // additional bit. + // + if ((FirstNonAddress & (FirstNonAddress - 1)) != 0) { + ++PhysMemAddressWidth; + } + + // + // The minimum address width is 36 (covers up to and excluding 64 GB, which + // is the maximum for Ia32 + PAE). The theoretical architecture maximum for + // X64 long mode is 52 bits, but the DXE IPL clamps that down to 48 bits. We + // can simply assert that here, since 48 bits are good enough for 256 TB. + // + if (PhysMemAddressWidth <= 36) { + PhysMemAddressWidth = 36; + } + + ASSERT (PhysMemAddressWidth <= 48); + + return PhysMemAddressWidth; +} + +VOID +PlatformQemuInitializeRamBelow1gb ( + VOID + ) +{ + PlatformAddMemoryRangeHob (0, BASE_512KB + BASE_128KB); +} + +/** + Peform Memory Detection for QEMU / KVM + +**/ +VOID +QemuInitializeRam ( + UINT32 Uc32Base, + UINT16 HostBridgeDevId, + EFI_BOOT_MODE BootMode, + BOOLEAN SmmSmramRequire, + UINT32 LowerMemorySize, + UINT16 Q35TsegMbytes + ) +{ + UINT64 UpperMemorySize; + MTRR_SETTINGS MtrrSettings; + EFI_STATUS Status; + + DEBUG ((DEBUG_INFO, "%a called\n", __FUNCTION__)); + + if (BootMode == BOOT_ON_S3_RESUME) { + // + // Create the following memory HOB as an exception on the S3 boot path. + // + // Normally we'd create memory HOBs only on the normal boot path. However, + // CpuMpPei specifically needs such a low-memory HOB on the S3 path as + // well, for "borrowing" a subset of it temporarily, for the AP startup + // vector. + // + // CpuMpPei saves the original contents of the borrowed area in permanent + // PEI RAM, in a backup buffer allocated with the normal PEI services. + // CpuMpPei restores the original contents ("returns" the borrowed area) at + // End-of-PEI. End-of-PEI in turn is emitted by S3Resume2Pei before + // transferring control to the OS's wakeup vector in the FACS. + // + // We expect any other PEIMs that "borrow" memory similarly to CpuMpPei to + // restore the original contents. Furthermore, we expect all such PEIMs + // (CpuMpPei included) to claim the borrowed areas by producing memory + // allocation HOBs, and to honor preexistent memory allocation HOBs when + // looking for an area to borrow. + // + PlatformQemuInitializeRamBelow1gb (); + } else { + // + // Create memory HOBs + // + PlatformQemuInitializeRamBelow1gb (); + + if (SmmSmramRequire) { + UINT32 TsegSize; + + TsegSize = Q35TsegMbytes * SIZE_1MB; + PlatformAddMemoryRangeHob (BASE_1MB, LowerMemorySize - TsegSize); + PlatformAddReservedMemoryBaseSizeHob ( + LowerMemorySize - TsegSize, + TsegSize, + TRUE + ); + } else { + PlatformAddMemoryRangeHob (BASE_1MB, LowerMemorySize); + } + + // + // If QEMU presents an E820 map, then create memory HOBs for the >=4GB RAM + // entries. Otherwise, create a single memory HOB with the flat >=4GB + // memory size read from the CMOS. + // + Status = ScanOrAdd64BitE820Ram (TRUE, NULL, NULL); + if (EFI_ERROR (Status)) { + UpperMemorySize = GetSystemMemorySizeAbove4gb (); + if (UpperMemorySize != 0) { + PlatformAddMemoryBaseSizeHob (BASE_4GB, UpperMemorySize); + } + } + } + + // + // We'd like to keep the following ranges uncached: + // - [640 KB, 1 MB) + // - [LowerMemorySize, 4 GB) + // + // Everything else should be WB. Unfortunately, programming the inverse (ie. + // keeping the default UC, and configuring the complement set of the above as + // WB) is not reliable in general, because the end of the upper RAM can have + // practically any alignment, and we may not have enough variable MTRRs to + // cover it exactly. + // + if (IsMtrrSupported () && (HostBridgeDevId != CLOUDHV_DEVICE_ID)) { + MtrrGetAllMtrrs (&MtrrSettings); + + // + // MTRRs disabled, fixed MTRRs disabled, default type is uncached + // + ASSERT ((MtrrSettings.MtrrDefType & BIT11) == 0); + ASSERT ((MtrrSettings.MtrrDefType & BIT10) == 0); + ASSERT ((MtrrSettings.MtrrDefType & 0xFF) == 0); + + // + // flip default type to writeback + // + SetMem (&MtrrSettings.Fixed, sizeof MtrrSettings.Fixed, 0x06); + ZeroMem (&MtrrSettings.Variables, sizeof MtrrSettings.Variables); + MtrrSettings.MtrrDefType |= BIT11 | BIT10 | 6; + MtrrSetAllMtrrs (&MtrrSettings); + + // + // Set memory range from 640KB to 1MB to uncacheable + // + Status = MtrrSetMemoryAttribute ( + BASE_512KB + BASE_128KB, + BASE_1MB - (BASE_512KB + BASE_128KB), + CacheUncacheable + ); + ASSERT_EFI_ERROR (Status); + + // + // Set the memory range from the start of the 32-bit MMIO area (32-bit PCI + // MMIO aperture on i440fx, PCIEXBAR on q35) to 4GB as uncacheable. + // + Status = MtrrSetMemoryAttribute ( + Uc32Base, + SIZE_4GB - Uc32Base, + CacheUncacheable + ); + ASSERT_EFI_ERROR (Status); + } +} + +/** + Publish system RAM and reserve memory regions + +**/ +VOID +EFIAPI +PlatformInitializeRamRegions ( + IN UINT32 Uc32Base, + IN UINT16 HostBridgeDevId, + IN BOOLEAN SmmSmramRequire, + IN EFI_BOOT_MODE BootMode, + IN BOOLEAN S3Supported, + IN UINT32 LowerMemorySize, + IN UINT16 Q35TsegMbytes + ) +{ + QemuInitializeRam ( + Uc32Base, + HostBridgeDevId, + BootMode, + SmmSmramRequire, + LowerMemorySize, + Q35TsegMbytes + ); + + if (BootMode != BOOT_ON_S3_RESUME) { + if (!SmmSmramRequire) { + // + // Reserve the lock box storage area + // + // Since this memory range will be used on S3 resume, it must be + // reserved as ACPI NVS. + // + // If S3 is unsupported, then various drivers might still write to the + // LockBox area. We ought to prevent DXE from serving allocation requests + // such that they would overlap the LockBox storage. + // + ZeroMem ( + (VOID *)(UINTN)FixedPcdGet32 (PcdOvmfLockBoxStorageBase), + (UINTN)FixedPcdGet32 (PcdOvmfLockBoxStorageSize) + ); + BuildMemoryAllocationHob ( + (EFI_PHYSICAL_ADDRESS)(UINTN)FixedPcdGet32 (PcdOvmfLockBoxStorageBase), + (UINT64)(UINTN)FixedPcdGet32 (PcdOvmfLockBoxStorageSize), + S3Supported ? EfiACPIMemoryNVS : EfiBootServicesData + ); + } + + #ifdef MDE_CPU_X64 + if (FixedPcdGet32 (PcdOvmfWorkAreaSize) != 0) { + // + // Reserve the work area. + // + // Since this memory range will be used by the Reset Vector on S3 + // resume, it must be reserved as ACPI NVS. + // + // If S3 is unsupported, then various drivers might still write to the + // work area. We ought to prevent DXE from serving allocation requests + // such that they would overlap the work area. + // + BuildMemoryAllocationHob ( + (EFI_PHYSICAL_ADDRESS)(UINTN)FixedPcdGet32 (PcdOvmfWorkAreaBase), + (UINT64)(UINTN)FixedPcdGet32 (PcdOvmfWorkAreaSize), + EfiBootServicesData + ); + } + + #endif + } +} diff --git a/OvmfPkg/Library/PlatformInitLib/PlatformInitLib.inf b/OvmfPkg/Library/PlatformInitLib/PlatformInitLib.inf index 21813458cb59..6ba1e59246d1 100644 --- a/OvmfPkg/Library/PlatformInitLib/PlatformInitLib.inf +++ b/OvmfPkg/Library/PlatformInitLib/PlatformInitLib.inf @@ -24,15 +24,31 @@ [Sources] Cmos.c + MemDetect.c Platform.c [Packages] MdeModulePkg/MdeModulePkg.dec MdePkg/MdePkg.dec OvmfPkg/OvmfPkg.dec + UefiCpuPkg/UefiCpuPkg.dec [LibraryClasses] BaseLib DebugLib IoLib HobLib + QemuFwCfgLib + QemuFwCfgSimpleParserLib + MtrrLib + PcdLib + +[FixedPcd] + gEfiMdePkgTokenSpaceGuid.PcdPciExpressBaseAddress + gUefiOvmfPkgTokenSpaceGuid.PcdOvmfWorkAreaBase + gUefiOvmfPkgTokenSpaceGuid.PcdOvmfWorkAreaSize + gUefiOvmfPkgTokenSpaceGuid.PcdOvmfLockBoxStorageBase + gUefiOvmfPkgTokenSpaceGuid.PcdOvmfLockBoxStorageSize + +[FeaturePcd] + gEfiMdeModulePkgTokenSpaceGuid.PcdDxeIplSwitchToLongMode -- 2.29.2.windows.2