linux-nfs.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Benny Halevy <bhalevy@panasas.com>
To: Trond Myklebust <Trond.Myklebust@netapp.com>
Cc: NFS list <linux-nfs@vger.kernel.org>,
	pNFS Mailing List <pnfs@linux-nfs.org>,
	Andy Adamson <andros@netapp.com>
Subject: [PATCH 0/7] nfs41-for-2.6.30 cleanup patches
Date: Mon, 30 Mar 2009 16:59:01 +0300	[thread overview]
Message-ID: <49D0D025.2080608@panasas.com> (raw)

Trond,

The following cleanup patches address your review comments.
They are tested to compile and I'll test them on hardware ASAP.
Once ready, I'd like to squash them in to the patchset and
resend a new version.

[PATCH 1/7] SQUASHME: nfs41: remove ifdefs around nfs_client_initdata.minorversion
[PATCH 2/7] SQUASHME: nfs41: use kcalloc to allocate slot table
[PATCH 3/7] SQUASHME: nfs41: alloc session with GFP_KERNEL
[PATCH 4/7] SQUASHME: nfs41: embed resp_len in nfs_getaclres
[PATCH 5/7] SQUASHME: Revert "NFS: fix decode_fs_locations_maxsz"
[PATCH 6/7] SQUASHME: nfs41: get fs_locations replen before encoding the GETATTR
[PATCH 7/7] SQUASHME: nfs41: get getacl replen before encoding the GETATTR

Other than that I've made the following changes to be included
in the next version:

- fixed bisectability issues
- got rid of rpc_clnt.cl_private, use *session in seq args/res instead
- moved nfs4_init_slot_table's definition from "sessions client infrastructure"
  to first usage, in "create_session operations".
- moved new xdr op args/res definitions away from "client xdr definitions",
  to where they're first used.
- move find_slot and free_slot closer to where they're first used.

Benny

             reply	other threads:[~2009-03-30 13:59 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-03-30 13:59 Benny Halevy [this message]
2009-03-30 14:00 ` [PATCH 1/7] SQUASHME: nfs41: remove ifdefs around nfs_client_initdata.minorversion Benny Halevy
2009-03-30 14:00 ` [PATCH 2/7] SQUASHME: nfs41: use kcalloc to allocate slot table Benny Halevy
2009-03-30 14:00 ` [PATCH 3/7] SQUASHME: nfs41: alloc session with GFP_KERNEL Benny Halevy
2009-03-30 14:00 ` [PATCH 4/7] SQUASHME: nfs41: embed resp_len in nfs_getaclres Benny Halevy
2009-03-30 14:00 ` [PATCH 5/7] SQUASHME: Revert "NFS: fix decode_fs_locations_maxsz" Benny Halevy
2009-03-30 14:00 ` [PATCH 6/7] SQUASHME: nfs41: get fs_locations replen before encoding the GETATTR Benny Halevy
2009-03-30 14:00 ` [PATCH 7/7] SQUASHME: nfs41: get getacl " Benny Halevy

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=49D0D025.2080608@panasas.com \
    --to=bhalevy@panasas.com \
    --cc=Trond.Myklebust@netapp.com \
    --cc=andros@netapp.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).