From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1030182AbXCFVcZ (ORCPT ); Tue, 6 Mar 2007 16:32:25 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1030200AbXCFVcZ (ORCPT ); Tue, 6 Mar 2007 16:32:25 -0500 Received: from smtp-outbound-1.vmware.com ([65.113.40.141]:35646 "EHLO smtp-outbound-1.vmware.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1030182AbXCFVcY (ORCPT ); Tue, 6 Mar 2007 16:32:24 -0500 Message-ID: <45EDDDE7.9060408@vmware.com> Date: Tue, 06 Mar 2007 13:32:23 -0800 From: Dan Hecht User-Agent: Thunderbird 1.5.0.2 (X11/20060420) MIME-Version: 1.0 To: Andi Kleen Cc: tglx@linutronix.de, Zachary Amsden , Ingo Molnar , akpm@linux-foundation.org, Virtualization Mailing List , Jeremy Fitzhardinge , Rusty Russell , LKML , john stultz Subject: Re: + stupid-hack-to-make-mainline-build.patch added to -mm tree References: <200703060654.l266sVxr014860@shell0.pdx.osdl.net> <45ED16D2.3000202@vmware.com> <20070306084258.GA15745@elte.hu> <20070306084647.GA16280@elte.hu> <45ED2C82.3080008@vmware.com> <1173178774.24738.311.camel@localhost.localdomain> <45EDD82F.90204@vmware.com> <20070306222157.GJ22829@bingen.suse.de> In-Reply-To: <20070306222157.GJ22829@bingen.suse.de> Content-Type: text/plain; charset=us-ascii; format=flowed Content-Transfer-Encoding: 7bit X-OriginalArrivalTime: 06 Mar 2007 21:32:22.0997 (UTC) FILETIME=[ED5A5850:01C76036] Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org On 03/06/2007 02:21 PM, Andi Kleen wrote: >> I believe this was just a quick fix in response to Ingo breaking the VMI >> build yesterday by disabling NO_IDLE_HZ on us. There is no technical >> reason why NO_IDLE_HZ=y can't coexist with NO_HZ. > > Well it's nasty that you force NO_IDLE_HZ on all of paravirt ops users. The only thing NO_IDLE_HZ=y "forces" on other users is some extra code (which you are going to get no matter what with CONFIG_PARAVIRT). It doesn't force them to use this code. It just provides a few extra routines that a paravirt_ops backend might want to call back into (I think both vmi and xen backends use these routines and that is why it became associated with CONFIG_PARAVIRT rather than CONFIG_VMI).