<div dir="ltr">Thanks Jan.</div><div class="gmail_extra"> <div class="gmail_quote">On 4 December 2015 at 17:30, Ján Tomko <<a href="mailto:jtomko@redhat.com" target="_blank">jtomko@redhat.com</a>> wrote: <blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">On Fri, Dec 04, 2015 at 10:42:30AM +0800, Tashi Lu wrote: > Hi everyone, > > I incidentally configured two IDE controller for my machine-1.2 VM(it > might be virsh attach-disk to blame), and I got the error message > "Only a single IDE controller is unsupported...". I can't understand > this error message, I think it means I HAVE to configure two or more > IDE controllers until I looked it up in the source. > > At libvirt-1.2.22/src/qemu/qemu_command.c: > > 4916 case VIR_DOMAIN_CONTROLLER_TYPE_IDE: > 4917 /* Since we currently only support the integrated IDE controller > 4918 * on 440fx, if we ever get to here, it's because some other > 4919 * machinetype had an IDE controller specified, or a 440fx had > 4920 * multiple ide controllers. > 4921 */ > 4922 if (qemuDomainMachineIsI440FX(domainDef)) > 4923 virReportError(VIR_ERR_CONFIG_UNSUPPORTED, "%s", > 4924 _("Only a single IDE controller is > unsupported " > 4925 "for this machine type")); > > I didn't dig VIR_DOMAIN_CONTROLLER_TYPE_IDE up, but reasoning from the > comment, I think this error message should be something like "More > than one IDE controllers is not supported for this machine type" > </div></div>The following commit replaced "unsupported" by "supported" in the error message: <a href="http://libvirt.org/git/?p=libvirt.git;a=commitdiff;h=e4ab3b5d" rel="noreferrer" target="_blank">http://libvirt.org/git/?p=libvirt.git;a=commitdiff;h=e4ab3b5d</a> commit e4ab3b5d38bfbaef3cfe9da07ffafca0904e69da Author: Guido Günther <<a href="mailto:agx@sigxcpu.org">agx@sigxcpu.org</a>> CommitDate: 2015-11-23 09:39:29 +0100 qemu: handle more machines with a single builtin IDE controller like I440FX by moving the condition into qemuDomainMachineHasBuiltinIDE and adding more machines. Reference: <a href="http://bugs.debian.org/805189" rel="noreferrer" target="_blank">http://bugs.debian.org/805189</a> git describe: v1.2.21-112-ge4ab3b5 contains: v1.3.0-rc1~91 It will be released in libvirt 1.3.0. Jan </blockquote></div> </div>