Re: [PATCH 3/3] pnfs: use size_t for LAYOUTGET response pages count

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

 



On Mon, 2012-07-23 at 13:05 +0300, Idan Kedar wrote:
> Signed-off-by: Idan Kedar <idank@xxxxxxxxxx>
> Signed-off-by: Benny Halevy <bhalevy@xxxxxxxxxx>
> ---
>  fs/nfs/nfs4proc.c |   10 +++++-----
>  1 files changed, 5 insertions(+), 5 deletions(-)
> 
> diff --git a/fs/nfs/nfs4proc.c b/fs/nfs/nfs4proc.c
> index 3db9ce7..056c3f0 100644
> --- a/fs/nfs/nfs4proc.c
> +++ b/fs/nfs/nfs4proc.c
> @@ -6164,13 +6164,13 @@ static void nfs4_layoutget_done(struct rpc_task *task, void *calldata)
>  	dprintk("<-- %s\n", __func__);
>  }
>  
> -static u32 max_response_pages(struct nfs_server *server)
> +static size_t max_response_pages(struct nfs_server *server)
>  {
>  	u32 max_resp_sz = server->nfs_client->cl_session->fc_attrs.max_resp_sz;
>  	return nfs_page_array_len(0, max_resp_sz);
>  }
>  
> -static void free_pagevec(struct page **pages, u32 size)
> +static void free_pagevec(struct page **pages, size_t size)
>  {
>  	int i;
>  
> @@ -6185,7 +6185,7 @@ static void free_pagevec(struct page **pages, u32 size)
>  	kfree(pages);
>  }
>  
> -static struct page **alloc_pagevec(u32 size, gfp_t gfp_flags)
> +static struct page **alloc_pagevec(size_t size, gfp_t gfp_flags)
>  {
>  	struct page **pages;
>  	int i;
> @@ -6212,7 +6212,7 @@ static void nfs4_layoutget_release(void *calldata)
>  {
>  	struct nfs4_layoutget *lgp = calldata;
>  	struct nfs_server *server = NFS_SERVER(lgp->args.inode);
> -	u32 max_pages = max_response_pages(server);
> +	size_t max_pages = max_response_pages(server);
>  
>  	dprintk("--> %s\n", __func__);
>  	free_pagevec(lgp->args.layout.pages, max_pages);
> @@ -6230,7 +6230,7 @@ static const struct rpc_call_ops nfs4_layoutget_call_ops = {
>  void nfs4_proc_layoutget(struct nfs4_layoutget *lgp, gfp_t gfp_flags)
>  {
>  	struct nfs_server *server = NFS_SERVER(lgp->args.inode);
> -	u32 max_pages = max_response_pages(server);
> +	size_t max_pages = max_response_pages(server);
>  	struct rpc_task *task;
>  	struct rpc_message msg = {
>  		.rpc_proc = &nfs4_procedures[NFSPROC4_CLNT_LAYOUTGET],

There is no reason why you can't fold this patch into 1/3.

-- 
Trond Myklebust
Linux NFS client maintainer

NetApp
Trond.Myklebust@xxxxxxxxxx
www.netapp.com

��.n��������+%������w��{.n�����{��w���jg��������ݢj����G�������j:+v���w�m������w�������h�����٥



[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