hi,
On Fri Dec 17 2010 at 22:34:04 +, YAMAMOTO Takashi wrote:
Module Name: src
Committed By:yamt
Date:Fri Dec 17 22:34:04 UTC 2010
Modified Files:
src/sys/kern: vfs_lookup.c
Log Message:
- lookup_once: when crossing a mount point, don't keep the parent vnode
locked.
ie. don't lock a vnode while holding another vnode which belongs to a
different filesystem. otherwise we propagate slowness (or deadness) of a
filesystem to another via vnode lock chain.
- lookup_parsepath: don't alter vnode states. let the caller do it instead.
- add comments and assertions.
Hi, this breaks the state-dp != ndp-ni_dvp invariant in at least 3
places in vfs_lookup.c:
http://www.gson.org/netbsd/bugs/build/build/2010.12.18.09.26.57/test.html#failed-tcs-summary
Can you have a look?
i reverted the change as i currently have no time to take a look.
i'll take a look later. thanks.
YAMAMOTO Takashi
--
dld karot toivorikkauttas, kyl rdtei ja lumpui piisaa