diff options
| author | Ross Lagerwall <ross.lagerwall@citrix.com> | 2022-08-26 15:38:41 +0100 |
|---|---|---|
| committer | Stefan Berger <stefanb@linux.ibm.com> | 2022-09-09 17:55:59 -0400 |
| commit | f0ccce6a95f6ff947040692ef941230918181562 (patch) | |
| tree | 3797c96277bc3d44d20ec32c87df669fcc0f59cf | |
| parent | a0bcec03761477371ff7c2e80dc07fff14222d92 (diff) | |
| download | qemu-f0ccce6a95f6ff947040692ef941230918181562.zip qemu-f0ccce6a95f6ff947040692ef941230918181562.tar.gz qemu-f0ccce6a95f6ff947040692ef941230918181562.tar.bz2 | |
tpm_crb: Avoid backend startup just before shutdown under Xen
When running under Xen and the guest reboots, it boots into a new domain
with a new QEMU process (and a new swtpm process if using the emulator
backend). The existing reset function is triggered just before the old
QEMU process exists which causes QEMU to startup the TPM backend and
then immediately shut it down. This is probably harmless but when using
the emulated backend, it wastes CPU and IO time reloading state, etc.
Fix this by calling the reset function directly from realize() when
running under Xen. During a reboot, this will be called by the QEMU
process for the new domain.
Signed-off-by: Ross Lagerwall <ross.lagerwall@citrix.com>
Reviewed-by: Stefan Berger <stefanb@linux.ibm.com>
Message-id: 20220826143841.1515326-1-ross.lagerwall@citrix.com
Signed-off-by: Stefan Berger <stefanb@linux.ibm.com>
| -rw-r--r-- | hw/tpm/tpm_crb.c | 7 |
1 files changed, 6 insertions, 1 deletions
diff --git a/hw/tpm/tpm_crb.c b/hw/tpm/tpm_crb.c index 67db594..ea930da 100644 --- a/hw/tpm/tpm_crb.c +++ b/hw/tpm/tpm_crb.c @@ -26,6 +26,7 @@ #include "sysemu/tpm_backend.h" #include "sysemu/tpm_util.h" #include "sysemu/reset.h" +#include "sysemu/xen.h" #include "tpm_prop.h" #include "tpm_ppi.h" #include "trace.h" @@ -308,7 +309,11 @@ static void tpm_crb_realize(DeviceState *dev, Error **errp) TPM_PPI_ADDR_BASE, OBJECT(s)); } - qemu_register_reset(tpm_crb_reset, dev); + if (xen_enabled()) { + tpm_crb_reset(dev); + } else { + qemu_register_reset(tpm_crb_reset, dev); + } } static void tpm_crb_class_init(ObjectClass *klass, void *data) |