From: "Wu, Hao A" <hao.a.wu@intel.com>
To: edk2-devel-groups-io <devel@edk2.groups.io>,
"'lersek@redhat.com'" <lersek@redhat.com>
Cc: Achin Gupta <achin.gupta@arm.com>, Andrew Fish <afish@apple.com>,
"Anthony Perard" <anthony.perard@citrix.com>,
Ard Biesheuvel <ard.biesheuvel@linaro.org>,
"You, Benjamin" <benjamin.you@intel.com>,
"Zhang, Chao B" <chao.b.zhang@intel.com>,
"Bi, Dandan" <dandan.bi@intel.com>,
David Woodhouse <dwmw2@infradead.org>,
"Dong, Eric" <eric.dong@intel.com>,
"Dong, Guo" <guo.dong@intel.com>,
"Carsey, Jaben" <jaben.carsey@intel.com>,
"Wang, Jian J" <jian.j.wang@intel.com>,
"Wu, Jiaxin" <jiaxin.wu@intel.com>,
"Yao, Jiewen" <jiewen.yao@intel.com>,
"Justen, Jordan L" <jordan.l.justen@intel.com>,
Julien Grall <julien.grall@arm.com>,
"Leif Lindholm" <leif.lindholm@linaro.org>,
"Gao, Liming" <liming.gao@intel.com>,
"Ma, Maurice" <maurice.ma@intel.com>,
"Kinney, Michael D" <michael.d.kinney@intel.com>,
"Ni, Ray" <ray.ni@intel.com>, "Fu, Siyuan" <siyuan.fu@intel.com>,
Supreeth Venkatesh <supreeth.venkatesh@arm.com>,
"Gao, Zhichao" <zhichao.gao@intel.com>
Subject: Re: [edk2-devel] [PATCH 00/35] edk2: clean up the usage of standardized (VOID*) typedefs
Date: Thu, 19 Sep 2019 00:32:36 +0000 [thread overview]
Message-ID: <B80AF82E9BFB8E4FBD8C89DA810C6A093C93392D@SHSMSX104.ccr.corp.intel.com> (raw)
In-Reply-To: <20190917194935.24322-1-lersek@redhat.com>
> -----Original Message-----
> From: devel@edk2.groups.io [mailto:devel@edk2.groups.io] On Behalf Of
> Laszlo Ersek
> Sent: Wednesday, September 18, 2019 3:49 AM
> To: edk2-devel-groups-io
> Cc: Achin Gupta; Andrew Fish; Anthony Perard; Ard Biesheuvel; You,
> Benjamin; Zhang, Chao B; Bi, Dandan; David Woodhouse; Dong, Eric; Dong,
> Guo; Wu, Hao A; Carsey, Jaben; Wang, Jian J; Wu, Jiaxin; Yao, Jiewen; Justen,
> Jordan L; Julien Grall; Leif Lindholm; Gao, Liming; Ma, Maurice; Kinney,
> Michael D; Ni, Ray; Fu, Siyuan; Supreeth Venkatesh; Gao, Zhichao
> Subject: [edk2-devel] [PATCH 00/35] edk2: clean up the usage of
> standardized (VOID*) typedefs
>
> Repository: https://github.com/lersek/edk2.git
> Branch: voidptr
>
> The UEFI / PI / Shell specifications define a number of standard types
> as pointers to VOID. This is arguably a design mistake; those types
> should have been pointers to distinct incomplete union or structure
> types. Here's why:
>
> Roughly paraphrasing the constraints from ISO C99 "6.5.16.1 Simple
> assignment" and "6.5.4 Cast operators", any pointer-to-object type
> converts implicitly to, and from, pointer-to-void, provided const /
> volatile qualifications are not relaxed. Such implicit conversions
> prevent compilers from catching at least the following two kinds of
> coding mistakes:
>
> - mixing up one type with another (for example, EFI_HANDLE with
> EFI_EVENT),
>
> - getting the depth of indirection wrong (for example, mixing up
> (EFI_HANDLE*) with EFI_HANDLE).
>
> This series first separates these standard types from each other, in the
> first patch, which is *not* being proposed for merging. This unmasks a
> number of warts (semantic issues, or actual bugs) in the source code, in
> the form of build breakages. The rest of the series works through those
> breakages, cleaning and fixing the code.
>
> Every DSC file in the edk2 tree was built for at least one of the NOOPT,
> DEBUG, RELEASE targets (NOOPT being preferred), with the GCC48 toolchain
> (for IA32 / X64) and the GCC5 toolchain (for ARM / AARCH64). Of course,
> the build arches were restricted to the SUPPORTED_ARCHITECTURES stated
> in the individual DSC files.
>
> There were two exceptions to the above rule: DynamicTablesPkg was only
> build-tested with AARCH64 (despite its SUPPORTED_ARCHITECTURES), given
> that 32-bit ARM has no ACPI bindings. StandaloneMmPkg too was only
> build-tested with AARCH64; it doesn't actually support IA32/X64 yet.
>
> Regarding boot & runtime tests, ArmVirtQemu on AARCH64 was tested with
> booting to the OS (RHEL7). Furthermore, I exercised OVMF with my usual
> boot and S3 tests, covering IA32, IA32X64, and X64. Finally, some other
> individual tests (noted per patch) were done with OVMF.
>
> Cc: Achin Gupta <achin.gupta@arm.com>
> Cc: Andrew Fish <afish@apple.com>
> Cc: Anthony Perard <anthony.perard@citrix.com>
> Cc: Ard Biesheuvel <ard.biesheuvel@linaro.org>
> Cc: Benjamin You <benjamin.you@intel.com>
> Cc: Chao Zhang <chao.b.zhang@intel.com>
> Cc: Dandan Bi <dandan.bi@intel.com>
> Cc: David Woodhouse <dwmw2@infradead.org>
> Cc: Eric Dong <eric.dong@intel.com>
> Cc: Guo Dong <guo.dong@intel.com>
> Cc: Hao A Wu <hao.a.wu@intel.com>
> Cc: Jaben Carsey <jaben.carsey@intel.com>
> Cc: Jian J Wang <jian.j.wang@intel.com>
> Cc: Jian Wang <jian.j.wang@intel.com>
> Cc: Jiaxin Wu <jiaxin.wu@intel.com>
> Cc: Jiewen Yao <jiewen.yao@intel.com>
> Cc: Jordan Justen <jordan.l.justen@intel.com>
> Cc: Julien Grall <julien.grall@arm.com>
> Cc: Leif Lindholm <leif.lindholm@linaro.org>
> Cc: Liming Gao <liming.gao@intel.com>
> Cc: Maurice Ma <maurice.ma@intel.com>
> Cc: Michael D Kinney <michael.d.kinney@intel.com>
> Cc: Ray Ni <ray.ni@intel.com>
> Cc: Siyuan Fu <siyuan.fu@intel.com>
> Cc: Supreeth Venkatesh <supreeth.venkatesh@arm.com>
> Cc: Zhichao Gao <zhichao.gao@intel.com>
>
> Thanks
> Laszlo
>
> Laszlo Ersek (35):
> DO NOT APPLY: edk2: turn standard handle types into pointers to
> non-VOID
> EmbeddedPkg: add missing EFIAPI calling convention specifiers
> EmbeddedPkg/AndroidFastbootTransportTcpDxe: fix DestroyChild() call
> EmbeddedPkg/Universal/MmcDxe: "fix" CloseProtocol() call in
> BindingStop()
> EmulatorPkg/DxeTimerLib: drop superfluous cast
> EmulatorPkg: stop abusing EFI_HANDLE for keystroke notify registration
> MdeModulePkg: fix cast in GetModuleInfoFromHandle() calls
> MdeModulePkg/UefiHiiLib: stop using EFI_HANDLE in place of
> EFI_HII_HANDLE
> MdeModulePkg: stop abusing EFI_EVENT for protocol notify registration
> MdeModulePkg/PlatformVarCleanupLib: fix HiiConstructConfigHdr() call
> MdeModulePkg: document workaround for EFI_RUNTIME_EVENT_ENTRY PI
> spec
> bug
> MdeModulePkg: stop abusing EFI_HANDLE for keystroke notify
> registration
> MdeModulePkg: PEI Core: clean up "AprioriFile" handling in
> FindFileEx()
> MdeModulePkg: fix UninstallMultipleProtocolInterfaces() calls
> MdeModulePkg/PiSmmCore: make type punning consistent
> MdeModulePkg/S3SaveState: cast Position for S3BootScriptLib explicitly
For the patches made to MdeModulePkg (patch 7~16),
Reviewed-by: Hao A Wu <hao.a.wu@intel.com>
Best Regards,
Hao Wu
> MdePkg/DxeServicesLib: remove bogus cast
> NetworkPkg/DxeNetLib: fix type typo in NetLibGetMacAddress()
> NetworkPkg: fix CloseProtocol & UninstallMultipleProtocolInterfaces
> calls
> NetworkPkg/Ip4Dxe: fix NetLibDestroyServiceChild() call
> NetworkPkg/TcpDxe: fix SockFreeFoo() parameter list
> OvmfPkg/XenBusDxe: fix UninstallMultipleProtocolInterfaces() call
> OvmfPkg/VirtioNetDxe: fix SignalEvent() call
> OvmfPkg/PlatformDxe: fix EFI_HII_HANDLE parameters of internal
> functions
> OvmfPkg/VideoDxe: document EFI_EDID_OVERRIDE_PROTOCOL.GetEdid()
> call
> SecurityPkg: fix UninstallMultipleProtocolInterfaces() calls
> SecurityPkg: stop abusing EFI_EVENT for protocol notify registration
> ShellPkg/UefiShellDriver1CommandsLib: fix parameter list typo
> ShellPkg: stop using EFI_HANDLE in place of EFI_HII_HANDLE
> ShellPkg: stop taking EFI_HANDLE in place of SHELL_FILE_HANDLE
> ShellPkg/UefiShellDebug1CommandsLib: fix ShellCloseFile() call
> ShellPkg/UefiShellLib: clarify workaround for unfixable EdkShell bug
> StandaloneMmPkg/Core: stop abusing EFI_HANDLE for FwVolHeader
> tracking
> UefiPayloadPkg/BlSupportPei: fix MMCONFIG assignment from XSDT
> UefiPayloadPkg/BlSupportDxe: fix ReserveResourceInGcd() calls
>
>
> EmbeddedPkg/Drivers/AndroidFastbootTransportTcpDxe/FastbootTranspor
> tTcp.c | 2 +-
> EmbeddedPkg/Drivers/ConsolePrefDxe/ConsolePrefDxe.c | 1
> +
> EmbeddedPkg/Drivers/Lan9118Dxe/Lan9118Dxe.c | 1 +
> EmbeddedPkg/Drivers/SataSiI3132Dxe/SataSiI3132.h | 32
> ++++++--
> EmbeddedPkg/Drivers/SataSiI3132Dxe/SiI3132AtaPassThru.c | 8
> ++
> EmbeddedPkg/GdbStub/GdbStubInternal.h | 9 +++
> EmbeddedPkg/MetronomeDxe/Metronome.c | 1 +
> EmbeddedPkg/Universal/MmcDxe/Mmc.c | 5 +-
> EmulatorPkg/EmuGopDxe/GopInput.c | 4 +-
> EmulatorPkg/Library/DxeTimerLib/DxeTimerLib.c | 2 +-
> MdeModulePkg/Bus/I2c/I2cDxe/I2cBus.c | 2 +-
> MdeModulePkg/Bus/Pci/NvmExpressDxe/NvmExpress.c | 2
> +-
> MdeModulePkg/Bus/Pci/PciBusDxe/PciDeviceSupport.c | 6
> +-
> MdeModulePkg/Bus/Pci/PciSioSerialDxe/Serial.c | 2 +-
> MdeModulePkg/Bus/Sd/EmmcDxe/EmmcDxe.c | 2 +-
> MdeModulePkg/Bus/Usb/UsbBusDxe/UsbEnumer.c | 2 +-
> MdeModulePkg/Bus/Usb/UsbMassStorageDxe/UsbMassImpl.c
> | 2 +-
> MdeModulePkg/Core/Dxe/Event/Event.c | 8 ++
> MdeModulePkg/Core/Dxe/Event/Event.h | 2 +-
> MdeModulePkg/Core/Dxe/Hand/Handle.h | 2 +-
> MdeModulePkg/Core/Pei/FwVol/FwVol.c | 2 +-
> MdeModulePkg/Core/Pei/FwVol/FwVol.h | 2 +-
> MdeModulePkg/Core/PiSmmCore/PiSmmCore.h | 2 +-
> MdeModulePkg/Core/PiSmmCore/Smi.c | 8 +-
> MdeModulePkg/Core/RuntimeDxe/Runtime.c | 10 ++-
>
> MdeModulePkg/Library/DxeCorePerformanceLib/DxeCorePerformanceLib.c
> | 12 +--
> MdeModulePkg/Library/PlatformVarCleanupLib/PlatVarCleanupLib.c
> | 6 +-
>
> MdeModulePkg/Library/SmmCorePerformanceLib/SmmCorePerformanceLi
> b.c | 8 +-
> MdeModulePkg/Library/UefiHiiLib/HiiString.c | 4 +-
> MdeModulePkg/Universal/Acpi/AcpiTableDxe/AcpiSdt.h | 2
> +-
> MdeModulePkg/Universal/Acpi/S3SaveStateDxe/S3SaveState.c |
> 4 +-
> MdeModulePkg/Universal/Acpi/SmmS3SaveState/SmmS3SaveState.c
> | 4 +-
> MdeModulePkg/Universal/Console/ConSplitterDxe/ConSplitter.c
> | 2 +-
> MdeModulePkg/Universal/DriverSampleDxe/DriverSample.c |
> 4 +-
>
> MdeModulePkg/Universal/FaultTolerantWriteDxe/FaultTolerantWriteDxe.c
> | 2 +-
>
> MdeModulePkg/Universal/FaultTolerantWriteDxe/FaultTolerantWriteSmm.
> c | 2 +-
> MdeModulePkg/Universal/HiiDatabaseDxe/HiiDatabase.h | 2
> +-
> MdeModulePkg/Universal/LoadFileOnFv2/LoadFileOnFv2.c | 2
> +-
> MdeModulePkg/Universal/Variable/RuntimeDxe/VariableDxe.c
> | 2 +-
> MdePkg/Include/Pi/PiPeiCis.h | 6 +-
> MdePkg/Include/Protocol/AcpiSystemDescriptionTable.h | 3
> +-
> MdePkg/Include/Protocol/Bis.h | 3 +-
> MdePkg/Include/Protocol/Eap.h | 3 +-
> MdePkg/Include/Protocol/HiiFont.h | 3 +-
> MdePkg/Include/Protocol/MmMp.h | 3 +-
> MdePkg/Include/Protocol/S3SaveState.h | 2 +-
> MdePkg/Include/Protocol/Shell.h | 3 +-
> MdePkg/Include/Protocol/UserManager.h | 9 ++-
> MdePkg/Include/Uefi/UefiBaseType.h | 6 +-
> MdePkg/Include/Uefi/UefiInternalFormRepresentation.h | 3
> +-
> MdePkg/Library/DxeServicesLib/DxeServicesLib.c | 2 +-
> NetworkPkg/DnsDxe/DnsDriver.c | 4 +-
> NetworkPkg/IScsiDxe/IScsiConfig.c | 2 +-
> NetworkPkg/Ip4Dxe/Ip4Driver.c | 2 +-
> NetworkPkg/Ip4Dxe/Ip4If.c | 4 +-
> NetworkPkg/Ip6Dxe/Ip6Driver.c | 2 +-
> NetworkPkg/Library/DxeNetLib/DxeNetLib.c | 2 +-
> NetworkPkg/Mtftp4Dxe/Mtftp4Driver.c | 2 +-
> NetworkPkg/TcpDxe/SockImpl.c | 4 +-
> NetworkPkg/TlsAuthConfigDxe/TlsAuthConfigDxe.c | 2 +-
> OvmfPkg/Csm/BiosThunk/VideoDxe/BiosVideo.c | 6 +-
> OvmfPkg/PlatformDxe/Platform.c | 4 +-
> OvmfPkg/VirtioNetDxe/Events.c | 2 +-
> OvmfPkg/XenBusDxe/XenBus.c | 2 +-
> SecurityPkg/HddPassword/HddPasswordDxe.c | 2 +-
> SecurityPkg/Tcg/Tcg2Config/Tcg2ConfigDriver.c | 2 +-
> SecurityPkg/Tcg/TcgConfigDxe/TcgConfigDriver.c | 2 +-
>
> SecurityPkg/VariableAuthenticated/SecureBootConfigDxe/SecureBootConfi
> gDriver.c | 2 +-
> ShellPkg/DynamicCommand/DpDynamicCommand/Dp.c |
> 6 +-
> ShellPkg/DynamicCommand/DpDynamicCommand/Dp.h |
> 4 +-
> ShellPkg/DynamicCommand/TftpDynamicCommand/Tftp.c |
> 6 +-
> ShellPkg/DynamicCommand/TftpDynamicCommand/Tftp.h |
> 4 +-
> ShellPkg/Include/Library/ShellCommandLib.h | 2 +-
> ShellPkg/Include/Library/ShellLib.h | 4 +-
> ShellPkg/Library/UefiHandleParsingLib/UefiHandleParsingLib.c | 2
> +-
> ShellPkg/Library/UefiShellBcfgCommandLib/UefiShellBcfgCommandLib.c
> | 2 +-
> ShellPkg/Library/UefiShellCommandLib/UefiShellCommandLib.c
> | 2 +-
> ShellPkg/Library/UefiShellCommandLib/UefiShellCommandLib.h
> | 2 +-
> ShellPkg/Library/UefiShellDebug1CommandsLib/Edit/FileBuffer.c |
> 2 +-
>
> ShellPkg/Library/UefiShellDebug1CommandsLib/UefiShellDebug1Comman
> dsLib.c | 2 +-
>
> ShellPkg/Library/UefiShellDebug1CommandsLib/UefiShellDebug1Comman
> dsLib.h | 2 +-
> ShellPkg/Library/UefiShellDriver1CommandsLib/Connect.c | 4
> +-
>
> ShellPkg/Library/UefiShellDriver1CommandsLib/UefiShellDriver1Command
> sLib.c | 2 +-
>
> ShellPkg/Library/UefiShellDriver1CommandsLib/UefiShellDriver1Command
> sLib.h | 2 +-
>
> ShellPkg/Library/UefiShellLevel1CommandsLib/UefiShellLevel1CommandsL
> ib.c | 2 +-
>
> ShellPkg/Library/UefiShellLevel1CommandsLib/UefiShellLevel1CommandsL
> ib.h | 2 +-
> ShellPkg/Library/UefiShellLevel2CommandsLib/Rm.c | 2 +-
>
> ShellPkg/Library/UefiShellLevel2CommandsLib/UefiShellLevel2CommandsL
> ib.c | 2 +-
>
> ShellPkg/Library/UefiShellLevel2CommandsLib/UefiShellLevel2CommandsL
> ib.h | 2 +-
> ShellPkg/Library/UefiShellLevel3CommandsLib/Touch.c | 2 +-
>
> ShellPkg/Library/UefiShellLevel3CommandsLib/UefiShellLevel3CommandsL
> ib.c | 2 +-
>
> ShellPkg/Library/UefiShellLevel3CommandsLib/UefiShellLevel3CommandsL
> ib.h | 2 +-
> ShellPkg/Library/UefiShellLib/UefiShellLib.c | 26 ++++++-
>
> ShellPkg/Library/UefiShellNetwork1CommandsLib/UefiShellNetwork1Com
> mandsLib.c | 2 +-
>
> ShellPkg/Library/UefiShellNetwork1CommandsLib/UefiShellNetwork1Com
> mandsLib.h | 2 +-
>
> ShellPkg/Library/UefiShellNetwork2CommandsLib/UefiShellNetwork2Com
> mandsLib.c | 2 +-
>
> ShellPkg/Library/UefiShellNetwork2CommandsLib/UefiShellNetwork2Com
> mandsLib.h | 2 +-
> StandaloneMmPkg/Core/Dispatcher.c | 80
> +++++++++++---------
> StandaloneMmPkg/Core/FwVol.c | 16 ++--
> StandaloneMmPkg/Core/StandaloneMmCore.h | 4 +-
> UefiPayloadPkg/BlSupportDxe/BlSupportDxe.c | 4 +-
> UefiPayloadPkg/BlSupportPei/BlSupportPei.c | 19 +++--
> 102 files changed, 294 insertions(+), 194 deletions(-)
>
> --
> 2.19.1.3.g30247aa5d201
>
>
> -=-=-=-=-=-=
> Groups.io Links: You receive all messages sent to this group.
>
> View/Reply Online (#47387):
> https://edk2.groups.io/g/devel/message/47387
> Mute This Topic: https://groups.io/mt/34180197/1768737
> Group Owner: devel+owner@edk2.groups.io
> Unsubscribe: https://edk2.groups.io/g/devel/unsub [hao.a.wu@intel.com]
> -=-=-=-=-=-=
next prev parent reply other threads:[~2019-09-19 0:32 UTC|newest]
Thread overview: 155+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-09-17 19:49 [PATCH 00/35] edk2: clean up the usage of standardized (VOID*) typedefs Laszlo Ersek
2019-09-17 19:49 ` [PATCH 01/35] DO NOT APPLY: edk2: turn standard handle types into pointers to non-VOID Laszlo Ersek
2019-09-17 20:06 ` [edk2-devel] " Ni, Ray
2019-09-17 20:22 ` Andrew Fish
2019-09-17 20:28 ` Ni, Ray
2019-09-17 21:07 ` Andrew Fish
2019-09-17 21:11 ` Michael D Kinney
2019-09-18 8:41 ` Laszlo Ersek
2019-09-18 15:16 ` Michael D Kinney
2019-09-18 17:41 ` Laszlo Ersek
2019-09-18 15:55 ` Andrew Fish
2019-09-18 16:16 ` Leif Lindholm
2019-09-18 17:45 ` Laszlo Ersek
2019-09-18 8:36 ` Laszlo Ersek
2019-09-17 19:49 ` [PATCH 02/35] EmbeddedPkg: add missing EFIAPI calling convention specifiers Laszlo Ersek
2019-09-18 17:41 ` Leif Lindholm
2019-09-17 19:49 ` [PATCH 03/35] EmbeddedPkg/AndroidFastbootTransportTcpDxe: fix DestroyChild() call Laszlo Ersek
2019-09-24 10:47 ` [edk2-devel] " Philippe Mathieu-Daudé
2019-09-26 12:12 ` Laszlo Ersek
2019-09-26 12:16 ` Ard Biesheuvel
2019-09-17 19:49 ` [PATCH 04/35] EmbeddedPkg/Universal/MmcDxe: "fix" CloseProtocol() call in BindingStop() Laszlo Ersek
2019-09-25 15:52 ` Ard Biesheuvel
2019-09-17 19:49 ` [PATCH 05/35] EmulatorPkg/DxeTimerLib: drop superfluous cast Laszlo Ersek
2019-09-17 20:02 ` Ni, Ray
2019-09-20 15:00 ` [edk2-devel] " Philippe Mathieu-Daudé
2019-09-17 19:49 ` [PATCH 06/35] EmulatorPkg: stop abusing EFI_HANDLE for keystroke notify registration Laszlo Ersek
2019-09-17 20:01 ` Ni, Ray
2019-09-24 10:44 ` [edk2-devel] " Philippe Mathieu-Daudé
2019-09-17 19:49 ` [PATCH 07/35] MdeModulePkg: fix cast in GetModuleInfoFromHandle() calls Laszlo Ersek
2019-09-19 1:46 ` Dandan Bi
2019-09-24 10:43 ` [edk2-devel] " Philippe Mathieu-Daudé
2019-09-17 19:49 ` [PATCH 08/35] MdeModulePkg/UefiHiiLib: stop using EFI_HANDLE in place of EFI_HII_HANDLE Laszlo Ersek
2019-09-19 1:46 ` Dandan Bi
2019-09-24 10:49 ` [edk2-devel] " Philippe Mathieu-Daudé
2019-09-17 19:49 ` [PATCH 09/35] MdeModulePkg: stop abusing EFI_EVENT for protocol notify registration Laszlo Ersek
2019-09-17 20:17 ` Ni, Ray
2019-09-25 16:02 ` [edk2-devel] " Philippe Mathieu-Daudé
2019-09-26 12:10 ` Laszlo Ersek
2019-09-17 19:49 ` [PATCH 10/35] MdeModulePkg/PlatformVarCleanupLib: fix HiiConstructConfigHdr() call Laszlo Ersek
2019-09-23 11:45 ` [edk2-devel] " Philippe Mathieu-Daudé
2019-09-24 17:28 ` Laszlo Ersek
2019-09-17 19:49 ` [PATCH 11/35] MdeModulePkg: document workaround for EFI_RUNTIME_EVENT_ENTRY PI spec bug Laszlo Ersek
2019-09-19 1:47 ` Dandan Bi
2019-09-17 19:49 ` [PATCH 12/35] MdeModulePkg: stop abusing EFI_HANDLE for keystroke notify registration Laszlo Ersek
2019-09-17 20:16 ` [edk2-devel] " Ni, Ray
2019-09-19 1:47 ` Dandan Bi
2019-09-24 10:54 ` [edk2-devel] " Philippe Mathieu-Daudé
2019-09-17 19:49 ` [PATCH 13/35] MdeModulePkg: PEI Core: clean up "AprioriFile" handling in FindFileEx() Laszlo Ersek
2019-09-19 1:46 ` Dandan Bi
2019-09-24 15:40 ` [edk2-devel] " Philippe Mathieu-Daudé
2019-09-17 19:49 ` [PATCH 14/35] MdeModulePkg: fix UninstallMultipleProtocolInterfaces() calls Laszlo Ersek
2019-09-17 20:16 ` Ni, Ray
2019-09-17 19:49 ` [PATCH 15/35] MdeModulePkg/PiSmmCore: make type punning consistent Laszlo Ersek
2019-09-18 2:38 ` Dong, Eric
2019-09-17 19:49 ` [PATCH 16/35] MdeModulePkg/S3SaveState: cast Position for S3BootScriptLib explicitly Laszlo Ersek
2019-09-19 1:47 ` [edk2-devel] " Dandan Bi
2019-09-17 19:49 ` [PATCH 17/35] MdePkg/DxeServicesLib: remove bogus cast Laszlo Ersek
2019-09-18 4:47 ` [edk2-devel] " Liming Gao
2019-09-24 15:38 ` Philippe Mathieu-Daudé
2019-09-17 19:49 ` [PATCH 18/35] NetworkPkg/DxeNetLib: fix type typo in NetLibGetMacAddress() Laszlo Ersek
2019-09-24 11:00 ` [edk2-devel] " Philippe Mathieu-Daudé
2019-10-08 0:32 ` Siyuan, Fu
2019-10-08 23:36 ` Laszlo Ersek
2019-09-26 12:14 ` Laszlo Ersek
2019-10-03 11:05 ` Laszlo Ersek
2019-10-04 19:18 ` Laszlo Ersek
2019-10-07 18:15 ` Michael D Kinney
2019-09-17 19:49 ` [PATCH 19/35] NetworkPkg: fix CloseProtocol & UninstallMultipleProtocolInterfaces calls Laszlo Ersek
2019-09-26 12:14 ` [edk2-devel] " Laszlo Ersek
2019-09-26 12:42 ` Philippe Mathieu-Daudé
2019-09-30 20:16 ` Laszlo Ersek
2019-10-01 7:18 ` Philippe Mathieu-Daudé
2019-09-27 0:03 ` Siyuan, Fu
2019-09-17 19:49 ` [PATCH 20/35] NetworkPkg/Ip4Dxe: fix NetLibDestroyServiceChild() call Laszlo Ersek
2019-09-23 16:03 ` [edk2-devel] " Philippe Mathieu-Daudé
2019-09-27 0:04 ` Siyuan, Fu
2019-09-26 12:14 ` Laszlo Ersek
2019-09-17 19:49 ` [PATCH 21/35] NetworkPkg/TcpDxe: fix SockFreeFoo() parameter list Laszlo Ersek
2019-09-23 15:09 ` [edk2-devel] " Philippe Mathieu-Daudé
2019-09-26 12:14 ` Laszlo Ersek
2019-09-27 0:06 ` Siyuan, Fu
2019-09-17 19:49 ` [PATCH 22/35] OvmfPkg/XenBusDxe: fix UninstallMultipleProtocolInterfaces() call Laszlo Ersek
2019-09-18 9:32 ` Anthony PERARD
2019-09-18 10:30 ` Julien Grall
2019-09-23 15:03 ` [edk2-devel] " Philippe Mathieu-Daudé
2019-09-17 19:49 ` [PATCH 23/35] OvmfPkg/VirtioNetDxe: fix SignalEvent() call Laszlo Ersek
2019-09-20 14:59 ` [edk2-devel] " Philippe Mathieu-Daudé
2019-09-26 12:16 ` Laszlo Ersek
2019-09-26 12:17 ` Ard Biesheuvel
2019-09-17 19:49 ` [PATCH 24/35] OvmfPkg/PlatformDxe: fix EFI_HII_HANDLE parameters of internal functions Laszlo Ersek
2019-09-20 14:56 ` [edk2-devel] " Philippe Mathieu-Daudé
2019-09-25 15:57 ` Ard Biesheuvel
2019-09-17 19:49 ` [PATCH 25/35] OvmfPkg/VideoDxe: document EFI_EDID_OVERRIDE_PROTOCOL.GetEdid() call Laszlo Ersek
2019-09-23 15:59 ` [edk2-devel] " Philippe Mathieu-Daudé
2019-09-26 12:43 ` Laszlo Ersek
2019-10-04 20:01 ` Laszlo Ersek
2019-09-17 19:49 ` [PATCH 26/35] SecurityPkg: fix UninstallMultipleProtocolInterfaces() calls Laszlo Ersek
2019-09-23 9:55 ` [edk2-devel] " Philippe Mathieu-Daudé
2019-09-26 12:45 ` Laszlo Ersek
2019-10-03 11:06 ` Laszlo Ersek
2019-10-04 0:05 ` Yao, Jiewen
2019-10-04 13:14 ` Zhang, Chao B
2019-10-04 18:15 ` Laszlo Ersek
2019-10-05 14:28 ` Zhang, Chao B
2019-10-07 18:14 ` Laszlo Ersek
2019-09-17 19:49 ` [PATCH 27/35] SecurityPkg: stop abusing EFI_EVENT for protocol notify registration Laszlo Ersek
2019-09-23 9:56 ` [edk2-devel] " Philippe Mathieu-Daudé
2019-09-26 12:46 ` Laszlo Ersek
2019-10-03 11:06 ` Laszlo Ersek
2019-10-04 0:05 ` Yao, Jiewen
2019-10-04 13:16 ` Zhang, Chao B
2019-10-05 14:28 ` Zhang, Chao B
2019-09-17 19:49 ` [PATCH 28/35] ShellPkg/UefiShellDriver1CommandsLib: fix parameter list typo Laszlo Ersek
2019-09-24 15:44 ` [edk2-devel] " Philippe Mathieu-Daudé
2019-09-26 2:46 ` Gao, Zhichao
2019-09-17 19:49 ` [PATCH 29/35] ShellPkg: stop using EFI_HANDLE in place of EFI_HII_HANDLE Laszlo Ersek
2019-09-25 18:04 ` [edk2-devel] " Philippe Mathieu-Daudé
2019-09-26 11:48 ` Laszlo Ersek
2019-09-26 2:43 ` Gao, Zhichao
2019-09-17 19:49 ` [PATCH 30/35] ShellPkg: stop taking EFI_HANDLE in place of SHELL_FILE_HANDLE Laszlo Ersek
2019-09-23 9:58 ` [edk2-devel] " Philippe Mathieu-Daudé
2019-09-26 2:53 ` Gao, Zhichao
2019-09-26 12:08 ` Laszlo Ersek
2019-09-26 14:43 ` Gao, Zhichao
2019-09-30 19:52 ` Laszlo Ersek
2019-09-17 19:49 ` [PATCH 31/35] ShellPkg/UefiShellDebug1CommandsLib: fix ShellCloseFile() call Laszlo Ersek
2019-09-23 10:01 ` [edk2-devel] " Philippe Mathieu-Daudé
2019-09-23 14:28 ` Carsey, Jaben
2019-09-24 1:18 ` Gao, Zhichao
2019-09-17 19:49 ` [PATCH 32/35] ShellPkg/UefiShellLib: clarify workaround for unfixable EdkShell bug Laszlo Ersek
2019-09-26 12:47 ` [edk2-devel] " Laszlo Ersek
2019-09-26 14:05 ` Gao, Zhichao
2019-09-26 14:58 ` Carsey, Jaben
2019-09-17 19:49 ` [PATCH 33/35] StandaloneMmPkg/Core: stop abusing EFI_HANDLE for FwVolHeader tracking Laszlo Ersek
2019-09-26 12:48 ` [edk2-devel] " Laszlo Ersek
2019-10-03 11:10 ` Laszlo Ersek
2019-10-03 11:17 ` Achin Gupta
2019-10-04 0:10 ` Yao, Jiewen
2019-10-04 14:53 ` Achin Gupta
2019-09-17 19:49 ` [PATCH 34/35] UefiPayloadPkg/BlSupportPei: fix MMCONFIG assignment from XSDT Laszlo Ersek
2019-09-23 2:30 ` Guo Dong
2019-09-26 13:17 ` [edk2-devel] " Philippe Mathieu-Daudé
2019-09-17 19:49 ` [PATCH 35/35] UefiPayloadPkg/BlSupportDxe: fix ReserveResourceInGcd() calls Laszlo Ersek
2019-09-23 2:28 ` [edk2-devel] " Guo Dong
2019-09-23 15:08 ` Philippe Mathieu-Daudé
2019-09-23 16:02 ` Guo Dong
2019-09-23 16:04 ` Philippe Mathieu-Daudé
2019-09-24 17:29 ` Laszlo Ersek
2019-09-19 0:32 ` Wu, Hao A [this message]
2019-09-23 16:27 ` [edk2-devel] [PATCH 00/35] edk2: clean up the usage of standardized (VOID*) typedefs Marvin Häuser
2019-09-24 20:26 ` Laszlo Ersek
2019-09-25 8:13 ` Marvin Häuser
2019-09-25 15:54 ` Laszlo Ersek
2019-10-08 23:49 ` Laszlo Ersek
2019-10-09 9:50 ` 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=B80AF82E9BFB8E4FBD8C89DA810C6A093C93392D@SHSMSX104.ccr.corp.intel.com \
--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