Re: [libvirt] [PATCH v3 3/4] qemu_command: Drop IDE validation code in qemuBuildControllerDevStr

On Fri, Dec 01, 2017 at 07:31:31PM +0800, Lin Ma wrote: > The corresponding code will be moved to qemuDomainControllerDefValidate > by next patch. > > Signed-off-by: Lin Ma <lma(a)suse.com&gt; > --- > src/qemu/qemu_command.c | 10 +--------- > 1 file changed, 1 insertion(+), 9 deletions(-) > > diff --git a/src/qemu/qemu_command.c b/src/qemu/qemu_command.c > index 6a8da1d..2074ff6 100644 > --- a/src/qemu/qemu_command.c > +++ b/src/qemu/qemu_command.c > @@ -3127,17 +3127,9 @@ qemuBuildControllerDevStr(const virDomainDef *domainDef, > /* Since we currently only support the integrated IDE > * controller on various boards, if we ever get to here, it's > * because some other machinetype had an IDE controller > - * specified, or one with a single IDE contraller had multiple > + * specified, or one with a single IDE controller had multiple > * ide controllers specified. > */ > - if (qemuDomainHasBuiltinIDE(domainDef)) > - virReportError(VIR_ERR_CONFIG_UNSUPPORTED, "%s", > - _("Only a single IDE controller is supported " > - "for this machine type")); > - else > - virReportError(VIR_ERR_CONFIG_UNSUPPORTED, "%s", > - _("IDE controllers are unsupported for " > - "this QEMU binary or machine type")); This is not correct, we shouldn't error out without setting any error message. This whole case should be removed and IDE controller should be handled by the "default" path if it ever happen to reach this code.

Pavel > goto error; > > default: > -- > 2.9.2 > > -- > libvir-list mailing list > libvir-list(a)redhat.com > https://www.redhat.com/mailman/listinfo/libvir-list > > > -- > libvir-list mailing list > libvir-list(a)redhat.com > https://www.redhat.com/mailman/listinfo/libvir-list