From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754387Ab2BWQ6m (ORCPT ); Thu, 23 Feb 2012 11:58:42 -0500 Received: from mga02.intel.com ([134.134.136.20]:27059 "EHLO mga02.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752339Ab2BWQ6l convert rfc822-to-8bit (ORCPT ); Thu, 23 Feb 2012 11:58:41 -0500 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="4.67,351,1309762800"; d="scan'208";a="114072284" From: "Liu, Jinsong" To: Jan Beulich , "konrad.wilk@oracle.com" CC: "keir.xen@gmail.com" , "Brown, Len" , "Li, Shaohua" , "lenb@kernel.org" , "xen-devel@lists.xensource.com" , "linux-acpi@vger.kernel.org" , "linux-kernel@vger.kernel.org" Subject: RE: [PATCH 1/2] RFC: Prepare PAD for native and xen platform Thread-Topic: [PATCH 1/2] RFC: Prepare PAD for native and xen platform Thread-Index: AQHM8jr7m4MGy1d3TzK01xgwJYdnxZZKoT+Q Date: Thu, 23 Feb 2012 16:58:34 +0000 Message-ID: References: <4F46530B020000780007F751@nat28.tlf.novell.com> In-Reply-To: <4F46530B020000780007F751@nat28.tlf.novell.com> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-originating-ip: [10.239.127.40] Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 8BIT MIME-Version: 1.0 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Jan Beulich wrote: >>>> "Liu, Jinsong" 02/23/12 2:29 PM >>> >> --- a/drivers/acpi/Kconfig >> +++ b/drivers/acpi/Kconfig >> @@ -213,10 +213,11 @@ config ACPI_HOTPLUG_CPU >> default y > > >> config ACPI_PROCESSOR_AGGREGATOR >> - tristate "Processor Aggregator" >> + bool "Processor Aggregator" > > There must be ways to address whatever strange problem you see without > making this piece of code non-modular. > Yes, another approach is x86_init approach, defining acpi_pad_ops at x86_init.c and overwritten when xen_start_kernel. This patch is just a RFC patch, to evaluate which approch is more reasonable :-) >> depends on ACPI_PROCESSOR >> depends on EXPERIMENTAL >> depends on X86 >> + default n > > This is pointless. Elaborate more? just a little curious why Kconfig has so many default n. > >> help >> ACPI 4.0 defines processor Aggregator, which enables OS to >> perform specific processor configuration and control that >> applies to all > > Jan