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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, 14 Dec 2011 10:23:15 -0500
Steve Dickson <SteveD@xxxxxxxxxx> 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?

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.

> 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@xxxxxxxxxxxxxxx
> > More majordomo info at  http://vger.kernel.org/majordomo-info.html


-- 
Jeff Layton <jlayton@xxxxxxxxxx>
--
To unsubscribe from this list: send the line "unsubscribe linux-nfs" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux Filesystem Development]     [Linux USB Development]     [Linux Media Development]     [Video for Linux]     [Linux NILFS]     [Linux Audio Users]     [Yosemite Info]     [Linux SCSI]

  Powered by Linux