lkml.org 
[lkml]   [2009]   [Apr]   [16]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH -tip] remove the BKL: Replace BKL in mount/umount syscalls with a mutex

* Al Viro <viro@ZenIV.linux.org.uk> wrote:

> remount is potentially nastier, but then it *is* nasty. Again,
> it's only per-fs stuff, so the obvious first step is taking BKL
> down into the instances. It doesn't protect anything in VFS; all
> uses are fs internal, so that'll take review of individual
> filesystems.
>
> NOTE: do not assume that code in fs/foo/* is correct; "it doesn't
> take BKL elsewhere" does _not_ mean that we don't have races.
> IOW, the same review ought to look for such beasts and deal with
> them. Mere "oh, no BKL anywhere in that fs" is not enough to
> discard the ->remount_fs() instance.

what kind of races do you mean? Timing sensitive ones that are there
just are not easy to trigger with the BKL held?

Or actual locking interaction between that body of BKL code and all
other BKL using code?

Ingo


\
 
 \ /
  Last update: 2009-04-17 02:05    [W:0.059 / U:0.800 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site