lkml.org 
[lkml]   [2004]   [Oct]   [28]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
SubjectRe: [PATCH] clustered apic patch missing APIC_DFR_CLUSTER def
Date
Hmmm...  The patch containing APIC_DFR_CLUSTER and friends went
into the -mm tree in the June/July timeframe. It must not have
been pushed with the main cluster patch.

You're right, we're using the generic version of
cpu_present_to_apicid(). The cluster one can go.


On Thursday 28 October 2004 11:27 am, Chris Wright wrote:
> The cluster apic support does not compile. I added the
> APIC_DFR_CLUSTER def to asm-x86_64/apicdef.h (stolen from i386 one).
> Also added APIC_DFR_FLAT while there. This may only be papering over
> real fix.
>
> Additionally, cluster_cpu_present_to_apicid() is defined but not used
> anywhere. So remove it.
>
> Signed-off-by: Chris Wright <chrisw@osdl.org>
>
> ===== include/asm-x86_64/apicdef.h 1.6 vs edited =====
> --- 1.6/include/asm-x86_64/apicdef.h 2004-10-28 00:39:50 -07:00
> +++ edited/include/asm-x86_64/apicdef.h 2004-10-28 11:15:19 -07:00
> @@ -32,6 +32,8 @@
> #define SET_APIC_LOGICAL_ID(x) (((x)<<24))
> #define APIC_ALL_CPUS 0xFFu
> #define APIC_DFR 0xE0
> +#define APIC_DFR_CLUSTER 0x0FFFFFFFu
> +#define APIC_DFR_FLAT 0xFFFFFFFFu
> #define APIC_SPIV 0xF0
> #define APIC_SPIV_FOCUS_DISABLED (1<<9)
> #define APIC_SPIV_APIC_ENABLED (1<<8)
> ===== arch/x86_64/kernel/genapic_cluster.c 1.1 vs edited =====
> --- 1.1/arch/x86_64/kernel/genapic_cluster.c 2004-10-28 00:39:50
> -07:00 +++ edited/arch/x86_64/kernel/genapic_cluster.c 2004-10-28
> 11:18:10 -07:00 @@ -57,14 +57,6 @@
> apic_write_around(APIC_LDR, val);
> }
>
> -static int cluster_cpu_present_to_apicid(int mps_cpu)
> -{
> - if ((unsigned)mps_cpu < NR_CPUS)
> - return (int)bios_cpu_apicid[mps_cpu];
> - else
> - return BAD_APICID;
> -}
> -
> /* Start with all IRQs pointing to boot CPU. IRQ balancing will
> shift them. */
>
> static cpumask_t cluster_target_cpus(void)

--
James Cleverdon
IBM LTC (xSeries Linux Solutions)
{jamesclv(Unix, preferred), cleverdj(Notes)} at us dot ibm dot comm

-
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/

\
 
 \ /
  Last update: 2005-03-22 14:07    [W:0.056 / U:0.148 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site