> >       default:
> > -        xenXMError(VIR_ERR_XML_ERROR,
> > -                   "%s", _("unknown device"));
> > +        xenXMError(VIR_ERR_OPERATION_INVALID, "%s",
> > +                   _("Xm driver only supports adding disk or network 
> > devices"));
> >           goto cleanup;
> >       }
> >
> 
> It looks like many other similar errors in other drivers use 
> VIR_ERR_CONFIG_UNSUPPORTED. Would that maybe be a better choice than 
> VIR_ERR_OPERATION_INVALID

Yeah, right, I guess it's not worth a v2, is it?

Jirka

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list

Reply via email to