login
Header Space

 
 

Re: x86 merge - a little feedback

Score:
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
To: Andi Kleen <andi@...>
Cc: Adrian Bunk <bunk@...>, Andi Kleen <ak@...>, Sam Ravnborg <sam@...>, Thomas Gleixner <tglx@...>, Ingo Molnar <mingo@...>, Linus Torvalds <torvalds@...>, LKML <linux-kernel@...>
Date: Sunday, September 16, 2007 - 1:08 am

On Sat, 15 Sep 2007 20:36:23 +0200 Andi Kleen <andi@firstfloor.org> wrote:


It's more complex, obviously.  More surprising.  It used to be the case that
arch/x86^4 files were xx86_64 and arch/i386 files were i386 and possibly
x86_64.  Now it's the case that arch/x86_64 files are x86_64 and maybe i386
and arch/i386 files are i386 and maybe x86_64.  Additional and quite
unnecessary complexity.

I mean, how often do x86_64 changes in your tree break i386?  Once every
3ish weeks would be my guess.  Often this will be because the person making
(and reviewing) the x86_64 change didn't know (or forgot) that the file is
also used by x86_64.


Doing something like that would reduce complexity, reduce surprise and
increase maintainability.  That's more than warm-and-fuzzies.
-
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
x86 merge - a little feedback, Sam Ravnborg, (Tue Sep 11, 4:12 pm)
Re: x86 merge - a little feedback, Linus Torvalds, (Tue Sep 11, 5:21 pm)
Re: x86 merge - a little feedback, Sam Ravnborg, (Wed Sep 12, 3:09 pm)
Re: x86 merge - a little feedback, Andi Kleen, (Tue Sep 11, 4:38 pm)
Re: x86 merge - a little feedback, Adrian Bunk, (Tue Sep 11, 5:14 pm)
Re: x86 merge - a little feedback, Andrew Morton, (Sat Sep 15, 5:32 am)
Re: x86 merge - a little feedback, Andi Kleen, (Sat Sep 15, 2:36 pm)
Re: x86 merge - a little feedback, Andrew Morton, (Sun Sep 16, 1:08 am)
Re: x86 merge - a little feedback, Andi Kleen, (Tue Sep 11, 5:34 pm)
Re: x86 merge - a little feedback, Paul Mundt, (Tue Sep 11, 8:29 pm)
Re: x86 merge - a little feedback, Andrew Morton, (Sat Sep 15, 6:55 am)
Re: x86 merge - a little feedback, Linus Torvalds, (Tue Sep 11, 5:51 pm)
Re: x86 merge - a little feedback, Jan Engelhardt, (Wed Sep 12, 2:14 pm)
Re: x86 merge - a little feedback, Adrian Bunk, (Tue Sep 11, 5:51 pm)
Re: x86 merge - a little feedback, Adrian Bunk, (Tue Sep 11, 4:34 pm)
Re: x86 merge - a little feedback, Christoph Hellwig, (Wed Sep 12, 5:27 am)
Re: x86 merge - a little feedback, Lennart Sorensen, (Wed Sep 12, 8:45 am)
Re: x86 merge - a little feedback, Sam Ravnborg, (Tue Sep 11, 5:05 pm)
Re: x86 merge - a little feedback, Adrian Bunk, (Tue Sep 11, 5:09 pm)
Re: x86 merge - a little feedback, Thomas Gleixner, (Tue Sep 11, 4:25 pm)
Re: x86 merge - a little feedback, Linus Torvalds, (Tue Sep 11, 5:24 pm)
speck-geostationary