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 loong64 support #8

Merged
merged 1 commit into from
Oct 24, 2023

Conversation

yzewei
Copy link
Contributor

@yzewei yzewei commented Oct 20, 2023

Protoc temporarily provides Loongarch support, and this submission adds a Protoc supported by Loongarch support.

The LoongArch architecture (LoongArch) is an Instruction Set Architecture (ISA) that has a RISC style.

@yzewei
Copy link
Contributor Author

yzewei commented Oct 20, 2023

@thallgren thanks!

@thallgren
Copy link
Member

@yzewei , not a requirement, but is there any way we could add unit tests for this architecture? I don't suppose GitHub as a runner for it.

@yzewei
Copy link
Contributor Author

yzewei commented Oct 21, 2023

Hi, @thallgren

One of the most important thing is there is a docker image where go-fuseftp tests can be done. You can use it on x86 machine with docker run -it --rm merore/archlinux-loongarch64.
About this image:

  • Boot an Loongarch64-based Arch Linux with qemu-system-loongarch64
  • The default user/password is root/loongarch64
  • All dependcies required by criu build and test are pre-installed
  • You must configure your own dependence on the environment, Golang's installation package is located at http://ftp.loongnix.cn/toolchain/golang/go-1.20/abi2.0/go1.20.2.linux-loong64.tar.gz
    Finally, thank you for your patience !

@thallgren thallgren merged commit 9ea3c0f into telepresenceio:master Oct 24, 2023
3 checks passed
@thallgren
Copy link
Member

Thanks for providing this together with info on how it can be tested. Much appreciated.

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