Re: [PATCH] Partially revert patch that encloses asm-offset.h numbersin brackets

From: Linus Torvalds
Date: Mon Oct 25 2010 - 11:18:30 EST


On Mon, Oct 25, 2010 at 8:05 AM, Jan Beulich <JBeulich@xxxxxxxxxx> wrote:
>
> Alexander, you seem to be having access to a worst case system -
> could you give this a try?

This all seems very wrong.

The whole "add parenthesis" seems to break several architectures
(Power apparently had problems too due to redefining one of the
macros), and it is ugly.

And the whole thing was added for some stupid gas bug for a very
specific case, and now we're adding _more_ crap to an already ugly
workaround.

How about just removing the idiotic workaround entirely, and then on
x86 passing "-traditional" to cpp for asm input so that it doesn't add
spaces in argument expansion? Make the workaround more targeted to the
actual problem, rather than working around it in some unrelated
thing..

Linus
--
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/