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

solved #262

Closed
wants to merge 1 commit into from
Closed

solved #262

wants to merge 1 commit into from

Conversation

yzewei
Copy link

@yzewei yzewei commented Jan 18, 2024

Similar problems have occurred under the Loongarch64 #260

# github.com/minio/madmin-go/kernel ../../go-workspace/pkg/mod/github.com/minio/[email protected]/kernel/kernel.go:73:32: undefined: utsnameStr

what did I do:

  • update madmin-go/kernel/kernel_utsname_int8.go
    add loong64
  • update go.mod
    Part of the dependent package needs to be updated to obtain Loongarch64 architecture support
  • about ci test
    Loongarch64 temporarily not supported -race

The test was completed under Openeuler 2203.
Openeuler system is a derivative version of centos.

@yzewei
Copy link
Author

yzewei commented Jan 18, 2024

ptal @klauspost @harshavardhana

//go:build (linux && 386) || (linux && amd64) || (linux && arm64) || (linux && loong64) || (linux && mips64) || (linux && mips64le) || (linux && mips)
// +build linux,386 linux,amd64 linux,arm64 linux,loong64 linux,mips64 linux,mips64le linux,mips
//go:build (linux && 386) || (linux && amd64) || (linux && arm64) || (linux && loong64) || (linux && mips64) || (linux && mips64le) || (linux && mips) || (linux && loong64)
// +build linux,386 linux,amd64 linux,arm64 linux,loong64 linux,mips64 linux,mips64le linux,mips linux,loong64
Copy link
Contributor

@jiuker jiuker Jan 18, 2024

Choose a reason for hiding this comment

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

there have linux,loong64. Why this pr fix this? @yzewei

@yzewei yzewei closed this Jan 18, 2024
@yzewei yzewei changed the title Add Loongarch64 support solved Jan 18, 2024
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