Skip to content
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

Enable foreign image loading on LoongArch64 and synchronize the changes about LoongArch64 in section 18.2.5 of UEFI 2.11 #6524

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

kilaterlee
Copy link
Contributor

Description

  1. Added LoongArch64 loading foreign image support.
  2. Synchronized UEFI 2.11 LoongArch64 changes.

How This Was Tested

Build OvmfPkg/LoongArchVirt/LoongArchVirtQemu.dsc and run it.

Integration Instructions

N/A

@tianocore-assign-reviewers
Copy link

WARNING: Cannot add some reviewers: A user specified as a reviewer for this PR is not a collaborator of the repository. Please add them as a collaborator to the repository so they can be requested in the future.

Non-collaborators requested:

Attn Admins:


Admin Instructions:

  • Add the non-collaborators as collaborators to the appropriate team(s) listed in teams
  • If they are no longer needed as reviewers, remove them from Maintainers.txt

@kilaterlee
Copy link
Contributor Author

@mdkinney @lgao4 Hi Mike and Liming, please help invite bibo-mao and lixianglai to join the Reviewer team, thanks! :)

kilaterlee and others added 4 commits December 12, 2024 10:24
Enable foreign images loading on LoongArch64 if the
EDKII_PECOFF_IMAGE_EMULATOR_PROTOCOL is present.

Cc: Michael D Kinney <[email protected]>
Cc: Liming Gao <[email protected]>
Cc: Zhiguang Liu <[email protected]>
Signed-off-by: Chao Li <[email protected]>
Co-Authored-by: Dongyan Qian <[email protected]>
Synchronize the changes about LoongArch64 in section 18.2.5 of UEFI2.11.

Cc: Michael D Kinney <[email protected]>
Cc: Liming Gao <[email protected]>
Cc: Zhiguang Liu <[email protected]>
Signed-off-by: Chao Li <[email protected]>
Since the UEFI 2.11 has been released, the macro
MAX_LOONGARCH_EXCEPTION has been added in MdePkg, so it is deleted in
LoongArch folder header file.

Cc: Ray Ni <[email protected]>
Cc: Jiaxin Wu <[email protected]>
Cc: Zhiguang Liu <[email protected]>
Cc: Dun Tan <[email protected]>
Cc: Rahul Kumar <[email protected]>
Cc: Gerd Hoffmann <[email protected]>
Cc: Jiaxin Wu <[email protected]>
Signed-off-by: Chao Li <[email protected]>
Since the UEFI 2.11 has been released, the macro
MAX_LOONGARCH_EXCEPTION has been added in MdePkg, so it is deleted in
LoongArchVirt/Sec/LoongArch64/Start.S

Cc: Ard Biesheuvel <[email protected]>
Cc: Jiewen Yao <[email protected]>
Cc: Gerd Hoffmann <[email protected]>
Signed-off-by: Chao Li <[email protected]>
Copy link

@AydenMeng AydenMeng left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@kilaterlee
Copy link
Contributor Author

@mdkinney @lgao4 @niruiyu Hi Mike, Liming and Ray, Do you have any objections, if not, can I set the push lable?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants