]> git.ipfire.org Git - thirdparty/kernel/stable.git/commit
x86/hyperv: Fix the detection of E820_TYPE_PRAM in a Gen2 VM
authorSaurabh Sengar <ssengar@linux.microsoft.com>
Sat, 11 Nov 2023 08:37:47 +0000 (00:37 -0800)
committerWei Liu <wei.liu@kernel.org>
Sun, 12 Nov 2023 22:50:30 +0000 (22:50 +0000)
commit7e8037b099c0bbe8f2109dc452dbcab8d400fc53
treee0726a05f9646086ec6d1fd72cb0c7d3d9a3a3cd
parentc3803203bc5ec910a3eb06172cf6fb368e0e4390
x86/hyperv: Fix the detection of E820_TYPE_PRAM in a Gen2 VM

A Gen2 VM doesn't support legacy PCI/PCIe, so both raw_pci_ops and
raw_pci_ext_ops are NULL, and pci_subsys_init() -> pcibios_init()
doesn't call pcibios_resource_survey() -> e820__reserve_resources_late();
as a result, any emulated persistent memory of E820_TYPE_PRAM (12) via
the kernel parameter memmap=nn[KMG]!ss is not added into iomem_resource
and hence can't be detected by register_e820_pmem().

Fix this by directly calling e820__reserve_resources_late() in
hv_pci_init(), which is called from arch_initcall(pci_arch_init).

It's ok to move a Gen2 VM's e820__reserve_resources_late() from
subsys_initcall(pci_subsys_init) to arch_initcall(pci_arch_init) because
the code in-between doesn't depend on the E820 resources.
e820__reserve_resources_late() depends on e820__reserve_resources(),
which has been called earlier from setup_arch().

For a Gen-2 VM, the new hv_pci_init() also adds any memory of
E820_TYPE_PMEM (7) into iomem_resource, and acpi_nfit_register_region() ->
acpi_nfit_insert_resource() -> region_intersects() returns
REGION_INTERSECTS, so the memory of E820_TYPE_PMEM won't get added twice.

Changed the local variable "int gen2vm" to "bool gen2vm".

Signed-off-by: Saurabh Sengar <ssengar@linux.microsoft.com>
Signed-off-by: Dexuan Cui <decui@microsoft.com>
Signed-off-by: Wei Liu <wei.liu@kernel.org>
Message-ID: <1699691867-9827-1-git-send-email-ssengar@linux.microsoft.com>
arch/x86/hyperv/hv_init.c