Re: [PATCH] nfsd: Check the nfsd support version when enable or disable the minorversion

[fanchaoting <fanchaoting@cn.fujitsu.com>]

bfields@fieldses.org 写道:
> On Wed, Jan 23, 2013 at 05:58:18PM +0800, fanchaoting wrote:
>> when enable or disable the minorversion throw /proc/fs/nfsd/versions,it
>> should check the support version.
>>
>> for example
>> if we do like following:
>>
>> echo "+2 +3 +4 +7.1">/proc/fs/nfsd/versions
>> echo "+2 +3 +4 -7.1">/proc/fs/nfsd/versions
>> it will enable or disable nfs4.1, but now the number nfs7 didn't support.
> 
> Actually, who knows whether a hypothetical NFSv7 would even support
> minor versioning.  Let's just do this:
> 
> diff --git a/fs/nfsd/nfsctl.c b/fs/nfsd/nfsctl.c
> index 7493428..65889ec 100644
> --- a/fs/nfsd/nfsctl.c
> +++ b/fs/nfsd/nfsctl.c
> @@ -534,7 +534,7 @@ static ssize_t __write_versions(struct file *file, char *buf, size_t size)
>  			else
>  				num = simple_strtol(vers, &minorp, 0);
>  			if (*minorp == '.') {
> -				if (num < 4)
> +				if (num != 4)
>  					return -EINVAL;
>  				minor = simple_strtoul(minorp+1, NULL, 0);
>  				if (minor == 0)
> 

thanks ,maybe you are right.

but if the kernel supports nfs5.1,nfs6.1. it may don't work.
people don't know whether the kernel can support nfs5.1,nfs6.1,nfs7.1 or not. 
if the kernel don't support nfs5.1,nfs6.1,nfs7.1 in the further. this change
maybe right. 

> --b.
> 
>> Signed-off-by: Fan Chaoting <fanchaoting@cn.fujitsu.com>
>>
>> ---
>>  fs/nfsd/nfsctl.c |    2 +-
>>  fs/nfsd/nfsd.h   |    1 +
>>  fs/nfsd/nfssvc.c |    4 ++++
>>  3 files changed, 6 insertions(+), 1 deletion(-)
>>
>> diff --git a/fs/nfsd/nfsctl.c b/fs/nfsd/nfsctl.c
>> index dab350d..1a6e89c 100644
>> --- a/fs/nfsd/nfsctl.c
>> +++ b/fs/nfsd/nfsctl.c
>> @@ -532,7 +532,7 @@ static ssize_t __write_versions(struct file *file, char *buf, size_t size)
>>  			else
>>  				num = simple_strtol(vers, &minorp, 0);
>>  			if (*minorp == '.') {
>> -				if (num < 4)
>> +				if (num < 4 || num >= nfsd_number_vers())
>>  					return -EINVAL;
>>  				minor = simple_strtoul(minorp+1, NULL, 0);
>>  				if (minor == 0)
>> diff --git a/fs/nfsd/nfsd.h b/fs/nfsd/nfsd.h
>> index 80d5ce4..c9d7203 100644
>> --- a/fs/nfsd/nfsd.h
>> +++ b/fs/nfsd/nfsd.h
>> @@ -104,6 +104,7 @@ int nfsd_vers(int vers, enum vers_op change);
>>  int nfsd_minorversion(u32 minorversion, enum vers_op change);
>>  void nfsd_reset_versions(void);
>>  int nfsd_create_serv(void);
>> +int nfsd_number_vers(void);
>>  
>>  extern int nfsd_max_blksize;
>>  
>> diff --git a/fs/nfsd/nfssvc.c b/fs/nfsd/nfssvc.c
>> index 2013aa00..072cdb9 100644
>> --- a/fs/nfsd/nfssvc.c
>> +++ b/fs/nfsd/nfssvc.c
>> @@ -670,3 +670,7 @@ int nfsd_pool_stats_release(struct inode *inode, struct file *file)
>>  	mutex_unlock(&nfsd_mutex);
>>  	return ret;
>>  }
>> +
>> +int nfsd_number_vers(void) {
>> +	return NFSD_NRVERS;
>> +}
>> -- 
>> 1.7.10.1
>>
> 
>