]> Cypherpunks repositories - gostls13.git/commit
runtime: eliminate poisonStack checks
authorAustin Clements <austin@google.com>
Fri, 17 Jun 2016 15:07:56 +0000 (11:07 -0400)
committerAustin Clements <austin@google.com>
Fri, 17 Jun 2016 15:18:39 +0000 (15:18 +0000)
commit9e8fa1e99c2003cee53a6630aea9d8a3627492ab
tree75c9c834bb67f8196533281977a83b07fce60ad0
parentfca9fc52c831ab6af56e30f8c48062a99ded2580
runtime: eliminate poisonStack checks

We haven't used poisonStack since we switched to 1-bit stack maps
(4d0f3a1), but the checks are still there. However, nothing prevents
us from genuinely allocating an object at this address on 32-bit and
causing the runtime to crash claiming that it's found a bad pointer.

Since we're not using poisonStack anyway, just pull it out.

Fixes #15831.

Change-Id: Ia6ef604675b8433f75045e369f5acd4644a5bb38
Reviewed-on: https://go-review.googlesource.com/24211
Run-TryBot: Austin Clements <austin@google.com>
Reviewed-by: Keith Randall <khr@golang.org>
src/runtime/mbarrier.go
src/runtime/stack.go