-
Notifications
You must be signed in to change notification settings - Fork 13
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Emulator.dsc: Change BaseStackCheckLib to StackCheckLibNull #67
base: main
Are you sure you want to change the base?
Conversation
Since EDKII PR: tianocore/edk2#5957 has been removed the BaseStackCheckLib, and the new StackCheckLib dependency library StackCheckFailureHookLib has not yet been made public, so we can only use StackCheckLibNull. Signed-off-by: Chao Li <[email protected]>
@andreiw Hi Andrei, this repo will not build if we update the edk2 code base to a9b38305b64ef5997d0ba5f7d2797a75edd1f9ef or newer, I have fixed it, PLS check, THX! :) |
@ardbiesheuvel Hi Ard, could you please review this change? |
LGTM |
Gonna pull this in here -> https://github.com/andreiw/MultiArchUefiPkg/ |
@kilaterlee can you also send some documentation patches for LoongArch, including building/testing with Qemu if possible? |
Okey, I will fork the new repo and submit it again. |
Here are LoongArch SPEC web: https://github.com/loongson/LoongArch-Documentation. |
If you want to building the MultiArchUefiPkg and unicorn-for-efi for LoongArch, it not different from ARM or RV:
|
Another question: |
Since EDKII PR: tianocore/edk2#5957 has been removed the BaseStackCheckLib, and the new StackCheckLib dependency library StackCheckFailureHookLib has not yet been made public, so we can only use StackCheckLibNull.