Re: xfs_file_ioctl / xfs_freeze: BUG: warning at kernel/mutex-debug.c:80/debug_mutex_unlock()

From: Ingo Molnar
Date: Tue Jan 09 2007 - 01:46:01 EST



* Andrew Morton <akpm@xxxxxxxx> wrote:

> > Revert bd_mount_mutex back to a semaphore so that xfs_freeze -f
> > /mnt/newtest; xfs_freeze -u /mnt/newtest works safely and doesn't
> > produce lockdep warnings.
>
> Sad. The alternative would be to implement
> mutex_unlock_dont_warn_if_a_different_task_did_it(). Ingo? Possible?

i'd like to avoid it as much as i'd like to avoid having to add
spin_unlock_dont_warn_if_a_different_task_did_it(). Unlocking by a
different task is usually a sign of messy locking and bugs lurking. Is
it really true that XFS's use of bd_mount_mutex is safe and justified?

Ingo
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/