Re: VFS locking changes in 2.5.7

Alexander Viro (viro@math.psu.edu)
Sun, 31 Mar 2002 15:58:44 -0500 (EST)


On Sun, 31 Mar 2002, Richard Gooch wrote:

> Hi, Al. I've noticed that you've made changes to the locking rules
> for the VFS (for example, the inode lookup() method no longer has the
> BKL).
>
> I also note that you've thrown calls to lock_kernel() and
> unlock_kernel() into the devfs code. Did you check each place you
> added the BKL to devfs that it was needed, or was this just a blind
> global operation? At first glance, it appears that many of the places
> where the BKL was inserted are not needed.

See Documentation/filesystems/porting

BKL had been shifted inside several methods, so that filesystem code itself
had the same locking as it used to (i.e. code that used to be under BKL
stayed under it). If your code doesn't need BKL - feel free to shrink the
area, but keep in mind that it used to be under BKL.

I didn't _add_ BKL - neither in devfs nor anywhere else. lock_kernel() is
the boundary of the protected area and all that had happened is that this
area had slightly shrunken, so its boundaries are inside the method instead
of being around its caller.

Again, further shrinking is up to maintainers of the filesystems.

While we are at it, the changes in question had happened quite a couple
of months ago - none of them in 2.5.7...

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