diff options
author | Paul Durrant <paul.durrant@citrix.com> | 2011-11-25 15:30:41 +0000 |
---|---|---|
committer | Paul Durrant <paul.durrant@citrix.com> | 2011-11-25 15:30:41 +0000 |
commit | 26ddefddf050f692444bde22226f9726556c5a20 (patch) | |
tree | 23ec0efce002770e08d3d6ec6b47de7707c188f6 /tools/libxc/xc_domain_save.c | |
parent | 5e0f79980e4ace10de74923f604ba523ede0ef90 (diff) | |
download | xen-26ddefddf050f692444bde22226f9726556c5a20.tar.gz xen-26ddefddf050f692444bde22226f9726556c5a20.tar.bz2 xen-26ddefddf050f692444bde22226f9726556c5a20.zip |
Fix save/restore for HVM domains with viridian=1
xc_domain_save/restore currently pay no attention to
HVM_PARAM_VIRIDIAN which results in an HVM domain running a recent
version on Windows (post-Vista) locking up on a domain restore due to
EOIs (done via a viridian MSR write) being silently dropped. This
patch adds an extra save entry for the viridian parameter and also
adds code in the viridian kernel module to catch attempted use of
viridian functionality when the HVM parameter has not been set.
Signed-off-by: Paul Durrant <paul.durrant@citrix.com>
Committed-by: Keir Fraser <keir@xen.org>
Diffstat (limited to 'tools/libxc/xc_domain_save.c')
-rw-r--r-- | tools/libxc/xc_domain_save.c | 12 |
1 files changed, 12 insertions, 0 deletions
diff --git a/tools/libxc/xc_domain_save.c b/tools/libxc/xc_domain_save.c index 76a377acd4..88edd37fe0 100644 --- a/tools/libxc/xc_domain_save.c +++ b/tools/libxc/xc_domain_save.c @@ -1506,6 +1506,18 @@ int xc_domain_save(xc_interface *xch, int io_fd, uint32_t dom, uint32_t max_iter PERROR("Error when writing the firmware ioport version"); goto out; } + + chunk.id = XC_SAVE_ID_HVM_VIRIDIAN; + chunk.data = 0; + xc_get_hvm_param(xch, dom, HVM_PARAM_VIRIDIAN, + (unsigned long *)&chunk.data); + + if ( (chunk.data != 0) && + wrexact(io_fd, &chunk, sizeof(chunk)) ) + { + PERROR("Error when writing the viridian flag"); + goto out; + } } if ( !callbacks->checkpoint ) |