All of lore.kernel.org
 help / color / mirror / Atom feed
From: Steve Dickson <SteveD@redhat.com>
To: Kevin Coffman <kwc@citi.umich.edu>
Cc: "J. Bruce Fields" <bfields@fieldses.org>, linux-nfs@vger.kernel.org
Subject: Re: [PATCH 3/7] gssd: add support for callback authentication
Date: Mon, 18 May 2009 11:18:57 -0400	[thread overview]
Message-ID: <4A117C61.9000504@RedHat.com> (raw)
In-Reply-To: <4d569c330905061422t53d4a96as712678f2805c81ac-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>

Since I'm just getting around to this now... just
replace this patch please....

tia,

steved

Kevin Coffman wrote:
> On Thu, Apr 30, 2009 at 5:24 PM, J. Bruce Fields <bfields@fieldses.org> wrote:
>> On Wed, Apr 29, 2009 at 05:56:26PM -0400, Kevin Coffman wrote:
>>> From: Olga Kornievskaia <aglo@citi.umich.edu>
>>>
>>> Add support for handling upcalls on the new "nfsd4_cb" directory pipes.
>>> Only new kernels (2.6.29) have support for this new pipe directory.
>>> (The need for this new pipe directory will go away with NFSv4.1 where
>>> the callback can be done on the same connection as the fore-channel.)
>> My only complaint is that the code would be robust (and more
>> future-proof) if instead of specifically looking for "nfs" and
>> "nfsd4_cb", we just look at all top-level rpc_pipefs directories and
>> handed directories under any of them in the same way.
>>
>> --b.
> 
> In our offline discussion, Bruce convinced me that we should just
> treat all the directories under the rpc_pipefs directory as equal, and
> process any clnt directories that show up within them.  (This
> currently includes, "lockd  mount  nfs  nfsd4_cb  portmap  statd".)
> Any new directories appearing in the future will automatically get the
> same treatment.
> 
> Steve, I don't know what you might have already done with these
> patches.  Would you prefer a replacement for this patch, or patch on
> top of this?
> 
> K.C.

  parent reply	other threads:[~2009-05-18 15:21 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-04-29 21:56 [PATCH 0/7] nfs-utils: add support for authenticated callbacks Kevin Coffman
     [not found] ` <20090429214300.25811.81332.stgit-zTNJhAanYLVZN1qrTdtDg5Vzexx5G7lz@public.gmane.org>
2009-04-29 21:56   ` [PATCH 1/7] svcgssd: add client's principal name to downcall information Kevin Coffman
2009-04-29 21:56   ` [PATCH 2/7] gssd: refactor update_client_list() Kevin Coffman
2009-04-29 21:56   ` [PATCH 3/7] gssd: add support for callback authentication Kevin Coffman
     [not found]     ` <20090429215626.25811.13927.stgit-zTNJhAanYLVZN1qrTdtDg5Vzexx5G7lz@public.gmane.org>
2009-04-30 21:24       ` J. Bruce Fields
2009-05-06 21:22         ` Kevin Coffman
     [not found]           ` <4d569c330905061422t53d4a96as712678f2805c81ac-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2009-05-18 15:18             ` Steve Dickson [this message]
     [not found]               ` <4A117C61.9000504-AfCzQyP5zfLQT0dZR+AlfA@public.gmane.org>
2009-05-18 15:40                 ` Kevin Coffman
2009-04-29 21:56   ` [PATCH 4/7] gssd: print full client directory being handled Kevin Coffman
2009-04-29 21:56   ` [PATCH 5/7] gssd: handle new client upcall Kevin Coffman
2009-04-29 21:56   ` [PATCH 6/7] gssd: process target= attribute in new upcall Kevin Coffman
2009-04-29 21:56   ` [PATCH 7/7] gssd: process service= " Kevin Coffman
2009-04-30 14:52 ` [PATCH 0/7] nfs-utils: add support for authenticated callbacks Kevin Coffman

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=4A117C61.9000504@RedHat.com \
    --to=steved@redhat.com \
    --cc=bfields@fieldses.org \
    --cc=kwc@citi.umich.edu \
    --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.