[sheepdog] [PATCH v4 3/4] sheep: use refcnt_t in various data structures

Hitoshi Mitake mitake.hitoshi at gmail.com
Fri Jul 12 03:53:47 CEST 2013


At Thu, 11 Jul 2013 11:08:56 +0800,
Liu Yuan wrote:
> 
> On Thu, Jul 11, 2013 at 10:48:12AM +0900, Hitoshi Mitake wrote:
> > This patch lets various data structures employ refcnt_t for its
> > reference counting of cache entries. Changed structs are: vnode_info,
> > client_info, request, and object_cache_entry.
> > 
> > Signed-off-by: Hitoshi Mitake <mitake.hitoshi at lab.ntt.co.jp>
> > ---
> > v3:
> >  - remove redundant assert()s
> > 
> >  include/sheep.h      |    3 ++-
> >  sheep/group.c        |   10 +++-------
> >  sheep/object_cache.c |    8 ++++----
> >  sheep/request.c      |   16 ++++++++--------
> >  sheep/sheep_priv.h   |    4 ++--
> >  sheep/vdi.c          |    2 +-
> >  6 files changed, 20 insertions(+), 23 deletions(-)
> > 
> > diff --git a/include/sheep.h b/include/sheep.h
> > index cd23b75..44e23f2 100644
> > --- a/include/sheep.h
> > +++ b/include/sheep.h
> > @@ -17,6 +17,7 @@
> >  #include "list.h"
> >  #include "net.h"
> >  #include "logger.h"
> > +#include "work.h"
> 
> I have no idea why you added work.h here. I manually removed it.

Thanks for your fix and sorry for the include line. This would be a
garbage which I forgot to clean.

Thanks,
Hitoshi



More information about the sheepdog mailing list