OpenAFS Master Repository branch, openafs-stable-1_8_x, updated. openafs-stable-1_8_9-70-g1da00df
Gerrit Code Review
gerrit@openafs.org
Thu, 30 Mar 2023 11:26:39 -0400
The following commit has been merged in the openafs-stable-1_8_x branch:
commit 1da00dfa9446c4c881e8659ec48a5e00262828ca
Author: Andrew Deason <adeason@sinenomine.net>
Date: Mon Apr 12 18:21:23 2021 -0500
afs: Assert avc->lock is held in afs_IAS_once
Commit 3be5880d (afs: Avoid panics in afs_InvalidateAllSegments) added
an assert to check that vcache->lock is write-locked before we call
afs_InvalidateAllSegments_once from a background operation.
However, afs_InvalidateAllSegments_once should always be called with
vcache->lock write-locked; there's nothing specific about the
backgrounded call that requires this. So to make sure we catch all
cases, move this assert to afs_InvalidateAllSegments_once itself.
Also remove the conditional check for WriteLocked(&avc->lock) in here,
since clearly avc->lock must be write-locked (and actually is, since
change Ic309e4006bf47bcb38fa2b53bf103e0c645a856d "afs: write-lock
vcache->lock in afs_InactiveVCache").
Add some comments to this function while we're here, to more clearly
indicate what locks are needed.
Reviewed-on: https://gerrit.openafs.org/14592
Tested-by: BuildBot <buildbot@rampaginggeek.com>
Reviewed-by: Mark Vitale <mvitale@sinenomine.net>
Reviewed-by: Benjamin Kaduk <kaduk@mit.edu>
(cherry picked from commit 72223e0958c2d7cddd968970547dd73fc3cc1351)
Change-Id: I9acf5599a8ef1e724a00cf362695fe34ca87c0bf
Reviewed-on: https://gerrit.openafs.org/15331
Tested-by: BuildBot <buildbot@rampaginggeek.com>
Reviewed-by: Mark Vitale <mvitale@sinenomine.net>
Reviewed-by: Michael Meffie <mmeffie@sinenomine.net>
Reviewed-by: Andrew Deason <adeason@sinenomine.net>
Reviewed-by: Stephan Wiesand <stephan.wiesand@desy.de>
src/afs/afs_daemons.c | 1 -
src/afs/afs_segments.c | 17 +++++++++++++++--
2 files changed, 15 insertions(+), 3 deletions(-)
--
OpenAFS Master Repository