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

add LOONGARCH64 compilation support #43

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

MarsDoge
Copy link

@MarsDoge MarsDoge commented Nov 4, 2023

This is a RISC processor architecture - LOONGARCH64, and the upstream EDK2 and GRUB repositories have merged related architecture patches, embracing open source ...

@pbatard
Copy link
Owner

pbatard commented Nov 4, 2023

Thanks for this.

The builds cannot run however because it appears that you committed the grub submodule after you applied the 0001-GRUB-fixes.patch, so you need to revert your changes to the grub submodules and update your pull request so that it can be validated for builds. Then you can re-apply the patch in your local repository.

@pbatard
Copy link
Owner

pbatard commented Nov 6, 2023

Hmmm, it looks like gcc-loongarch64-linux-gnu is not yet available for ubuntu-latest (the base images used by GitHub for Linux builds) per: https://github.com/pbatard/efifs/actions/runs/6770275752/job/18398546553?pr=43#step:2:20

I also don't think the gnu-efi compilation for LOONGARCH64 will work, since we can't update to a newer version of gnu-efi, and the ones we use has no biding for LOONGARCH64, so the linux_gcc_gnu-efi.yml GitHub actions file you modified is most likely bound to fail, even if gcc-loongarch64-linux-gnu is available.

I guess we can drop support for gnu-efi/LOONGARCH64, but obviously, since the binaries I publish are build through GitHub actions, we're going to have to wait until Ubuntu adds gcc-loongarch64-linux-gnu to their standard packages before we can add support for it in EfiFs. Right now, it seems like only binutils for LOONGARCH64 are available...

@MarsDoge
Copy link
Author

MarsDoge commented Nov 6, 2023

Hmmm, it looks like gcc-loongarch64-linux-gnu is not yet available for ubuntu-latest (the base images used by GitHub for Linux builds) per: https://github.com/pbatard/efifs/actions/runs/6770275752/job/18398546553?pr=43#step:2:20

I also don't think the gnu-efi compilation for LOONGARCH64 will work, since we can't update to a newer version of gnu-efi, and the ones we use has no biding for LOONGARCH64, so the linux_gcc_gnu-efi.yml GitHub actions file you modified is most likely bound to fail, even if gcc-loongarch64-linux-gnu is available.

I guess we can drop support for gnu-efi/LOONGARCH64, but obviously, since the binaries I publish are build through GitHub actions, we're going to have to wait until Ubuntu adds gcc-loongarch64-linux-gnu to their standard packages before we can add support for it in EfiFs. Right now, it seems like only binutils for LOONGARCH64 are available...

Thank you very much for your reply.
It is true that the latest ubuntu release version does not yet integrate the LoongArch compilation tool chain.
Can I add efi support first?

@MarsDoge
Copy link
Author

MarsDoge commented Nov 6, 2023

Looking forward to Ubuntu adds gcc-loongarch64-linux-gnu to their standard packages, I will request this PR again.

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.

2 participants