On Sat, Oct 08, 2005 at 12:28:38PM +0200, Andi Kleen wrote:
> On Saturday 08 October 2005 02:52, Siddha, Suresh B wrote:
> > On Fri, Oct 07, 2005 at 11:52:00AM +0200, Andi Kleen wrote:
> > > > I can fix the API mess. Is there anything else you want me to do?
> > >
> > > I think you overdid the sharing. Can you limit it to one file
> > > and copy the stuff that doesn't fit easily?
> >
> > Andi, This stuff is very much common to x86 and x86_64. Shared code is
> > split into two files because setting up sibling map code is generic and
> > HT/core detection code is very specific to Intel.
> >
> > How about the appended patch?
>
> I would prefer if the Intel CPU detection support wasn't distributed over so
> many small files. If you prefer to share it put it all into a single file and
> share that. But please only for code that can be cleanly shared without
> ifdefs.
Lets defer this code sharing to some other time. I want to make sure that
-mm tree (and finally 2.6.15) picks up these enhancements first, before
I start my vacation :)
> Also in general it would be better if you first did the cleanup and then
> as separate patches the various functionality enhancements.That makes
> the changes easier to be reviewed and it helps in binary search when something
> goes wrong.
I am going to send two follow up patches which addresses the functionality
enhancements.
thanks,
suresh
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
[Index of Archives]
[Kernel Newbies]
[Netfilter]
[Bugtraq]
[Photo]
[Stuff]
[Gimp]
[Yosemite News]
[MIPS Linux]
[ARM Linux]
[Linux Security]
[Linux RAID]
[Video 4 Linux]
[Linux for the blind]
[Linux Resources]
