[Qemu-commits] [qemu/qemu] f8b332: tpm: tpm_spapr: Exit on TPM backend failures

[Peter Maydell]

  Branch: refs/heads/master
  Home:   https://github.com/qemu/qemu
  Commit: f8b332a1ff107dc014a52eaf9bf547995205f18a
      
https://github.com/qemu/qemu/commit/f8b332a1ff107dc014a52eaf9bf547995205f18a
  Author: Stefan Berger <stefanb@linux.vnet.ibm.com>
  Date:   2020-07-15 (Wed, 15 Jul 2020)

  Changed paths:
    M hw/tpm/tpm_spapr.c

  Log Message:
  -----------
  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


  Commit: df8a7568932e4c3c930fdfeb228dd72b4bb71a1f
      
https://github.com/qemu/qemu/commit/df8a7568932e4c3c930fdfeb228dd72b4bb71a1f
  Author: Stefan Berger <stefanb@linux.vnet.ibm.com>
  Date:   2020-07-15 (Wed, 15 Jul 2020)

  Changed paths:
    M tests/qtest/tpm-util.c

  Log Message:
  -----------
  tests: tpm: Skip over pcrUpdateCounter byte in result comparison

The TPM 2 code in libtpms was fixed to handle the PCR 'TCB group' according
to the PCClient profile. The change of the PCRs belonging to the 'TCB group'
now affects the pcrUpdateCounter in the TPM2_PCRRead() responses where its
value is now different (typically lower by '1') than what it was before. To
not fail the tests, we skip the comparison of the 14th byte, which
represents the pcrUpdateCounter.

Signed-off-by: Stefan Berger <stefanb@linux.ibm.com>
Reviewed-by: Marc-André Lureau <marcandre.lureau@redhat.com>
Message-id: 20200707201625.4177419-3-stefanb@linux.vnet.ibm.com


  Commit: 175788d4eb91a840e9505d84ff2c6bedf4380143
      
https://github.com/qemu/qemu/commit/175788d4eb91a840e9505d84ff2c6bedf4380143
  Author: Peter Maydell <peter.maydell@linaro.org>
  Date:   2020-07-16 (Thu, 16 Jul 2020)

  Changed paths:
    M hw/tpm/tpm_spapr.c
    M tests/qtest/tpm-util.c

  Log Message:
  -----------
  Merge remote-tracking branch 
'remotes/stefanberger/tags/pull-tpm-2020-07-15-1' into staging

Merge tpm 2020/07/15 v1

# gpg: Signature made Wed 15 Jul 2020 20:16:21 BST
# gpg:                using RSA key B818B9CADF9089C2D5CEC66B75AD65802A0B4211
# gpg: Good signature from "Stefan Berger <stefanb@linux.vnet.ibm.com>" 
[unknown]
# gpg: WARNING: This key is not certified with a trusted signature!
# gpg:          There is no indication that the signature belongs to the owner.
# Primary key fingerprint: B818 B9CA DF90 89C2 D5CE  C66B 75AD 6580 2A0B 4211

* remotes/stefanberger/tags/pull-tpm-2020-07-15-1:
  tests: tpm: Skip over pcrUpdateCounter byte in result comparison
  tpm: tpm_spapr: Exit on TPM backend failures

Signed-off-by: Peter Maydell <peter.maydell@linaro.org>


Compare: https://github.com/qemu/qemu/compare/ee5128bb00f9...175788d4eb91