Date: Sat, 15 Jul 2006 00:43:37 -0400 From: John Baldwin <jhb@freebsd.org> To: SANETO Takanori <sanewo@ba2.so-net.ne.jp> Cc: bp@freebsd.org, freebsd-current@freebsd.org, Jiawei Ye <leafy7382@gmail.com> Subject: Re: lockmgr: thread 0xc21e2780 unlocking unheld lock Message-ID: <200607150043.37683.jhb@freebsd.org> In-Reply-To: <44B85EBA.3080207@ba2.so-net.ne.jp> References: <c21e92e20607110629t490a7abv1cd6f4eb84f19e66@mail.gmail.com> <200607111101.47747.jhb@freebsd.org> <44B85EBA.3080207@ba2.so-net.ne.jp>
next in thread | previous in thread | raw e-mail | index | archive | help
On Friday 14 July 2006 23:19, SANETO Takanori wrote: > I think that "==" should be "!=" in the patched code. Yeah, I fixed that in an updated patch to the OP that includes another fix for a missing lock in smbfs_unmount(). > John Baldwin wrote: > > On Tuesday 11 July 2006 09:29, Jiawei Ye wrote: > >> I was doing "sudo mount_smbfs //user@server/share /mnt" when the > >> following msg appeared. smbfs.ko was not loaded. > > > > Does building with INVARIANTS fix it? If so, try this patch (it looks > > like it has been broken since 1.1 of the smbfs code): > > > > Index: smb_conn.c > > =================================================================== > > RCS file: /usr/cvs/src/sys/netsmb/smb_conn.c,v > > retrieving revision 1.16 > > diff -u -r1.16 smb_conn.c > > --- smb_conn.c 31 Oct 2005 15:41:27 -0000 1.16 > > +++ smb_conn.c 11 Jul 2006 15:01:10 -0000 > > @@ -233,7 +233,8 @@ > > lockinit(&cp->co_lock, PZERO, objname, 0, 0); > > cp->co_level = level; > > cp->co_usecount = 1; > > - KASSERT(smb_co_lock(cp, LK_EXCLUSIVE, td) == 0, ("smb_co_init: lock > > failed")); > > + if (smb_co_lock(cp, LK_EXCLUSIVE, td) == 0) > > + panic("smb_co_init: lock failed")); > > } > > > > static void > > > > -- John Baldwin
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?200607150043.37683.jhb>