diff options
author | Jan Beulich <jbeulich@suse.com> | 2013-09-26 10:11:00 +0200 |
---|---|---|
committer | Jan Beulich <jbeulich@suse.com> | 2013-09-26 10:11:00 +0200 |
commit | 573fd5bd2d8bd068a6cea7ae5fcc58b962d43a78 (patch) | |
tree | 5140d0ac2d32456dcaaec3d681fd3f84582610c4 | |
parent | 7099bd90e1b84cb6a0143835ba050a445161715d (diff) | |
download | xen-573fd5bd2d8bd068a6cea7ae5fcc58b962d43a78.tar.gz xen-573fd5bd2d8bd068a6cea7ae5fcc58b962d43a78.tar.bz2 xen-573fd5bd2d8bd068a6cea7ae5fcc58b962d43a78.zip |
x86: fix rdrand asm()
Just learned the hard way that at least for non-volatile asm()s gcc
indeed does what the documentation says: It may move it across jumps
(i.e. ahead of the cpu_has() check). While the documentation claims
that this can also happen for volatile asm()s, if that was the case
we'd have many more problems in our code (and e,g, Linux would too).
Signed-off-by: Jan Beulich <jbeulich@suse.com>
Reviewed-by: Andrew Cooper <andrew.cooper3@citrix.com>
Acked-by: Keir Fraser <keir@xen.org>
-rw-r--r-- | xen/include/asm-x86/random.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/xen/include/asm-x86/random.h b/xen/include/asm-x86/random.h index 6198e72a7b..9e1fe0bc1d 100644 --- a/xen/include/asm-x86/random.h +++ b/xen/include/asm-x86/random.h @@ -8,7 +8,7 @@ static inline unsigned int arch_get_random(void) unsigned int val = 0; if ( cpu_has(¤t_cpu_data, X86_FEATURE_RDRAND) ) - asm ( ".byte 0x0f,0xc7,0xf0" : "+a" (val) ); + asm volatile ( ".byte 0x0f,0xc7,0xf0" : "+a" (val) ); return val; } |