From: "J. Bruce Fields" <bfields@fieldses.org>
To: Anna Schumaker <bjschuma@netapp.com>
Cc: linux-nfs@vger.kernel.org
Subject: Re: [PATCH v2 0/3] NFSD: Implement SEEK
Date: Tue, 12 Nov 2013 14:45:29 -0500 [thread overview]
Message-ID: <20131112194529.GA26341@fieldses.org> (raw)
In-Reply-To: <1384283048-7699-1-git-send-email-bjschuma@netapp.com>
On Tue, Nov 12, 2013 at 02:04:05PM -0500, Anna Schumaker wrote:
> These patches implement just the SEEK NFS v4.2 operation. WRITE_PLUS is
> still under discussion with the IETF after my last series of patches, so I
> am holding off on resubmitting until after spec discussion dies down.
>
> Questions? Comments? Thoughts?
> Anna
>
> Anna Schumaker (3):
> NFSD: Update error codes
I don't think I got this first patch.
> NFSD: Create nfs v4.2 decode ops
> NFSD: Implement SEEK
I'd like to be reassured the protocol is reasonably stable before we
commit this. I haven't been following the ietf wg discussion closely.
And this should initially be disabled by default. So, probably either:
- Introduce a new NFSD_V4_SEEK option, or
- Combine this and NFSD_V4_SECURITY_LABEL and this into a single
NFSD_V4_2 option.
And recommend "N" for now.
Probably the latter I guess, to be consistent with the client. And
because otherwise we could end up with an awful lot of config options.
--b.
>
> fs/nfsd/nfs4proc.c | 56 ++++++++++++++++++++++++++++++++++++++++++++++++++
> fs/nfsd/nfs4xdr.c | 58 ++++++++++++++++++++++++++++++++++++++++++++++++++++
> fs/nfsd/nfsd.h | 2 +-
> fs/nfsd/xdr4.h | 16 +++++++++++++++
> include/linux/nfs4.h | 23 ++++++++++++++++++---
> 5 files changed, 151 insertions(+), 4 deletions(-)
>
> --
> 1.8.4.2
>
> --
> 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
next prev parent reply other threads:[~2013-11-12 19:45 UTC|newest]
Thread overview: 21+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-11-12 19:04 [PATCH v2 0/3] NFSD: Implement SEEK Anna Schumaker
2013-11-12 19:04 ` [PATCH v2 1/3] NFSD: Update error codes Anna Schumaker
2013-11-12 19:04 ` [PATCH v2 2/3] NFSD: Create nfs v4.2 decode ops Anna Schumaker
2013-11-12 19:04 ` [PATCH v2 3/3] NFSD: Implement SEEK Anna Schumaker
2013-11-12 19:45 ` J. Bruce Fields [this message]
2013-11-12 19:54 ` [PATCH v2 0/3] " Anna Schumaker
2013-11-12 19:59 ` J. Bruce Fields
2013-11-13 16:01 ` Anna Schumaker
2013-11-13 16:07 ` Anna Schumaker
2013-11-13 16:15 ` Christoph Hellwig
2013-11-13 16:49 ` J. Bruce Fields
2013-11-13 16:52 ` Christoph Hellwig
2013-11-13 18:46 ` J. Bruce Fields
2013-11-13 17:02 ` Joshuah Hurst
2013-11-13 17:07 ` Anna Schumaker
2013-11-12 22:44 ` Marc Eshel
2013-11-12 23:02 ` J. Bruce Fields
2013-11-13 0:32 ` Marc Eshel
2013-11-13 13:37 ` J. Bruce Fields
2013-11-12 20:42 ` [PATCH v2 1/3] NFSD: Update error codes Anna Schumaker
2013-11-12 22:44 ` Marc Eshel
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=20131112194529.GA26341@fieldses.org \
--to=bfields@fieldses.org \
--cc=bjschuma@netapp.com \
--cc=linux-nfs@vger.kernel.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.