All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] powerpc: Make KVM_GUEST default to n
@ 2011-08-05  3:23 Anton Blanchard
  2011-08-09 11:58 ` Alexander Graf
  0 siblings, 1 reply; 2+ messages in thread
From: Anton Blanchard @ 2011-08-05  3:23 UTC (permalink / raw)
  To: benh, paulus, agraf; +Cc: linuxppc-dev


KVM_GUEST adds a 1 MB array to the kernel (kvm_tmp) which grew
my kernel enough to cause it to fail to boot.

Dynamically allocating or reducing the size of this array is a
good idea, but in the meantime I think it makes sense to make
KVM_GUEST default to n in order to minimise surprises.

Signed-off-by: Anton Blanchard <anton@samba.org>
---

Index: linux-powerpc/arch/powerpc/platforms/Kconfig
===================================================================
--- linux-powerpc.orig/arch/powerpc/platforms/Kconfig	2011-08-01 17:33:46.120121554 +1000
+++ linux-powerpc/arch/powerpc/platforms/Kconfig	2011-08-01 17:35:06.921772044 +1000
@@ -24,7 +24,7 @@ source "arch/powerpc/platforms/wsp/Kconf
 
 config KVM_GUEST
 	bool "KVM Guest support"
-	default y
+	default n
 	---help---
 	  This option enables various optimizations for running under the KVM
 	  hypervisor. Overhead for the kernel when not running inside KVM should

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: [PATCH] powerpc: Make KVM_GUEST default to n
  2011-08-05  3:23 [PATCH] powerpc: Make KVM_GUEST default to n Anton Blanchard
@ 2011-08-09 11:58 ` Alexander Graf
  0 siblings, 0 replies; 2+ messages in thread
From: Alexander Graf @ 2011-08-09 11:58 UTC (permalink / raw)
  To: Anton Blanchard; +Cc: paulus, linuxppc-dev

On 08/05/2011 05:23 AM, Anton Blanchard wrote:
> KVM_GUEST adds a 1 MB array to the kernel (kvm_tmp) which grew
> my kernel enough to cause it to fail to boot.
>
> Dynamically allocating or reducing the size of this array is a
> good idea, but in the meantime I think it makes sense to make
> KVM_GUEST default to n in order to minimise surprises.
>
> Signed-off-by: Anton Blanchard<anton@samba.org>

Hrm - the idea was that the code is so little intrusive that it could 
simply always be enabled. But yeah, maybe this should be default n.

Ben, could you please pull this into your ppc tree for now? I don't 
think I'll manage to clean up my tree and send out a pull request before 
KVM Forum.


Alex

> ---
>
> Index: linux-powerpc/arch/powerpc/platforms/Kconfig
> ===================================================================
> --- linux-powerpc.orig/arch/powerpc/platforms/Kconfig	2011-08-01 17:33:46.120121554 +1000
> +++ linux-powerpc/arch/powerpc/platforms/Kconfig	2011-08-01 17:35:06.921772044 +1000
> @@ -24,7 +24,7 @@ source "arch/powerpc/platforms/wsp/Kconf
>
>   config KVM_GUEST
>   	bool "KVM Guest support"
> -	default y
> +	default n
>   	---help---
>   	  This option enables various optimizations for running under the KVM
>   	  hypervisor. Overhead for the kernel when not running inside KVM should

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2011-08-09 11:59 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2011-08-05  3:23 [PATCH] powerpc: Make KVM_GUEST default to n Anton Blanchard
2011-08-09 11:58 ` Alexander Graf

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.