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.

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

runtime,cmd/compile: apparent memory corruption in compress/flate and x/net/publicsuffix starting 2021-10-05 #48846

Closed
bcmills opened this issue Oct 7, 2021 · 3 comments
Closed

runtime,cmd/compile: apparent memory corruption in compress/flate and x/net/publicsuffix starting 2021-10-05 #48846

bcmills opened this issue Oct 7, 2021 · 3 comments
Labels
FrozenDueToAge NeedsInvestigation Someone must examine and confirm this is a valid issue and not a duplicate of an existing one. release-blocker
Milestone
Go1.18

Comments

@bcmills
Copy link
Contributor

bcmills commented Oct 7, 2021

greplogs --dashboard -md -l -e \(\?ms\)runtime\\.throw.\*FAIL\\s+compress/flate

2021-10-07T14:01:52-ecb2f23/android-386-emu
2021-10-06T21:06:48-197b328/darwin-arm64-11_0-toothrot
2021-10-06T19:45:35-e38ec96/darwin-arm64-11_0-toothrot
2021-10-06T13:47:50-810b08b/darwin-arm64-11_0-toothrot
2021-10-05T21:25:51-6ae3afa/darwin-arm64-11_0-toothrot
2021-10-05T20:35:41-75773b0/android-arm64-corellium
2021-10-05T20:35:41-75773b0/darwin-arm64-11_0-toothrot
2021-10-05T19:28:36-7ae83c8/darwin-arm64-11_0-toothrot
2021-02-19T00:41:36-40765ff/netbsd-amd64-9_0
2020-12-05T12:36:42-be9379f/plan9-arm
2020-12-02T14:20:12-0433845/plan9-arm
2020-11-28T18:42:44-4ce0a7c/plan9-arm
2020-11-24T19:53:18-65dcd15/plan9-arm
2020-11-11T20:48:31-4c174a7/plan9-arm

I'm inclined to chalk the plan9-arm and failures up to platform-specific bugs, and the netbsd failure looks like #44269. But I have no explanation for the darwin-arm64 and android-*-emu failures starting October 5 — that looks like a separate regression to me.

@jeremyfaller, can you suggest someone to route this to?
@bcmills bcmills added NeedsInvestigation Someone must examine and confirm this is a valid issue and not a duplicate of an existing one. release-blocker labels Oct 7, 2021
@bcmills bcmills added this to the Go1.18 milestone Oct 7, 2021
@jeremyfaller
Copy link
Contributor

Seems a fishy one. While probably not GC, I'd still start with @mknyszek.

@bcmills
Copy link
Contributor Author

bcmills commented Oct 7, 2021

Looks like it affects x/net/publicsuffix too:

greplogs --dashboard -md -l -e \(\?ms\)runtime\\.throw.\*FAIL\\s+golang\\.org/x/net/publicsuffix --since=2021-01-01

