Re: [PATCH v4 00/10] tests: Add test cases for TPM 1.2 ACPI tables

[Stefan Berger]

On 7/13/21 11:19 AM, Michael S. Tsirkin wrote:
> On Mon, Jul 12, 2021 at 04:47:26PM -0400, Stefan Berger wrote:
> > This series of patches adds test case for TPM 1.2 ACPI tables.
>
> Acked-by: Michael S. Tsirkin <mst@redhat.com>
>
> Pls feel free to merge through tpm tree.

Thanks. I will have to post a v5 since this one had issues on s390x.

  Stefan


> >    Stefan
> >
> > v4:
> >    - Added patch 10 that checks for availability of a TPM device model
> >      using QMP and if not available skips the ACPI table test
> >
> > v3:
> >    - Define enum TPMVersion for when CONFIG_TPM is not defined
> >      affected patches 2 and 6
> >
> > v2:
> >    - Proper handling of renaming of files holding expected ACPI data
> >
> >
> > Stefan Berger (10):
> >    tests: Rename TestState to TPMTestState
> >    tests: Add tpm_version field to TPMTestState and fill it
> >    tests: acpi: Prepare for renaming of TPM2 related ACPI files
> >    tests: Add suffix 'tpm2' or 'tpm12' to ACPI table files
> >    tests: acpi: tpm2: Add the renamed ACPI files and drop old ones
> >    tests: tpm: Create TPM 1.2 response in TPM emulator
> >    tests: acpi: prepare for new TPM 1.2 related tables
> >    tests: acpi: Add test cases for TPM 1.2 with TCPA table
> >    tests: acpi: tpm1.2: Add expected TPM 1.2 ACPI blobs
> >    tests: Use QMP to check whether a TPM device model is available
> >
> >   tests/data/acpi/q35/DSDT.tis.tpm12            | Bin 0 -> 8465 bytes
> >   .../data/acpi/q35/{DSDT.tis => DSDT.tis.tpm2} | Bin
> >   tests/data/acpi/q35/TCPA.tis.tpm12            | Bin 0 -> 50 bytes
> >   .../data/acpi/q35/{TPM2.tis => TPM2.tis.tpm2} | Bin
> >   tests/qtest/bios-tables-test.c                |  30 ++++++---
> >   tests/qtest/tpm-crb-test.c                    |   5 +-
> >   tests/qtest/tpm-emu.c                         |  61 ++++++++++++++++--
> >   tests/qtest/tpm-emu.h                         |  20 +++++-
> >   tests/qtest/tpm-tis-device-test.c             |   3 +-
> >   tests/qtest/tpm-tis-test.c                    |   3 +-
> >   tests/qtest/tpm-tis-util.c                    |   2 +-
> >   11 files changed, 100 insertions(+), 24 deletions(-)
> >   create mode 100644 tests/data/acpi/q35/DSDT.tis.tpm12
> >   rename tests/data/acpi/q35/{DSDT.tis => DSDT.tis.tpm2} (100%)
> >   create mode 100644 tests/data/acpi/q35/TCPA.tis.tpm12
> >   rename tests/data/acpi/q35/{TPM2.tis => TPM2.tis.tpm2} (100%)
> >
> > --
> > 2.31.1