[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20071220222429.GA28029@elte.hu>
Date: Thu, 20 Dec 2007 23:24:29 +0100
From: Ingo Molnar <mingo@...e.hu>
To: Jeremy Fitzhardinge <jeremy@...p.org>
Cc: LKML <linux-kernel@...r.kernel.org>, Andi Kleen <ak@...e.de>,
Thomas Gleixner <tglx@...utronix.de>,
Glauber de Oliveira Costa <glommer@...il.com>,
Jan Beulich <jbeulich@...ell.com>
Subject: Re: [PATCH 0/5] x86: another attempt at x86 pagetable unification
* Jeremy Fitzhardinge <jeremy@...p.org> wrote:
> But byte-for-byte identity isn't (necessarily) possible when actually
> unifying. If the same function exists in different forms on 32- and
> 64-bit, then unifying requires I pick one of them (or perhaps a new
> superset) to use in the unified form. That function may generate
> different code compared to the one that it replaced...
it's still possible: you can do preparatory patches that bring one
architecture in sync with the other one, in small, per function steps.
Then the actual unification is still an identity transformation. (and
all the preparatory patches are small and bisectable)
it's also a lot less frustrating and a lot more enjoyable that way IMO.
If it's 50 small patches, then so be it ... 50 patches only take ~2
seconds more for me to apply to x86.git (which time is immediately saved
by the vastly improved reviewability and testability of a 50 patches
set), so dont worry about any overhead on the maintainers side. And
you'll end up moving up on the v2.6.25 contributors top-list on LWN as
well ;-) The worst aspect of it is writing up the 50 changelogs (i use
pre-created templates for that) and figuring out how to script a
patch-bomb to lkml. In every other aspect it's a win-win scenario for
everyone involved.
Ingo
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists