From: Steve Dickson <SteveD@redhat.com>
To: Benny Halevy <bhalevy@panasas.com>
Cc: NFS list <linux-nfs@vger.kernel.org>,
pNFS Mailing List <pnfs@linux-nfs.org>,
"J. Bruce Fields" <bfields@citi.umich.edu>
Subject: Re: [RFC 0/4] nfs-utils: nfsd support for minor version
Date: Thu, 16 Apr 2009 14:13:16 -0400 [thread overview]
Message-ID: <49E7753C.4010300@RedHat.com> (raw)
In-Reply-To: <49E7721E.2070601@panasas.com>
Benny Halevy wrote:
>> Benny Halevy wrote:
>>> Steve, please review the following patch that add support
>>> for controlling the nfsv4 minor version support via
>>> /proc/fs/nfsd/versions.
>>>
>>> [PATCH 1/4] utils/nfsd: fix -N optarg error printout
>>> This patch fixes an existing bug.
>> This is a bug... and has been committed...
>>
>>> [RFC 2/4] utils/nfsd: add support for minorvers4
>>> Under-the-cover support for minorvers4
>> I see you let minorvers4 default to zero, which means 4.1
>> support is off by default. Why? As long as we have away to
>> turn of 4.1 processing (i.e. your 4/4 patch), then I see
>> no reason we should have the support enabled by default.
>
> I was also thinking about using the new nfs-utils with old kernels.
> Though, these should not puke on seeing [-+]4.1, they'll
> just interpret it as enabling/disabling v4.
> I'll test that...
I just did... Using the '-N 4.1' flag to rpc.nfsd on an older
kernel simply turns off all v4 processing... which is fine, IMHO...
steved.
next prev parent reply other threads:[~2009-04-16 18:16 UTC|newest]
Thread overview: 23+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-04-13 8:25 [RFC 0/4] nfs-utils: nfsd support for minor version Benny Halevy
2009-04-13 8:29 ` [PATCH 1/4] utils/nfsd: fix -N optarg error printout Benny Halevy
2009-04-13 8:29 ` [RFC 2/4] utils/nfsd: add support for minorvers4 Benny Halevy
2009-04-13 8:29 ` [RFC 3/4] utils/nfsd: add -n --nfs-version option Benny Halevy
2009-04-13 8:29 ` [RFC 4/4] utils/nfsd: enable/disable minorvers4 via command line Benny Halevy
2009-04-16 17:24 ` [RFC 0/4] nfs-utils: nfsd support for minor version Steve Dickson
[not found] ` <49E769C5.6010902-AfCzQyP5zfLQT0dZR+AlfA@public.gmane.org>
2009-04-16 17:59 ` Benny Halevy
2009-04-16 18:13 ` Steve Dickson [this message]
[not found] ` <49E7753C.4010300-AfCzQyP5zfLQT0dZR+AlfA@public.gmane.org>
2009-04-16 18:23 ` [pnfs] " J. Bruce Fields
2009-04-16 18:37 ` Benny Halevy
2009-04-16 19:01 ` Steve Dickson
[not found] ` <49E7809B.2020002-AfCzQyP5zfLQT0dZR+AlfA@public.gmane.org>
2009-04-16 19:18 ` J. Bruce Fields
2009-04-17 12:35 ` Steve Dickson
[not found] ` <49E87798.8090308-AfCzQyP5zfLQT0dZR+AlfA@public.gmane.org>
2009-04-17 15:42 ` J. Bruce Fields
2009-04-17 16:18 ` Chuck Lever
2009-04-17 16:40 ` Benny Halevy
2009-04-22 12:06 ` [PATCH 0/3] nfs-utils: nfsd support for minor version, take 2 Benny Halevy
2009-04-22 12:10 ` [PATCH 1/3] utils/nfsd: add support for minorvers4 Benny Halevy
2009-04-22 12:10 ` [PATCH 2/3] utils/nfsd: disable minorvers4 via command line Benny Halevy
2009-04-22 12:10 ` [PATCH 3/3] utils/nfsd: enable nfs minorvers4 by default Benny Halevy
2009-04-22 21:54 ` [PATCH 0/3] nfs-utils: nfsd support for minor version, take 2 J. Bruce Fields
2009-04-23 8:58 ` Benny Halevy
2009-05-18 14:49 ` Steve Dickson
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=49E7753C.4010300@RedHat.com \
--to=steved@redhat.com \
--cc=bfields@citi.umich.edu \
--cc=bhalevy@panasas.com \
--cc=linux-nfs@vger.kernel.org \
--cc=pnfs@linux-nfs.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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.