Skip to content

cmd/compile: SIGBUS unaligned access on mips64 via qemu-mips64 [1.22 backport] #67212

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.

Sign up for GitHub

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

Jump to bottom
Closed
randall77 opened this issue May 6, 2024 · 5 comments
Closed

cmd/compile: SIGBUS unaligned access on mips64 via qemu-mips64 [1.22 backport] #67212

randall77 opened this issue May 6, 2024 · 5 comments
Labels
CherryPickApproved Used during the release process for point releases compiler/runtime Issues related to the Go compiler and/or runtime.
Milestone
Go1.22.4

Comments

@randall77
Copy link
Contributor

@randall77 requested issue #67160 to be considered for backport to the next 1.22 minor release.

This is a bug that miscompiles some type equality functions on architectures that can't do unaligned loads. There is no easy workaround.

(Backport issue created manually.)
@randall77 randall77 added CherryPickCandidate Used during the release process for point releases compiler/runtime Issues related to the Go compiler and/or runtime. labels May 6, 2024
@randall77 randall77 added this to the Go1.22.3 milestone May 6, 2024
@gopherbot
Copy link
Contributor

Change https://go.dev/cl/583417 mentions this issue: [release-branch.go1.22] cmd/compile: don't combine loads in generated equality functions

gopherbot pushed a commit that referenced this issue May 6, 2024
@randall77
[release-branch.go1.22] cmd/compile: don't combine loads in generated…
9d2e285 
… equality functions

... if the architecture can't do unaligned loads.
We already handle this in a few places, but this particular place
was added in CL 399542 and missed this additional restriction.

Fixes #67212

Change-Id: I45988f11ff3ed45df1c4da3f0931ab1fdb22dbfe
Reviewed-on: https://go-review.googlesource.com/c/go/+/583175
Reviewed-by: Cuong Manh Le <[email protected]>
Auto-Submit: Keith Randall <[email protected]>
Reviewed-by: Keith Randall <[email protected]>
LUCI-TryBot-Result: Go LUCI <[email protected]>
Reviewed-by: Derek Parker <[email protected]>
Reviewed-by: Cherry Mui <[email protected]>
(cherry picked from commit 3c72dd5)
Reviewed-on: https://go-review.googlesource.com/c/go/+/583417
@gopherbot
Copy link
Contributor

Closed by merging 9d2e285 to release-branch.go1.22.

@cherrymui
Copy link
Member

cherrymui commented May 6, 2024 •
edited
Loading

CL submitted reverted. Reopen for cherry-pick review (the dashboard only shows open issues).

@cherrymui cherrymui reopened this May 6, 2024
@gopherbot gopherbot modified the milestones: Go1.22.3, Go1.22.4 May 7, 2024
@gopherbot
Copy link
Contributor

Change https://go.dev/cl/583798 mentions this issue: [release-branch.go1.22] cmd/compile: don't combine loads in generated equality functions

@cherrymui cherrymui added the CherryPickApproved Used during the release process for point releases label May 8, 2024
@gopherbot gopherbot removed the CherryPickCandidate Used during the release process for point releases label May 8, 2024
gopherbot pushed a commit that referenced this issue May 10, 2024
@randall77 @cherrymui
[release-branch.go1.22] cmd/compile: don't combine loads in generated…
dc8976d 
… equality functions

... if the architecture can't do unaligned loads.
We already handle this in a few places, but this particular place
was added in CL 399542 and missed this additional restriction.

Fixes #67212

(Reroll of CL 583417)

Change-Id: I8ee74909341e0af0d51d3761e17ff92e744ecd38
Reviewed-on: https://go-review.googlesource.com/c/go/+/583798
Reviewed-by: Keith Randall <[email protected]>
Reviewed-by: Keith Randall <[email protected]>
LUCI-TryBot-Result: Go LUCI <[email protected]>
@cherrymui
Copy link
Member

CL submitted.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
CherryPickApproved Used during the release process for point releases compiler/runtime Issues related to the Go compiler and/or runtime.
Projects
None yet
Milestone
Go1.22.4
Development

No branches or pull requests
3 participants
@randall77 @gopherbot @cherrymui