lkml.org 
[lkml]   [2009]   [Aug]   [6]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: linux-next: manual merge of the edac-amd tree with the rr tree
Hi,

On Thu, Aug 06, 2009 at 05:51:52PM +1000, Stephen Rothwell wrote:
> Hi Borislav,
>
> Today's linux-next merge of the edac-amd tree got a conflict in
> arch/x86/kernel/smpboot.c between commit
> d39044d21e1a2d3713f1329ad22d23df5d20140b ("cpumask: use zalloc_cpumask_var
> () where possible") from the rr tree and commit
> a763af6587fbead6cee75ad3ae14709fa67e7dbb ("x86: provide CPU topology
> information for multi-node processors") from the edac-amd tree.
>
> I fixed it up (see below) and can carry the fix as necessary.

I guess this is the best solution for now. Andreas' latest patches went
out yesterday and they too contain the

alloc_cpumask_var
cpumask_clear

order. Which means, this fix will have to be applied then between the
tip and the rr trees after Ingo merges Andreas' stuff, so please hold on
to it until after the merge window.

Thanks.

> --
> Cheers,
> Stephen Rothwell sfr@canb.auug.org.au
>
> diff --cc arch/x86/kernel/smpboot.c
> index 70bd79b,5428337..0000000
> --- a/arch/x86/kernel/smpboot.c
> +++ b/arch/x86/kernel/smpboot.c
> @@@ -1057,9 -1068,14 +1068,10 @@@ void __init native_smp_prepare_cpus(uns
> #endif
> current_thread_info()->cpu = 0; /* needed? */
> for_each_possible_cpu(i) {
> - alloc_cpumask_var(&per_cpu(cpu_sibling_map, i), GFP_KERNEL);
> - alloc_cpumask_var(&per_cpu(cpu_core_map, i), GFP_KERNEL);
> - alloc_cpumask_var(&per_cpu(cpu_node_map, i), GFP_KERNEL);
> - alloc_cpumask_var(&cpu_data(i).llc_shared_map, GFP_KERNEL);
> - cpumask_clear(per_cpu(cpu_core_map, i));
> - cpumask_clear(per_cpu(cpu_node_map, i));
> - cpumask_clear(per_cpu(cpu_sibling_map, i));
> - cpumask_clear(cpu_data(i).llc_shared_map);
> + zalloc_cpumask_var(&per_cpu(cpu_sibling_map, i), GFP_KERNEL);
> + zalloc_cpumask_var(&per_cpu(cpu_core_map, i), GFP_KERNEL);
> ++ zalloc_cpumask_var(&per_cpu(cpu_node_map, i), GFP_KERNEL);
> + zalloc_cpumask_var(&cpu_data(i).llc_shared_map, GFP_KERNEL);
> }
> set_cpu_sibling_map(0);
>
> --
> 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/

--
Regards/Gruss,
Boris.


\
 
 \ /
  Last update: 2009-08-06 10:09    [W:0.055 / U:0.480 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site