LKML Archive on lore.kernel.org
help / color / mirror / Atom feed
* [patch] paravirt: allow NO_HZ, remove NO_IDLE_HZ dependency
@ 2007-03-05 11:29 Ingo Molnar
2007-03-05 11:46 ` Thomas Gleixner
0 siblings, 1 reply; 2+ messages in thread
From: Ingo Molnar @ 2007-03-05 11:29 UTC (permalink / raw)
To: Andrew Morton, Linus Torvalds; +Cc: linux-kernel, Thomas Gleixner, Andi Kleen
Subject: [patch] paravirt: remove NO_IDLE_HZ dependency
From: Ingo Molnar <mingo@elte.hu>
PARAVIRT forces CONFIG_NO_HZ off - this is bad and prevents sane
paravirtual guests like KVM from being able to use dynticks-enabled
Linux guests. Remove this. Vmi-timer needs to be updated properly to
make use of dynticks.
furthermore, remove the mistaken turning on of NO_IDLE_HZ on
x86+PARAVIRT kernels, it's an obsolete, limited form of dynticks.
must-have for v2.6.21.
Signed-off-by: Ingo Molnar <mingo@elte.hu>
---
arch/i386/Kconfig | 11 +----------
1 file changed, 1 insertion(+), 10 deletions(-)
Index: linux/arch/i386/Kconfig
===================================================================
--- linux.orig/arch/i386/Kconfig
+++ linux/arch/i386/Kconfig
@@ -224,7 +224,7 @@ config PARAVIRT
config VMI
bool "VMI Paravirt-ops support"
- depends on PARAVIRT && !NO_HZ
+ depends on PARAVIRT
default y
help
VMI provides a paravirtualized interface to multiple hypervisors
@@ -1291,12 +1291,3 @@ config X86_TRAMPOLINE
config KTIME_SCALAR
bool
default y
-
-config NO_IDLE_HZ
- bool
- depends on PARAVIRT
- default y
- help
- Switches the regular HZ timer off when the system is going idle.
- This helps a hypervisor detect that the Linux system is idle,
- reducing the overhead of idle systems.
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: [patch] paravirt: allow NO_HZ, remove NO_IDLE_HZ dependency
2007-03-05 11:29 [patch] paravirt: allow NO_HZ, remove NO_IDLE_HZ dependency Ingo Molnar
@ 2007-03-05 11:46 ` Thomas Gleixner
0 siblings, 0 replies; 2+ messages in thread
From: Thomas Gleixner @ 2007-03-05 11:46 UTC (permalink / raw)
To: Ingo Molnar; +Cc: Andrew Morton, Linus Torvalds, linux-kernel, Andi Kleen
On Mon, 2007-03-05 at 12:29 +0100, Ingo Molnar wrote:
> Subject: [patch] paravirt: remove NO_IDLE_HZ dependency
> From: Ingo Molnar <mingo@elte.hu>
>
> PARAVIRT forces CONFIG_NO_HZ off - this is bad and prevents sane
> paravirtual guests like KVM from being able to use dynticks-enabled
> Linux guests. Remove this. Vmi-timer needs to be updated properly to
> make use of dynticks.
>
> furthermore, remove the mistaken turning on of NO_IDLE_HZ on
> x86+PARAVIRT kernels, it's an obsolete, limited form of dynticks.
>
> must-have for v2.6.21.
>
> Signed-off-by: Ingo Molnar <mingo@elte.hu>
Acked-by: Thomas Gleixner <tglx@linutronix.de>
> ---
> arch/i386/Kconfig | 11 +----------
> 1 file changed, 1 insertion(+), 10 deletions(-)
>
> Index: linux/arch/i386/Kconfig
> ===================================================================
> --- linux.orig/arch/i386/Kconfig
> +++ linux/arch/i386/Kconfig
> @@ -224,7 +224,7 @@ config PARAVIRT
>
> config VMI
> bool "VMI Paravirt-ops support"
> - depends on PARAVIRT && !NO_HZ
> + depends on PARAVIRT
> default y
> help
> VMI provides a paravirtualized interface to multiple hypervisors
> @@ -1291,12 +1291,3 @@ config X86_TRAMPOLINE
> config KTIME_SCALAR
> bool
> default y
> -
> -config NO_IDLE_HZ
> - bool
> - depends on PARAVIRT
> - default y
> - help
> - Switches the regular HZ timer off when the system is going idle.
> - This helps a hypervisor detect that the Linux system is idle,
> - reducing the overhead of idle systems.
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2007-03-05 11:40 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2007-03-05 11:29 [patch] paravirt: allow NO_HZ, remove NO_IDLE_HZ dependency Ingo Molnar
2007-03-05 11:46 ` Thomas Gleixner
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).