Re: [RFC PATCH v2 -tip 0/4] x86: signal handler improvement

From: Ingo Molnar
Date: Sat Sep 27 2008 - 13:14:39 EST



* H. Peter Anvin <hpa@xxxxxxxxx> wrote:

> Hiroshi Shimamoto wrote:
>>>>
>>> This is a gcc failure, and should be reported to the gcc people.
>>
>> Does this gcc failure mean unnecessary storing into stack?
>
> Yes, it should be able to process this in a register, instead of
> storing to the stack and then merging later.
>
> It's possible it's trying to do that to hide latency, but that's
> clearly a lose in this case.
>
> I'd hate to obfuscate the code, and I'd *really* hate to obfuscate the
> code to work around gcc brokenness, and then not even bothering to
> tell the gcc folks.

assuming it's reported to the gcc folks too, do you have any objections
against including this in tip/x86/signal? The 1% improvement in lmbench
is quite nice.

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/