diff options
author | Jan Beulich <jbeulich@suse.com> | 2012-10-30 16:46:07 +0100 |
---|---|---|
committer | Jan Beulich <jbeulich@suse.com> | 2012-10-30 16:46:07 +0100 |
commit | ecf9846a6a200e8c8cfb26d409d18fd9615df9b9 (patch) | |
tree | 9f3cf4c722a6991a8dc032a050984f9ba8014923 /tools/tests | |
parent | 4d246723a85a03406e4969a260291e11b8e05960 (diff) | |
download | xen-ecf9846a6a200e8c8cfb26d409d18fd9615df9b9.tar.gz xen-ecf9846a6a200e8c8cfb26d409d18fd9615df9b9.tar.bz2 xen-ecf9846a6a200e8c8cfb26d409d18fd9615df9b9.zip |
x86: save/restore only partial register state where possible
... and make restore conditional not only upon having saved the state,
but also upon whether saved state was actually modified (and register
values are known to have been preserved).
Note that RBP is unconditionally considered a volatile register (i.e.
irrespective of CONFIG_FRAME_POINTER), since the RBP handling would
become overly complicated due to the need to save/restore it on the
compat mode hypercall path [6th argument].
Note further that for compat mode code paths, saving/restoring R8...R15
is entirely unnecessary - we don't allow those guests to enter 64-bit
mode, and hence they have no way of seeing these registers' contents
(and there consequently also is no information leak, except if the
context saving domctl would be considered such).
Finally, note that this may not properly deal with gdbstub's needs, yet
(but if so, I can't really suggest adjustments, as I don't know that
code).
Signed-off-by: Jan Beulich <jbeulich@suse.com>
Acked-by: Keir Fraser <keir@xen.org>
Diffstat (limited to 'tools/tests')
-rw-r--r-- | tools/tests/x86_emulator/x86_emulate.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/tools/tests/x86_emulator/x86_emulate.c b/tools/tests/x86_emulator/x86_emulate.c index 0306f8ebc6..f18f615209 100644 --- a/tools/tests/x86_emulator/x86_emulate.c +++ b/tools/tests/x86_emulator/x86_emulate.c @@ -10,6 +10,7 @@ typedef bool bool_t; #define BUG() abort() #define cpu_has_amd_erratum(nr) 0 +#define mark_regs_dirty(r) ((void)(r)) #include "x86_emulate/x86_emulate.h" #include "x86_emulate/x86_emulate.c" |