Steve Lord wrote:
>
> On Mon, 2002-10-14 at 01:34, Andrew Morton wrote:
>
> >
> > The number of filsystems which do not take the bkl in truncate/setattr
> > is in fact quite small. Here's the patch which removes all doubt:
> >
> >
> >
> >
> > fs/affs/file.c | 13 ++++++++-----
> > fs/attr.c | 2 --
> > fs/cifs/inode.c | 7 ++++++-
> > fs/jfs/file.c | 3 +++
> > fs/reiserfs/file.c | 2 ++
> > fs/smbfs/proc.c | 18 +++++++++++++++---
> > fs/sysv/itree.c | 6 +++++-
> > fs/xfs/linux/xfs_iops.c | 11 +++++++++--
> > 8 files changed, 48 insertions(+), 14 deletions(-)
>
> XFS deliberately does not take the BKL - anywhere. Our setattr
> code is doing its own locking. You just added the BKL to a
> bunch of xfs operations which do not need it. Now, vmtruncate
> may need it, itself, but if vmtruncate does not, then the xfs
> callout from vmtruncate certainly does not.
>
Sorry, but that is standard "bkl migration" methodology. You had it
before, so you get it after. It is not my role to change XFS locking.
Anyway, I don't think these patches are going anywhere.
-
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/
This archive was generated by hypermail 2b29 : Tue Oct 15 2002 - 22:00:50 EST