On Mon, Apr 08, 2019 at 05:50:57PM +0200, Laszlo Ersek wrote: > On 04/08/19 16:23, Anthony PERARD wrote: > > 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.
Thanks for the feedback, that sounds good. > > 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. I'll rename it in v3. > (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). Done: https://bugzilla.tianocore.org/show_bug.cgi?id=1689 I'll add the links for the patch series once there is one for v2. > (5) The BZ should be referenced in all the commit messages. I'll do that in v3. > (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@edk2.groups.io>. 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). I'm about to do that. Thanks, -- Anthony PERARD -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#38704): https://edk2.groups.io/g/devel/message/38704 Mute This Topic: https://groups.io/mt/30964532/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
