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

[Andi Kleen]

> So from my point of view, none of the changelogs explain why the
> changes needed. Now, I'm not an expert on the x86 code so maybe I'm
> that you need it some unspecified follow-up patchkit. HTH.

Proper asmlinkages are imho a cleanliness feature that stands
on its own. They have been used in the past for all kinds of things,
but unfortunately have bitrotted recently on x86.

If nothing else they are useful documentation. For example
if you do grep for *.[ch] only and you hit an asmlinkage it's a good
hint to look into the .S too. In general you need to be much more
careful about any changes to those, because assembler is more fragile.

I have been encouraged by other people to post those early, so I did.
and at least in my traditional Linux development world view posting 
patches early is the right thing to do.

-Andi
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/