mirror of
https://github.com/xen-project/xen.git
synced 2026-10-29 18:50:05 +01:00
x86emul: correct stub invocation constraints
Stub invocations need to have the space the stub occupies as an input, to prevent the compiler from re-ordering (or omitting) writes to it. Signed-off-by: Jan Beulich <jbeulich@suse.com> Reviewed-by: Andrew Cooper <andrew.cooper3@citrix.com> Release-acked-by: Julien Grall <julien.grall@arm.com>
This commit is contained in:
1 parent
144aec4140
commit
cd91ab08ea
1 file changed
+4
-2
@@ -837,7 +837,8 @@ do{ asm volatile ( \
|
||||
".popsection\n\t" \
|
||||
_ASM_EXTABLE(.Lret%=, .Lfix%=) \
|
||||
: [exn] "+g" (res_), constraints, \
|
||||
[stub] "rm" (stub.func) ); \
|
||||
[stub] "rm" (stub.func), \
|
||||
"m" (*(uint8_t(*)[MAX_INST_LEN + 1])stub.ptr) ); \
|
||||
if ( unlikely(~res_.raw) ) \
|
||||
{ \
|
||||
gprintk(XENLOG_WARNING, \
|
||||
@@ -853,7 +854,8 @@ do{ asm volatile ( \
|
||||
#else
|
||||
# define invoke_stub(pre, post, constraints...) \
|
||||
asm volatile ( pre "\n\tcall *%[stub]\n\t" post \
|
||||
: constraints, [stub] "rm" (stub.func) )
|
||||
: constraints, [stub] "rm" (stub.func), \
|
||||
"m" (*(uint8_t(*)[MAX_INST_LEN + 1])stub.buf) )
|
||||
#endif
|
||||
|
||||
#define emulate_stub(dst, src...) do { \
|
||||
|
||||