* [PATCH] nfsd: Check the nfsd support version when enable or disable the minorversion @ 2013-01-23 9:58 fanchaoting 2013-01-23 23:24 ` bfields 0 siblings, 1 reply; 5+ messages in thread From: fanchaoting @ 2013-01-23 9:58 UTC (permalink / raw) To: bfields@fieldses.org; +Cc: linux-nfs@vger.kernel.org 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. 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 ^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH] nfsd: Check the nfsd support version when enable or disable the minorversion 2013-01-23 9:58 [PATCH] nfsd: Check the nfsd support version when enable or disable the minorversion fanchaoting @ 2013-01-23 23:24 ` bfields 2013-01-24 1:22 ` fanchaoting 2013-01-25 8:52 ` fanchaoting 0 siblings, 2 replies; 5+ messages in thread From: bfields @ 2013-01-23 23:24 UTC (permalink / raw) To: fanchaoting; +Cc: linux-nfs@vger.kernel.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) --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 > ^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH] nfsd: Check the nfsd support version when enable or disable the minorversion 2013-01-23 23:24 ` bfields @ 2013-01-24 1:22 ` fanchaoting 2013-01-25 8:52 ` fanchaoting 1 sibling, 0 replies; 5+ messages in thread From: fanchaoting @ 2013-01-24 1:22 UTC (permalink / raw) To: bfields@fieldses.org; +Cc: linux-nfs@vger.kernel.org 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 >> > > ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] nfsd: Check the nfsd support version when enable or disable the minorversion 2013-01-23 23:24 ` bfields 2013-01-24 1:22 ` fanchaoting @ 2013-01-25 8:52 ` fanchaoting 2013-01-29 18:13 ` bfields 1 sibling, 1 reply; 5+ messages in thread From: fanchaoting @ 2013-01-25 8:52 UTC (permalink / raw) To: bfields@fieldses.org; +Cc: linux-nfs@vger.kernel.org when enable or disable the minorversion throw /proc/fs/nfsd/versions, it should check the support version. now only nfs4 suppport minorversion. Signed-off-by: Fan Chaoting <fanchaoting@cn.fujitsu.com> --- fs/nfsd/nfsctl.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) 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) -- 1.7.10.1 ^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH] nfsd: Check the nfsd support version when enable or disable the minorversion 2013-01-25 8:52 ` fanchaoting @ 2013-01-29 18:13 ` bfields 0 siblings, 0 replies; 5+ messages in thread From: bfields @ 2013-01-29 18:13 UTC (permalink / raw) To: fanchaoting; +Cc: linux-nfs@vger.kernel.org On Fri, Jan 25, 2013 at 04:52:08PM +0800, fanchaoting wrote: > when enable or disable the minorversion throw /proc/fs/nfsd/versions, > it should check the support version. now only nfs4 suppport minorversion. > > Signed-off-by: Fan Chaoting <fanchaoting@cn.fujitsu.com> Apologies, I'd already committed the same fix (and credited you with reported-by:) before seeing this.--b. > > --- > fs/nfsd/nfsctl.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > 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) > -- > 1.7.10.1 > > -- > To unsubscribe from this list: send the line "unsubscribe linux-nfs" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2013-01-29 18:13 UTC | newest] Thread overview: 5+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2013-01-23 9:58 [PATCH] nfsd: Check the nfsd support version when enable or disable the minorversion fanchaoting 2013-01-23 23:24 ` bfields 2013-01-24 1:22 ` fanchaoting 2013-01-25 8:52 ` fanchaoting 2013-01-29 18:13 ` bfields
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.