[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

RE: [Xen-devel] [PATCH] Clear device-model information when destroying an HVM domain with stubdom-v3



Jun Zhu (Intern) writes ("RE: [Xen-devel] [PATCH] Clear device-model 
information when destroying an HVM domain with stubdom-v3"):
> Thanks for your suggesion. I use goto out in both wrong condition,
> and remove the unnecessory logging. The vim tabs has been converted
> to 4 spaces.

Thanks.  You still need to put { and } on the same line as the else :-).

Also, more seriously:

> +        } else {
> +            XL_LOG_ERRNO(ctx, XL_LOG_ERROR, "failed to kill Device Model 
> [%d]",
> +                    atoi(pid));
> +            goto out;

This should set ret to something appropriate.  ERROR_FAIL, probably.

Ian.

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel


 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.