Re: [Xen-devel] [PATCH v4 --for 4.6 COLOPre 24/25] tools/libxl: move remus state into a seperate structure

[Yang Hongyang <yanghy@xxxxxxxxxxxxxx>]

On 07/15/2015 11:08 PM, Ian Jackson wrote:
> Yang Hongyang writes ("[Xen-devel] [PATCH v4 --for 4.6 COLOPre 24/25] tools/libxl: 
> move remus state into a seperate structure"):
> > Add a new structure remus state, and move concrete layer's private
> > member to remus state.
> > it is pure refactoring and no functional changes.
> Thanks.  I don't have much to add to what Ian Campbell has said, but
>
> >       if (dss->checkpointed_stream == LIBXL_CHECKPOINTED_STREAM_REMUS) {
> > -        dss->interval = r_info->interval;
> >           if (libxl_defbool_val(r_info->compression))
> >               dss->xcflags |= XCFLAGS_CHECKPOINT_COMPRESS;
> In your next version it would be worth mentioning the movement of this
> initialisation in the commit message.
Ok, thanks!

> Ian.
> .
--
Thanks,
Yang.

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel