hwrng: via_rng - Fix asm constraints
authorHerbert Xu <herbert@gondor.apana.org.au>
Fri, 7 Jan 2011 03:48:57 +0000 (14:48 +1100)
committerHerbert Xu <herbert@gondor.apana.org.au>
Fri, 7 Jan 2011 03:48:57 +0000 (14:48 +1100)
The inline asm to invoke xstore did not specify the constraints
correctly.  In particular, dx/di should have been marked as output
registers as well as input as they're modified by xstore.

Thanks to Mario Holbe for creating this patch and testing it.

Tested-by: Mario 'BitKoenig' Holbe <Mario.Holbe@TU-Ilmenau.DE>
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
drivers/char/hw_random/via-rng.c

index 794aacb715c1d85002808f6a7d194b012f85abaf..7f86666bb393abcf6743343b7af19e6096ba1564 100644 (file)
@@ -81,8 +81,7 @@ static inline u32 xstore(u32 *addr, u32 edx_in)
        ts_state = irq_ts_save();
 
        asm(".byte 0x0F,0xA7,0xC0 /* xstore %%edi (addr=%0) */"
-               :"=m"(*addr), "=a"(eax_out)
-               :"D"(addr), "d"(edx_in));
+               : "=m" (*addr), "=a" (eax_out), "+d" (edx_in), "+D" (addr));
 
        irq_ts_restore(ts_state);
        return eax_out;