[sheepdog] [PATCH v3 2/3] lib: move sockfd cache from sheep to lib

Liu Yuan namei.unix at gmail.com
Fri Jul 19 17:27:02 CEST 2013


On Sat, Jul 20, 2013 at 12:24:18AM +0900, Hitoshi Mitake wrote:
> At Fri, 19 Jul 2013 23:21:54 +0800,
> Liu Yuan wrote:
> > 
> > On Sat, Jul 20, 2013 at 12:14:33AM +0900, Hitoshi Mitake wrote:
> > > At Fri, 19 Jul 2013 20:31:57 +0800,
> > > Liu Yuan wrote:
> > > > 
> > > > On Fri, Jul 19, 2013 at 03:18:23PM +0900, Hitoshi Mitake wrote:
> > > > > On some subcommands, collie also issues lots of request to sheeps. So
> > > > > collie can enjoy sockfd caching. For this purpose, this patch moves
> > > > > sockfd from sheep to lib and generalize the interfaces of sockfd.
> > > > > 
> > > > > This patch doesn't change anything related to IO nic in sockfd
> > > > > cache. Because collie can know address nad port of IO nic and
> > > > > potentially use them.
> > > > 
> > > > I think you should strip out node management code. Only sheep daemon need to
> > > > manage node.
> > > 
> > > What does "node management code" mean? sockfd_node_del() and
> > > sockfd_cache_add_group()?
> > 
> > Yes, only sheep needs these code.
> 
> OK, I agree. I'll move them to sheep again in v4.

Well, seems you need to add nodes in collie too? I didn't see the code in collie
to add nodes to sockfd cache.

Thanks
Yuan



More information about the sheepdog mailing list