[v1] RISCV: Fix InternalLongJump

[edk2-devel] [PATCH v1 0/1] RISCV: Fix InternalLongJump

Andrei Warkentin posted 1 patch 7 months, 1 week ago

Diff against v2
Download series mbox

Failed in applying to current master (apply log)

There is a newer version of this series

CryptoPkg/Library/OpensslLib/openssl                  | 2 +-
MdePkg/Library/BaseLib/RiscV64/RiscVSetJumpLongJump.S | 7 ++-----
2 files changed, 3 insertions(+), 6 deletions(-)

Expand all Fold all

[edk2-devel] [PATCH v1 0/1] RISCV: Fix InternalLongJump

Posted by Andrei Warkentin 7 months, 1 week ago

I was playing around with building code with -Os and ran into
weird crashes, that I ended up chasing down to an obviously
incorrect InternalLongJump implementation, which has never
been correctly passing its 2nd parameter down as the return
value of SetJump.

You can find the pull at https://github.com/tianocore/edk2/pull/4836,
with the failing tests seemingly unrelated (at least one of these
was related to CryptoPkg not being found).

Andrei Warkentin (1):
  RISCV: Fix InternalLongJump to return correct value

 CryptoPkg/Library/OpensslLib/openssl                  | 2 +-
 MdePkg/Library/BaseLib/RiscV64/RiscVSetJumpLongJump.S | 7 ++-----
 2 files changed, 3 insertions(+), 6 deletions(-)

-- 
2.34.1



-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#108815): https://edk2.groups.io/g/devel/message/108815
Mute This Topic: https://groups.io/mt/101450444/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-