On Fri, Jul 17, 2015 at 12:24:11AM +0800, Yang Hongyang wrote:
> 
> 
> On 07/17/2015 12:10 AM, Wei Liu wrote:
> >On Wed, Jul 15, 2015 at 03:45:39PM +0800, Yang Hongyang wrote:
> >>Pass checkpointed_stream from libxl to libxc.
> >>It won't affact legacy migration because legacy migration
> >>won't use this param.
> >>
> >>Signed-off-by: Yang Hongyang <yan...@cn.fujitsu.com>
> >>CC: Ian Campbell <ian.campb...@citrix.com>
> >>CC: Ian Jackson <ian.jack...@eu.citrix.com>
> >>CC: Wei Liu <wei.l...@citrix.com>
> >>CC: Andrew Cooper <andrew.coop...@citrix.com>
> >>---
> >[...]
> >>
> >>+    if (dss->checkpointed_stream && !r_info) {
> >
> >Please explicitly check for _NONE type instead of relying it of being 0.
> >Arguably the actual value is not going to change in the future but it's
> >better to be explicit.
> 
> I think you mean check xxx != _NONE here, right? will fix.
> 

Yes, that's what I meant.

Wei.

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

Reply via email to