-
Notifications
You must be signed in to change notification settings - Fork 933
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 (from Incus) #13696
Add loongarch64 (from Incus) #13696
Conversation
Heads up @ru-fu - the "Documentation" label was applied to this issue. |
Pretty sure that LoongArch needs to be added to |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍 for the docs
Signed-off-by: zhaixiaojuan <zhaixiaojuan@loongson.cn> (cherry picked from commit 387f397a2537adfab3db260a70fe9b600a13c165) Signed-off-by: Thomas Parrott <thomas.parrott@canonical.com> License: Apache-2.0
Signed-off-by: zhaixiaojuan <zhaixiaojuan@loongson.cn> (cherry picked from commit ed32f61d9dd659966b175ad65da844bf492d28bd) Signed-off-by: Thomas Parrott <thomas.parrott@canonical.com> License: Apache-2.0
From zhaixiaojuan <zhaixiaojuan@loongson.cn> Signed-off-by: Thomas Parrott <thomas.parrott@canonical.com>
Signed-off-by: Thomas Parrott <thomas.parrott@canonical.com>
Looks good to me. At the same time, it worth mentioning that saying that loongarch64 is a supported architecture is overestimation, just because we don't have any ways to actually check that it works. But this change is safe and as author of it works for Loongson, I hope it was tested internally on a real hardware. |
Based on #13691 and lxc/incus#370