RE: [Xen-devel] [PATCH] Clear device-model information when dest

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

<Prev in Thread]	Current Thread	[Next in Thread>
RE: [Xen-devel] [PATCH] Clear device-model information when destroying an HVM domain with stubdom-v3, Jun Zhu (Intern) RE: [Xen-devel] [PATCH] Clear device-model information when destroying an HVM domain with stubdom-v3, Ian Jackson <=

Previous by Date:

[Xen-devel] Re: [PATCH] ocaml: xc bindings: use libxenctrl and libxenguest, Ian Jackson

Next by Date:

RE: [Xen-devel] [PATCH] libxl: make libxl communicate with xenstored by socket or xenbus driver-V3, Ian Jackson

Previous by Thread:

RE: [Xen-devel] [PATCH] Clear device-model information when destroying an HVM domain with stubdom-v3, Jun Zhu (Intern)

Next by Thread:

[Xen-devel] VM hung after running sometime, MaoXiaoyun

Indexes:

[Date] [Thread] [Top] [All Lists]