[sheepdog] [PATCH] sheep: update node in cluster driver
MORITA Kazutaka
morita.kazutaka at gmail.com
Fri May 31 19:27:09 CEST 2013
At Fri, 31 May 2013 21:45:31 +0800,
Liu Yuan wrote:
>
> On 05/31/2013 08:52 PM, MORITA Kazutaka wrote:
> > At Fri, 31 May 2013 18:35:59 +0800,
> > Liu Yuan wrote:
> >>
> >> diff --git a/sheep/group.c b/sheep/group.c
> >> index b9399ee..f74ef10 100644
> >> --- a/sheep/group.c
> >> +++ b/sheep/group.c
> >> @@ -1242,6 +1242,7 @@ void update_node_size(struct sd_node *node)
> >> int idx = get_node_idx(cur_vinfo, node);
> >> assert(idx != -1);
> >> cur_vinfo->nodes[idx].space = node->space;
> >> + sys->cdrv->update_node(node);
> >> }
> >
> > The current cluster driver model is:
> >
> > - Sheep triggers a node event with a cluster driver interface.
> > - The cluster driver updates its internal state, and invokes a
> > callback to notify the change to sheep.
> >
> > In that sense, this fix is a bit weird to me.
> >
> > I think a better fix is:
> >
> > 1. make SD_OP_REWEIGHT a local request.
> > 2. call cdrv->update_node() in cluster_reweight().
> > 3. order the update_node event with other nonblocking events and call
> > sd_update_node() as a callback of the update_node event.
> > 4. call update_node_size() and kick_node_recover() in sd_update_node()
> >
> > Then, we can remove a SD_OP_UPDATE_SIZE operation and FIXME in the
> > local driver.
> >
> > That being said, the current fix also looks correct to me (except the
> > local driver), so I'm fine with merging this code without changes.
> >
>
> Yeah, your fix looks better integrated but this patch win as a simpler
> with less change approach and the node can be updated inside a ordered
> notify event. Local driver just a test driver so there is nothing to worry.
>
> That said, I'd like to merge my fix for 0.6.0 and prepare a another
> patch as you described.
Okay, I've applied this patch, thanks!
Kazutaka
More information about the sheepdog
mailing list