Create an almost verbatim copy of the "OvmfPkg/AcpiPlatformDxe/AcpiPlatformDxe.inf" driver for the OvmfXen platform. We're going to trim the driver in subsequent patches. Ultimately, the XenAcpiPlatformDxe driver will duplicate a negligible amount of code that is currently present in the QemuFwCfgAcpiPlatformDxe driver.
List the new driver in "Maintainers.txt", in the "OvmfPkg: Xen-related modules" section. Switch the OvmfXen platform to the new driver at once. This patch should be reviewed with "git show --find-copies-harder". Cc: Andrew Fish <af...@apple.com> Cc: Anthony Perard <anthony.per...@citrix.com> Cc: Ard Biesheuvel <ardb+tianoc...@kernel.org> Cc: Jordan Justen <jordan.l.jus...@intel.com> Cc: Julien Grall <jul...@xen.org> Cc: Leif Lindholm <l...@nuviainc.com> Cc: Michael D Kinney <michael.d.kin...@intel.com> Cc: Philippe Mathieu-Daudé <phi...@redhat.com> Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=2122 Signed-off-by: Laszlo Ersek <ler...@redhat.com> --- OvmfPkg/OvmfXen.dsc | 2 +- OvmfPkg/OvmfXen.fdf | 2 +- OvmfPkg/{AcpiPlatformDxe/AcpiPlatformDxe.inf => XenAcpiPlatformDxe/XenAcpiPlatformDxe.inf} | 7 ++++--- Maintainers.txt | 1 + OvmfPkg/{AcpiPlatformDxe => XenAcpiPlatformDxe}/AcpiPlatform.h | 3 ++- OvmfPkg/{AcpiPlatformDxe => XenAcpiPlatformDxe}/AcpiPlatform.c | 3 ++- OvmfPkg/{AcpiPlatformDxe => XenAcpiPlatformDxe}/BootScript.c | 2 +- OvmfPkg/{AcpiPlatformDxe => XenAcpiPlatformDxe}/EntryPoint.c | 4 ++-- OvmfPkg/{AcpiPlatformDxe => XenAcpiPlatformDxe}/PciDecoding.c | 2 +- OvmfPkg/{AcpiPlatformDxe => XenAcpiPlatformDxe}/Qemu.c | 3 +-- OvmfPkg/{AcpiPlatformDxe => XenAcpiPlatformDxe}/QemuFwCfgAcpi.c | 2 +- OvmfPkg/{AcpiPlatformDxe => XenAcpiPlatformDxe}/Xen.c | 1 + 12 files changed, 18 insertions(+), 14 deletions(-) diff --git a/OvmfPkg/OvmfXen.dsc b/OvmfPkg/OvmfXen.dsc index e535503e385d..4d5171cd43c1 100644 --- a/OvmfPkg/OvmfXen.dsc +++ b/OvmfPkg/OvmfXen.dsc @@ -649,13 +649,13 @@ [Components] OvmfPkg/SmbiosPlatformDxe/SmbiosPlatformDxe.inf # # ACPI Support # MdeModulePkg/Universal/Acpi/AcpiTableDxe/AcpiTableDxe.inf - OvmfPkg/AcpiPlatformDxe/AcpiPlatformDxe.inf + OvmfPkg/XenAcpiPlatformDxe/XenAcpiPlatformDxe.inf OvmfPkg/AcpiTables/AcpiTables.inf MdeModulePkg/Universal/Acpi/S3SaveStateDxe/S3SaveStateDxe.inf MdeModulePkg/Universal/Acpi/BootScriptExecutorDxe/BootScriptExecutorDxe.inf MdeModulePkg/Universal/Acpi/BootGraphicsResourceTableDxe/BootGraphicsResourceTableDxe.inf # diff --git a/OvmfPkg/OvmfXen.fdf b/OvmfPkg/OvmfXen.fdf index c7d4d1853027..85306245b50f 100644 --- a/OvmfPkg/OvmfXen.fdf +++ b/OvmfPkg/OvmfXen.fdf @@ -349,13 +349,13 @@ [FV.DXEFV] INF MdeModulePkg/Bus/Isa/Ps2KeyboardDxe/Ps2KeyboardDxe.inf INF MdeModulePkg/Universal/SmbiosDxe/SmbiosDxe.inf INF OvmfPkg/SmbiosPlatformDxe/SmbiosPlatformDxe.inf INF MdeModulePkg/Universal/Acpi/AcpiTableDxe/AcpiTableDxe.inf -INF OvmfPkg/AcpiPlatformDxe/AcpiPlatformDxe.inf +INF OvmfPkg/XenAcpiPlatformDxe/XenAcpiPlatformDxe.inf INF RuleOverride=ACPITABLE OvmfPkg/AcpiTables/AcpiTables.inf INF MdeModulePkg/Universal/Acpi/S3SaveStateDxe/S3SaveStateDxe.inf INF MdeModulePkg/Universal/Acpi/BootScriptExecutorDxe/BootScriptExecutorDxe.inf INF MdeModulePkg/Universal/Acpi/BootGraphicsResourceTableDxe/BootGraphicsResourceTableDxe.inf INF FatPkg/EnhancedFatDxe/Fat.inf diff --git a/OvmfPkg/AcpiPlatformDxe/AcpiPlatformDxe.inf b/OvmfPkg/XenAcpiPlatformDxe/XenAcpiPlatformDxe.inf similarity index 85% copy from OvmfPkg/AcpiPlatformDxe/AcpiPlatformDxe.inf copy to OvmfPkg/XenAcpiPlatformDxe/XenAcpiPlatformDxe.inf index 77607298cbea..379b5d56d565 100644 --- a/OvmfPkg/AcpiPlatformDxe/AcpiPlatformDxe.inf +++ b/OvmfPkg/XenAcpiPlatformDxe/XenAcpiPlatformDxe.inf @@ -1,18 +1,19 @@ ## @file -# OVMF ACPI Platform Driver +# OVMF ACPI Platform Driver for Xen guests # +# Copyright (C) 2021, Red Hat, Inc. # Copyright (c) 2008 - 2019, Intel Corporation. All rights reserved.<BR> # SPDX-License-Identifier: BSD-2-Clause-Patent # ## [Defines] INF_VERSION = 0x00010005 - BASE_NAME = AcpiPlatform - FILE_GUID = 49970331-E3FA-4637-9ABC-3B7868676970 + BASE_NAME = XenAcpiPlatform + FILE_GUID = fa0a48ac-767d-4c88-b70c-ec54c8b900c4 MODULE_TYPE = DXE_DRIVER VERSION_STRING = 1.0 ENTRY_POINT = AcpiPlatformEntryPoint # # The following information is for reference only and not required by the build tools. diff --git a/Maintainers.txt b/Maintainers.txt index cafe6b1ab85d..50c87d1ac29a 100644 --- a/Maintainers.txt +++ b/Maintainers.txt @@ -503,12 +503,13 @@ F: OvmfPkg/Library/XenRealTimeClockLib/ F: OvmfPkg/OvmfXen.* F: OvmfPkg/OvmfXenElfHeaderGenerator.c F: OvmfPkg/PlatformPei/MemDetect.c F: OvmfPkg/PlatformPei/Platform.* F: OvmfPkg/PlatformPei/Xen.* F: OvmfPkg/SmbiosPlatformDxe/*Xen.c +F: OvmfPkg/XenAcpiPlatformDxe/ F: OvmfPkg/XenBusDxe/ F: OvmfPkg/XenIoPciDxe/ F: OvmfPkg/XenIoPvhDxe/ F: OvmfPkg/XenPlatformPei/ F: OvmfPkg/XenPvBlkDxe/ F: OvmfPkg/XenResetVector/ diff --git a/OvmfPkg/AcpiPlatformDxe/AcpiPlatform.h b/OvmfPkg/XenAcpiPlatformDxe/AcpiPlatform.h similarity index 91% copy from OvmfPkg/AcpiPlatformDxe/AcpiPlatform.h copy to OvmfPkg/XenAcpiPlatformDxe/AcpiPlatform.h index d98b79baab40..6259697c4b2d 100644 --- a/OvmfPkg/AcpiPlatformDxe/AcpiPlatform.h +++ b/OvmfPkg/XenAcpiPlatformDxe/AcpiPlatform.h @@ -1,9 +1,10 @@ /** @file - OVMF ACPI Platform Driver + OVMF ACPI Platform Driver for Xen guests + Copyright (C) 2021, Red Hat, Inc. Copyright (c) 2008 - 2012, Intel Corporation. All rights reserved.<BR> SPDX-License-Identifier: BSD-2-Clause-Patent **/ #ifndef ACPI_PLATFORM_H_ diff --git a/OvmfPkg/AcpiPlatformDxe/AcpiPlatform.c b/OvmfPkg/XenAcpiPlatformDxe/AcpiPlatform.c similarity index 95% copy from OvmfPkg/AcpiPlatformDxe/AcpiPlatform.c copy to OvmfPkg/XenAcpiPlatformDxe/AcpiPlatform.c index efbbfab89d2b..2b2dc576754b 100644 --- a/OvmfPkg/AcpiPlatformDxe/AcpiPlatform.c +++ b/OvmfPkg/XenAcpiPlatformDxe/AcpiPlatform.c @@ -1,9 +1,10 @@ /** @file - OVMF ACPI Platform Driver + OVMF ACPI Platform Driver for Xen guests + Copyright (C) 2021, Red Hat, Inc. Copyright (c) 2008 - 2012, Intel Corporation. All rights reserved.<BR> SPDX-License-Identifier: BSD-2-Clause-Patent **/ #include <Library/DebugLib.h> // ASSERT_EFI_ERROR() diff --git a/OvmfPkg/AcpiPlatformDxe/BootScript.c b/OvmfPkg/XenAcpiPlatformDxe/BootScript.c similarity index 96% copy from OvmfPkg/AcpiPlatformDxe/BootScript.c copy to OvmfPkg/XenAcpiPlatformDxe/BootScript.c index f2d1089e91ce..14d1e68694cb 100644 --- a/OvmfPkg/AcpiPlatformDxe/BootScript.c +++ b/OvmfPkg/XenAcpiPlatformDxe/BootScript.c @@ -1,11 +1,11 @@ /** @file Append an ACPI S3 Boot Script fragment from the QEMU_LOADER_WRITE_POINTER commands of QEMU's fully processed table linker/loader script. - Copyright (C) 2017, Red Hat, Inc. + Copyright (C) 2017-2021, Red Hat, Inc. SPDX-License-Identifier: BSD-2-Clause-Patent **/ #include <Library/BaseLib.h> // CpuDeadLoop() #include <Library/DebugLib.h> // DEBUG() diff --git a/OvmfPkg/AcpiPlatformDxe/EntryPoint.c b/OvmfPkg/XenAcpiPlatformDxe/EntryPoint.c similarity index 93% copy from OvmfPkg/AcpiPlatformDxe/EntryPoint.c copy to OvmfPkg/XenAcpiPlatformDxe/EntryPoint.c index bb5fe82c18d4..b6d0835fe3d4 100644 --- a/OvmfPkg/AcpiPlatformDxe/EntryPoint.c +++ b/OvmfPkg/XenAcpiPlatformDxe/EntryPoint.c @@ -1,10 +1,10 @@ /** @file - Entry point of OVMF ACPI Platform Driver + Entry point of OVMF ACPI Platform Driver for Xen guests - Copyright (C) 2015, Red Hat, Inc. + Copyright (C) 2015-2021, Red Hat, Inc. Copyright (c) 2008 - 2015, Intel Corporation. All rights reserved.<BR> SPDX-License-Identifier: BSD-2-Clause-Patent **/ #include <Guid/RootBridgesConnectedEventGroup.h> // gRootBridgesConnectedEve... diff --git a/OvmfPkg/AcpiPlatformDxe/PciDecoding.c b/OvmfPkg/XenAcpiPlatformDxe/PciDecoding.c similarity index 96% copy from OvmfPkg/AcpiPlatformDxe/PciDecoding.c copy to OvmfPkg/XenAcpiPlatformDxe/PciDecoding.c index 6ba4c936ad8e..00fc57eb13eb 100644 --- a/OvmfPkg/AcpiPlatformDxe/PciDecoding.c +++ b/OvmfPkg/XenAcpiPlatformDxe/PciDecoding.c @@ -1,11 +1,11 @@ /** @file Temporarily enable IO and MMIO decoding for all PCI devices while QEMU regenerates the ACPI tables. - Copyright (C) 2016, Red Hat, Inc. + Copyright (C) 2016-2021, Red Hat, Inc. SPDX-License-Identifier: BSD-2-Clause-Patent **/ #include <Library/DebugLib.h> // DEBUG() #include <Library/MemoryAllocationLib.h> // AllocatePool() diff --git a/OvmfPkg/AcpiPlatformDxe/Qemu.c b/OvmfPkg/XenAcpiPlatformDxe/Qemu.c similarity index 96% copy from OvmfPkg/AcpiPlatformDxe/Qemu.c copy to OvmfPkg/XenAcpiPlatformDxe/Qemu.c index bdcc46b7186a..b4a407c41b71 100644 --- a/OvmfPkg/AcpiPlatformDxe/Qemu.c +++ b/OvmfPkg/XenAcpiPlatformDxe/Qemu.c @@ -1,13 +1,12 @@ /** @file OVMF ACPI QEMU support + Copyright (C) 2012-2021, Red Hat, Inc. Copyright (c) 2008 - 2014, Intel Corporation. All rights reserved.<BR> - Copyright (C) 2012-2014, Red Hat, Inc. - SPDX-License-Identifier: BSD-2-Clause-Patent **/ #include <IndustryStandard/Acpi.h> // EFI_ACPI_1_0_IO_APIC_STRUCTURE #include <Library/BaseMemoryLib.h> // CopyMem() diff --git a/OvmfPkg/AcpiPlatformDxe/QemuFwCfgAcpi.c b/OvmfPkg/XenAcpiPlatformDxe/QemuFwCfgAcpi.c similarity index 97% copy from OvmfPkg/AcpiPlatformDxe/QemuFwCfgAcpi.c copy to OvmfPkg/XenAcpiPlatformDxe/QemuFwCfgAcpi.c index df800b149275..521c06cf546f 100644 --- a/OvmfPkg/AcpiPlatformDxe/QemuFwCfgAcpi.c +++ b/OvmfPkg/XenAcpiPlatformDxe/QemuFwCfgAcpi.c @@ -1,11 +1,11 @@ /** @file OVMF ACPI support using QEMU's fw-cfg interface + Copyright (C) 2012-2021, Red Hat, Inc. Copyright (c) 2008 - 2014, Intel Corporation. All rights reserved.<BR> - Copyright (C) 2012-2014, Red Hat, Inc. SPDX-License-Identifier: BSD-2-Clause-Patent **/ #include <IndustryStandard/Acpi.h> // EFI_ACPI_DESCRIPTION_HEADER diff --git a/OvmfPkg/AcpiPlatformDxe/Xen.c b/OvmfPkg/XenAcpiPlatformDxe/Xen.c similarity index 96% copy from OvmfPkg/AcpiPlatformDxe/Xen.c copy to OvmfPkg/XenAcpiPlatformDxe/Xen.c index be2c5d695846..e8395db548d9 100644 --- a/OvmfPkg/AcpiPlatformDxe/Xen.c +++ b/OvmfPkg/XenAcpiPlatformDxe/Xen.c @@ -1,9 +1,10 @@ /** @file OVMF ACPI Xen support + Copyright (C) 2021, Red Hat, Inc. Copyright (c) 2008 - 2012, Intel Corporation. All rights reserved.<BR> Copyright (c) 2012, Bei Guan <gbtj...@gmail.com> SPDX-License-Identifier: BSD-2-Clause-Patent **/ -- 2.19.1.3.g30247aa5d201 -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#75680): https://edk2.groups.io/g/devel/message/75680 Mute This Topic: https://groups.io/mt/83110483/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-