]> git.michaelhowe.org Git - packages/o/openafs.git/commit
LINUX: Avoid infinite d_invalidate loop
authorAndrew Deason <adeason@sinenomine.net>
Thu, 13 Dec 2012 22:36:20 +0000 (16:36 -0600)
committerStephan Wiesand <stephan.wiesand@desy.de>
Fri, 21 Dec 2012 18:43:12 +0000 (10:43 -0800)
commitf69aaf6914096077108b534aff8fd147cd6a65ae
tree68792704352ca17241e364e793ea123dcc347712
parent5f7cf5fc22ac3ca62175c69c1a25139dde4b22b0
LINUX: Avoid infinite d_invalidate loop

If a reference is grabbed after we d_invalidate() a dentry, but before
we dput() that dentry, the dentry can still be on the alias list for
its inode. This will cause us to loop forever, since we'll get the
same dentry back from d_find_alias, and d_invalidate() will return
success, since it is a no-op.

Avoid this by just limiting the number of times we loop. Leaving
dentry aliases around is non-critical as long as we clear some each
time.

This change is not applicable to master, which will get a more robust
fix for this issue. This change was discussed during the 19 Dec 2012
release-team meeting.

Change-Id: Ief22ae9f8f6de62518604383c421772c25c5dc26
Reviewed-on: http://gerrit.openafs.org/8795
Reviewed-by: Paul Smeddle <paul.smeddle@gmail.com>
Reviewed-by: Stephan Wiesand <stephan.wiesand@desy.de>
Tested-by: BuildBot <buildbot@rampaginggeek.com>
src/afs/LINUX/osi_vnodeops.c
src/afs/LINUX24/osi_vnodeops.c