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)