Re: [PATCH -mm] rescue large xfs preferred iosize from the inode diet patch

[David Chinner <dgc@xxxxxxx>]

On Thu, Sep 21, 2006 at 05:33:24PM -0500, Eric Sandeen wrote:
> The inode diet patch in -mm unhooked xfs_preferred_iosize from the stat call:
....
> Signed-off-by: Eric Sandeen <sandeen@xxxxxxxxxxx>
> 
> XFS guys, does this look ok?
>
> Index: linux-2.6.18/fs/xfs/linux-2.6/xfs_iops.c
> ===================================================================
> --- linux-2.6.18.orig/fs/xfs/linux-2.6/xfs_iops.c
> +++ linux-2.6.18/fs/xfs/linux-2.6/xfs_iops.c
> @@ -623,12 +623,16 @@ xfs_vn_getattr(
>  {
>       struct inode    *inode = dentry->d_inode;
>       bhv_vnode_t     *vp = vn_from_inode(inode);
> +     xfs_inode_t     *ip;
>       int             error = 0;
>  
>       if (unlikely(vp->v_flag & VMODIFIED))
>               error = vn_revalidate(vp);
> -     if (!error)
> +     if (!error) {
>               generic_fillattr(inode, stat);
> +             ip = xfs_vtoi(vp);
> +             stat->blksize = xfs_preferred_iosize(ip->i_mount);
> +     }
>       return -error;
>  }

ACK. Looks good, Eric. Good catch.

Cheers,

Dave.
-- 
Dave Chinner
Principal Engineer
SGI Australian Software Group