[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH v2 00/31] Specific platform to run OVMF in Xen PVH and HVM guests



On 04/08/19 17:50, Laszlo Ersek wrote:
> On 04/08/19 16:23, Anthony PERARD wrote:
>> Patch series available in this git branch:
>> https://xenbits.xen.org/git-http/people/aperard/ovmf.git 
>> br.platform-xen-pvh-v2
>>
>> Hi,
>>
>> I've started to create a Xen specific platform, in OvmfPkg/XenOvmf.dsc
>> with the goal to make it work on both Xen HVM and Xen PVH.
> 
> The previous version of this feature dates back to Dec 2016 / Jan 2017;
> I haven't forgotten about it, and I'm happy that you are adding the
> feature as a separate platform DSC, as I requested.
> 
>> The first few patches only create the platform and duplicate some code
>> from OvmfPkg and the later patches makes OVMF boot in a Xen PVH guest
>> and can boot a Linux guest.
>>
>> After this patch series, I'd like to wait a bit before removing Xen
>> support from the OvmfPkg*.dsc, to allow time to switch to the new Xen
>> only platform, maybe 1 year.
> 
> I welcome this proposal very much. This will eliminate a significant
> amount of complexity in the current modules, allow for
> easier-to-understand maintenance assignments/responsibilities, and pave
> the way for features that are unique to either Xen or QEMU (in
> particular in the variable storage area, for which Xen has just received
> a dedicated userspace daemon IIRC, and for which -- on QEMU -- I had
> attempted to make pflash a conditional hard requirement, in the past,
> but failed due to OvmfPkg*.dsc targeting Xen too).
> 
>> Question:
>>
>> Should we start moving these to a different *Pkg? Like it's done for
>> ArmPkg and ArmVirtPkg?  Maybe XenPkg.
> 
> I'm pretty happy with the current package structure. ArmPkg is for both
> physical and virtual hardware. ArmVirtPkg is virt-only, and we already
> have separate platform DSCs/FDFs between Xen (ArmVirtXen) and QEMU/KVM
> (ArmVirtQemu[Kernel]). Xen- and QEMU/KVM-specific drivers and library
> instances peacefully coexist under ArmVirtPkg; the DSCs/FDFs include
> them as appropriate.
> 
> The same should map nicely to OvmfPkg. x86 stuff that targets both
> physical and virtual hardware belongs under PcAtChipsetPkg and
> UefiCpuPkg. Virt-only stuff should go under OvmfPkg; Xen-specific and
> QEMU/KVM-specific modules can coexist under OvmfPkg. It's sufficient if
> the platform DSCs cherry-pick them as appropriate.
> 
> And, if there are Xen-specific (but not arch-specific) PEIMs / drivers /
> libraries that work alike on ARM and x86, they should go under OvmfPkg,
> as ArmVirtPkg can (and already does) pull Xen-only modules from OvmfPkg.
> See for example, in ArmVirtXen.dsc:
> 
>   
> SerialPortLib|OvmfPkg/Library/XenConsoleSerialPortLib/XenConsoleSerialPortLib.inf
>   XenHypercallLib|OvmfPkg/Library/XenHypercallLib/XenHypercallLib.inf
> 
>   OvmfPkg/XenBusDxe/XenBusDxe.inf
>   OvmfPkg/XenPvBlkDxe/XenPvBlkDxe.inf
> 
>> To build and boot:
>>
>> To build, simply run OvmfPkg/build.sh -p OvmfPkg/XenOvmf.dsc
> 
> (1) To stick with the ArmVirt pattern, we should initially call this
> platform OvmfXen.
> 
> (2) And, once you remove the Xen bits from the current
> OvmfPkg*.dsc files, we should likely rename those to OvmfQemu*dsc.
> 
> I'll try to process this series in a timely manner. My main focus will
> be possible regressions on QEMU/KVM, and formalities (license blocks,
> signoffs, edk2 coding style, build issues that I might notice in
> review).
> 
> I can't provide testing feedback, but xen-devel subscribers (and any
> other Xen users too, obviously) are super welcome to test & report
> results!
> 
> (3) Please file a BZ at <https://bugzilla.tianocore.org/> about this
> feature, and assign it to yourself. The BZ should keep track of all
> versions of the patch series (from the mailing list archive).
> 
> (4) Ideally, the release notes for the edk2 stable release in which the
> feature will appear, should mention the feature (via linking the BZ):
> <https://github.com/tianocore/tianocore.github.io/wiki/EDK-II-Release-Planning#proposed-features>
> 
> (5) The BZ should be referenced in all the commit messages.
> 
> (6) The new edk2 development mailing list is at:
> 
> https://edk2.groups.io/g/devel
> 
> Please subscribe there, and resend this series to that address, i.e.
> <devel@xxxxxxxxxxxxxx>. I'm CC'ing the new address myself, for this
> initial response, but I'd prefer the rest of my comments to go only to
> the new list (without manually updating the CC list on every response).

