[edk2-devel] [PATCH V3 0/4] Reserve shared memory for DMA operation

[Yao, Jiewen]

Merged https://github.com/tianocore/edk2/pull/3784

> -----Original Message-----
> From: Tom Lendacky <thomas.lendacky at amd.com>
> Sent: Saturday, December 17, 2022 10:18 PM
> To: Yao, Jiewen <jiewen.yao at intel.com>; Xu, Min M <min.m.xu at intel.com>;
> devel at edk2.groups.io
> Cc: Aktas, Erdem <erdemaktas at google.com>; James Bottomley
> <jejb at linux.ibm.com>; Gerd Hoffmann <kraxel at redhat.com>; Kinney,
> Michael D <michael.d.kinney at intel.com>; Gao, Liming
> <gaoliming at byosoft.com.cn>
> Subject: Re: [PATCH V3 0/4] Reserve shared memory for DMA operation
> 
> On 12/16/22 21:11, Yao, Jiewen wrote:
> > Hi Tom
> > Any further discussion?
> >
> > I would like to merge if no other feedback.
> 
> No other feedback from me. For the series:
> 
> Reviewed-by: Tom Lendacky <thomas.lendacky at amd.com>
> 
> >
> >> -----Original Message-----
> >> From: Xu, Min M <min.m.xu at intel.com>
> >> Sent: Thursday, December 15, 2022 1:18 PM
> >> To: devel at edk2.groups.io
> >> Cc: Xu, Min M <min.m.xu at intel.com>; Aktas, Erdem
> >> <erdemaktas at google.com>; James Bottomley <jejb at linux.ibm.com>; Yao,
> >> Jiewen <jiewen.yao at intel.com>; Gerd Hoffmann <kraxel at redhat.com>;
> >> Kinney, Michael D <michael.d.kinney at intel.com>; Gao, Liming
> >> <gaoliming at byosoft.com.cn>; Tom Lendacky
> <thomas.lendacky at amd.com>
> >> Subject: [PATCH V3 0/4] Reserve shared memory for DMA operation
> >>
> >> BZ: https://bugzilla.tianocore.org/show_bug.cgi?id=4171
> >>
> >> This patch-set introduces the feature of reserving shared memory
> >> for DMA operation. Its intention is to reduce the allocation and
> >> conversion of private/shared memory, so that boot performance
> >> can be improved significantly. Detailed information is in Patch#1.
> >>
> >> Patch#2 renames AmdSevIoMmu.* to CcIoMmu.* because these 2 files
> >> support both SEV and TDX guest.
> >>
> >> Patch#3 is provided by Tom Lendacky which add SEV support for reserved
> >> shared memory.
> >>
> >> Patch#4 updates the related section in Maintainers.txt.
> >>
> >> Code: https://github.com/mxu9/edk2/tree/IoMmu.v3
> >>
> >> v3 changes:
> >>   - Check mReservedSharedMemSupported in
> >> IoMmuReleaseReservedSharedMem
> >>     before release the pre-allocated shared memory.
> >>   - Delete a line of comment in Patch#3 which is suggested by Tom
> Lendacky.
> >>
> >> v2 changes:
> >>   - Add Patch#3 which is provided by Tom Lendacky. It adds SEV support
> for
> >>     reserved shared memory.
> >>   - Add more description for mReservedMemRanges. It describes:
> >>     1) How the pre-allocated memory is managed.
> >>     2) What if the pre-allocated memory is used up.
> >>
> >> Cc: Erdem Aktas <erdemaktas at google.com>
> >> Cc: James Bottomley <jejb at linux.ibm.com>
> >> Cc: Jiewen Yao <jiewen.yao at intel.com>
> >> Cc: Gerd Hoffmann <kraxel at redhat.com>
> >> Cc: Michael D Kinney <michael.d.kinney at intel.com>
> >> Cc: Liming Gao <gaoliming at byosoft.com.cn>
> >> Signed-off-by: Tom Lendacky <thomas.lendacky at amd.com>
> >> Signed-off-by: Min Xu <min.m.xu at intel.com>
> >>
> >> Min M Xu (3):
> >>    OvmfPkg/IoMmuDxe: Reserve shared memory region for DMA
> operation
> >>    OvmfPkg/IoMmuDxe: Rename AmdSevIoMmu to CcIoMmu
> >>    Maintainers: Update OvmfPkg/IoMmuDxe
> >>
> >> Tom Lendacky (1):
> >>    OvmfPkg/IoMmuDxe: Add SEV support for reserved shared memory
> >>
> >>   Maintainers.txt                               |   2 +-
> >>   OvmfPkg/IoMmuDxe/{AmdSevIoMmu.c => CcIoMmu.c} | 204 ++++----
> >>   OvmfPkg/IoMmuDxe/{AmdSevIoMmu.h => CcIoMmu.h} |   0
> >>   OvmfPkg/IoMmuDxe/IoMmuBuffer.c                | 495
> ++++++++++++++++++
> >>   OvmfPkg/IoMmuDxe/IoMmuDxe.c                   |   2 +-
> >>   OvmfPkg/IoMmuDxe/IoMmuDxe.inf                 |   6 +-
> >>   OvmfPkg/IoMmuDxe/IoMmuInternal.h              | 179 +++++++
> >>   7 files changed, 781 insertions(+), 107 deletions(-)
> >>   rename OvmfPkg/IoMmuDxe/{AmdSevIoMmu.c => CcIoMmu.c} (85%)
> >>   rename OvmfPkg/IoMmuDxe/{AmdSevIoMmu.h => CcIoMmu.h} (100%)
> >>   create mode 100644 OvmfPkg/IoMmuDxe/IoMmuBuffer.c
> >>   create mode 100644 OvmfPkg/IoMmuDxe/IoMmuInternal.h
> >>
> >> --
> >> 2.29.2.windows.2
> >


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#97525): https://edk2.groups.io/g/devel/message/97525
Mute This Topic: https://groups.io/mt/95683528/1813853
Group Owner: devel+owner at edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [edk2-devel-archive at redhat.com]
-=-=-=-=-=-=-=-=-=-=-=-