Date: Mon, 4 Aug 2008 09:19:07 -0400 From: John Baldwin <jhb@freebsd.org> To: Konstantin Belousov <kib@freebsd.org> Cc: cvs-src@freebsd.org, src-committers@freebsd.org, cvs-all@freebsd.org Subject: Re: cvs commit: src/sys/kern link_elf.c link_elf_obj.c Message-ID: <200808040919.08314.jhb@freebsd.org> In-Reply-To: <200808031334.m73DY4fR030018@repoman.freebsd.org> References: <200808031334.m73DY4fR030018@repoman.freebsd.org>
next in thread | previous in thread | raw e-mail | index | archive | help
On Sunday 03 August 2008 09:33:45 am Konstantin Belousov wrote: > kib 2008-08-03 13:33:45 UTC > > FreeBSD src repository > > Modified files: > sys/kern link_elf.c link_elf_obj.c > Log: > SVN rev 181235 on 2008-08-03 13:33:45Z by kib > > Calling linker_load_dependencies() while holding the module' > vnode lock may cause a LOR between kld_sx lock and vnode lock. > linker_load_dependencies() drops kld_sx, and another thread may attempt > to load the same kld. > > Reported and tested by: pjd > MFC after: 1 week If it is safe to drop the vnode lock here, is there any reason we can't just drop it earlier and not bother reacquiring it? -- John Baldwin
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?200808040919.08314.jhb>