[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20070525101248.GA7547@elte.hu>
Date: Fri, 25 May 2007 12:12:48 +0200
From: Ingo Molnar <mingo@...e.hu>
To: Andi Kleen <andi@...stfloor.org>
Cc: Andrew Morton <akpm@...ux-foundation.org>,
Satyam Sharma <satyam.sharma@...il.com>,
linux-kernel@...r.kernel.org,
Linus Torvalds <torvalds@...ux-foundation.org>,
Peter Zijlstra <a.p.zijlstra@...llo.nl>
Subject: Re: [patch] x86_64: fix sched_clock()
* Andi Kleen <andi@...stfloor.org> wrote:
> > please indicate that you've picked up my style cleanups, i dont want
> > to redo all this a few days/weeks down the line ...
>
> It's done slightly differently now due to conflicting earlier changes,
> but the end result should be about what you intended. [...]
please send me your current sched-clock.c, i'll redo any remaining
cleanups.
But ... i find your approach curious, why didnt you just apply the
cleanups i sent? You clearly started working on this as a reaction to my
cleanup patches and to the bugfixes i sent ontop of the cleanup patches.
Your "I'll do this differently" approach is totally unnecessary from a
commit management point of view (this is new code after all and
will/should go upstream in a single clean chunk anyway), the only effect
this has is that that you are discouraging contributors like me from
contributing cleanups to the x86_64 tree.
To further underline this feeling i got, none of your reactions to any
of my patches showed even a single positive thought that you are happy
about people fixing code you are introducing (in fact you didnt even
indicate which patch you took, only at my repeated prodding did you say
anything about the cleanup patch i sent!), so to me the impression is
that deep in yourself you are (subconsciously) not happy about others
contributing to the x86_64 tree. Please tell me that i'm wrong :-(
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