diff options
| author | Stefan Berger <stefanb@linux.vnet.ibm.com> | 2020-07-07 16:16:24 -0400 |
|---|---|---|
| committer | Stefan Berger <stefanb@linux.vnet.ibm.com> | 2020-07-15 14:57:33 -0400 |
| commit | f8b332a1ff107dc014a52eaf9bf547995205f18a (patch) | |
| tree | 27e3968b780d36ce95550c7a9616b930d2104583 | |
| parent | 8746309137ba470d1b2e8f5ce86ac228625db940 (diff) | |
| download | qemu-f8b332a1ff107dc014a52eaf9bf547995205f18a.zip qemu-f8b332a1ff107dc014a52eaf9bf547995205f18a.tar.gz qemu-f8b332a1ff107dc014a52eaf9bf547995205f18a.tar.bz2 | |
tpm: tpm_spapr: Exit on TPM backend failures
Exit on TPM backend failures in the same way as the TPM CRB and TIS device
models do. With this change we now get an error report when the backend
did not start up properly:
error: internal error: qemu unexpectedly closed the monitor:
2020-07-07T12:49:28.333928Z qemu-system-ppc64: tpm-emulator: \
TPM result for CMD_INIT: 0x101 operation failed
Signed-off-by: Stefan Berger <stefanb@linux.ibm.com>
Reviewed-by: Marc-André Lureau <marcandre.lureau@redhat.com>
Message-id: 20200707201625.4177419-2-stefanb@linux.vnet.ibm.com
| -rw-r--r-- | hw/tpm/tpm_spapr.c | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/hw/tpm/tpm_spapr.c b/hw/tpm/tpm_spapr.c index cb4dfd1..8288ab0 100644 --- a/hw/tpm/tpm_spapr.c +++ b/hw/tpm/tpm_spapr.c @@ -306,7 +306,10 @@ static void tpm_spapr_reset(SpaprVioDevice *dev) TPM_SPAPR_BUFFER_MAX); tpm_backend_reset(s->be_driver); - tpm_spapr_do_startup_tpm(s, s->be_buffer_size); + + if (tpm_spapr_do_startup_tpm(s, s->be_buffer_size) < 0) { + exit(1); + } } static enum TPMVersion tpm_spapr_get_version(TPMIf *ti) |