Hi Ray, I think nesting may be possible in physical platforms, but very hard to induce.
One option is to consolidate to a single LocalApicTimerDxe implementation in the UefiCpuPkg, but allow the platform DSC to either specify a Null NestedInterruptTplLib for physical platforms or the full one from the OvmfPkg for VM use cases. The other changes could be included for OvmfPkg use cases. If the VM does not support the CPUID leafs, then the PCD value should be used. Mike > -----Original Message----- > From: Pedro Falcato <pedro.falc...@gmail.com> > Sent: Monday, January 15, 2024 1:00 AM > To: devel@edk2.groups.io; Ni, Ray <ray...@intel.com> > Cc: Kinney, Michael D <michael.d.kin...@intel.com>; Desimone, Nathaniel > L <nathaniel.l.desim...@intel.com>; Laszlo Ersek <ler...@redhat.com>; > Kumar, Rahul R <rahul.r.ku...@intel.com>; Gerd Hoffmann > <kra...@redhat.com> > Subject: Re: [edk2-devel] [PATCH 1/6] UefiCpuPkg/LocalApicTimerDxe: > Duplicate OvmfPkg/LocalApicTimerDxe driver > > On Mon, Jan 15, 2024 at 8:04 AM Ni, Ray <ray...@intel.com> wrote: > > > > This commit only duplicates the OvmfPkg/LocalApicTimerDxe. > > Following commits will enhance the driver. > > Hi, > > Please describe why you're doing this change. i.e what's your use case > for LocalApicTimerDxe, and why are you duplicating this instead of > moving OvmfPkg's (why do we need to maintain 2 separate versions of > what is essentially the same driver)? > > Thanks, > Pedro -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#113841): https://edk2.groups.io/g/devel/message/113841 Mute This Topic: https://groups.io/mt/103734961/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/leave/9847357/21656/1706620634/xyzzy [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
