Re: [PATCH] Shift BKL into ->statfs()

Alexander Viro (viro@math.psu.edu)
Wed, 3 Jul 2002 02:25:02 -0400 (EDT)


On Tue, 2 Jul 2002, Paul Menage wrote:

>
> This patch removes BKL protection from the invocation of the
> super_operations ->statfs() method, and shifts it into the filesystems
> where necessary. Any out-of-tree filesystems may need to take the BKL in
> their statfs() methods if they were relying on it for synchronisation.
>
> All ->statfs() implementations have been modified to take the BKL,
> except for the following, which don't reference any external mutable
> data:
>
> simple_statfs
> isofs_statfs
> ncp_statfs
> cramfs_statfs
> romfs_statfs
>
> Additionally, capifs is changed to use simple_statfs rather than its
> own home-grown version.

Patch makes sense. However

1) please, let's keep Documentation/filesystems/porting more or less
in chronological order. I.e. somebody who wants to check what's new should
be able to check the end of the list, not hunt through the middle for changes.
IOW, I'd rather see duplicate sections (or even "->statfs() had lost BKL; see
section on ->write_super()")

2) ext2, shmem, FAT, minix and sysv ->statfs() don't need BKL.

3) efs and vxfs are read-only.

-
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/