David Miller wrote:
> From: Tejun Heo <tj@xxxxxxxxxx>
> Date: Wed, 12 Aug 2009 15:10:29 +0900
>
>> Tejun Heo wrote:
>>> Ingo Molnar wrote:
>>>> for the x86 bits:
>>>>
>>>> Acked-by: Ingo Molnar <mingo@xxxxxxx>
>>>>
>>>> Once Dave acks it i suspect you can send it to Linus directly?
>> Aieeeee, no response.
>
> These patches look fine, and I've tested them also on my
> Niagara-2 box.
>
> Acked-by: David S. Miller <davem@xxxxxxxxxxxxx>
>
> Sorry for the delay :)
Great. Oops, last review pass revealed a bug in the following chunk.
@@ -1259,9 +1259,16 @@ ssize_t __init pcpu_embed_first_chunk(size_t static_size, size_t reserved_size,
}
/* return the leftover and copy */
- for_each_possible_cpu(cpu) {
+ for (cpu = 0; cpu < nr_cpu_ids; cpu++) {
void *ptr = pcpue_ptr + cpu * pcpue_unit_size;
+ if (cpu_possible(cpu)) {
+ free_bootmem(__pa(ptr + pcpue_size),
+ pcpue_unit_size - pcpue_size);
+ memcpy(ptr, __per_cpu_load, static_size);
+ } else
+ free_bootmem(__pa(ptr), pcpue_unit_size);
+
free_bootmem(__pa(ptr + pcpue_size),
pcpue_unit_size - pcpue_size);
memcpy(ptr, __per_cpu_load, static_size);
The if-else block should replace the original free_bootmem()/memcpy()
not added in front of it. Fixed. Will reply with updated patch.
Thanks.
--
tejun
--
To unsubscribe from this list: send the line "unsubscribe sparclinux" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
