From mboxrd@z Thu Jan 1 00:00:00 1970 Authentication-Results: mx.groups.io; dkim=missing; spf=pass (domain: redhat.com, ip: 209.132.183.28, mailfrom: lersek@redhat.com) Received: from mx1.redhat.com (mx1.redhat.com [209.132.183.28]) by groups.io with SMTP; Fri, 05 Jul 2019 07:24:42 -0700 Received: from smtp.corp.redhat.com (int-mx04.intmail.prod.int.phx2.redhat.com [10.5.11.14]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mx1.redhat.com (Postfix) with ESMTPS id 5EDF53086204; Fri, 5 Jul 2019 14:24:42 +0000 (UTC) Received: from lacos-laptop-7.usersys.redhat.com (ovpn-116-244.ams2.redhat.com [10.36.116.244]) by smtp.corp.redhat.com (Postfix) with ESMTP id D16E6968C9; Fri, 5 Jul 2019 14:24:40 +0000 (UTC) Subject: Re: [PATCH v3 07/35] OvmfPkg/XenResetVector: Saving start of day pointer for PVH guests To: Anthony PERARD , devel@edk2.groups.io Cc: xen-devel@lists.xenproject.org, Ard Biesheuvel , Jordan Justen , Julien Grall References: <20190704144233.27968-1-anthony.perard@citrix.com> <20190704144233.27968-8-anthony.perard@citrix.com> From: "Laszlo Ersek" Message-ID: <20f29521-4ad5-fcb4-65dd-c6ee975cd6b8@redhat.com> Date: Fri, 5 Jul 2019 16:24:34 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.9.1 MIME-Version: 1.0 In-Reply-To: <20190704144233.27968-8-anthony.perard@citrix.com> X-Scanned-By: MIMEDefang 2.79 on 10.5.11.14 X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.5.16 (mx1.redhat.com [10.5.110.42]); Fri, 05 Jul 2019 14:24:42 +0000 (UTC) Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 7bit On 07/04/19 16:42, Anthony PERARD wrote: > As described in the Xen PVH documentation [1], "ebx: contains the > physical memory address where the loader has placed the boot start info > structure". To have this pointer saved to be able to use it later in the > PEI phase, we allocate some space in the MEMFD for it. We use 'XPVH' as > a signature (for "Xen PVH"). > > [1] https://xenbits.xenproject.org/docs/unstable/misc/pvh.html > > Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=1689 > Signed-off-by: Anthony PERARD > Acked-by: Laszlo Ersek > --- > > Notes: > v3: > - rename PcdXenStartOfDay* to PcdXenPvhStartOfDay* > - Use first available token value for those (0x17 and 0x28) You ended up using 0x32 rather than 0x28, but that was necessary (and correct) due to other patches meanwhile. Also there's been a minimal comment style fix in "XenPVHMain.asm". My Acked-by stands. Thanks! Laszlo > > OvmfPkg/OvmfPkg.dec | 3 +++ > OvmfPkg/OvmfXen.fdf | 4 ++++ > OvmfPkg/XenResetVector/XenResetVector.inf | 3 +++ > OvmfPkg/XenResetVector/Ia32/XenPVHMain.asm | 6 ++++++ > OvmfPkg/XenResetVector/XenResetVector.nasmb | 2 ++ > 5 files changed, 18 insertions(+) > > diff --git a/OvmfPkg/OvmfPkg.dec b/OvmfPkg/OvmfPkg.dec > index 9640360f62..c2a2ebfb95 100644 > --- a/OvmfPkg/OvmfPkg.dec > +++ b/OvmfPkg/OvmfPkg.dec > @@ -218,6 +218,9 @@ [PcdsFixedAtBuild] > # The value should be a multiple of 4KB. > gUefiOvmfPkgTokenSpaceGuid.PcdHighPmmMemorySize|0x400000|UINT32|0x31 > > + gUefiOvmfPkgTokenSpaceGuid.PcdXenPvhStartOfDayStructPtr|0x0|UINT32|0x17 > + gUefiOvmfPkgTokenSpaceGuid.PcdXenPvhStartOfDayStructPtrSize|0x0|UINT32|0x32 > + > [PcdsDynamic, PcdsDynamicEx] > gUefiOvmfPkgTokenSpaceGuid.PcdEmuVariableEvent|0|UINT64|2 > gUefiOvmfPkgTokenSpaceGuid.PcdOvmfFlashVariablesEnable|FALSE|BOOLEAN|0x10 > diff --git a/OvmfPkg/OvmfXen.fdf b/OvmfPkg/OvmfXen.fdf > index 43c268f6cb..49997fee9b 100644 > --- a/OvmfPkg/OvmfXen.fdf > +++ b/OvmfPkg/OvmfXen.fdf > @@ -172,6 +172,10 @@ [FD.MEMFD] > 0x007000|0x001000 > gEfiMdePkgTokenSpaceGuid.PcdGuidedExtractHandlerTableAddress|gUefiOvmfPkgTokenSpaceGuid.PcdGuidedExtractHandlerTableSize > > +0x008000|0x001000 > +# Used by XenResetVector to communicate with XenPlatformPei > +gUefiOvmfPkgTokenSpaceGuid.PcdXenPvhStartOfDayStructPtr|gUefiOvmfPkgTokenSpaceGuid.PcdXenPvhStartOfDayStructPtrSize > + > 0x010000|0x010000 > gUefiOvmfPkgTokenSpaceGuid.PcdOvmfSecPeiTempRamBase|gUefiOvmfPkgTokenSpaceGuid.PcdOvmfSecPeiTempRamSize > > diff --git a/OvmfPkg/XenResetVector/XenResetVector.inf b/OvmfPkg/XenResetVector/XenResetVector.inf > index 097fc9b5b4..46b133a834 100644 > --- a/OvmfPkg/XenResetVector/XenResetVector.inf > +++ b/OvmfPkg/XenResetVector/XenResetVector.inf > @@ -36,3 +36,6 @@ [BuildOptions] > [Pcd] > gUefiOvmfPkgTokenSpaceGuid.PcdOvmfSecPageTablesBase > gUefiOvmfPkgTokenSpaceGuid.PcdOvmfSecPageTablesSize > + > + gUefiOvmfPkgTokenSpaceGuid.PcdXenPvhStartOfDayStructPtr > + gUefiOvmfPkgTokenSpaceGuid.PcdXenPvhStartOfDayStructPtrSize > diff --git a/OvmfPkg/XenResetVector/Ia32/XenPVHMain.asm b/OvmfPkg/XenResetVector/Ia32/XenPVHMain.asm > index 2a17fed52f..f42df3dba2 100644 > --- a/OvmfPkg/XenResetVector/Ia32/XenPVHMain.asm > +++ b/OvmfPkg/XenResetVector/Ia32/XenPVHMain.asm > @@ -22,6 +22,12 @@ xenPVHMain: > ; > xor esp, esp > > + ; > + ; Store "Start of day" struct pointer for later use > + ; > + mov dword[PVH_SPACE (0)], ebx > + mov dword[PVH_SPACE (4)], 'XPVH' > + > mov ebx, ADDR_OF(gdtr) > lgdt [ebx] > > diff --git a/OvmfPkg/XenResetVector/XenResetVector.nasmb b/OvmfPkg/XenResetVector/XenResetVector.nasmb > index 0dbc4f2c1d..b2cb405d54 100644 > --- a/OvmfPkg/XenResetVector/XenResetVector.nasmb > +++ b/OvmfPkg/XenResetVector/XenResetVector.nasmb > @@ -34,6 +34,8 @@ > > %include "CommonMacros.inc" > > +%define PVH_SPACE(Offset) (FixedPcdGet32 (PcdXenPvhStartOfDayStructPtr) + (Offset)) > + > %include "PostCodes.inc" > > %ifdef DEBUG_PORT80 >