hi, > On Fri Dec 17 2010 at 22:34:04 +0000, 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
