Re: [PATCH 1/3] x86: Readd asmregparm

[Ingo Molnar <mingo@elte.hu>]

* Andi Kleen <andi@firstfloor.org> wrote:

> > That's a totally useless changelog (you are deliberately hiding 
> > information again) so the series is obviously not acceptable.
> > Please fix and resend.
> 
> I reread it and it seems complete to me. Please be more specific.
> 
> If you refer to the followon patchkit - it's not ready for posting 
> yet. [...]

You contradict yourself here - why do you claim that it's "complete" 
while you readily admit in the *next* sentence that the "followon 
patchkit" (with no specifics provided whatsoever) is "not ready for 
posting yet"?

You are not a newbie anymore, you know what the rules are for 
*everyone*: if it's not ready for posting yet then don't send a 
half-done series for inclusion, wasting everyone's time.

> [...] However I believe these changes are reasonable cleanups that 
> are useful on their own. That's why I posted them now.

Uhm, that's trivially untrue crazy talk and you should know it: the 
patches add code and typing complexity for no good stated reason 
whatsoever.

If we want the maintenance overhead of that then you better come up 
with your deliberately kept secret "project" and we'll be able to 
weigh whether the benefits overweigh the maintenance costs ...

In any case, i'd like to warn you that you are very close to me 
starting to ignore your mails summarily due to absymal low quality: 
in the past few months 70-80% of the cases your first patch 
submissions were trivially flawed, and in many cases you should have 
been well aware of the fact that they were trivially flawed. And 
that's not from a newbie but from a long-time kernel contributor!

Thanks,

	Ingo