xfs
[Top] [All Lists]

Re: [PATCH] xfs: remove a dmapi leftover

To: Christoph Hellwig <hch@xxxxxxxxxxxxx>
Subject: Re: [PATCH] xfs: remove a dmapi leftover
From: Alex Elder <aelder@xxxxxxx>
Date: Fri, 09 Jul 2010 07:52:39 -0500
Cc: xfs@xxxxxxxxxxx
In-reply-to: <20100703092116.GA27669@xxxxxxxxxxxxx>
References: <20100703092116.GA27669@xxxxxxxxxxxxx>
Reply-to: aelder@xxxxxxx
On Sat, 2010-07-03 at 05:21 -0400, Christoph Hellwig wrote:
> The open_exec file operation is only added by the external dmapi patch.
> 
> Signed-off-by: Christoph Hellwig <hch@xxxxxx>

Looks good.

Reviewed-by: Alex Elder <aelder@xxxxxxx>

> Index: xfs/fs/xfs/linux-2.6/xfs_file.c
> ===================================================================
> --- xfs.orig/fs/xfs/linux-2.6/xfs_file.c      2010-07-03 10:54:07.323263955 
> +0200
> +++ xfs/fs/xfs/linux-2.6/xfs_file.c   2010-07-03 10:54:11.636128320 +0200
> @@ -921,9 +921,6 @@ const struct file_operations xfs_file_op
>       .open           = xfs_file_open,
>       .release        = xfs_file_release,
>       .fsync          = xfs_file_fsync,
> -#ifdef HAVE_FOP_OPEN_EXEC
> -     .open_exec      = xfs_file_open_exec,
> -#endif
>  };
>  
>  const struct file_operations xfs_dir_file_operations = {
> 
> _______________________________________________
> xfs mailing list
> xfs@xxxxxxxxxxx
> http://oss.sgi.com/mailman/listinfo/xfs



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