[libvirt] [PATCH 11/14] Remove the network backend if NIC hotplug fails

Mark McLoughlin markmc at redhat.com
Mon Jul 20 13:10:37 UTC 2009


On Mon, 2009-07-20 at 14:56 +0200, Daniel Veillard wrote:
> On Mon, Jul 20, 2009 at 12:51:21PM +0100, Mark McLoughlin wrote:
> >      if (qemudMonitorCommand(vm, cmd, &reply) < 0) {
> > -        /* FIXME: try and remove the backend again */
> >          qemudReportError(conn, dom, NULL, VIR_ERR_OPERATION_FAILED,
> >                           _("failed to add NIC with '%s'"), cmd);
> >          VIR_FREE(cmd);
> > -        return -1;
> > +        goto try_remove;
> >      }
> 
>   idem, maybe patch 10 and 11  don't need to be separated

Just trying to make things easier to review by adding stuff iteratively.

(I probably should have mentioned in 10 that the problems mentioned in
the patch description are tackled in subsequent patches)

Cheers,
Mark.




More information about the libvir-list mailing list