]> Cypherpunks repositories - gostls13.git/commit
runtime: clear upper half of sec in ARM walltime
authorMichael Pratt <mpratt@google.com>
Wed, 3 Nov 2021 20:28:51 +0000 (16:28 -0400)
committerMichael Pratt <mpratt@google.com>
Thu, 4 Nov 2021 17:07:48 +0000 (17:07 +0000)
commit577287728a8e8abbcbc6c96fdd49feb5a99ced0c
tree4cfc73dd07524359df49af834e8016c51ba9afd2
parent6e7b82a4bbd08e5724a29e204faf97342a0b6a15
runtime: clear upper half of sec in ARM walltime

https://golang.org/cl/246763 accidentally changed this to assign the old
value of vdsoPC to the upper half of sec as that is the last value in
R1.

The old value of vdsoPC is usually zero, so this typically works.
However, the reentrant case will have a non-zero value, resulting in a
bogus returned time. I am unsure if walltime is reachable from a signal
handler, so this may never occur.

Change-Id: I8562d6ec2a845fcffee9618d994b8ea57fbd199e
Reviewed-on: https://go-review.googlesource.com/c/go/+/361159
Trust: Michael Pratt <mpratt@google.com>
Run-TryBot: Michael Pratt <mpratt@google.com>
TryBot-Result: Go Bot <gobot@golang.org>
Reviewed-by: Cherry Mui <cherryyz@google.com>
src/runtime/sys_linux_arm.s