[lustre-devel] [PATCH] lustre: accept suffix when parsing max_pages_per_rpc

[James Simmons]

> upstream lustre allows size suffixes (K, M, G, etc) for
> max_pages_per_rpc.  We should too.
> 
> Signed-off-by: NeilBrown <neilb at suse.com>
> ---
> 
> This fixes test 101g.

Reviewed-by: James Simmons <jsimmons at infradead.org>
 
>  drivers/staging/lustre/lustre/obdclass/lprocfs_status.c | 7 ++++---
>  1 file changed, 4 insertions(+), 3 deletions(-)
> 
> diff --git a/drivers/staging/lustre/lustre/obdclass/lprocfs_status.c b/drivers/staging/lustre/lustre/obdclass/lprocfs_status.c
> index edac88440b41..feba2ef5a3bc 100644
> --- a/drivers/staging/lustre/lustre/obdclass/lprocfs_status.c
> +++ b/drivers/staging/lustre/lustre/obdclass/lprocfs_status.c
> @@ -1868,11 +1868,12 @@ ssize_t max_pages_per_rpc_store(struct kobject *kobj, struct attribute *attr,
>  	struct obd_connect_data *ocd;
>  	unsigned long long val;
>  	int chunk_mask;
> +	char *endp;
>  	int rc;
>  
> -	rc = kstrtoull(buffer, 10, &val);
> -	if (rc)
> -		return rc;
> +	val = memparse(buffer, &endp);
> +	if (*endp)
> +		return -EINVAL;
>  
>  	/* if the max_pages is specified in bytes, convert to pages */
>  	if (val >= ONE_MB_BRW_SIZE)
> -- 
> 2.14.0.rc0.dirty
> 
>