Re: [PATCH 2/3] nfs: rename members of nfs_pgio_data

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On May 1, 2014, at 15:14, Weston Andros Adamson <dros@xxxxxxxxxxxxxxx> wrote:

> diff --git a/include/linux/nfs_xdr.h b/include/linux/nfs_xdr.h
> index a7ccc8c..8564539 100644
> --- a/include/linux/nfs_xdr.h
> +++ b/include/linux/nfs_xdr.h
> @@ -1261,13 +1261,13 @@ struct nfs_pgio_data {
> 	struct list_head	list;
> 	struct rpc_task		task;
> 	struct nfs_fattr	fattr;
> -	struct nfs_writeverf	verf;		/* Used for writes */
> +	struct nfs_writeverf	rpc_verf;	/* Used for writes */

Can we just make this “writeverf"? It isn’t really an rpc object, so the ‘rpc_’ prefix seems a little misleading.

> 	struct nfs_pgio_args	args;		/* argument struct */
> 	struct nfs_pgio_res	res;		/* result struct */
> 	unsigned long		timestamp;	/* For lease renewal */
> 	int (*pgio_done_cb)(struct rpc_task *task, struct nfs_pgio_data *data);
> 	__u64			mds_offset;	/* Filelayout dense stripe */
> -	struct nfs_page_array	pages;
> +	struct nfs_page_array	page_array;
> 	struct nfs_client	*ds_clp;	/* pNFS data server */
> 	int			ds_idx;		/* ds index if ds_clp is set */
> };
> -- 
> 1.8.5.2 (Apple Git-48)
> 

_________________________________
Trond Myklebust
Linux NFS client maintainer, PrimaryData
trond.myklebust@xxxxxxxxxxxxxxx

--
To unsubscribe from this list: send the line "unsubscribe linux-nfs" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Linux Filesystem Development]     [Linux USB Development]     [Linux Media Development]     [Video for Linux]     [Linux NILFS]     [Linux Audio Users]     [Yosemite Info]     [Linux SCSI]

  Powered by Linux