On Apr 27, 2013, at 12:27 AM, Michael van Elst <mlel...@netbsd.org> 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 - hann...@eis.cs.tu-bs.de - TU Braunschweig (Germany)

Reply via email to