On Thu, Nov 02, 2023 at 05:09:55PM +0800, sunceping wrote: > REF: https://bugzilla.tianocore.org/show_bug.cgi?id=4572 > > According to section 3.2 of the [GHCI] spec, if the result is > "TDG.VP.VMCALL_RETRY" for TDG.VP.VMCALL.MapGPA, TD must retry the > mapping for the pages in the region starting at the GPA specified in r11. > > Currently, TDVF does not properly handle the retry results of MapGPA. > For this, TDVF should update the TdVmCall to return the value in R11 > and must retry the mapping for the pages by the value. > > How to verify the retry for MapGPA in TDVF: > Note: Since the range size of MapGPA in QEMU is limited to 64MB and > TDVF always maps 1.5GB( 2GB~3.5GB) MMIO to shared-memory for TD guest, > the retry action is triggered always. > Pre-Config: > QEMU: > https://github.com/intel/qemu-tdx/tree/tdx-qemu-upstream | tag: > tdx-qemu-upstream-2023.10.20-v8.1.0 > KERNEL: > https://github.com/intel/tdx/tree/kvm-upstream-2023.10.16-v6.6-rc2 > > Step: > Boot with TD guest and check the log with TdVmcall(MAPGPA), as below: > TdxDxe:SetMemorySharedOrPrivate: Cr3Base=0x0 Physical=0x80000000 > Length=0x60000000 Mode=Shared > SetOrClearSharedBit: TdVmcall(MAPGPA) Retry PhysicalAddress is 8000080000000, > MapGpaRetryaddr is 8000084000000 > > Reference: > [GHCI]: TDX Guest-Host-Communication Interface v1.0 > https://cdrdv2.intel.com/v1/dl/getContent/726790 > > v2 changes: > - Update the code based on the comments of v1 reviewer > - Update TdVmcall to instead of the extra API file
Series: Acked-by: Gerd Hoffmann <kra...@redhat.com> -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#110825): https://edk2.groups.io/g/devel/message/110825 Mute This Topic: https://groups.io/mt/102337973/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