2021-10-07T18:58:33-59d4e92-7286502/android-amd64-emu
2021-10-07T18:45:53-59d4e92-77f2750/android-amd64-emu
2021-10-07T18:34:24-59d4e92-2043b3b/android-amd64-emu
2021-10-07T18:02:14-59d4e92-019ad98/android-amd64-emu
2021-10-07T16:38:24-59d4e92-be571a3/aix-ppc64
2021-10-07T14:37:45-59d4e92-ebeab63/darwin-arm64-11_0-toothrot
2021-10-07T14:01:52-59d4e92-ecb2f23/aix-ppc64
2021-10-07T12:55:05-59d4e92-6f74ed0/aix-ppc64
2021-10-07T03:41:40-62292e8-6f74ed0/aix-ppc64
2021-10-07T03:41:40-62292e8-6f74ed0/android-amd64-emu
2021-10-07T03:41:40-62292e8-6f74ed0/darwin-arm64-11_0-toothrot
2021-10-07T02:30:47-62292e8-39bbf08/aix-ppc64
2021-10-07T02:30:47-62292e8-39bbf08/android-amd64-emu
2021-10-07T02:30:47-62292e8-39bbf08/darwin-arm64-11_0-toothrot
2021-10-07T00:39:17-62292e8-812a33d/aix-ppc64
2021-10-07T00:39:17-62292e8-812a33d/android-amd64-emu
2021-10-07T00:39:17-62292e8-812a33d/darwin-arm64-11_0-toothrot
2021-10-06T23:26:57-62292e8-c8dd89e/aix-ppc64
2021-10-06T23:26:57-62292e8-c8dd89e/darwin-arm64-11_0-toothrot
2021-10-06T23:01:01-62292e8-4ffa2f1/aix-ppc64
2021-10-06T23:01:01-62292e8-4ffa2f1/android-amd64-emu
2021-10-06T23:01:01-62292e8-4ffa2f1/darwin-arm64-11_0-toothrot
2021-10-06T22:42:28-62292e8-4002616/aix-ppc64
2021-10-06T22:42:28-62292e8-4002616/darwin-arm64-11_0-toothrot
2021-10-06T21:17:08-62292e8-2e107b4/aix-ppc64
2021-10-06T21:17:08-62292e8-2e107b4/android-amd64-emu
2021-10-06T21:17:08-62292e8-2e107b4/darwin-arm64-11_0-toothrot
2021-10-06T21:06:48-62292e8-197b328/aix-ppc64
2021-10-06T21:06:48-62292e8-197b328/android-amd64-emu
2021-10-06T20:30:12-62292e8-4a37a1d/aix-ppc64
2021-10-06T20:30:12-62292e8-4a37a1d/android-amd64-emu
2021-10-06T20:21:49-62292e8-f580b75/darwin-arm64-11_0-toothrot
2021-10-06T19:59:27-62292e8-17c513e/aix-ppc64
2021-10-06T19:59:27-62292e8-17c513e/android-amd64-emu
2021-10-06T19:59:27-62292e8-17c513e/darwin-arm64-11_0-toothrot
2021-10-06T19:08:15-62292e8-6487b15/aix-ppc64
2021-10-06T19:08:15-62292e8-6487b15/darwin-arm64-11_0-toothrot
2021-10-06T19:02:31-62292e8-195945a/aix-ppc64
2021-10-06T18:44:56-d2e5035-195945a/aix-ppc64
2021-10-06T18:44:56-d2e5035-195945a/android-amd64-emu
2021-10-06T17:38:01-d2e5035-415f0a3/aix-ppc64
2021-10-06T17:38:01-d2e5035-415f0a3/android-amd64-emu
2021-10-06T16:46:09-d2e5035-3160571/android-amd64-emu
2021-10-06T16:11:20-d2e5035-aecf4b1/android-amd64-emu
2021-10-05T23:58:29-d2e5035-ac60900/android-amd64-emu
2021-10-05T23:36:43-d2e5035-e82ed0c/android-amd64-emu
2021-10-05T22:42:11-d2e5035-96fface/android-amd64-emu
2021-10-05T21:50:30-d2e5035-6ae3afa/aix-ppc64
2021-10-05T21:50:30-d2e5035-6ae3afa/android-amd64-emu
2021-10-05T21:25:51-d4b1ae0-6ae3afa/aix-ppc64
2021-10-05T21:25:51-d4b1ae0-6ae3afa/android-amd64-emu
2021-10-05T21:25:51-d4b1ae0-6ae3afa/darwin-arm64-11_0-toothrot
2021-10-05T20:57:56-d4b1ae0-55e7f7e/aix-ppc64
2021-10-05T20:53:02-d4b1ae0-695a59b/android-amd64-emu
2021-10-05T20:35:54-d4b1ae0-990c9c6/android-amd64-emu
2021-10-05T20:35:54-d4b1ae0-990c9c6/darwin-arm64-11_0-toothrot
2021-10-05T20:15:01-d4b1ae0-0b4d499/android-amd64-emu
2021-10-05T20:15:01-d4b1ae0-0b4d499/darwin-arm64-11_0-toothrot
2021-10-05T19:28:36-d4b1ae0-7ae83c8/android-amd64-emu
2021-02-19T00:41:47-5f4716e-5f2e24e/netbsd-amd64-9_0

@bcmills bcmills changed the title runtime,cmd/compile: apparent memory corruption in compress/flate test starting 2021-10-05 runtime,cmd/compile: apparent memory corruption in compress/flate and x/net/publicsuffix starting 2021-10-05 Oct 7, 2021
@cherrymui
Copy link
Member

Fixed in CL https://go-review.googlesource.com/c/go/+/354793

@bcmills bcmills mentioned this issue Aug 22, 2022
Closed
@golang golang locked and limited conversation to collaborators Oct 8, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
FrozenDueToAge NeedsInvestigation Someone must examine and confirm this is a valid issue and not a duplicate of an existing one. release-blocker
Projects
None yet
Milestone
Go1.18
Development

No branches or pull requests
4 participants
@jeremyfaller @bcmills @gopherbot @cherrymui