-
Notifications
You must be signed in to change notification settings - Fork 17.8k
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
test/fixedbugs: issue11656.go fails on Go1.17.3 for armv7l (Odroid-HC1) #49508
Comments
That instruction looks reasonable, if odd. It is
But more generally, that doesn't really look like an instruction stream. The following instructions are all zeros, which would be
which are useless instructions. Can you build the test directly, using |
issue11656.go intentionally writes an artificial function in memory, which includes an |
So then this just looks like the heap is marked as non-executable? |
Yeah. I think if the heap is not executable we expect to get a SEGV at the call instruction. If it is executable, we expect to get a SEGV from the heap. SIGILL would be weird. Maybe some I-cache coherence thing? (So we read the data in the SIGILL handler (from D-cache, perhaps) and get the correct instruction) |
Can I skip that one test somehow? Or I shouldn't because it's actually something wrong? |
@Talkless depends on what you want to do. You can skip the test by adding an |
If you want to help us fix the Go distribution for all users, we'll need to figure out what exactly is different about your system. |
I just want to built some Go-based software, not as developer, as a user.
IDK, I was just building Go, please let me know how could I answer these questions. |
Then you can just run make.bash, instead of all.bash. |
Sorry, I've missed that question. So I did:
and the output is:
|
FYI, I've tested on qemu arm (go 1.17.1) and it works fine. |
@Talkless Go 1.11 is very old. Could you try newer version of Go, Go 1.16, 1.17, or tip? Thanks. |
Wait, I did it wrong I guess. I had to build Go 1.17.3 from source using Debian Buser's 1.11, and THEN using newly built 1.17.3 build that unit test, right? |
Yes, that sounds right. Or you can download it from https://go.dev/dl/ . |
|
What version of Go are you using (
go version
)?Does this issue reproduce with the latest release?
Yes, 1.17.3 tag.
What operating system and processor architecture are you using (
go env
)?go env
OutputWhat did you do?
Building 1.17.3 under Debian 10 Buster using script:
What did you expect to see?
Successfull build, as 1.15.x was built before.
What did you see instead?
The text was updated successfully, but these errors were encountered: