]> git.michaelhowe.org Git - packages/o/openafs.git/commit
Prevent VLRUQ race in ShakeLooseVCaches
authorSimon Wilkinson <sxw@inf.ed.ac.uk>
Wed, 4 Nov 2009 23:40:39 +0000 (23:40 +0000)
committerDerrick Brashear <shadow|account-1000005@unknown>
Thu, 17 Dec 2009 14:00:14 +0000 (06:00 -0800)
commit893233f4574e9c50dda930ace0403ecb66f8bdef
tree6d8633e0f957f203fc1166000a564b6b3c8ec5f3
parent4abf9ae08fbb93d98ae5ebda850683ae06e5ab56
Prevent VLRUQ race in ShakeLooseVCaches

When ShakeLooseVCaches is called from afs_Daemon, the xvcache lock
is not held. This means that if the GLOCK is dropped for any reason
(for example, whilst purging the dentry cache), then
ShakeLooseVCaches can be raced, end we can end up attempting to
flush the same vcache twice.

The symptoms of this in Linux are that we oops in clear_inode.

Get the xvcache lock in afs_Daemon(), before calling
ShakeLooseVCaches. Also, remove the conditional GLOCK code from
that function. If we don't have the GLOCK on entry, then we're really
in trouble (and both code paths - afs_Daemon and afs_NewVCache should
get the GLOCK for us, anyway)

FIXES 125589

(cherry picked from 95770bf95ad766207252ea0c2d2b5ab2415c896f)
Change-Id: I3fe5b41a661cd162ec73c51492925ad87c6d4c13
Reviewed-on: http://gerrit.openafs.org/781
Reviewed-by: Marc Dionne <marc.c.dionne@gmail.com>
Tested-by: Derrick Brashear <shadow@dementia.org>
Reviewed-by: Derrick Brashear <shadow@dementia.org>
Reviewed-on: http://gerrit.openafs.org/986
src/afs/afs_daemons.c
src/afs/afs_vcache.c