On 08/22/19 16:52, Gao, Liming wrote: >> -----Original Message----- >> From: Laszlo Ersek [mailto:[email protected]] >> Sent: Thursday, August 22, 2019 7:56 PM >> To: Gao, Liming <[email protected]> >> Cc: [email protected]; Kinney, Michael D <[email protected]>; >> Mike Turner <[email protected]>; Wang, Jian J >> <[email protected]>; Wu, Hao A <[email protected]>; Bi, Dandan >> <[email protected]>; Anthony Perard >> <[email protected]>; Justen, Jordan L <[email protected]> >> Subject: Re: [edk2-devel] [Patch] MdeModulePkg DxeCore: Fix for missing MAT >> update
>> This is the current status: >> >> - OvmfXen: >> - runs in Xen HVM guests >> - runs in Xen PVH guests >> >> - traditional OVMF >> - runs in Xen HVM guests >> - runs in QEMU/KVM guests >> >> The desired (and agreed upon) end-status is the following: >> >> - OvmfXen: >> - runs in Xen HVM guests >> - runs in Xen PVH guests >> >> - traditional OVMF >> - runs in QEMU/KVM guests >> >> (This will match the platform separation that we have under ArmVirtPkg too.) >> >> Anthony plans to remove the Xen HVM code from traditional OVMF in a year >> or so, if I understand correctly. That's when "traditional OVMF" will >> become QEMU/KVM-only, completing the separation. And that's when I >> expect we can fix TianoCore#386 for QEMU/KVM *only*, without Jordan >> NACKing the patch set. >> > Ok. Is there BZ for this change? The contributor can propose his work > planning. > Then, I will update edk2 feature planning wiki. I've now reported: https://bugzilla.tianocore.org/show_bug.cgi?id=2122 and assigned it to Anthony. (Anthony, I hope that's OK with you.) >> Basically, "PcdMemVarstoreEmuEnable" would be constant TRUE in OvmfXen >> (inherited from the OVMF DEC file), and it would be exposed to the >> platform builder via "-D MEM_VARSTORE_EMU_ENABLE=FALSE" in the "OVMF for >> QEMU/KVM" platform. FaultTolerantWritePei and VariablePei would be >> included in the "OVMF for QEMU/KVM" DSC files only, and so on. >> > I think this design meets with the request BZ386. I've also reopened https://bugzilla.tianocore.org/show_bug.cgi?id=386 now (moving it back to CONFIRMED state, from RESOLVED|WONTFIX), and I've made it dependent on TianoCore#2122. Thanks, Laszlo -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#46301): https://edk2.groups.io/g/devel/message/46301 Mute This Topic: https://groups.io/mt/32821535/21656 Group Owner: [email protected] Unsubscribe: https://edk2.groups.io/g/devel/unsub [[email protected]] -=-=-=-=-=-=-=-=-=-=-=-
