xfs
[Top] [All Lists]

Re: 2.6.2-rc2 nfsd+xfs spins in i_size_read()

To: Christoph Hellwig <hch@xxxxxxxxxxxxx>, Miquel van Smoorenburg <miquels@xxxxxxxxxx>, Andrew Morton <akpm@xxxxxxxx>, Nathan Scott <nathans@xxxxxxx>, linux-kernel@xxxxxxxxxxxxxxx, linux-xfs@xxxxxxxxxxx
Subject: Re: 2.6.2-rc2 nfsd+xfs spins in i_size_read()
From: David Weinehall <tao@xxxxxxxxxx>
Date: Wed, 4 Feb 2004 01:06:23 +0100
In-reply-to: <20040204000315.A12127@xxxxxxxxxxxxx>
Mail-followup-to: Christoph Hellwig <hch@xxxxxxxxxxxxx>, Miquel van Smoorenburg <miquels@xxxxxxxxxx>, Andrew Morton <akpm@xxxxxxxx>, Nathan Scott <nathans@xxxxxxx>, linux-kernel@xxxxxxxxxxxxxxx, linux-xfs@xxxxxxxxxxx
References: <bv8qr7$m2v$1@xxxxxxxxxxxxxxx> <20040129063009.GD2474@frodo> <bv8qr7$m2v$1@xxxxxxxxxxxxxxx> <20040128222521.75a7d74f.akpm@xxxxxxxx> <20040129063009.GD2474@frodo> <20040129232033.GA10541@xxxxxxxxxx> <20040204000315.A12127@xxxxxxxxxxxxx>
Sender: linux-xfs-bounce@xxxxxxxxxxx
User-agent: Mutt/1.4.1i
On Wed, Feb 04, 2004 at 12:03:15AM +0000, Christoph Hellwig wrote:
> Okay, what about this little patch?:
> 
> 
> Index: fs/xfs/linux-2.6/xfs_iops.c
> ===================================================================
> RCS file: /cvs/linux-2.6-xfs/fs/xfs/linux-2.6/xfs_iops.c,v
> retrieving revision 1.212
> diff -u -p -r1.212 xfs_iops.c
> --- fs/xfs/linux-2.6/xfs_iops.c       12 Dec 2003 04:17:52 -0000      1.212
> +++ fs/xfs/linux-2.6/xfs_iops.c       3 Feb 2004 23:56:17 -0000
> @@ -80,11 +80,15 @@ validate_fields(
>       vattr_t         va;
>       int             error;
>  
> -     va.va_mask = XFS_AT_NLINK|XFS_AT_SIZE|XFS_AT_NBLOCKS;
> +     va.va_mask = XFS_AT_NLINK|XFS_AT_SIZE|XFS_AT_NBLOCKS|XFS_AT_SIZE;

Huh? XFS_AT_SIZE twice?!

[snip]


Regards: David Weinehall
-- 
 /) David Weinehall <tao@xxxxxxxxxx> /) Northern lights wander      (\
//  Maintainer of the v2.0 kernel   //  Dance across the winter sky //
\)  http://www.acc.umu.se/~tao/    (/   Full colour fire           (/


<Prev in Thread] Current Thread [Next in Thread>