Subject: Re: CVS commit: src/sys/kern
To: Juergen Hannken-Illjes <hannken@netbsd.org>
From: Antti Kantee <pooka@cs.hut.fi>
List: source-changes
Date: 06/07/2007 13:34:08
On Thu Jun 07 2007 at 10:03:12 +0000, Juergen Hannken-Illjes wrote:
> 
> Module Name:	src
> Committed By:	hannken
> Date:		Thu Jun  7 10:03:12 UTC 2007
> 
> Modified Files:
> 	src/sys/kern: vfs_syscalls.c
> 
> Log Message:
> Dounmount(): rearrange mountlist_slock.  vfs_allocate_syncvnode() may sleep
> getting a new vnode so it must not be called with this simple_lock taken.
> 
> Fixes PR #36395

Why is mountlist_slock needed in the error branch at all?

-- 
Antti Kantee <pooka@iki.fi>                     Of course he runs NetBSD
http://www.iki.fi/pooka/                          http://www.NetBSD.org/
    "la qualité la plus indispensable du cuisinier est l'exactitude"