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

Stefan Berger posted 10 patches 4 years, 7 months ago
Test checkpatch failed
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20210712204736.365349-1-stefanb@linux.vnet.ibm.com
Maintainers: Laurent Vivier <lvivier@redhat.com>, Paolo Bonzini <pbonzini@redhat.com>, "Michael S. Tsirkin" <mst@redhat.com>, Thomas Huth <thuth@redhat.com>, Stefan Berger <stefanb@linux.vnet.ibm.com>, Igor Mammedov <imammedo@redhat.com>
There is a newer version of this series
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%)
[PATCH v4 00/10] tests: Add test cases for TPM 1.2 ACPI tables
Posted by Stefan Berger 4 years, 7 months ago
This series of patches adds test case for TPM 1.2 ACPI tables.

  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


Re: [PATCH v4 00/10] tests: Add test cases for TPM 1.2 ACPI tables
Posted by Michael S. Tsirkin 4 years, 6 months ago
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.

>   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
> 
> 


Re: [PATCH v4 00/10] tests: Add test cases for TPM 1.2 ACPI tables
Posted by Stefan Berger 4 years, 6 months ago
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
>>
>>