diff options
author | Andi Kleen <ak@suse.de> | 2006-09-27 15:38:56 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@g5.osdl.org> | 2006-09-27 17:39:51 -0400 |
commit | 00463c1633b6d6a2178d2dc794c0a70ac2f9ce6b (patch) | |
tree | 83d02451139608ad70fb33c5f0e3bb90019841b2 /include | |
parent | cdb8355add9b1d87ecfcb58b12879897dc1e3e36 (diff) |
[PATCH] i386: Use early clobbers for semaphores now
The new code does clobber the result early, so make sure to tell
gcc to not put it into the same register as a input argument
Signed-off-by: Andi Kleen <ak@suse.de>
Cc: Andrew Morton <akpm@osdl.org>
Acked-by: Kyle McMartin <kyle@parisc-linux.org>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
Diffstat (limited to 'include')
-rw-r--r-- | include/asm-i386/semaphore.h | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/include/asm-i386/semaphore.h b/include/asm-i386/semaphore.h index e63b6a68f04c..4e34a468c383 100644 --- a/include/asm-i386/semaphore.h +++ b/include/asm-i386/semaphore.h | |||
@@ -126,7 +126,7 @@ static inline int down_interruptible(struct semaphore * sem) | |||
126 | "lea %1,%%eax\n\t" | 126 | "lea %1,%%eax\n\t" |
127 | "call __down_failed_interruptible\n" | 127 | "call __down_failed_interruptible\n" |
128 | "2:" | 128 | "2:" |
129 | :"=a" (result), "+m" (sem->count) | 129 | :"=&a" (result), "+m" (sem->count) |
130 | : | 130 | : |
131 | :"memory"); | 131 | :"memory"); |
132 | return result; | 132 | return result; |
@@ -148,7 +148,7 @@ static inline int down_trylock(struct semaphore * sem) | |||
148 | "lea %1,%%eax\n\t" | 148 | "lea %1,%%eax\n\t" |
149 | "call __down_failed_trylock\n\t" | 149 | "call __down_failed_trylock\n\t" |
150 | "2:\n" | 150 | "2:\n" |
151 | :"=a" (result), "+m" (sem->count) | 151 | :"=&a" (result), "+m" (sem->count) |
152 | : | 152 | : |
153 | :"memory"); | 153 | :"memory"); |
154 | return result; | 154 | return result; |