All of lore.kernel.org
 help / color / mirror / Atom feed
From: Adrian Bunk <bunk@stusta.de>
To: Keith Owens <kaos@sgi.com>
Cc: tony.luck@intel.com, linux-ia64@vger.kernel.org,
	linux-kernel@vger.kernel.org
Subject: [2.6 patch] let IA64_GENERIC select more stuff
Date: Tue, 07 Feb 2006 23:17:13 +0000	[thread overview]
Message-ID: <20060207231713.GG3524@stusta.de> (raw)
In-Reply-To: <9883.1139351831@ocs3>

On Wed, Feb 08, 2006 at 09:37:11AM +1100, Keith Owens wrote:
> 
> A generic IA64 kernel requires (at least) the ACPI and NUMA options in
> order to run on all the IA64 platforms out there.  Omitting those
> options and relying on the user to set them by hand is going to cause
> more problems.
> 
> If anything, there should be more options being set as a side effect of
> selecting IA64_GENERIC, including ARCH_DISCONTIGMEM_ENABLE,
> ARCH_SPARSEMEM_ENABLE, PCI and even SMP.

IOW, you want the patch below?

Not that I'm a big fan of this approach, but if it should be done this 
way, it should be done right.

cu
Adrian


<--  snip  -->


Let IA64_GENERIC select more stuff (as wanted by the ia64 developers).


Signed-off-by: Adrian Bunk <bunk@stusta.de>

--- linux-2.6.16-rc1-mm5-ia64/arch/ia64/Kconfig.old	2006-02-07 23:07:29.000000000 +0100
+++ linux-2.6.16-rc1-mm5-ia64/arch/ia64/Kconfig	2006-02-08 00:13:58.000000000 +0100
@@ -73,10 +73,12 @@
 config IA64_GENERIC
 	bool "generic"
 	select ACPI
 	select NUMA
 	select ACPI_NUMA
+	select PCI
+	select SMP
 	help
 	  This selects the system type of your hardware.  A "generic" kernel
 	  will run on any supported IA-64 system.  However, if you configure
 	  a kernel for your specific system, it will be faster and smaller.
 
@@ -132,10 +134,11 @@
 	  This choice is safe for all IA-64 systems, but may not perform
 	  optimally on systems with, say, Itanium 2 or newer processors.
 
 config MCKINLEY
 	bool "Itanium 2"
+	depends on IA64_GENERIC=n
 	help
 	  Select this to configure for an Itanium 2 (McKinley) processor.
 
 endchoice
 
@@ -318,11 +321,11 @@
 	  for architectures which are either NUMA (Non-Uniform Memory Access)
 	  or have huge holes in the physical address space for other reasons.
  	  See <file:Documentation/vm/numa> for more.
 
 config ARCH_FLATMEM_ENABLE
-	def_bool y
+	def_bool y if IA64_GENERIC=n
 
 config ARCH_SPARSEMEM_ENABLE
 	def_bool y
 	depends on ARCH_DISCONTIGMEM_ENABLE
 


WARNING: multiple messages have this Message-ID (diff)
From: Adrian Bunk <bunk@stusta.de>
To: Keith Owens <kaos@sgi.com>
Cc: tony.luck@intel.com, linux-ia64@vger.kernel.org,
	linux-kernel@vger.kernel.org
Subject: [2.6 patch] let IA64_GENERIC select more stuff
Date: Wed, 8 Feb 2006 00:17:13 +0100	[thread overview]
Message-ID: <20060207231713.GG3524@stusta.de> (raw)
In-Reply-To: <9883.1139351831@ocs3>

On Wed, Feb 08, 2006 at 09:37:11AM +1100, Keith Owens wrote:
> 
> A generic IA64 kernel requires (at least) the ACPI and NUMA options in
> order to run on all the IA64 platforms out there.  Omitting those
> options and relying on the user to set them by hand is going to cause
> more problems.
> 
> If anything, there should be more options being set as a side effect of
> selecting IA64_GENERIC, including ARCH_DISCONTIGMEM_ENABLE,
> ARCH_SPARSEMEM_ENABLE, PCI and even SMP.

IOW, you want the patch below?

Not that I'm a big fan of this approach, but if it should be done this 
way, it should be done right.

cu
Adrian


<--  snip  -->


Let IA64_GENERIC select more stuff (as wanted by the ia64 developers).


Signed-off-by: Adrian Bunk <bunk@stusta.de>

--- linux-2.6.16-rc1-mm5-ia64/arch/ia64/Kconfig.old	2006-02-07 23:07:29.000000000 +0100
+++ linux-2.6.16-rc1-mm5-ia64/arch/ia64/Kconfig	2006-02-08 00:13:58.000000000 +0100
@@ -73,10 +73,12 @@
 config IA64_GENERIC
 	bool "generic"
 	select ACPI
 	select NUMA
 	select ACPI_NUMA
+	select PCI
+	select SMP
 	help
 	  This selects the system type of your hardware.  A "generic" kernel
 	  will run on any supported IA-64 system.  However, if you configure
 	  a kernel for your specific system, it will be faster and smaller.
 
