[Qemu-commits] [qemu/qemu] 83e83c: specs: Extend TPM spec with TPM emulator descripti...

[GitHub]

  Branch: refs/heads/master
  Home:   https://github.com/qemu/qemu
  Commit: 83e83c34b2102153682729531b9c9d3e6659f037
      
https://github.com/qemu/qemu/commit/83e83c34b2102153682729531b9c9d3e6659f037
  Author: Stefan Berger <address@hidden>
  Date:   2017-11-15 (Wed, 15 Nov 2017)

  Changed paths:
    M docs/specs/tpm.txt

  Log Message:
  -----------
  specs: Extend TPM spec with TPM emulator description

Following the recent extension of QEMU with a TPM emulator device,
update the specs describing for how to interact with the device.

The results of commands run inside a Linux VM are expected to be
similar to those when the TPM passthrough device is used, so we
just reuse that.

Signed-off-by: Stefan Berger <address@hidden>
Reviewed-by: Marc-André Lureau <address@hidden>


  Commit: 17b1af773ed81a9167abfa09749ab92bd05740b0
      
https://github.com/qemu/qemu/commit/17b1af773ed81a9167abfa09749ab92bd05740b0
  Author: Marc-André Lureau <address@hidden>
  Date:   2017-11-15 (Wed, 15 Nov 2017)

  Changed paths:
    M hw/tpm/tpm_emulator.c

  Log Message:
  -----------
  tpm-emulator: protect concurrent ctrl_chr access

The control chardev is being used from the data thread to set the
locality of the next request. Altough the chr has a write mutex, we
may potentially read the reply from another thread request.

Add a mutex to protect from concurrent control commands.

Signed-off-by: Marc-André Lureau <address@hidden>
Reviewed-by: Stefan Berger <address@hidden>
Signed-off-by: Stefan Berger <address@hidden>


  Commit: ad4aca69bbd40663ca93a3eb1d8042c023b9b407
      
https://github.com/qemu/qemu/commit/ad4aca69bbd40663ca93a3eb1d8042c023b9b407
  Author: Stefan Berger <address@hidden>
  Date:   2017-11-15 (Wed, 15 Nov 2017)

  Changed paths:
    M hw/tpm/tpm_tis.c

  Log Message:
  -----------
  tpm_tis: Return TPM_VERSION_UNSPEC in case of BE failure

In case the backend has a failure, such as the tpm_emulator's CMD_INIT
failing, the TIS goes into failure mode and does not respond to reads
or writes to MMIO registers. In this case we need to prevent the ACPI
table from being added and the straight-forward way is to indicate that
there's no known TPM version being used.

Signed-off-by: Stefan Berger <address@hidden>
Reviewed-by: Marc-André Lureau <address@hidden>


  Commit: 6cd65969da57eaa9bdff07b80ecca2becce0597a
      
https://github.com/qemu/qemu/commit/6cd65969da57eaa9bdff07b80ecca2becce0597a
  Author: Stefan Berger <address@hidden>
  Date:   2017-11-15 (Wed, 15 Nov 2017)

  Changed paths:
    M hw/tpm/tpm_tis.c

  Log Message:
  -----------
  tpm_tis: Return 0 for every register in case of failure mode

Rather than returning ~0, return 0 for every register in case of failure
mode. The '0' is better to indicate that there's no device there. It avoids
SeaBIOS detecting a device and getting stuck on it trying to read and write
its registers.

Signed-off-by: Stefan Berger <address@hidden>
Reviewed-by: Marc-André Lureau <address@hidden>


  Commit: 8048082f7a11040a366942a2de8abb4c3d0020c9
      
https://github.com/qemu/qemu/commit/8048082f7a11040a366942a2de8abb4c3d0020c9
  Author: Peter Maydell <address@hidden>
  Date:   2017-11-16 (Thu, 16 Nov 2017)

  Changed paths:
    M docs/specs/tpm.txt
    M hw/tpm/tpm_emulator.c
    M hw/tpm/tpm_tis.c

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

Merge tpm 2017/11/15 v1

# gpg: Signature made Wed 15 Nov 2017 11:51:47 GMT
# gpg:                using RSA key 0x75AD65802A0B4211
# gpg: Good signature from "Stefan Berger <address@hidden>"
# 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-2017-11-15-1:
  tpm_tis: Return 0 for every register in case of failure mode
  tpm_tis: Return TPM_VERSION_UNSPEC in case of BE failure
  tpm-emulator: protect concurrent ctrl_chr access
  specs: Extend TPM spec with TPM emulator description

Signed-off-by: Peter Maydell <address@hidden>


Compare: https://github.com/qemu/qemu/compare/1fa0f627d03c...8048082f7a11