diff options
author | Ian Lance Taylor <iant@golang.org> | 2023-05-16 16:03:38 -0700 |
---|---|---|
committer | Gopher Robot <gobot@golang.org> | 2023-05-17 20:12:00 +0000 |
commit | a1674f3ee30bf46d22fcac115529ce830f8c9ac9 (patch) | |
tree | dc71025bd11a51a2d2233b6bb84314578f59f1be /src | |
parent | 8c822ac7e8e136f079359be1772c1ba3f06399e6 (diff) | |
download | go-git-a1674f3ee30bf46d22fcac115529ce830f8c9ac9.tar.gz |
runtime: publish netpoll info after incrementing fdseq
I think there is a theoretical possibility of a mistake before this CL.
pollCache.free would increment fdseq, but would not update atomicInfo.
The epoll code could compare to fdseq before the increment, but suspend
before calling setEventErr. The pollCache could get reallocated,
and pollOpen could clear eventErr. Then the setEventErr could continue
and set it again. Then pollOpen could call publishInfo.
Avoid this rather remote possibility by calling publishInfo after
incrementing fdseq. That ensures that delayed setEventErr will not
modify the eventErr flag.
Fixes #60133
Change-Id: I69e336535312544690821c9fd53f3023ff15b80c
Reviewed-on: https://go-review.googlesource.com/c/go/+/495297
Run-TryBot: Ian Lance Taylor <iant@golang.org>
TryBot-Result: Gopher Robot <gobot@golang.org>
Reviewed-by: Ian Lance Taylor <iant@google.com>
Run-TryBot: Ian Lance Taylor <iant@google.com>
Reviewed-by: Bryan Mills <bcmills@google.com>
Auto-Submit: Ian Lance Taylor <iant@google.com>
Diffstat (limited to 'src')
-rw-r--r-- | src/runtime/netpoll.go | 8 |
1 files changed, 8 insertions, 0 deletions
diff --git a/src/runtime/netpoll.go b/src/runtime/netpoll.go index b1b3766e11..a2b0be2261 100644 --- a/src/runtime/netpoll.go +++ b/src/runtime/netpoll.go @@ -287,12 +287,20 @@ func poll_runtime_pollClose(pd *pollDesc) { } func (c *pollCache) free(pd *pollDesc) { + // pd can't be shared here, but lock anyhow because + // that's what publishInfo documents. + lock(&pd.lock) + // Increment the fdseq field, so that any currently // running netpoll calls will not mark pd as ready. fdseq := pd.fdseq.Load() fdseq = (fdseq + 1) & (1<<taggedPointerBits - 1) pd.fdseq.Store(fdseq) + pd.publishInfo() + + unlock(&pd.lock) + lock(&c.lock) pd.link = c.first c.first = pd |