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.lenda...@amd.com>
-----Original Message-----
From: Xu, Min M <min.m...@intel.com>
Sent: Thursday, December 15, 2022 1:18 PM
To: devel@edk2.groups.io
Cc: Xu, Min M <min.m...@intel.com>; Aktas, Erdem
<erdemak...@google.com>; James Bottomley <j...@linux.ibm.com>; Yao,
Jiewen <jiewen....@intel.com>; Gerd Hoffmann <kra...@redhat.com>;
Kinney, Michael D <michael.d.kin...@intel.com>; Gao, Liming
<gaolim...@byosoft.com.cn>; Tom Lendacky <thomas.lenda...@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 <erdemak...@google.com>
Cc: James Bottomley <j...@linux.ibm.com>
Cc: Jiewen Yao <jiewen....@intel.com>
Cc: Gerd Hoffmann <kra...@redhat.com>
Cc: Michael D Kinney <michael.d.kin...@intel.com>
Cc: Liming Gao <gaolim...@byosoft.com.cn>
Signed-off-by: Tom Lendacky <thomas.lenda...@amd.com>
Signed-off-by: Min Xu <min.m...@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 (#97524): https://edk2.groups.io/g/devel/message/97524
Mute This Topic: https://groups.io/mt/95683528/21656
Group Owner: devel+ow...@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-
