From: "Corvin Köhne" <corvink@FreeBSD.org>
To: Ard Biesheuvel <ardb@kernel.org>, "Yao, Jiewen" <jiewen.yao@intel.com>
Cc: Laszlo Ersek <lersek@redhat.com>,
"devel@edk2.groups.io" <devel@edk2.groups.io>,
Anatol Belski <anbelski@linux.microsoft.com>,
"Warkentin, Andrei" <andrei.warkentin@intel.com>,
Anthony Perard <anthony.perard@citrix.com>,
Ard Biesheuvel <ardb+tianocore@kernel.org>,
"Aktas, Erdem" <erdemaktas@google.com>,
Gerd Hoffmann <kraxel@redhat.com>,
Jianyong Wu <jianyong.wu@arm.com>,
Michael Roth <michael.roth@amd.com>,
"Xu, Min M" <min.m.xu@intel.com>,
Rebecca Cran <rebecca@bsdio.com>,
Sunil V L <sunilvl@ventanamicro.com>,
Tom Lendacky <thomas.lendacky@amd.com>
Subject: Re: [edk2-devel] [PATCH 00/37] OvmfPkg: remove the CSM (after edk2-stable202311)
Date: Mon, 13 Nov 2023 08:52:14 +0100 [thread overview]
Message-ID: <1d29f69c31a20d9897e75a1219b499bfeccc5c00.camel@FreeBSD.org> (raw)
In-Reply-To: <CAMj1kXFA6N=GB1EzB43cQRbJYFRU8NbQk+RC08Zk=mq48Kzy4Q@mail.gmail.com>
[-- Attachment #1: Type: text/plain, Size: 24474 bytes --]
On Sat, 2023-11-11 at 20:54 +1000, Ard Biesheuvel wrote:
> On Sat, 11 Nov 2023 at 12:12, Yao, Jiewen <jiewen.yao@intel.com>
> wrote:
> >
> > Glad to see we can get rid of the legacy burden.
> >
> > All: Reviewed-by: Jiewen Yao <Jiewen.yao@intel.com>
> >
> >
>
> Agreed, thanks for taking care of this.
>
> Reviewed-by: Ard Biesheuvel <ardb@kernel.org>
Acked-by: Corvin Köhne <corvink@FreeBSD.org>
>
> > > -----Original Message-----
> > > From: Laszlo Ersek <lersek@redhat.com>
> > > Sent: Saturday, November 11, 2023 7:58 AM
> > > To: devel@edk2.groups.io
> > > Cc: Anatol Belski <anbelski@linux.microsoft.com>; Warkentin,
> > > Andrei
> > > <andrei.warkentin@intel.com>; Anthony Perard
> > > <anthony.perard@citrix.com>;
> > > Ard Biesheuvel <ardb+tianocore@kernel.org>; Corvin Köhne
> > > <corvink@freebsd.org>; Aktas, Erdem <erdemaktas@google.com>; Gerd
> > > Hoffmann <kraxel@redhat.com>; Jianyong Wu <jianyong.wu@arm.com>;
> > > Yao,
> > > Jiewen <jiewen.yao@intel.com>; Michael Roth
> > > <michael.roth@amd.com>; Xu,
> > > Min M <min.m.xu@intel.com>; Rebecca Cran <rebecca@bsdio.com>;
> > > Sunil V L
> > > <sunilvl@ventanamicro.com>; Tom Lendacky
> > > <thomas.lendacky@amd.com>
> > > Subject: [PATCH 00/37] OvmfPkg: remove the CSM (after edk2-
> > > stable202311)
> > >
> > > BZ: https://bugzilla.tianocore.org/show_bug.cgi?id=4588
> > > CI: https://github.com/tianocore/edk2/pull/5031 (@ 961d5add9f03)
> > >
> > > Remove the Compatibility Support Module (CSM) from OVMF (after
> > > edk2-stable202311).
> > >
> > > Modify the following platforms:
> > >
> > > OvmfPkg/AmdSev/AmdSevX64.dsc
> > > OvmfPkg/Bhyve/BhyveX64.dsc
> > > OvmfPkg/CloudHv/CloudHvX64.dsc
> > > OvmfPkg/IntelTdx/IntelTdxX64.dsc
> > > OvmfPkg/Microvm/MicrovmX64.dsc
> > > OvmfPkg/OvmfPkgIa32.dsc
> > > OvmfPkg/OvmfPkgIa32X64.dsc
> > > OvmfPkg/OvmfPkgX64.dsc
> > > OvmfPkg/OvmfXen.dsc
> > >
> > > Each of those platforms builds at every stage of the series.
> > >
> > > Follow a gradual approach. Peel off CSM components in (reverse)
> > > dependency order:
> > >
> > > - exclude a high-level CSM component (library or driver) from the
> > > OVMF
> > > platforms, without breaking dependencies of low-level
> > > components;
> > >
> > > - delete the high-level component from OvmfPkg;
> > >
> > > - add, to a removal queue, any source code artifacts (protocols,
> > > GUIDs,
> > > headers, PCDs) that the high-level component's deletion
> > > *unreferences*;
> > >
> > > - delete all entries of the removal queue (protocols, GUIDs,
> > > headers,
> > > PCDs) from the edk2 source tree that are now completely
> > > unreferenced
> > > (... and extend the removal queue recursively, if needed);
> > >
> > > - advance to the next component that now qualifies as "high-
> > > level"
> > > (because nothing consumes the services it provides any longer),
> > > and
> > > exclude that one.
> > >
> > > Regression-test the traditional platforms as needed; see the
> > > notes in
> > > the following patches:
> > >
> > > - OvmfPkg: remove PcdCsmEnable
> > > - OvmfPkg/IncompatiblePciDeviceSupportDxe: ignore CSM presence
> > > - OvmfPkg: exclude 8254TimerDxe
> > >
> > > Cc: Anatol Belski <anbelski@linux.microsoft.com>
> > > Cc: Andrei Warkentin <andrei.warkentin@intel.com>
> > > Cc: Anthony Perard <anthony.perard@citrix.com>
> > > Cc: Ard Biesheuvel <ardb+tianocore@kernel.org>
> > > Cc: Corvin Köhne <corvink@freebsd.org>
> > > Cc: Erdem Aktas <erdemaktas@google.com>
> > > Cc: Gerd Hoffmann <kraxel@redhat.com>
> > > Cc: Jianyong Wu <jianyong.wu@arm.com>
> > > Cc: Jiewen Yao <jiewen.yao@intel.com>
> > > Cc: Michael Roth <michael.roth@amd.com>
> > > Cc: Min Xu <min.m.xu@intel.com>
> > > Cc: Rebecca Cran <rebecca@bsdio.com>
> > > Cc: Sunil V L <sunilvl@ventanamicro.com>
> > > Cc: Tom Lendacky <thomas.lendacky@amd.com>
> > >
> > > Thanks
> > > Laszlo
> > >
> > > Laszlo Ersek (37):
> > > OvmfPkg: cripple CSM_ENABLE macro
> > > OvmfPkg: remove PcdCsmEnable
> > > OvmfPkg: unplug LegacyBootManagerLib from BdsDxe and UiApp
> > > OvmfPkg: remove LegacyBootManagerLib
> > > OvmfPkg: unplug LegacyBootMaintUiLib from UiApp
> > > OvmfPkg: remove LegacyBootMaintUiLib
> > > OvmfPkg: remove gEfiLegacyDevOrderVariableGuid
> > > OvmfPkg: exclude the CSM-based VideoDxe driver
> > > OvmfPkg: remove Csm/BiosThunk/VideoDxe
> > > OvmfPkg: remove gEfiVgaMiniPortProtocolGuid
> > > OvmfPkg: remove Bios Video PCDs
> > > OvmfPkg: exclude LegacyBiosDxe
> > > OvmfPkg/IncompatiblePciDeviceSupportDxe: ignore CSM presence
> > > Revert "OvmfPkg: don't assign PCI BARs above 4GiB when CSM
> > > enabled"
> > > OvmfPkg: remove LegacyBiosDxe
> > > OvmfPkg: exclude NullMemoryTestDxe driver
> > > OvmfPkg: remove gEfiIsaIoProtocolGuid
> > > OvmfPkg: remove gEfiIsaAcpiProtocolGuid
> > > OvmfPkg: remove gEfiLegacyBiosGuid
> > > OvmfPkg: remove LegacyBiosDxe PCDs
> > > OvmfPkg: unplug CsmSupportLib from BdsDxe
> > > OvmfPkg: remove CsmSupportLib
> > > OvmfPkg: remove gEfiFirmwareVolumeProtocolGuid
> > > OvmfPkg: remove gEfiLegacyBiosPlatformProtocolGuid
> > > OvmfPkg: remove gEfiLegacyBiosProtocolGuid
> > > OvmfPkg: remove gEfiLegacyInterruptProtocolGuid
> > > OvmfPkg: remove <FrameworkDxe.h>
> > > OvmfPkg: exclude Csm16.inf / Csm16.bin
> > > OvmfPkg: remove Rule.Common.USER_DEFINED.CSM from all FDF files
> > > OvmfPkg: remove Csm16
> > > OvmfPkg: exclude 8254TimerDxe
> > > OvmfPkg: remove 8254TimerDxe
> > > OvmfPkg: exclude 8259InterruptControllerDxe
> > > OvmfPkg: remove 8259InterruptControllerDxe
> > > OvmfPkg: remove gEfiLegacy8259ProtocolGuid
> > > OvmfPkg: remove Pcd8259LegacyModeEdgeLevel and
> > > Pcd8259LegacyModeMask
> > > OvmfPkg: remove CSM_ENABLE build macro
> > >
> > > OvmfPkg/8254TimerDxe/8254Timer.inf
> > > | 43 -
> > > OvmfPkg/8254TimerDxe/Timer.c
> > > | 406 ---
> > > OvmfPkg/8254TimerDxe/Timer.h
> > > | 186 --
> > > OvmfPkg/8254TimerDxe/Timer.uni
> > > | 16 -
> > > OvmfPkg/8254TimerDxe/TimerExtra.uni
> > > | 14 -
> > > OvmfPkg/8259InterruptControllerDxe/8259.c
> > > | 622 ----
> > > OvmfPkg/8259InterruptControllerDxe/8259.h
> > > | 218 --
> > > OvmfPkg/8259InterruptControllerDxe/8259.inf
> > > | 45 -
> > > OvmfPkg/8259InterruptControllerDxe/Legacy8259.uni
> > > | 16 -
> > > OvmfPkg/8259InterruptControllerDxe/Legacy8259Extra.uni
> > > | 14 -
> > > OvmfPkg/AmdSev/AmdSevX64.dsc
> > > | 4 -
> > > OvmfPkg/AmdSev/AmdSevX64.fdf
> > > | 1 -
> > > OvmfPkg/Bhyve/BhyveRfbDxe/GopDriver.c
> > > | 2 -
> > > OvmfPkg/Bhyve/BhyveX64.dsc
> > > | 24 +-
> > > OvmfPkg/Bhyve/BhyveX64.fdf
> > > | 14 -
> > > OvmfPkg/CloudHv/CloudHvX64.dsc
> > > | 1 -
> > > OvmfPkg/CloudHv/CloudHvX64.fdf
> > > | 6 -
> > > OvmfPkg/Csm/BiosThunk/VideoDxe/BiosVideo.c
> > > | 3357 ---------
> > > -----------
> > > OvmfPkg/Csm/BiosThunk/VideoDxe/BiosVideo.h
> > > | 524 ---
> > > OvmfPkg/Csm/BiosThunk/VideoDxe/BiosVideoDxe.uni
> > > | 17 -
> > > OvmfPkg/Csm/BiosThunk/VideoDxe/BiosVideoDxeExtra.uni
> > > | 14 -
> > > OvmfPkg/Csm/BiosThunk/VideoDxe/ComponentName.c
> > > | 304 --
> > > OvmfPkg/Csm/BiosThunk/VideoDxe/VesaBiosExtensions.h
> > > | 459 ---
> > > OvmfPkg/Csm/BiosThunk/VideoDxe/VideoDxe.inf
> > > | 78 -
> > > OvmfPkg/Csm/Csm16/Csm16.inf
> > > | 17 -
> > > OvmfPkg/Csm/Csm16/ReadMe.txt
> > > | 12 -
> > > OvmfPkg/Csm/CsmSupportLib/CsmSupportLib.c
> > > | 31 -
> > > OvmfPkg/Csm/CsmSupportLib/CsmSupportLib.h
> > > | 48 -
> > > OvmfPkg/Csm/CsmSupportLib/CsmSupportLib.inf
> > > | 55 -
> > > OvmfPkg/Csm/CsmSupportLib/LegacyInterrupt.c
> > > | 212 --
> > > OvmfPkg/Csm/CsmSupportLib/LegacyInterrupt.h
> > > | 115 -
> > > OvmfPkg/Csm/CsmSupportLib/LegacyPlatform.c
> > > | 1086 -------
> > > OvmfPkg/Csm/CsmSupportLib/LegacyPlatform.h
> > > | 97 -
> > > OvmfPkg/Csm/CsmSupportLib/LegacyRegion.c
> > > | 506 ---
> > > OvmfPkg/Csm/CsmSupportLib/LegacyRegion.h
> > > | 202 --
> > > OvmfPkg/Csm/Include/Framework/BootScript.h
> > > | 40 -
> > > OvmfPkg/Csm/Include/Framework/DxeCis.h
> > > | 169 -
> > > OvmfPkg/Csm/Include/Framework/FirmwareVolumeHeader.h
> > > | 79 -
> > > OvmfPkg/Csm/Include/Framework/FirmwareVolumeImageFormat.h
> > > |
> > > 32 -
> > > OvmfPkg/Csm/Include/Framework/FrameworkInternalFormRepresentatio
> > > n.h
> > > > 396 ---
> > > OvmfPkg/Csm/Include/Framework/Hob.h
> > > | 28 -
> > > OvmfPkg/Csm/Include/Framework/StatusCode.h
> > > | 155 -
> > > OvmfPkg/Csm/Include/FrameworkDxe.h
> > > | 26 -
> > > OvmfPkg/Csm/Include/Guid/LegacyBios.h
> > > | 29 -
> > > OvmfPkg/Csm/Include/Guid/LegacyDevOrder.h
> > > | 39 -
> > > OvmfPkg/Csm/Include/Protocol/FirmwareVolume.h
> > > | 339 --
> > > OvmfPkg/Csm/Include/Protocol/IsaAcpi.h
> > > | 298 --
> > > OvmfPkg/Csm/Include/Protocol/IsaIo.h
> > > | 356 ---
> > > OvmfPkg/Csm/Include/Protocol/LegacyBios.h
> > > | 1551 ---------
> > > OvmfPkg/Csm/Include/Protocol/LegacyBiosPlatform.h
> > > | 753 -----
> > > OvmfPkg/Csm/Include/Protocol/LegacyInterrupt.h
> > > | 121 -
> > > OvmfPkg/Csm/Include/Protocol/VgaMiniPort.h
> > > | 88 -
> > > OvmfPkg/Csm/LegacyBiosDxe/IA32/InterruptTable.nasm
> > > | 63 -
> > > OvmfPkg/Csm/LegacyBiosDxe/LegacyBbs.c
> > > | 530 ---
> > > OvmfPkg/Csm/LegacyBiosDxe/LegacyBda.c
> > > | 62 -
> > > OvmfPkg/Csm/LegacyBiosDxe/LegacyBios.c
> > > | 1227 -------
> > > OvmfPkg/Csm/LegacyBiosDxe/LegacyBiosDxe.inf
> > > | 129 -
> > > OvmfPkg/Csm/LegacyBiosDxe/LegacyBiosDxe.uni
> > > | 16 -
> > > OvmfPkg/Csm/LegacyBiosDxe/LegacyBiosDxeExtra.uni
> > > | 14 -
> > > OvmfPkg/Csm/LegacyBiosDxe/LegacyBiosInterface.h
> > > | 1435 -------
> > > --
> > > OvmfPkg/Csm/LegacyBiosDxe/LegacyBootSupport.c
> > > | 2212 -------
> > > ------
> > > OvmfPkg/Csm/LegacyBiosDxe/LegacyCmos.c
> > > | 117 -
> > > OvmfPkg/Csm/LegacyBiosDxe/LegacyIde.c
> > > | 315 --
> > > OvmfPkg/Csm/LegacyBiosDxe/LegacyPci.c
> > > | 3114 -------------
> > > -----
> > > OvmfPkg/Csm/LegacyBiosDxe/LegacySio.c
> > > | 489 ---
> > > OvmfPkg/Csm/LegacyBiosDxe/Thunk.c
> > > | 422 ---
> > > OvmfPkg/Csm/LegacyBiosDxe/X64/InterruptTable.nasm
> > > | 64 -
> > > OvmfPkg/Csm/LegacyBootMaintUiLib/LegacyBootMaintUi.c
> > > | 1500 -
> > > --------
> > > OvmfPkg/Csm/LegacyBootMaintUiLib/LegacyBootMaintUi.h
> > > | 242 --
> > > OvmfPkg/Csm/LegacyBootMaintUiLib/LegacyBootMaintUiLib.inf
> > > | 62 -
> > > OvmfPkg/Csm/LegacyBootMaintUiLib/LegacyBootMaintUiLib.uni
> > > | 20 -
> > > OvmfPkg/Csm/LegacyBootMaintUiLib/LegacyBootMaintUiStrings.uni
> > > |
> > > 43 -
> > > OvmfPkg/Csm/LegacyBootMaintUiLib/LegacyBootMaintUiVfr.Vfr
> > > | 67 -
> > > OvmfPkg/Csm/LegacyBootMaintUiLib/LegacyBootMaintUiVfr.h
> > > | 73 -
> > > OvmfPkg/Csm/LegacyBootManagerLib/InternalLegacyBm.h
> > > | 60 -
> > > OvmfPkg/Csm/LegacyBootManagerLib/LegacyBm.c
> > > | 1573 ------
> > > ---
> > > OvmfPkg/Csm/LegacyBootManagerLib/LegacyBootManagerLib.inf
> > > | 57
> > > -
> > > OvmfPkg/Csm/LegacyBootManagerLib/LegacyBootManagerLib.uni
> > > |
> > > 20 -
> > > OvmfPkg/Include/Protocol/Legacy8259.h
> > > | 290 --
> > > OvmfPkg/IncompatiblePciDeviceSupportDxe/IncompatiblePciDeviceSup
> > > port.c |
> > > 141 +-
> > > OvmfPkg/IncompatiblePciDeviceSupportDxe/IncompatiblePciDeviceSup
> > > port.inf |
> > > 5 +-
> > > OvmfPkg/IntelTdx/IntelTdxX64.dsc
> > > | 9 -
> > > OvmfPkg/IntelTdx/IntelTdxX64.fdf
> > > | 6 -
> > > OvmfPkg/Microvm/MicrovmX64.dsc
> > > | 4 -
> > > OvmfPkg/Microvm/MicrovmX64.fdf
> > > | 6 -
> > > OvmfPkg/OvmfPkg.dec
> > > | 100 -
> > > OvmfPkg/OvmfPkgIa32.dsc
> > > | 33 -
> > > OvmfPkg/OvmfPkgIa32.fdf
> > > | 20 +-
> > > OvmfPkg/OvmfPkgIa32X64.dsc
> > > | 37 -
> > > OvmfPkg/OvmfPkgIa32X64.fdf
> > > | 20 +-
> > > OvmfPkg/OvmfPkgX64.dsc
> > > | 37 -
> > > OvmfPkg/OvmfPkgX64.fdf
> > > | 20 +-
> > > OvmfPkg/OvmfXen.dsc
> > > | 28 +-
> > > OvmfPkg/OvmfXen.fdf
> > > | 12 -
> > > OvmfPkg/PlatformPei/MemDetect.c
> > > | 36 +-
> > > OvmfPkg/PlatformPei/PlatformPei.inf
> > > | 1 -
> > > OvmfPkg/RiscVVirt/RiscVVirtQemu.fdf
> > > | 2 -
> > > 97 files changed, 24 insertions(+), 27974 deletions(-)
> > > delete mode 100644 OvmfPkg/8254TimerDxe/8254Timer.inf
> > > delete mode 100644 OvmfPkg/8254TimerDxe/Timer.c
> > > delete mode 100644 OvmfPkg/8254TimerDxe/Timer.h
> > > delete mode 100644 OvmfPkg/8254TimerDxe/Timer.uni
> > > delete mode 100644 OvmfPkg/8254TimerDxe/TimerExtra.uni
> > > delete mode 100644 OvmfPkg/8259InterruptControllerDxe/8259.c
> > > delete mode 100644 OvmfPkg/8259InterruptControllerDxe/8259.h
> > > delete mode 100644 OvmfPkg/8259InterruptControllerDxe/8259.inf
> > > delete mode 100644
> > > OvmfPkg/8259InterruptControllerDxe/Legacy8259.uni
> > > delete mode 100644
> > > OvmfPkg/8259InterruptControllerDxe/Legacy8259Extra.uni
> > > delete mode 100644 OvmfPkg/Csm/BiosThunk/VideoDxe/BiosVideo.c
> > > delete mode 100644 OvmfPkg/Csm/BiosThunk/VideoDxe/BiosVideo.h
> > > delete mode 100644
> > > OvmfPkg/Csm/BiosThunk/VideoDxe/BiosVideoDxe.uni
> > > delete mode 100644
> > > OvmfPkg/Csm/BiosThunk/VideoDxe/BiosVideoDxeExtra.uni
> > > delete mode 100644
> > > OvmfPkg/Csm/BiosThunk/VideoDxe/ComponentName.c
> > > delete mode 100644
> > > OvmfPkg/Csm/BiosThunk/VideoDxe/VesaBiosExtensions.h
> > > delete mode 100644 OvmfPkg/Csm/BiosThunk/VideoDxe/VideoDxe.inf
> > > delete mode 100644 OvmfPkg/Csm/Csm16/Csm16.inf
> > > delete mode 100644 OvmfPkg/Csm/Csm16/ReadMe.txt
> > > delete mode 100644 OvmfPkg/Csm/CsmSupportLib/CsmSupportLib.c
> > > delete mode 100644 OvmfPkg/Csm/CsmSupportLib/CsmSupportLib.h
> > > delete mode 100644 OvmfPkg/Csm/CsmSupportLib/CsmSupportLib.inf
> > > delete mode 100644 OvmfPkg/Csm/CsmSupportLib/LegacyInterrupt.c
> > > delete mode 100644 OvmfPkg/Csm/CsmSupportLib/LegacyInterrupt.h
> > > delete mode 100644 OvmfPkg/Csm/CsmSupportLib/LegacyPlatform.c
> > > delete mode 100644 OvmfPkg/Csm/CsmSupportLib/LegacyPlatform.h
> > > delete mode 100644 OvmfPkg/Csm/CsmSupportLib/LegacyRegion.c
> > > delete mode 100644 OvmfPkg/Csm/CsmSupportLib/LegacyRegion.h
> > > delete mode 100644 OvmfPkg/Csm/Include/Framework/BootScript.h
> > > delete mode 100644 OvmfPkg/Csm/Include/Framework/DxeCis.h
> > > delete mode 100644
> > > OvmfPkg/Csm/Include/Framework/FirmwareVolumeHeader.h
> > > delete mode 100644
> > > OvmfPkg/Csm/Include/Framework/FirmwareVolumeImageFormat.h
> > > delete mode 100644
> > > OvmfPkg/Csm/Include/Framework/FrameworkInternalFormRepresentation
> > > .h
> > > delete mode 100644 OvmfPkg/Csm/Include/Framework/Hob.h
> > > delete mode 100644 OvmfPkg/Csm/Include/Framework/StatusCode.h
> > > delete mode 100644 OvmfPkg/Csm/Include/FrameworkDxe.h
> > > delete mode 100644 OvmfPkg/Csm/Include/Guid/LegacyBios.h
> > > delete mode 100644 OvmfPkg/Csm/Include/Guid/LegacyDevOrder.h
> > > delete mode 100644 OvmfPkg/Csm/Include/Protocol/FirmwareVolume.h
> > > delete mode 100644 OvmfPkg/Csm/Include/Protocol/IsaAcpi.h
> > > delete mode 100644 OvmfPkg/Csm/Include/Protocol/IsaIo.h
> > > delete mode 100644 OvmfPkg/Csm/Include/Protocol/LegacyBios.h
> > > delete mode 100644
> > > OvmfPkg/Csm/Include/Protocol/LegacyBiosPlatform.h
> > > delete mode 100644
> > > OvmfPkg/Csm/Include/Protocol/LegacyInterrupt.h
> > > delete mode 100644 OvmfPkg/Csm/Include/Protocol/VgaMiniPort.h
> > > delete mode 100644
> > > OvmfPkg/Csm/LegacyBiosDxe/IA32/InterruptTable.nasm
> > > delete mode 100644 OvmfPkg/Csm/LegacyBiosDxe/LegacyBbs.c
> > > delete mode 100644 OvmfPkg/Csm/LegacyBiosDxe/LegacyBda.c
> > > delete mode 100644 OvmfPkg/Csm/LegacyBiosDxe/LegacyBios.c
> > > delete mode 100644 OvmfPkg/Csm/LegacyBiosDxe/LegacyBiosDxe.inf
> > > delete mode 100644 OvmfPkg/Csm/LegacyBiosDxe/LegacyBiosDxe.uni
> > > delete mode 100644
> > > OvmfPkg/Csm/LegacyBiosDxe/LegacyBiosDxeExtra.uni
> > > delete mode 100644
> > > OvmfPkg/Csm/LegacyBiosDxe/LegacyBiosInterface.h
> > > delete mode 100644 OvmfPkg/Csm/LegacyBiosDxe/LegacyBootSupport.c
> > > delete mode 100644 OvmfPkg/Csm/LegacyBiosDxe/LegacyCmos.c
> > > delete mode 100644 OvmfPkg/Csm/LegacyBiosDxe/LegacyIde.c
> > > delete mode 100644 OvmfPkg/Csm/LegacyBiosDxe/LegacyPci.c
> > > delete mode 100644 OvmfPkg/Csm/LegacyBiosDxe/LegacySio.c
> > > delete mode 100644 OvmfPkg/Csm/LegacyBiosDxe/Thunk.c
> > > delete mode 100644
> > > OvmfPkg/Csm/LegacyBiosDxe/X64/InterruptTable.nasm
> > > delete mode 100644
> > > OvmfPkg/Csm/LegacyBootMaintUiLib/LegacyBootMaintUi.c
> > > delete mode 100644
> > > OvmfPkg/Csm/LegacyBootMaintUiLib/LegacyBootMaintUi.h
> > > delete mode 100644
> > > OvmfPkg/Csm/LegacyBootMaintUiLib/LegacyBootMaintUiLib.inf
> > > delete mode 100644
> > > OvmfPkg/Csm/LegacyBootMaintUiLib/LegacyBootMaintUiLib.uni
> > > delete mode 100644
> > > OvmfPkg/Csm/LegacyBootMaintUiLib/LegacyBootMaintUiStrings.uni
> > > delete mode 100644
> > > OvmfPkg/Csm/LegacyBootMaintUiLib/LegacyBootMaintUiVfr.Vfr
> > > delete mode 100644
> > > OvmfPkg/Csm/LegacyBootMaintUiLib/LegacyBootMaintUiVfr.h
> > > delete mode 100644
> > > OvmfPkg/Csm/LegacyBootManagerLib/InternalLegacyBm.h
> > > delete mode 100644 OvmfPkg/Csm/LegacyBootManagerLib/LegacyBm.c
> > > delete mode 100644
> > > OvmfPkg/Csm/LegacyBootManagerLib/LegacyBootManagerLib.inf
> > > delete mode 100644
> > > OvmfPkg/Csm/LegacyBootManagerLib/LegacyBootManagerLib.uni
> > > delete mode 100644 OvmfPkg/Include/Protocol/Legacy8259.h
> > >
> > >
> > > base-commit: 589f2e49e5f9ff998bd4f08cbf28a1572ab7b544
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#111129): https://edk2.groups.io/g/devel/message/111129
Mute This Topic: https://groups.io/mt/102518622/7686176
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [rebecca@openfw.io]
-=-=-=-=-=-=-=-=-=-=-=-
[-- Attachment #2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 833 bytes --]
next prev parent reply other threads:[~2023-11-13 7:52 UTC|newest]
Thread overview: 44+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-11-10 23:57 [edk2-devel] [PATCH 00/37] OvmfPkg: remove the CSM (after edk2-stable202311) Laszlo Ersek
2023-11-10 23:57 ` [edk2-devel] [PATCH 01/37] OvmfPkg: cripple CSM_ENABLE macro Laszlo Ersek
2023-11-10 23:57 ` [edk2-devel] [PATCH 02/37] OvmfPkg: remove PcdCsmEnable Laszlo Ersek
2023-11-10 23:57 ` [edk2-devel] [PATCH 03/37] OvmfPkg: unplug LegacyBootManagerLib from BdsDxe and UiApp Laszlo Ersek
2023-11-10 23:57 ` [edk2-devel] [PATCH 04/37] OvmfPkg: remove LegacyBootManagerLib Laszlo Ersek
2023-11-10 23:57 ` [edk2-devel] [PATCH 05/37] OvmfPkg: unplug LegacyBootMaintUiLib from UiApp Laszlo Ersek
2023-11-10 23:57 ` [edk2-devel] [PATCH 06/37] OvmfPkg: remove LegacyBootMaintUiLib Laszlo Ersek
2023-11-10 23:57 ` [edk2-devel] [PATCH 07/37] OvmfPkg: remove gEfiLegacyDevOrderVariableGuid Laszlo Ersek
2023-11-10 23:57 ` [edk2-devel] [PATCH 08/37] OvmfPkg: exclude the CSM-based VideoDxe driver Laszlo Ersek
2023-11-10 23:57 ` [edk2-devel] [PATCH 09/37] OvmfPkg: remove Csm/BiosThunk/VideoDxe Laszlo Ersek
2023-11-10 23:57 ` [edk2-devel] [PATCH 10/37] OvmfPkg: remove gEfiVgaMiniPortProtocolGuid Laszlo Ersek
2023-11-10 23:57 ` [edk2-devel] [PATCH 11/37] OvmfPkg: remove Bios Video PCDs Laszlo Ersek
2023-11-10 23:57 ` [edk2-devel] [PATCH 12/37] OvmfPkg: exclude LegacyBiosDxe Laszlo Ersek
2023-11-10 23:57 ` [edk2-devel] [PATCH 13/37] OvmfPkg/IncompatiblePciDeviceSupportDxe: ignore CSM presence Laszlo Ersek
2023-11-10 23:57 ` [edk2-devel] [PATCH 14/37] Revert "OvmfPkg: don't assign PCI BARs above 4GiB when CSM enabled" Laszlo Ersek
2023-11-10 23:57 ` [edk2-devel] [PATCH 15/37] OvmfPkg: remove LegacyBiosDxe Laszlo Ersek
2023-11-10 23:57 ` [edk2-devel] [PATCH 16/37] OvmfPkg: exclude NullMemoryTestDxe driver Laszlo Ersek
2024-04-24 11:02 ` Corvin Köhne
2023-11-10 23:58 ` [edk2-devel] [PATCH 17/37] OvmfPkg: remove gEfiIsaIoProtocolGuid Laszlo Ersek
2023-11-10 23:58 ` [edk2-devel] [PATCH 18/37] OvmfPkg: remove gEfiIsaAcpiProtocolGuid Laszlo Ersek
2023-11-10 23:58 ` [edk2-devel] [PATCH 19/37] OvmfPkg: remove gEfiLegacyBiosGuid Laszlo Ersek
2023-11-10 23:58 ` [edk2-devel] [PATCH 20/37] OvmfPkg: remove LegacyBiosDxe PCDs Laszlo Ersek
2023-11-10 23:58 ` [edk2-devel] [PATCH 21/37] OvmfPkg: unplug CsmSupportLib from BdsDxe Laszlo Ersek
2023-11-10 23:58 ` [edk2-devel] [PATCH 22/37] OvmfPkg: remove CsmSupportLib Laszlo Ersek
2023-11-10 23:58 ` [edk2-devel] [PATCH 23/37] OvmfPkg: remove gEfiFirmwareVolumeProtocolGuid Laszlo Ersek
2023-11-10 23:58 ` [edk2-devel] [PATCH 24/37] OvmfPkg: remove gEfiLegacyBiosPlatformProtocolGuid Laszlo Ersek
2023-11-10 23:58 ` [edk2-devel] [PATCH 25/37] OvmfPkg: remove gEfiLegacyBiosProtocolGuid Laszlo Ersek
2023-11-10 23:58 ` [edk2-devel] [PATCH 26/37] OvmfPkg: remove gEfiLegacyInterruptProtocolGuid Laszlo Ersek
2023-11-10 23:58 ` [edk2-devel] [PATCH 27/37] OvmfPkg: remove <FrameworkDxe.h> Laszlo Ersek
2023-11-10 23:58 ` [edk2-devel] [PATCH 28/37] OvmfPkg: exclude Csm16.inf / Csm16.bin Laszlo Ersek
2023-11-10 23:58 ` [edk2-devel] [PATCH 29/37] OvmfPkg: remove Rule.Common.USER_DEFINED.CSM from all FDF files Laszlo Ersek
2023-11-10 23:58 ` [edk2-devel] [PATCH 30/37] OvmfPkg: remove Csm16 Laszlo Ersek
2023-11-10 23:58 ` [edk2-devel] [PATCH 31/37] OvmfPkg: exclude 8254TimerDxe Laszlo Ersek
2023-11-10 23:58 ` [edk2-devel] [PATCH 32/37] OvmfPkg: remove 8254TimerDxe Laszlo Ersek
2023-11-10 23:58 ` [edk2-devel] [PATCH 33/37] OvmfPkg: exclude 8259InterruptControllerDxe Laszlo Ersek
2023-11-10 23:58 ` [edk2-devel] [PATCH 34/37] OvmfPkg: remove 8259InterruptControllerDxe Laszlo Ersek
2023-11-10 23:58 ` [edk2-devel] [PATCH 35/37] OvmfPkg: remove gEfiLegacy8259ProtocolGuid Laszlo Ersek
2023-11-10 23:58 ` [edk2-devel] [PATCH 36/37] OvmfPkg: remove Pcd8259LegacyModeEdgeLevel and Pcd8259LegacyModeMask Laszlo Ersek
2023-11-10 23:58 ` [edk2-devel] [PATCH 37/37] OvmfPkg: remove CSM_ENABLE build macro Laszlo Ersek
2023-11-11 2:12 ` [edk2-devel] [PATCH 00/37] OvmfPkg: remove the CSM (after edk2-stable202311) Yao, Jiewen
2023-11-11 10:54 ` Ard Biesheuvel
2023-11-13 7:52 ` Corvin Köhne [this message]
2023-11-13 10:44 ` Gerd Hoffmann
2023-12-07 18:10 ` Laszlo Ersek
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-list from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1d29f69c31a20d9897e75a1219b499bfeccc5c00.camel@FreeBSD.org \
--to=devel@edk2.groups.io \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox