public inbox for linux-nfs@vger.kernel.org
 help / color / mirror / Atom feed
From: Chuck Lever <chuck.lever@oracle.com>
To: trond.myklebust@fys.uio.no
Cc: linux-nfs@vger.kernel.org
Subject: [PATCH 0/8] Mount XDR rewrite, and parser bugfix, take 2
Date: Sat, 13 Jun 2009 19:25:37 -0400	[thread overview]
Message-ID: <20090613232149.3225.92143.stgit@isabey.1015granger.net> (raw)

Hi Trond-

Redriven based on your review comments.  The patches start with the
enc_dirpath patch, for reference.

The parser fix patch is larger now, but the resulting parser is
simpler.  There's no need to actually count errors; simply raise a
flag whenever an invalid option is encountered.  Invalid values
cause parsing to stop immediately.

---

Chuck Lever (8):
      NFS: Invalid mount option values should always fail, even with "sloppy"
      NFS: Remove unused XDR decoder functions
      NFS: Update MNT and MNT3 reply decoding functions
      NFS: add XDR decoder for mountd version 3 auth-flavor lists
      NFS: add new file handle decoders to in-kernel mountd client
      NFS: Add separate mountd status code decoders for each mountd version
      NFS: remove unused function in fs/nfs/mount_clnt.c
      NFS: Use xdr_stream-based XDR encoder for MNT's dirpath argument


 fs/nfs/internal.h   |    8 +
 fs/nfs/mount_clnt.c |  305 +++++++++++++++++++++++++++++++++++++++++++++------
 fs/nfs/nfsroot.c    |    2 
 fs/nfs/super.c      |  159 +++++++++++----------------
 4 files changed, 340 insertions(+), 134 deletions(-)

-- 
Chuck Lever <chuck.lever@oracle.com>

             reply	other threads:[~2009-06-13 23:26 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-06-13 23:25 Chuck Lever [this message]
     [not found] ` <20090613232149.3225.92143.stgit-r85ClMMopbrwdl/1UfZZQIVfYA8g3rJ/@public.gmane.org>
2009-06-13 23:25   ` [PATCH 1/8] NFS: Use xdr_stream-based XDR encoder for MNT's dirpath argument Chuck Lever
2009-06-13 23:25   ` [PATCH 2/8] NFS: remove unused function in fs/nfs/mount_clnt.c Chuck Lever
2009-06-13 23:26   ` [PATCH 3/8] NFS: Add separate mountd status code decoders for each mountd version Chuck Lever
2009-06-13 23:26   ` [PATCH 4/8] NFS: add new file handle decoders to in-kernel mountd client Chuck Lever
2009-06-13 23:26   ` [PATCH 5/8] NFS: add XDR decoder for mountd version 3 auth-flavor lists Chuck Lever
2009-06-13 23:26   ` [PATCH 6/8] NFS: Update MNT and MNT3 reply decoding functions Chuck Lever
2009-06-13 23:26   ` [PATCH 7/8] NFS: Remove unused XDR decoder functions Chuck Lever
2009-06-13 23:26   ` [PATCH 8/8] NFS: Invalid mount option values should always fail, even with "sloppy" Chuck Lever

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=20090613232149.3225.92143.stgit@isabey.1015granger.net \
    --to=chuck.lever@oracle.com \
    --cc=linux-nfs@vger.kernel.org \
    --cc=trond.myklebust@fys.uio.no \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox