Re: CVS commit: src/sys

["J. Hannken-Illjes" <hannken%eis.cs.tu-bs.de@localhost>]

On Apr 27, 2013, at 12:27 AM, Michael van Elst <mlelstv%netbsd.org@localhost> 
wrote:

> Module Name:  src
> Committed By: mlelstv
> Date:         Fri Apr 26 22:27:17 UTC 2013
> 
> Modified Files:
>       src/sys/kern: vfs_mount.c
>       src/sys/sys: mount.h
> 
> Log Message:
> Correct umount semantics to return EBUSY when a filesystem is busy
> instead of failing filesystem operations with EBUSY when attempting
> an umount.
> This fixes kern/38141.

Who did the review or where has it been discussed?

We now get a deadlock between mountlist_lock and mnt_unmounting,
seen between dounmount() and do_sys_sync() for example.

The lock order is mountlist_lock->mnt_unmounting, but dounmount()
locks in reverse direction.

Please fix or revert.

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