[sheepdog] [PATCH RFC] sheep: free memory used for exceptional nodes

[Hitoshi Mitake]

At Mon, 06 May 2013 23:47:26 +0900,
Hitoshi Mitake wrote:
> 
> At Mon,  6 May 2013 23:46:04 +0900,
> Hitoshi Mitake wrote:
> > 
> > It seems that current clear_exceptional_node_lists() leaks memory used
> > for representing delayed and failed nodes.
> 
> 
> BTW, I have a question about the mechanism of dealing with exceptional
> nodes. In sd_join_handler(), if the condition: 
>        nr_local == nr + nr_failed - nr_delayed_nodes
> is true, status of sheepdog cluster becomes OK.
> 
> I couldn't understand the meaning of the above condition. Because
> failed nodes exit immediately, so they should not be counted as
> workable nodes.  (On the other hand, delayed nodes are not counted as 
> workable. It is also strange from my perspective.)
> 
> I'm glad if someone give me an explanation.

If nobody has a good explanation about this strategy, I'd like to work
on refactoring it.

Thanks,
Hitoshi