* [PATCH 1/1] i383 numa: fix numaq/summit apicid conflict
@ 2006-10-04 1:25 keith mannthey
2006-10-04 9:57 ` Andy Whitcroft
0 siblings, 1 reply; 3+ messages in thread
From: keith mannthey @ 2006-10-04 1:25 UTC (permalink / raw)
To: torvalds; +Cc: lkml, andrew, apw
From: Keith Mannthey <kmannth@us.ibm.com>
This patch allows numaq to properly align cpus to their given node
during boot. Pass logical apicid to apicid_to_node and allow the summit
sub-arch to use physical apicid (hard_smp_processor_id()).
Tested against numaq and summit based systems with no issues. against
2.6.18-git18.
Signed-off-by: Keith Mannthey <kmannth@us.ibm.com>
---
arch/i386/kernel/smpboot.c | 2 +-
include/asm-i386/mach-summit/mach_apic.h | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff -urN linux-2.6.18/arch/i386/kernel/smpboot.c linux-2.6.18-git18/arch/i386/kernel/smpboot.c
--- linux-2.6.18/arch/i386/kernel/smpboot.c 2006-10-02 02:59:49.000000000 -0700
+++ linux-2.6.18-git18/arch/i386/kernel/smpboot.c 2006-10-02 00:36:52.000000000 -0700
@@ -648,7 +648,7 @@
{
int cpu = smp_processor_id();
int apicid = logical_smp_processor_id();
- int node = apicid_to_node(hard_smp_processor_id());
+ int node = apicid_to_node(apicid);
if (!node_online(node))
node = first_online_node;
diff -urN linux-2.6.18/include/asm-i386/mach-summit/mach_apic.h linux-2.6.18-git18/include/asm-i386/mach-summit/mach_apic.h
--- linux-2.6.18/include/asm-i386/mach-summit/mach_apic.h 2006-10-02 02:59:54.000000000 -0700
+++ linux-2.6.18-git18/include/asm-i386/mach-summit/mach_apic.h 2006-10-02 00:51:24.000000000 -0700
@@ -88,7 +88,7 @@
static inline int apicid_to_node(int logical_apicid)
{
- return apicid_2_node[logical_apicid];
+ return apicid_2_node[hard_smp_processor_id()];
}
/* Mapping from cpu number to logical apicid */
^ permalink raw reply [flat|nested] 3+ messages in thread* Re: [PATCH 1/1] i383 numa: fix numaq/summit apicid conflict
2006-10-04 1:25 [PATCH 1/1] i383 numa: fix numaq/summit apicid conflict keith mannthey
@ 2006-10-04 9:57 ` Andy Whitcroft
2006-10-04 17:46 ` keith mannthey
0 siblings, 1 reply; 3+ messages in thread
From: Andy Whitcroft @ 2006-10-04 9:57 UTC (permalink / raw)
To: kmannth; +Cc: torvalds, lkml, andrew
keith mannthey wrote:
> From: Keith Mannthey <kmannth@us.ibm.com>
>
> This patch allows numaq to properly align cpus to their given node
> during boot. Pass logical apicid to apicid_to_node and allow the summit
> sub-arch to use physical apicid (hard_smp_processor_id()).
> Tested against numaq and summit based systems with no issues. against
> 2.6.18-git18.
>
> Signed-off-by: Keith Mannthey <kmannth@us.ibm.com>
> ---
> arch/i386/kernel/smpboot.c | 2 +-
> include/asm-i386/mach-summit/mach_apic.h | 2 +-
> 2 files changed, 2 insertions(+), 2 deletions(-)
>
> diff -urN linux-2.6.18/arch/i386/kernel/smpboot.c linux-2.6.18-git18/arch/i386/kernel/smpboot.c
> --- linux-2.6.18/arch/i386/kernel/smpboot.c 2006-10-02 02:59:49.000000000 -0700
> +++ linux-2.6.18-git18/arch/i386/kernel/smpboot.c 2006-10-02 00:36:52.000000000 -0700
> @@ -648,7 +648,7 @@
> {
> int cpu = smp_processor_id();
> int apicid = logical_smp_processor_id();
> - int node = apicid_to_node(hard_smp_processor_id());
> + int node = apicid_to_node(apicid);
>
> if (!node_online(node))
> node = first_online_node;
> diff -urN linux-2.6.18/include/asm-i386/mach-summit/mach_apic.h linux-2.6.18-git18/include/asm-i386/mach-summit/mach_apic.h
> --- linux-2.6.18/include/asm-i386/mach-summit/mach_apic.h 2006-10-02 02:59:54.000000000 -0700
> +++ linux-2.6.18-git18/include/asm-i386/mach-summit/mach_apic.h 2006-10-02 00:51:24.000000000 -0700
> @@ -88,7 +88,7 @@
>
> static inline int apicid_to_node(int logical_apicid)
> {
> - return apicid_2_node[logical_apicid];
> + return apicid_2_node[hard_smp_processor_id()];
> }
>
> /* Mapping from cpu number to logical apicid */
My worry here is that we might have users who are calling this about
other cpus. As you have effectivly ignored the parameter on summit here.
Can we not just map the hard_smp_processor_id to its logical apic id
when filling in the apicid_2_node array on summit? Such that it really
does have the logical id in there?
-apw
^ permalink raw reply [flat|nested] 3+ messages in thread* Re: [PATCH 1/1] i383 numa: fix numaq/summit apicid conflict
2006-10-04 9:57 ` Andy Whitcroft
@ 2006-10-04 17:46 ` keith mannthey
0 siblings, 0 replies; 3+ messages in thread
From: keith mannthey @ 2006-10-04 17:46 UTC (permalink / raw)
To: Andy Whitcroft; +Cc: torvalds, lkml, andrew
On Wed, 2006-10-04 at 10:57 +0100, Andy Whitcroft wrote:
> keith mannthey wrote:
> > From: Keith Mannthey <kmannth@us.ibm.com>
> >
> > This patch allows numaq to properly align cpus to their given node
> > during boot. Pass logical apicid to apicid_to_node and allow the summit
> > sub-arch to use physical apicid (hard_smp_processor_id()).
> > Tested against numaq and summit based systems with no issues. against
> > 2.6.18-git18.
> >
> > Signed-off-by: Keith Mannthey <kmannth@us.ibm.com>
> > ---
> > arch/i386/kernel/smpboot.c | 2 +-
> > include/asm-i386/mach-summit/mach_apic.h | 2 +-
> > 2 files changed, 2 insertions(+), 2 deletions(-)
> >
> > diff -urN linux-2.6.18/arch/i386/kernel/smpboot.c linux-2.6.18-git18/arch/i386/kernel/smpboot.c
> > --- linux-2.6.18/arch/i386/kernel/smpboot.c 2006-10-02 02:59:49.000000000 -0700
> > +++ linux-2.6.18-git18/arch/i386/kernel/smpboot.c 2006-10-02 00:36:52.000000000 -0700
> > @@ -648,7 +648,7 @@
> > {
> > int cpu = smp_processor_id();
> > int apicid = logical_smp_processor_id();
> > - int node = apicid_to_node(hard_smp_processor_id());
> > + int node = apicid_to_node(apicid);
> >
> > if (!node_online(node))
> > node = first_online_node;
> > diff -urN linux-2.6.18/include/asm-i386/mach-summit/mach_apic.h linux-2.6.18-git18/include/asm-i386/mach-summit/mach_apic.h
> > --- linux-2.6.18/include/asm-i386/mach-summit/mach_apic.h 2006-10-02 02:59:54.000000000 -0700
> > +++ linux-2.6.18-git18/include/asm-i386/mach-summit/mach_apic.h 2006-10-02 00:51:24.000000000 -0700
> > @@ -88,7 +88,7 @@
> >
> > static inline int apicid_to_node(int logical_apicid)
> > {
> > - return apicid_2_node[logical_apicid];
> > + return apicid_2_node[hard_smp_processor_id()];
> > }
> >
> > /* Mapping from cpu number to logical apicid */
>
> My worry here is that we might have users who are calling this about
> other cpus. As you have effectivly ignored the parameter on summit here.
I sort of thought about riping out the parameter to apicid_to_node and
letting summit and numaq fill in whatever they need. apicid_to_node
should only be used in map_cpu_to_logical_apicid. After this the kernel
has a nice cpu to node mapping and a great interface for it.
> Can we not just map the hard_smp_processor_id to its logical apic id
> when filling in the apicid_2_node array on summit? Such that it really
> does have the logical id in there?
The apicid_2_node array comes right from the SRAT table.... Without
constructing yet another table I don't seem to have a good way to
translate logical to physical or vice versa.
Maybe making the callers of apicid_to_node pass both logical and
physical apicid will relive the worry?
Sure we can create another bootime mapping but it seems overkill.
Really only summit and numaq do anything useful with apicid_to_node and
i386 numa for that matter and I don't expect any new numa sub
arches....
What do you think?
Thanks,
Keith
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2006-10-04 17:46 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2006-10-04 1:25 [PATCH 1/1] i383 numa: fix numaq/summit apicid conflict keith mannthey
2006-10-04 9:57 ` Andy Whitcroft
2006-10-04 17:46 ` keith mannthey
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox