On Tue, Oct 23, 2007 at 09:27:56PM +0200, Olli Hauer wrote:
quoted text
> lock order reversal:
>  1st 0xc4cea568 struct mount mtx (struct mount mtx) @=20
>  /usr/src/sys/modules/zfs/../../compat/opensolaris/kern/opensolaris_vfs.c=
:209
quoted text
>  2nd 0xc3ee9010 sleep mtxpool (sleep mtxpool) @=20
>  /usr/src/sys/kern/kern_resource.c:1266
> KDB: stack backtrace:
> db_trace_self_wrapper(c0a9c175,e7318734,c078510e,c0a9e63c,c3ee9010,...) a=
t=20
quoted text
> db_trace_self_wrapper+0x26
> kdb_backtrace(c0a9e63c,c3ee9010,c0a982df,c0a982df,c0a98e54,...) at=20
> kdb_backtrace+0x29
> witness_checkorder(c3ee9010,9,c0a98e54,4f2,38,...) at=20
> witness_checkorder+0x6de
> _mtx_lock_flags(c3ee9010,0,c0a98e54,4f2,c4c32d00,...) at=20
> _mtx_lock_flags+0xbc
> uifree(c3f08c20,c4967220,c4cea538,e73187d4,c431b9df,...) at uifree+0x2d
> crfree(c4c32d00,0,c439446c,d1,c3,...) at crfree+0x54
> domount(c48f8840,c4967220,c43995fb,c469e260,e7318810,...) at domount+0x20f
> zfsctl_snapdir_lookup(e7318aa0,e7318aa0,c48f8840,2,c4967330,...) at=20
> zfsctl_snapdir_lookup+0x362
> VOP_LOOKUP_APV(c439d5e0,e7318aa0,c48f8840,c0aa409d,19b,...) at=20
> VOP_LOOKUP_APV+0xa5
> lookup(e7318b48,c0aa409d,c6,bf,c4cd772c,...) at lookup+0x58e
> namei(e7318b48,c48f8840,c0bb51d4,c48f8840,e7318b4c,...) at namei+0x34b
> kern_lstat(c48f8840,282111b8,0,e7318c18,c0aa5520,...) at kern_lstat+0x4f
> lstat(c48f8840,e7318cfc,8,c0a9f13d,c0b469d0,...) at lstat+0x2f
> syscall(e7318d38) at syscall+0x2b3
> Xint0x80_syscall() at Xint0x80_syscall+0x20
> --- syscall (190, FreeBSD ELF32, lstat), eip =3D 0x2819c48b, esp =3D=20

Revert the previous patch, refetch and try again. The new one also
eliminates this LOR:

	http://people.freebsd.org/~pjd/patches/opensolaris_vfs.c.patch

--=20
Pawel Jakub Dawidek                       http://www.wheel.pl
pjd@FreeBSD.org                           http://www.FreeBSD.org
FreeBSD committer                         Am I Evil? Yes, I Am!