On Thu, Nov 21, 2013 at 01:52:02PM +0800, Li Zhang wrote: > From: Li Zhang <zhlcindy(a)linux.vnet.ibm.com&gt; > > It doesn't need to add a default memory balloon for PPC64. > Only if users want it, it can be added explicitly. > > Signed-off-by: Li Zhang <zhlcindy(a)linux.vnet.ibm.com&gt; > --- > src/qemu/qemu_command.c | 12 ------------ > src/qemu/qemu_domain.c | 12 ++++++++---- > 2 files changed, 8 insertions(+), 16 deletions(-) > > diff --git a/src/qemu/qemu_command.c b/src/qemu/qemu_command.c > index 8dc7e43..a1e5387 100644 > --- a/src/qemu/qemu_command.c > +++ b/src/qemu/qemu_command.c > @@ -11989,18 +11989,6 @@ qemuParseCommandLine(virCapsPtr qemuCaps, > def->videos[def->nvideos++] = vid; > } > > - /* > - * having a balloon is the default, define one with type="none" to avoid it > - */ > - if (!def->memballoon) { > - virDomainMemballoonDefPtr memballoon; > - if (VIR_ALLOC(memballoon) < 0) > - goto error; > - memballoon->model = VIR_DOMAIN_MEMBALLOON_MODEL_VIRTIO; > - > - def->memballoon = memballoon; > - } > - > VIR_FREE(nics); > > if (virDomainDefAddImplicitControllers(def) < 0) > diff --git a/src/qemu/qemu_domain.c b/src/qemu/qemu_domain.c > index 346fec3..0744e89 100644 > --- a/src/qemu/qemu_domain.c > +++ b/src/qemu/qemu_domain.c > @@ -724,13 +724,17 @@ qemuDomainDefPostParse(virDomainDefPtr def, > break; > > case VIR_ARCH_ARMV7L: > - addDefaultUSB = false; > - addDefaultMemballoon = false; > - break; > + addDefaultUSB = false; > + addDefaultMemballoon = false; > + break; > + > + case VIR_ARCH_PPC64: > + addPCIRoot = true; > + addDefaultMemballoon = false; > + break; > > case VIR_ARCH_ALPHA: > case VIR_ARCH_PPC: > - case VIR_ARCH_PPC64: > case VIR_ARCH_PPCEMB: > case VIR_ARCH_SH4: > case VIR_ARCH_SH4EB: Unless this is actively causing failures, then NACK to this patch. In retrospect we should not have enabled the balloon driver by default in the KVM driver. We do however have a way to turn this off in the XML, and I think it is better that the behaviour be consistent across architectures, instead of special casing PPC

Daniel