Re: [PATCH 4/14] i386 / Clean up asm and volatile keywords in desc

2005-08-16 Thread H. Peter Anvin

Andi Kleen wrote:

On Wed, Aug 10, 2005 at 09:53:51PM -0700, [EMAIL PROTECTED] wrote:


Stop using extra underscores on asm and volatiles, that is just silly.


Actually the volatiles might be still useful. Or if you drop them
at least add memory clobbers. I had sometimes bugs  on x86-64
with the compiler moving such assembly statements with invisible 
side effects around too aggressively and causing weird problems.


Agreed on the underscores, I hate them too :)



Well, put volatiles in *unless* the only effect of an asm statement is 
producing the output values.


-hpa

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


Re: [PATCH 4/14] i386 / Clean up asm and volatile keywords in desc

2005-08-16 Thread Chris Wright
* Andi Kleen ([EMAIL PROTECTED]) wrote:
> On Wed, Aug 10, 2005 at 09:53:51PM -0700, [EMAIL PROTECTED] wrote:
> > Stop using extra underscores on asm and volatiles, that is just silly.
> 
> Actually the volatiles might be still useful. Or if you drop them
> at least add memory clobbers.

They are still there, just the underscores on both asm and volatile got
pulled.

> I had sometimes bugs  on x86-64
> with the compiler moving such assembly statements with invisible 
> side effects around too aggressively and causing weird problems.
> 
> Agreed on the underscores, I hate them too :)

Heh, same here ;-)

thanks,
-chris
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [PATCH 4/14] i386 / Clean up asm and volatile keywords in desc

2005-08-16 Thread Andi Kleen
On Wed, Aug 10, 2005 at 09:53:51PM -0700, [EMAIL PROTECTED] wrote:
> Stop using extra underscores on asm and volatiles, that is just silly.

Actually the volatiles might be still useful. Or if you drop them
at least add memory clobbers. I had sometimes bugs  on x86-64
with the compiler moving such assembly statements with invisible 
side effects around too aggressively and causing weird problems.

Agreed on the underscores, I hate them too :)

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