Re: [libvirt] [PATCH] qemu: handle more machines with a single builtin IDE controller

2015-11-21 Thread Laine Stump
On 11/21/2015 02:00 PM, Guido Günther wrote: like I440FX by moving the condition into qemuDomainMachineHasBuiltinIDE and adding more machines. Nice! I like this better than the original 3 patch series, since it allows recognizing any new machinetypes with a single line change. ACK, and thank

[libvirt] [PATCH] qemu: handle more machines with a single builtin IDE controller

2015-11-21 Thread Guido Günther
like I440FX by moving the condition into qemuDomainMachineHasBuiltinIDE and adding more machines. Reference: http://bugs.debian.org/805189 --- src/qemu/qemu_command.c | 26 ++ src/qemu/qemu_domain.c | 10 ++ src/qemu/qemu_domain.h | 1 + 3 files changed, 25 inse

Re: [libvirt] [PATCH 1/3] qemu: PPCs G3Beige has a default IDE controller

2015-11-21 Thread Guido Günther
Hi Laine, On Fri, Nov 20, 2015 at 06:55:09PM -0500, Laine Stump wrote: > On 11/20/2015 03:20 PM, Guido Günther wrote: > >so handle it like I440FX > >--- > > src/qemu/qemu_command.c | 17 - > > src/qemu/qemu_domain.c | 7 +++ > > src/qemu/qemu_domain.h | 1 + > > 3 files cha