Re: [PATCH 0/7] clstated: add a daemon to track NFSv4 client names on stable storage (RFC)

[Steve Dickson <SteveD@redhat.com>]

On 12/14/2011 10:32 AM, Jeff Layton wrote:
> On Wed, 14 Dec 2011 10:23:15 -0500
> Steve Dickson <SteveD@redhat.com> wrote:
> 
>>
>>
>> On 12/14/2011 08:57 AM, Jeff Layton wrote:
>>> This patchset is the userspace portion of the knfsd client name tracking
>>> overhaul. See this patch series for an explanation:
>>>
>>>     nfsd: overhaul the client name tracking code (RFC)
>>>
>>> The daemon listens for upcalls on the rpc_pipefs pipe using libevent,
>>> and handles the requests.
>>>
>>> The data is stored using a sqlite database. The main reason for this is
>>> that it takes care of most of the fussy details and atomicity concerns
>>> of tracking the information on stable storage.
>> This will make nfs-utils dependent on the sqlite database... Any idea
>> what kinda of extra baggage this brings? 
>>
> 
> Depends on what you mean by "baggage". What is your concern?
In Fedora doing a 'yum install sqlite' which would require
a ton of other package needing to be install... The Required
for nfs-utils is getting pretty long at this point... 
 
> 
> For something like fedora, it'll mean a build-time dependency on
> sqlite-devel, and a runtime dependency on sqlite. We could consider
> embedding their single-file amalgamation code, but I'd prefer not to do
> that unless we really have to.
> 
>>>
>>> For now, the daemon is only suitable for single-host configurations.
>>> The plan is to later extend this to be suitable for clustered
>>> configurations as well.
>>>
>>> The code is still a little rough, so be gentle. It also lacks things
>>> like a manpage. I plan to add all that before doing a "formal" patch
>>> submission, but I wanted to get some early review of the overall design
>>> before to spend a lot of time knocking off the rough edges.
>> Finally the name of the daemon...  clstated does not make it clear that 
>> this is a nfs server daemon... maybe something like nfsdcld? 
>>
> 
> Sure, we could change the name if that's desirable. Maybe
> nfsd.clnamed ? At this point it's really just a client name tracking
> daemon.
How about nfsdcld ;-) short, to the point and easy to type... 8-) 

steved.

> 
>> steved.
>>
>>>
>>> Jeff Layton (7):
>>>   clstated: add clname tracking daemon stub
>>>   clstated: reattempt the pipe open if it fails on ENOENT
>>>   clstated: add autoconf goop for sqlite
>>>   clstated: add routines for a sqlite backend database
>>>   clstated: add remove functionality
>>>   clstated: add check/update functionality
>>>   clstated: add function to remove unreclaimed client records
>>>
>>>  aclocal/libsqlite3.m4      |   33 +++
>>>  configure.ac               |   23 ++
>>>  utils/Makefile.am          |    4 +
>>>  utils/clstated/Makefile.am |   14 +
>>>  utils/clstated/clstated.c  |  355 +++++++++++++++++++++++++++
>>>  utils/clstated/sqlite.c    |  572 ++++++++++++++++++++++++++++++++++++++++++++
>>>  utils/clstated/sqlite.h    |   30 +++
>>>  7 files changed, 1031 insertions(+), 0 deletions(-)
>>>  create mode 100644 aclocal/libsqlite3.m4
>>>  create mode 100644 utils/clstated/Makefile.am
>>>  create mode 100644 utils/clstated/clstated.c
>>>  create mode 100644 utils/clstated/sqlite.c
>>>  create mode 100644 utils/clstated/sqlite.h
>>>
>>> --
>>> 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
> 
>