@@ -132,10 +134,11 @@
 	  This choice is safe for all IA-64 systems, but may not perform
 	  optimally on systems with, say, Itanium 2 or newer processors.
 
 config MCKINLEY
 	bool "Itanium 2"
+	depends on IA64_GENERIC=n
 	help
 	  Select this to configure for an Itanium 2 (McKinley) processor.
 
 endchoice
 
@@ -318,11 +321,11 @@
 	  for architectures which are either NUMA (Non-Uniform Memory Access)
 	  or have huge holes in the physical address space for other reasons.
  	  See <file:Documentation/vm/numa> for more.
 
 config ARCH_FLATMEM_ENABLE
-	def_bool y
+	def_bool y if IA64_GENERIC=n
 
 config ARCH_SPARSEMEM_ENABLE
 	def_bool y
 	depends on ARCH_DISCONTIGMEM_ENABLE
 


  parent reply	other threads:[~2006-02-07 23:17 UTC|newest]

Thread overview: 70+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-02-07 22:11 [2.6 patch] IA64_GENERIC shouldn't select other stuff Adrian Bunk
2006-02-07 22:11 ` Adrian Bunk
2006-02-07 22:37 ` Keith Owens
2006-02-07 22:37   ` Keith Owens
2006-02-07 22:43   ` Matthew Wilcox
2006-02-07 22:43     ` Matthew Wilcox
2006-02-07 22:53     ` Adrian Bunk
2006-02-07 22:53       ` Adrian Bunk
2006-02-07 23:17   ` Adrian Bunk [this message]
2006-02-07 23:17     ` [2.6 patch] let IA64_GENERIC select more stuff Adrian Bunk
2006-02-08  0:52     ` Chen, Kenneth W
2006-02-08  0:52       ` Chen, Kenneth W
2006-02-08  1:19       ` Adrian Bunk
2006-02-08  1:19         ` Adrian Bunk
2006-02-08  1:40     ` Chen, Kenneth W
2006-02-08  1:40       ` Chen, Kenneth W
2006-02-08  2:08       ` Adrian Bunk
2006-02-08  2:08         ` Adrian Bunk
2006-02-08  2:17     ` Chen, Kenneth W
2006-02-08  2:17       ` Chen, Kenneth W
2006-02-08  2:43       ` Adrian Bunk
2006-02-08  2:43         ` Adrian Bunk
2006-02-08  2:48     ` Keith Owens
2006-02-08  2:48       ` Keith Owens
2006-02-08  2:54       ` Matthew Wilcox
2006-02-08  2:54         ` Matthew Wilcox
2006-02-08  2:58     ` Chen, Kenneth W
2006-02-08  2:58       ` Chen, Kenneth W
2006-02-08  3:51       ` Adrian Bunk
2006-02-08  3:51         ` Adrian Bunk
2006-02-08  5:52     ` Chen, Kenneth W
2006-02-08  5:52       ` Chen, Kenneth W
2006-02-08 11:59       ` Adrian Bunk
2006-02-08 11:59         ` Adrian Bunk
2006-02-08 13:38         ` Jes Sorensen
2006-02-08 13:38           ` Jes Sorensen
2006-02-08 21:38           ` Adrian Bunk
2006-02-08 21:38             ` Adrian Bunk
2006-02-09 12:53             ` Jes Sorensen
2006-02-09 12:53               ` Jes Sorensen
2006-02-09 13:18               ` Matthew Wilcox
2006-02-09 13:18                 ` Matthew Wilcox
2006-02-09 13:26                 ` Jes Sorensen
2006-02-09 13:26                   ` Jes Sorensen
2006-02-09 14:16                   ` Adrian Bunk
2006-02-09 14:16                     ` Adrian Bunk
2006-02-09 15:19                     ` Jes Sorensen
2006-02-09 15:19                       ` Jes Sorensen
2006-02-08 18:35     ` Luck, Tony
2006-02-08 18:35       ` Luck, Tony
2006-02-08 19:21       ` Alex Williamson
2006-02-08 19:21         ` Alex Williamson
2006-02-08 19:24         ` Jes Sorensen
2006-02-08 19:24           ` Jes Sorensen
2006-02-08 19:48     ` Luck, Tony
2006-02-08 19:48       ` Luck, Tony
2006-02-08 19:55     ` Chen, Kenneth W
2006-02-08 19:55       ` Chen, Kenneth W
2006-02-08 21:24       ` Matthew Wilcox
2006-02-08 21:24         ` Matthew Wilcox
2006-02-08 21:36     ` Chen, Kenneth W
2006-02-08 21:36       ` Chen, Kenneth W
2006-02-08 22:24     ` Chen, Kenneth W
2006-02-08 22:24       ` Chen, Kenneth W
2006-02-08 22:49       ` Adrian Bunk
2006-02-08 22:49         ` Adrian Bunk
2006-02-08 22:35     ` Luck, Tony
2006-02-08 22:35       ` Luck, Tony
2006-02-07 23:06 ` [2.6 patch] IA64_GENERIC shouldn't select other stuff Luck, Tony
2006-02-07 23:06   ` Luck, Tony

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20060207231713.GG3524@stusta.de \
    --to=bunk@stusta.de \
    --cc=kaos@sgi.com \
    --cc=linux-ia64@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=tony.luck@intel.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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.