Subject: Re: CVS commit: src/sys/kern
To: Antti Kantee <pooka@cs.hut.fi>
From: Juergen Hannken-Illjes <hannken@eis.cs.tu-bs.de>
List: source-changes
Date: 06/07/2007 13:25:27
On Thu, Jun 07, 2007 at 01:34:08PM +0300, Antti Kantee wrote:
> 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?

Didn't want to break anything.  mountlist_slock is not commented and it looks
like it also protects mnt_iflag/mnt_unmounter.

-- 
Juergen Hannken-Illjes - hannken@eis.cs.tu-bs.de - TU Braunschweig (Germany)