Re: [patch] per cpu MCA/INIT fixes.

From: Jesse Barnes <jbarnes_at_engr.sgi.com>
Date: 2005-01-07 04:14:06
On Wednesday, January 5, 2005 7:08 pm, Russ Anderson wrote:
>  #ifdef CONFIG_VIRTUAL_MEM_MAP
> Index: linux/arch/ia64/mm/discontig.c
> ===================================================================
> --- linux.orig/arch/ia64/mm/discontig.c 2005-01-05 15:26:42.455970944 -0600
> +++ linux/arch/ia64/mm/discontig.c 2005-01-05 15:33:50.644044221 -0600
> @@ -348,12 +348,12 @@
>    mem_data[node].node_data = __va(pernode);
>    pernode += L1_CACHE_ALIGN(sizeof(struct ia64_node_data));
>
> -  mca_data_phys = (void *)pernode;
> -  pernode += L1_CACHE_ALIGN(sizeof(ia64_mca_cpu_t)) * phys_cpus;
> -
>    mem_data[node].pgdat->bdata = bdp;
>    pernode += L1_CACHE_ALIGN(sizeof(pg_data_t));
>
> +  mca_data_phys = (void *)pernode;
> +  pernode += L1_CACHE_ALIGN(sizeof(ia64_mca_cpu_t)) * phys_cpus;
> +

The above will probably conflict with Jack's recent change to stagger the 
per-node structures by one cacheline.  You may have to rediff against one of 
Tony's test trees.

> +#ifdef CONFIG_SMP
> +#include <asm/mca.h>
> +#endif

Just a nit, header files should always be safe to include unconditionally.  If 
they're not, the headers need fixing (barring out of tree stuff like kdb of 
course).

Jesse

-
To unsubscribe from this list: send the line "unsubscribe linux-ia64" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Received on Thu Jan 6 12:15:00 2005

This archive was generated by hypermail 2.1.8 : 2005-08-02 09:20:34 EST