[OpenAFS-devel] OpenAFS for FreeBSD 5.2 patch
Jim Rees
rees@umich.edu
Fri, 19 Mar 2004 14:12:10 -0500
panic
lockmgr
vop_stdlock
vop_defaultop
vn_lock
osi_VM_TryToSmush
afs_TryToSmush
afsremove
afs_remove
...
It looks like osi_VM_TryToSmush used to only acquire the vnode interlock,
and Garrett went to great lengths to change it so that it locks the vnode
instead. In this case that's obviously wrong, since afsremove is entered
with the vnode locked.
My guess is that there is some other path that enters osi_VM_TryToSmush
without the vnode locked, and Garrett decided it should be.
I'm reluctant to change this without understanding why Garrett made this
change. Unfortunately he's busy with another project right now and probably
won't have time to look at this any time soon.