Re: [RFC PATCH] x86: Unify include/asm-x86/linkage_[32|64].h

From: Ingo Molnar
Date: Tue Dec 04 2007 - 16:33:22 EST



* Harvey Harrison <harvey.harrison@xxxxxxxxx> wrote:

> I'm not sure if the definition of asmlinkage and prevent_tail_call can
> be omitted as well and let the linux/linkage.h version get picked up
> instead.

no, we cannot remove them - asmlinkage is needed for the syscall entry
(and other entry code) to work, the and the prevent_tail_call works
around a compiler bug. (which might or might not be fixed in latest gcc
- but we generally dont remove workarounds unless we are really sure
it's fine.)

so your patch looks good to me, i've queued it up.

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