Re: not needed patch

From: Yinghai Lu
Date: Thu Dec 20 2007 - 14:45:42 EST


On Thursday 20 December 2007 06:29:06 am Ingo Molnar wrote:
>
> * Yinghai Lu <Yinghai.Lu@xxxxxxx> wrote:
>
> > Ingo.
> >
> > commit fbdcf18df73758b2e187ab94678b30cd5f6ff9f9 is not needed. another
> > patch (by you !! commit 699d934d5f958d7944d195c03c334f28cc0b3669 x86:
> > fixup cpu_info array conversion) already removed clearing of
> > c->cpu_index. in identify_cpu
> > also it is not consisent to smpboot_32.c. (it will assign id to
> > cpu_index right after
> >
> > *c = boot_cpu_data;
> > )
>
> well, it might in the worst-case be a superfluous change, but not cause
> any problems in 2.6.24, right?

now it is ok with 2.6.24.

>
> > by revert commit fbdcf18df73758b2e187ab94678b30cd5f6ff9f9, we could
> > use c->cpu_index in identify_cpu.
>
> but that's 2.6.25 stuff, right? Travis?
or at least before bewfore merging smpboot_32.c and smpboot_64.c

YH
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/