From: "J. Bruce Fields" <bfields@fieldses.org>
To: Trond Myklebust <Trond.Myklebust@netapp.com>
Cc: linux-nfs@vger.kernel.org
Subject: Re: [PATCH v2 0/4] Allow the admin to turn off NFSv4 uid/gid mapping
Date: Tue, 30 Nov 2010 14:24:41 -0500 [thread overview]
Message-ID: <20101130192441.GD29091@fieldses.org> (raw)
In-Reply-To: <1291124043-5074-1-git-send-email-Trond.Myklebust@netapp.com>
On Tue, Nov 30, 2010 at 08:33:59AM -0500, Trond Myklebust wrote:
> The following patches allow the admin to turn off NFSv4 uid/gid mapping
> if mounting using AUTH_SYS security.
Maybe that should be "when a mountpoint allows only AUTH_SYS security"?
(Once we start allowing auth_flavor_len > 1, presumably we don't want to
change mappings depending on which flavor an individual getattr used!)
Also, maybe it's right, but I'm a little weirded about by the dependency
on the auth flavor.
--b.
>
> Minor changes w.r.t. first revision:
> - Added documentation in kernel-parameters.txt
> - Print a warning if an NFSERR_BADOWNER causes us to turn idmapping on again
>
> Trond Myklebust (4):
> NFSv4: If the server sends us a numeric uid/gid then accept it
> NFSv4: Send unmapped uid/gids to the server if the idmapper fails
> NFSv4: cleanup idmapper functions to take an nfs_server argument
> NFSv4: Send unmapped uid/gids to the server when using auth_sys
>
> Documentation/kernel-parameters.txt | 8 +++
> fs/nfs/client.c | 16 ++++++
> fs/nfs/idmap.c | 87 +++++++++++++++++++++++++++-------
> fs/nfs/nfs4proc.c | 13 +++++-
> fs/nfs/nfs4xdr.c | 18 +++----
> include/linux/nfs_fs_sb.h | 1 +
> include/linux/nfs_idmap.h | 8 ++--
> 7 files changed, 118 insertions(+), 33 deletions(-)
>
> --
> 1.7.3.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:[~2010-11-30 19:24 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-11-30 13:33 [PATCH v2 0/4] Allow the admin to turn off NFSv4 uid/gid mapping Trond Myklebust
2010-11-30 13:34 ` [PATCH v2 1/4] NFSv4: If the server sends us a numeric uid/gid then accept it Trond Myklebust
2010-11-30 13:34 ` [PATCH v2 2/4] NFSv4: Send unmapped uid/gids to the server if the idmapper fails Trond Myklebust
2010-11-30 13:34 ` [PATCH v2 3/4] NFSv4: cleanup idmapper functions to take an nfs_server argument Trond Myklebust
2010-11-30 13:34 ` [PATCH v2 4/4] NFSv4: Send unmapped uid/gids to the server when using auth_sys Trond Myklebust
2010-11-30 19:24 ` J. Bruce Fields [this message]
2010-11-30 21:44 ` [PATCH v2 0/4] Allow the admin to turn off NFSv4 uid/gid mapping Trond Myklebust
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=20101130192441.GD29091@fieldses.org \
--to=bfields@fieldses.org \
--cc=Trond.Myklebust@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.