To: Jeff Layton <jeff.layton@primarydata.com>
Subject: Re: [PATCH 00/19] gssd improvements
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8;
 format=flowed
Date: Tue, 09 Dec 2014 14:52:56 +0100
From: =?UTF-8?Q?David_H=C3=A4rdeman?= <david@hardeman.nu>
Cc: linux-nfs@vger.kernel.org, SteveD@redhat.com
In-Reply-To: <20141209080923.2708eb4f@tlielax.poochiereds.net>
References: <20141209053828.24756.89941.stgit@zeus.muc.hardeman.nu>
 <20141209080923.2708eb4f@tlielax.poochiereds.net>
Message-ID: <4639bc17bcb236c23cfaf2bc57d98b67@hardeman.nu>
Sender: linux-nfs-owner@vger.kernel.org

On 2014-12-09 14:09, Jeff Layton wrote:
> On Tue, 09 Dec 2014 06:40:40 +0100
> David Härdeman <david@hardeman.nu> wrote:
> 
>> The following series converts gssd to use libevent and inotify instead
>> of a handrolled event loop and dnotify. Lots of cleanups in the 
>> process
>> (e.g. removing a lot of arbitrary limitations and fixed size buffers).
...
> 
> Looks like a nice cleanup at first glance, but I haven't gone over it
> in detail.
> 
> One thing that _would_ be nice while you're in here though would be to
> help parallelize more of process_krb5_upcall. Currently it forks before
> changing its identity and then the parent waits on that to exit which
> keeps everything serialized.
> 
> I did it that way so that we didn't need change how we close the fds
> afterward. Now that you're already doing some surgery there, that might
> be easier to do and may help performance.

Thanks, the current patchset doesn't really touch upon that code. After 
the patchset is applied, utils/gssd/gssd.c is basically everything that 
happens up to the point where data is available to read from the gss 
pipe but not after....

But it should certainly make it easier to add parallelism in the 
future...

BTW: when you say "how we close the fds"...what do you mean exactly?


//David