Re: [PATCH] sunrpc: fix comment in struct xdr_buf definition

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

 



On Mon, Feb 04, 2013 at 12:51:17PM -0500, Jeff Layton wrote:
> ...these pages aren't necessarily contiguous.

I guess you could say it was correct if you took "contiguous" to mean
that the xdr data they hold is meant to be contiguous.

But, yeah, at best I can't see how that comment would help anyone.

Applying absent any objections.

--b.

> 
> Signed-off-by: Jeff Layton <jlayton@xxxxxxxxxx>
> ---
>  include/linux/sunrpc/xdr.h | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/include/linux/sunrpc/xdr.h b/include/linux/sunrpc/xdr.h
> index 6398899..224d060 100644
> --- a/include/linux/sunrpc/xdr.h
> +++ b/include/linux/sunrpc/xdr.h
> @@ -56,7 +56,7 @@ struct xdr_buf {
>  	struct kvec	head[1],	/* RPC header + non-page data */
>  			tail[1];	/* Appended after page data */
>  
> -	struct page **	pages;		/* Array of contiguous pages */
> +	struct page **	pages;		/* Array of pages */
>  	unsigned int	page_base,	/* Start of page data */
>  			page_len,	/* Length of page data */
>  			flags;		/* Flags for data disposition */
> -- 
> 1.7.11.7
> 
--
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