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 <dhaval@rivosinc.com>
Sent: Friday, March 3, 2023 2:12 AM
To: devel@edk2.groups.io
Cc: Guo, Gua <gua.guo@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.guo@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