Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1030373AbVIJAaY (ORCPT ); Fri, 9 Sep 2005 20:30:24 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S932649AbVIJAaY (ORCPT ); Fri, 9 Sep 2005 20:30:24 -0400 Received: from colin.muc.de ([193.149.48.1]:57358 "EHLO mail.muc.de") by vger.kernel.org with ESMTP id S932648AbVIJAaY (ORCPT ); Fri, 9 Sep 2005 20:30:24 -0400 Date: 10 Sep 2005 02:30:22 +0200 Date: Sat, 10 Sep 2005 02:30:22 +0200 From: Andi Kleen To: Zwane Mwaikambo Cc: Ashok Raj , Andrew Morton , Linux Kernel Subject: Re: [patch 13/14] x86_64: Use common functions in cluster and physflat mode Message-ID: <20050910003022.GB61151@muc.de> References: <200509032135.j83LZ8gX020554@shell0.pdx.osdl.net> <20050905231628.GA16476@muc.de> <20050906161215.B19592@unix-os.sc.intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.4.1i Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1664 Lines: 44 On Fri, Sep 09, 2005 at 10:07:28AM -0700, Zwane Mwaikambo wrote: > On Tue, 6 Sep 2005, Ashok Raj wrote: > > > On Tue, Sep 06, 2005 at 01:16:28AM +0200, Andi Kleen wrote: > > > On Sat, Sep 03, 2005 at 02:33:30PM -0700, akpm@osdl.org wrote: > > > > > > > > From: Ashok Raj > > > > > > > > Newly introduced physflat_* shares way too much with cluster with only a very > > > > differences. So we introduce some common functions in that can be reused in > > > > both cases. > > On a slightly different topic, how come we're using physflat for hotplug > cpu? The original idea was to always use physflat mode for hotplug because that does all the sequencing stuff and avoids the shortcut races. But then Ashok decided it was better to add more ifdefs to flat mode instead and I gave up protesting at some point. -Andi > > -#ifndef CONFIG_CPU_HOTPLUG > /* In the CPU hotplug case we cannot use broadcast mode > because that opens a race when a CPU is removed. > - Stay at physflat mode in this case. > - It is bad to do this unconditionally though. Once > - we have ACPI platform support for CPU hotplug > - we should detect hotplug capablity from ACPI tables and > - only do this when really needed. -AK */ > + Stay at physflat mode in this case. - AK */ > +#ifdef CONFIG_HOTPLUG_CPU > if (num_cpus <= 8) > genapic = &apic_flat; > > Thanks, > Zwane > - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/