On Fri, Mar 3, 2023 at 6:05 AM Guo, Gua <gua....@intel.com> wrote: > What's the issue you encounter ? > I will add this context in the next patchset. But primarily I am enabling UPL for RiscV and during the process I found a few sequences which I believe are arch specific and showing up in the common boot path. While RV has a similar FP enabling concept, in my understanding this enabling is required to automate (HW based) FP save/restore during process context switches at OS runtime. Also OS (at least I confirmed with Linux) have their own enabling process for FP. And I do not see any instance in UPL/BDS that is making use of this feature. Also it appears that UPL spec suggests BL performs this step.
> > -----Original Message----- > From: Dhaval Sharma <dha...@rivosinc.com> > Sent: Friday, March 3, 2023 2:12 AM > To: devel@edk2.groups.io > Cc: Guo, Gua <gua....@intel.com> > Subject: [PATCH v1 0/1] Remove FP Init in UPL Entry > > Remove floating point initialization from UPL entry point > > Cc: Gua guo <gua....@intel.com> > Dhaval Sharma (1): > UefiPayloadPkg: Remove FP Init from UPL entryfunc > > UefiPayloadPkg/UefiPayloadEntry/UniversalPayloadEntry.c | 3 --- > 1 file changed, 3 deletions(-) > > -- > 2.40.0.rc0.57.g454dfcbddf > > -- Thanks! =D -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#100696): https://edk2.groups.io/g/devel/message/100696 Mute This Topic: https://groups.io/mt/97345073/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
