[PATCH v5 0/3] tpm: Fix default choices for CRB and SPAPR dev models

Stefan Berger posted 3 patches 3 years, 8 months ago
Test syntax-check failed
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/libvirt tags/patchew/20200709204620.156606-1-stefanb@linux.vnet.ibm.com
src/qemu/qemu_domain.c   | 12 +++++++++---
src/qemu/qemu_validate.c | 10 ++++++----
2 files changed, 15 insertions(+), 7 deletions(-)
[PATCH v5 0/3] tpm: Fix default choices for CRB and SPAPR dev models
Posted by Stefan Berger 3 years, 8 months ago
From: Stefan Berger <stefanb@linux.ibm.com>

This series of patches adds an additional check for the SPAPR device model
that prevents the choice of a TPM 1.2 backend and chooses a TPM 2 as default.
Also CRB now chooses a TPM 2 as default since TPM 1.2 wouldn't work with it,
either.

   Stefan

v4->v5:
  - Added R-b's

Stefan Berger (3):
  qemu: Move setting of TPM default to post parse function
  qemu: Set SPAPR TPM default to 2.0 and prevent 1.2 choice
  qemu: Choose TPM 2 for backend as default for CRB interface

 src/qemu/qemu_domain.c   | 12 +++++++++---
 src/qemu/qemu_validate.c | 10 ++++++----
 2 files changed, 15 insertions(+), 7 deletions(-)

-- 
2.17.1

Re: [PATCH v5 0/3] tpm: Fix default choices for CRB and SPAPR dev models
Posted by Marc-André Lureau 3 years, 8 months ago
Hi

On Fri, Jul 10, 2020 at 12:49 AM Stefan Berger <stefanb@linux.vnet.ibm.com>
wrote:

> From: Stefan Berger <stefanb@linux.ibm.com>
>
> This series of patches adds an additional check for the SPAPR device model
> that prevents the choice of a TPM 1.2 backend and chooses a TPM 2 as
> default.
> Also CRB now chooses a TPM 2 as default since TPM 1.2 wouldn't work with
> it,
> either.
>
>    Stefan
>
> v4->v5:
>   - Added R-b's
>
> Stefan Berger (3):
>   qemu: Move setting of TPM default to post parse function
>   qemu: Set SPAPR TPM default to 2.0 and prevent 1.2 choice
>   qemu: Choose TPM 2 for backend as default for CRB interface
>

Series:
Reviewed-by: Marc-André Lureau <marcandre.lureau@redhat.com>


>  src/qemu/qemu_domain.c   | 12 +++++++++---
>  src/qemu/qemu_validate.c | 10 ++++++----
>  2 files changed, 15 insertions(+), 7 deletions(-)
>
> --
> 2.17.1
>
>

-- 
Marc-André Lureau
Re: [PATCH v5 0/3] tpm: Fix default choices for CRB and SPAPR dev models
Posted by Peter Krempa 3 years, 8 months ago
On Tue, Jul 14, 2020 at 23:00:51 +0400, Marc-André Lureau wrote:
> Hi
> 
> On Fri, Jul 10, 2020 at 12:49 AM Stefan Berger <stefanb@linux.vnet.ibm.com>
> wrote:
> 
> > From: Stefan Berger <stefanb@linux.ibm.com>
> >
> > This series of patches adds an additional check for the SPAPR device model
> > that prevents the choice of a TPM 1.2 backend and chooses a TPM 2 as
> > default.
> > Also CRB now chooses a TPM 2 as default since TPM 1.2 wouldn't work with
> > it,
> > either.
> >
> >    Stefan
> >
> > v4->v5:
> >   - Added R-b's
> >
> > Stefan Berger (3):
> >   qemu: Move setting of TPM default to post parse function
> >   qemu: Set SPAPR TPM default to 2.0 and prevent 1.2 choice
> >   qemu: Choose TPM 2 for backend as default for CRB interface
> >
> 
> Series:
> Reviewed-by: Marc-André Lureau <marcandre.lureau@redhat.com>

I've added this tag and pushed the series.