(7) I forgot to mention that, due to
<https://bugzilla.tianocore.org/show_bug.cgi?id=1373> being very close
to being fixed / pushed, the the "Contributed-under" lines near your
signoffs will no longer be necessary.

Thanks
Laszlo

> 
> Thanks!
> Laszlo
> 
>> Then use OVMF.fd as a kernel of a pvh guest config file (with
>> xl/libxl).
>>
>> Patch series available in this git branch:
>> https://xenbits.xen.org/git-http/people/aperard/ovmf.git 
>> br.platform-xen-pvh-v2
>>
>> Anthony PERARD (31):
>>   OvmfPkg/ResetSystemLib: Add missing dependency on PciLib
>>   OvmfPkg: Create platform XenOvmf
>>   OvmfPkg: Introduce XenResetVector
>>   OvmfPkg: Introduce XenPlatformPei
>>   OvmfPkg/XenOvmf: Creating an ELF header
>>   OvmfPkg/XenResetVector: Add new entry point for Xen PVH
>>   OvmfPkg/XenResetVector: Saving start of day pointer for PVH guests
>>   OvmfPkg/XenResetVector: Allow to jumpstart from either hvmloader or
>>     PVH
>>   OvmfPkg/XenOvmf: use a TimerLib instance that depends only on the CPU
>>   OvmfPkg/XenPlatformPei: Detect OVMF_INFO from hvmloader
>>   OvmfPkg/XenPlatformPei: Use mXenHvmloaderInfo to get E820
>>   OvmfPkg/XenPlatformPei: Grab RSDP from PVH guest start of day struct
>>   OvmfPkg/Library/XenPlatformLib: New library
>>   OvmfPkg/AcpiPlatformDxe: Use PVH RSDP if exist
>>   OvmfPkg/XenHypercallLib: Enable it in PEIM
>>   OvmfPkg/XenPlatformPei: Introduce XenHvmloaderDetected
>>   OvmfPkg/XenPlatformPei: Reserve hvmloader's memory only when it as
>>     runned
>>   OvmfPkg/XenPlatformPei: Setup HyperPages earlier
>>   OvmfPkg/XenPlatformPei: Introduce XenPvhDetected
>>   OvmfPkg: Import XENMEM_memory_map hypercall to Xen/memory.h
>>   OvmfPkg/XenPlatformPei: Get E820 table via hypercall ...
>>   OvmfPkg/XenPlatformPei: Rework memory detection
>>   OvmfPkg/XenPlatformPei: Reserve VGA memory region, to boot Linux
>>   OvmfPkg/XenPlatformPei: Ignore missing PCI Host Bridge on Xen PVH
>>   OvmfPkg/PlatformBootManagerLib: Handle the absence of PCI bus on Xen
>>     PVH
>>   OvmfPkg/XenOvmf: Override PcdFSBClock to Xen vLAPIC timer frequency
>>   OvmfPkg/XenOvmf: Introduce XenTimerDxe
>>   OvmfPkg/PlatformBootManagerLib: Use a Xen console for ConOut/ConIn
>>   OvmfPkg: Introduce XenIoPvhDxe to initialize Grant Tables
>>   OvmfPkg: Move XenRealTimeClockLib from ArmVirtPkg
>>   OvmfPkg/XenOvmf: use RealTimeClockRuntimeDxe from EmbeddedPkg
>>
>>  OvmfPkg/OvmfPkg.dec                                                         
>>                                         |   4 +
>>  ArmVirtPkg/ArmVirtXen.dsc                                                   
>>                                         |   2 +-
>>  OvmfPkg/OvmfPkgIa32.dsc                                                     
>>                                         |   1 +
>>  OvmfPkg/OvmfPkgIa32X64.dsc                                                  
>>                                         |   1 +
>>  OvmfPkg/OvmfPkgX64.dsc                                                      
>>                                         |   1 +
>>  OvmfPkg/{OvmfPkgX64.dsc => XenOvmf.dsc}                                     
>>                                         | 236 ++------
>>  OvmfPkg/XenOvmf.fdf                                                         
>>                                         | 561 ++++++++++++++++++++
>>  OvmfPkg/AcpiPlatformDxe/AcpiPlatformDxe.inf                                 
>>                                         |   2 +-
>>  OvmfPkg/Library/PlatformBootManagerLib/PlatformBootManagerLib.inf           
>>                                         |   4 +
>>  OvmfPkg/Library/ResetSystemLib/ResetSystemLib.inf                           
>>                                         |   1 +
>>  OvmfPkg/Library/XenHypercallLib/XenHypercallLib.inf                         
>>                                         |   2 +-
>>  ArmVirtPkg/Library/XenRealTimeClockLib/XenRealTimeClockLib.inf => 
>> OvmfPkg/Library/XenPlatformLib/XenPlatformLib.inf |  30 +-
>>  {ArmVirtPkg => OvmfPkg}/Library/XenRealTimeClockLib/XenRealTimeClockLib.inf 
>>                                         |   0
>>  OvmfPkg/XenIoPvhDxe/XenIoPvhDxe.inf                                         
>>                                         |  35 ++
>>  OvmfPkg/XenPlatformPei/XenPlatformPei.inf                                   
>>                                         | 107 ++++
>>  OvmfPkg/XenResetVector/XenResetVector.inf                                   
>>                                         |  46 ++
>>  OvmfPkg/XenTimerDxe/XenTimerDxe.inf                                         
>>                                         |  49 ++
>>  OvmfPkg/AcpiPlatformDxe/AcpiPlatform.h                                      
>>                                         |   6 +-
>>  OvmfPkg/Include/Guid/XenInfo.h                                              
>>                                         |   8 +-
>>  OvmfPkg/Include/IndustryStandard/Xen/arch-x86/hvm/start_info.h              
>>                                         | 159 ++++++
>>  OvmfPkg/Include/IndustryStandard/Xen/memory.h                               
>>                                         |  23 +
>>  OvmfPkg/Include/Library/XenHypercallLib.h                                   
>>                                         |  12 +
>>  OvmfPkg/Include/Library/XenPlatformLib.h                                    
>>                                         |  59 ++
>>  OvmfPkg/Include/OvmfPlatforms.h                                             
>>                                         |   6 +
>>  OvmfPkg/Library/PlatformBootManagerLib/BdsPlatform.h                        
>>                                         |   1 +
>>  OvmfPkg/XenPlatformPei/Cmos.h                                               
>>                                         |  58 ++
>>  OvmfPkg/XenPlatformPei/Platform.h                                           
>>                                         | 135 +++++
>>  OvmfPkg/{PlatformPei => XenPlatformPei}/Xen.h                               
>>                                         |   0
>>  OvmfPkg/XenTimerDxe/XenTimerDxe.h                                           
>>                                         | 183 +++++++
>>  OvmfPkg/AcpiPlatformDxe/Xen.c                                               
>>                                         |  41 +-
>>  OvmfPkg/Library/PlatformBootManagerLib/BdsPlatform.c                        
>>                                         |  15 +-
>>  OvmfPkg/Library/PlatformBootManagerLib/PlatformData.c                       
>>                                         |  59 ++
>>  OvmfPkg/Library/ResetSystemLib/ResetSystemLib.c                             
>>                                         |   3 +-
>>  OvmfPkg/Library/XenHypercallLib/X86XenHypercall.c                           
>>                                         |  11 +
>>  OvmfPkg/Library/XenPlatformLib/XenPlatformLib.c                             
>>                                         |  75 +++
>>  {ArmVirtPkg => OvmfPkg}/Library/XenRealTimeClockLib/XenRealTimeClockLib.c   
>>                                         |   0
>>  OvmfPkg/PlatformPei/Xen.c                                                   
>>                                         |   3 -
>>  OvmfPkg/XenIoPvhDxe/XenIoPvhDxe.c                                           
>>                                         |  38 ++
>>  OvmfPkg/XenPlatformPei/AmdSev.c                                             
>>                                         |  70 +++
>>  OvmfPkg/XenPlatformPei/ClearCache.c                                         
>>                                         | 118 ++++
>>  OvmfPkg/XenPlatformPei/Cmos.c                                               
>>                                         |  66 +++
>>  OvmfPkg/XenPlatformPei/Fv.c                                                 
>>                                         |  82 +++
>>  OvmfPkg/XenPlatformPei/MemDetect.c                                          
>>                                         | 498 +++++++++++++++++
>>  OvmfPkg/XenPlatformPei/Platform.c                                           
>>                                         | 458 ++++++++++++++++
>>  OvmfPkg/XenPlatformPei/Xen.c                                                
>>                                         | 381 +++++++++++++
>>  OvmfPkg/XenTimerDxe/XenTimerDxe.c                                           
>>                                         | 361 +++++++++++++
>>  generate_elf_header.c                                                       
>>                                         |  78 +++
>>  OvmfPkg/XenResetVector/Ia16/Real16ToFlat32.asm                              
>>                                         | 144 +++++
>>  OvmfPkg/XenResetVector/Ia16/ResetVectorVtf0.asm                             
>>                                         |  87 +++
>>  OvmfPkg/XenResetVector/Ia32/Flat32ToFlat64.asm                              
>>                                         |  66 +++
>>  OvmfPkg/XenResetVector/Ia32/PageTables64.asm                                
>>                                         | 156 ++++++
>>  OvmfPkg/XenResetVector/Ia32/SearchForBfvBase.asm                            
>>                                         |  93 ++++
>>  OvmfPkg/XenResetVector/Ia32/XenPVHMain.asm                                  
>>                                         |  75 +++
>>  OvmfPkg/XenResetVector/XenResetVector.nasmb                                 
>>                                         |  78 +++
>>  54 files changed, 4526 insertions(+), 262 deletions(-)
>>  copy OvmfPkg/{OvmfPkgX64.dsc => XenOvmf.dsc} (78%)
>>  create mode 100644 OvmfPkg/XenOvmf.fdf
>>  copy ArmVirtPkg/Library/XenRealTimeClockLib/XenRealTimeClockLib.inf => 
>> OvmfPkg/Library/XenPlatformLib/XenPlatformLib.inf (50%)
>>  rename {ArmVirtPkg => 
>> OvmfPkg}/Library/XenRealTimeClockLib/XenRealTimeClockLib.inf (100%)
>>  create mode 100644 OvmfPkg/XenIoPvhDxe/XenIoPvhDxe.inf
>>  create mode 100644 OvmfPkg/XenPlatformPei/XenPlatformPei.inf
>>  create mode 100644 OvmfPkg/XenResetVector/XenResetVector.inf
>>  create mode 100644 OvmfPkg/XenTimerDxe/XenTimerDxe.inf
>>  create mode 100644 
>> OvmfPkg/Include/IndustryStandard/Xen/arch-x86/hvm/start_info.h
>>  create mode 100644 OvmfPkg/Include/Library/XenPlatformLib.h
>>  create mode 100644 OvmfPkg/XenPlatformPei/Cmos.h
>>  create mode 100644 OvmfPkg/XenPlatformPei/Platform.h
>>  copy OvmfPkg/{PlatformPei => XenPlatformPei}/Xen.h (100%)
>>  create mode 100644 OvmfPkg/XenTimerDxe/XenTimerDxe.h
>>  create mode 100644 OvmfPkg/Library/XenPlatformLib/XenPlatformLib.c
>>  rename {ArmVirtPkg => 
>> OvmfPkg}/Library/XenRealTimeClockLib/XenRealTimeClockLib.c (100%)
>>  create mode 100644 OvmfPkg/XenIoPvhDxe/XenIoPvhDxe.c
>>  create mode 100644 OvmfPkg/XenPlatformPei/AmdSev.c
>>  create mode 100644 OvmfPkg/XenPlatformPei/ClearCache.c
>>  create mode 100644 OvmfPkg/XenPlatformPei/Cmos.c
>>  create mode 100644 OvmfPkg/XenPlatformPei/Fv.c
>>  create mode 100644 OvmfPkg/XenPlatformPei/MemDetect.c
>>  create mode 100644 OvmfPkg/XenPlatformPei/Platform.c
>>  create mode 100644 OvmfPkg/XenPlatformPei/Xen.c
>>  create mode 100644 OvmfPkg/XenTimerDxe/XenTimerDxe.c
>>  create mode 100644 generate_elf_header.c
>>  create mode 100644 OvmfPkg/XenResetVector/Ia16/Real16ToFlat32.asm
>>  create mode 100644 OvmfPkg/XenResetVector/Ia16/ResetVectorVtf0.asm
>>  create mode 100644 OvmfPkg/XenResetVector/Ia32/Flat32ToFlat64.asm
>>  create mode 100644 OvmfPkg/XenResetVector/Ia32/PageTables64.asm
>>  create mode 100644 OvmfPkg/XenResetVector/Ia32/SearchForBfvBase.asm
>>  create mode 100644 OvmfPkg/XenResetVector/Ia32/XenPVHMain.asm
>>  create mode 100644 OvmfPkg/XenResetVector/XenResetVector.nasmb
>>
> 


_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/mailman/listinfo/xen-devel

 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.