On Fri, Mar 3, 2023 at 6:05 AM Guo, Gua 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 > Sent: Friday, March 3, 2023 2:12 AM > To: devel@edk2.groups.io > Cc: Guo, Gua > Subject: [PATCH v1 0/1] Remove FP Init in UPL Entry > > Remove floating point initialization from UPL entry point > > Cc: Gua guo > 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