Till Straumann <[EMAIL PROTECTED]> writes:

> /* Powerpc I/O barrier instruction */
> #define EIEIO(pmem) do { asm volatile("eieio":"=m"(*pmem):"m"(*pmem)); }
> while (0)

Looking closer, your asm statement has a bug.  The "m" constraint can
match memory addresses with side effects (auto inc/dec), but the insn
does not carry out that side effect.  On powerpc the side effect must be
encoded through the update form of the load/store insns.  If you don't
use a load or store insn with the operand the you must use the "o"
constraint to avoid the side effect.

Andreas.

-- 
Andreas Schwab, SuSE Labs, [EMAIL PROTECTED]
SuSE Linux Products GmbH, Maxfeldstraße 5, 90409 Nürnberg, Germany
PGP key fingerprint = 58CA 54C7 6D53 942B 1756  01D3 44D5 214B 8276 4ED5
"And now for something completely different."

Reply via email to