]> Cypherpunks repositories - gostls13.git/commit
hash/crc32: fix incorrect checksums with avx512+race
authorKlaus Post <klauspost@gmail.com>
Mon, 28 Jul 2025 15:26:52 +0000 (15:26 +0000)
committerGopher Robot <gobot@golang.org>
Mon, 28 Jul 2025 15:51:49 +0000 (08:51 -0700)
commite81eac19d30f373496cd1d08ce2f89c0469a21fd
tree2eddb800d3388056e4c439604442fbb4a12b7d1a
parent6fbad4be75e7746512bbe55794694ed788ea5c5b
hash/crc32: fix incorrect checksums with avx512+race

CRC value loaded from incorrect register, which happened
to line up with actual register on default compile.
Therefore failures would only show up with -race.

Add regression test with longer payloads.

Fix regression from CL 689435.

Fixes #74767.

Change-Id: Ib63ddade998a5630297b285f3d566361f36f28e5
GitHub-Last-Rev: 5074012d2cbc8fdb78c1fb2ada72e3bd98c985c2
GitHub-Pull-Request: golang/go#74775
Reviewed-on: https://go-review.googlesource.com/c/go/+/690855
Reviewed-by: Keith Randall <khr@google.com>
Reviewed-by: Keith Randall <khr@golang.org>
Auto-Submit: Keith Randall <khr@golang.org>
Reviewed-by: David Chase <drchase@google.com>
LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
src/hash/crc32/crc32_amd64.s
src/hash/crc32/crc32_test.go