* [PATCH] x86/Kconfig: simplify X86_UP_APIC handling
@ 2015-02-05 15:31 Jan Beulich
2015-02-06 0:00 ` Bryan O'Donoghue
2015-02-19 0:27 ` [tip:x86/build] x86/Kconfig: Simplify " tip-bot for Jan Beulich
0 siblings, 2 replies; 3+ messages in thread
From: Jan Beulich @ 2015-02-05 15:31 UTC (permalink / raw)
To: mingo, tglx, hpa; +Cc: pure.logic, linux-kernel
We don't really need a helper symbol for that. For one, it's
pointlessly getting set to Y for all configurations (even 64-bit ones).
And then the purpose can be fulfilled by suitably adjusting
X86_UP_APIC: Hide its prompt when PCI_MSI, and default it to PCI_MSI.
Signed-off-by: Jan Beulich <jbeulich@suse.com>
Cc: Bryan O'Donoghue <pure.logic@nexus-software.ie>
---
arch/x86/Kconfig | 7 ++-----
1 file changed, 2 insertions(+), 5 deletions(-)
--- 3.19-rc7/arch/x86/Kconfig
+++ 3.19-rc7-x86-Kconfig-UP-APIC-MSI/arch/x86/Kconfig
@@ -856,7 +856,8 @@ config SCHED_MC
source "kernel/Kconfig.preempt"
config X86_UP_APIC
- bool "Local APIC support on uniprocessors"
+ bool "Local APIC support on uniprocessors" if !PCI_MSI
+ default PCI_MSI
depends on X86_32 && !SMP && !X86_32_NON_STANDARD
---help---
A local APIC (Advanced Programmable Interrupt Controller) is an
@@ -868,10 +869,6 @@ config X86_UP_APIC
performance counters), and the NMI watchdog which detects hard
lockups.
-config X86_UP_APIC_MSI
- def_bool y
- select X86_UP_APIC if X86_32 && !SMP && !X86_32_NON_STANDARD && PCI_MSI
-
config X86_UP_IOAPIC
bool "IO-APIC support on uniprocessors"
depends on X86_UP_APIC
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] x86/Kconfig: simplify X86_UP_APIC handling
2015-02-05 15:31 [PATCH] x86/Kconfig: simplify X86_UP_APIC handling Jan Beulich
@ 2015-02-06 0:00 ` Bryan O'Donoghue
2015-02-19 0:27 ` [tip:x86/build] x86/Kconfig: Simplify " tip-bot for Jan Beulich
1 sibling, 0 replies; 3+ messages in thread
From: Bryan O'Donoghue @ 2015-02-06 0:00 UTC (permalink / raw)
To: Jan Beulich, mingo, tglx, hpa; +Cc: linux-kernel
On 05/02/15 15:31, Jan Beulich wrote:
> We don't really need a helper symbol for that. For one, it's
> pointlessly getting set to Y for all configurations (even 64-bit ones).
> And then the purpose can be fulfilled by suitably adjusting
> X86_UP_APIC: Hide its prompt when PCI_MSI, and default it to PCI_MSI.
ACK
tested works for me the same as the previous patch - plus it's one
symbol les.
Tested-by: Bryan O'Donoghue <pure.logic@nexus-software.ie>
^ permalink raw reply [flat|nested] 3+ messages in thread
* [tip:x86/build] x86/Kconfig: Simplify X86_UP_APIC handling
2015-02-05 15:31 [PATCH] x86/Kconfig: simplify X86_UP_APIC handling Jan Beulich
2015-02-06 0:00 ` Bryan O'Donoghue
@ 2015-02-19 0:27 ` tip-bot for Jan Beulich
1 sibling, 0 replies; 3+ messages in thread
From: tip-bot for Jan Beulich @ 2015-02-19 0:27 UTC (permalink / raw)
To: linux-tip-commits
Cc: tglx, jbeulich, JBeulich, mingo, hpa, linux-kernel, pure.logic
Commit-ID: 50849eefea3ba8aa6e540e0cbdc9533098f25656
Gitweb: http://git.kernel.org/tip/50849eefea3ba8aa6e540e0cbdc9533098f25656
Author: Jan Beulich <JBeulich@suse.com>
AuthorDate: Thu, 5 Feb 2015 15:31:56 +0000
Committer: Ingo Molnar <mingo@kernel.org>
CommitDate: Wed, 18 Feb 2015 22:10:54 +0100
x86/Kconfig: Simplify X86_UP_APIC handling
We don't really need a helper symbol for that. For one, it's
pointlessly getting set to Y for all configurations (even 64-bit
ones). And then the purpose can be fulfilled by suitably
adjusting X86_UP_APIC: Hide its prompt when PCI_MSI, and default
it to PCI_MSI.
Tested-by: Bryan O'Donoghue <pure.logic@nexus-software.ie>
Signed-off-by: Jan Beulich <jbeulich@suse.com>
Link: http://lkml.kernel.org/r/54D39AFC020000780005D684@mail.emea.novell.com
Signed-off-by: Ingo Molnar <mingo@kernel.org>
---
arch/x86/Kconfig | 7 ++-----
1 file changed, 2 insertions(+), 5 deletions(-)
diff --git a/arch/x86/Kconfig b/arch/x86/Kconfig
index afb75f5..c226c2b 100644
--- a/arch/x86/Kconfig
+++ b/arch/x86/Kconfig
@@ -859,7 +859,8 @@ config UP_LATE_INIT
depends on !SMP && X86_LOCAL_APIC
config X86_UP_APIC
- bool "Local APIC support on uniprocessors"
+ bool "Local APIC support on uniprocessors" if !PCI_MSI
+ default PCI_MSI
depends on X86_32 && !SMP && !X86_32_NON_STANDARD
---help---
A local APIC (Advanced Programmable Interrupt Controller) is an
@@ -871,10 +872,6 @@ config X86_UP_APIC
performance counters), and the NMI watchdog which detects hard
lockups.
-config X86_UP_APIC_MSI
- def_bool y
- select X86_UP_APIC if X86_32 && !SMP && !X86_32_NON_STANDARD && PCI_MSI
-
config X86_UP_IOAPIC
bool "IO-APIC support on uniprocessors"
depends on X86_UP_APIC
^ permalink raw reply related [flat|nested] 3+ messages in thread
end of thread, other threads:[~2015-02-19 0:27 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-02-05 15:31 [PATCH] x86/Kconfig: simplify X86_UP_APIC handling Jan Beulich
2015-02-06 0:00 ` Bryan O'Donoghue
2015-02-19 0:27 ` [tip:x86/build] x86/Kconfig: Simplify " tip-bot for Jan Beulich
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).