mbox series

[v3,0/2] sev: enable secret injection to a self described area in OVMF

Message ID 20210204193939.16617-1-jejb@linux.ibm.com (mailing list archive)
Headers show
Series sev: enable secret injection to a self described area in OVMF | expand

Message

James Bottomley Feb. 4, 2021, 7:39 p.m. UTC
v3: add bounds checking and remove misleading comment

v2: fix build issues and update comments.  Also drop the first patch
    which is now merged upstream

The two patches introduce a parser for the optional OVMF description
table which is placed just below the reset vector (the format of the
table is described in the patch itself) and also adds a hook to pull
out the description of the SEV secret area location and use it in
place of the sev-inject-launch-secret gpa.

James

---

James Bottomley (2):
  pc: add parser for OVMF reset block
  sev: update sev-inject-launch-secret to make gpa optional

 hw/i386/pc_sysfw.c    | 112 ++++++++++++++++++++++++++++++++++++++++++
 include/hw/i386/pc.h  |   4 ++
 qapi/misc-target.json |   2 +-
 target/i386/monitor.c |  23 ++++++++-
 4 files changed, 139 insertions(+), 2 deletions(-)

Comments

Paolo Bonzini Feb. 5, 2021, 10:58 a.m. UTC | #1
On 04/02/21 20:39, James Bottomley wrote:
> v3: add bounds checking and remove misleading comment
> 
> v2: fix build issues and update comments.  Also drop the first patch
>      which is now merged upstream
> 
> The two patches introduce a parser for the optional OVMF description
> table which is placed just below the reset vector (the format of the
> table is described in the patch itself) and also adds a hook to pull
> out the description of the SEV secret area location and use it in
> place of the sev-inject-launch-secret gpa.
> 
> James
> 
> ---
> 
> James Bottomley (2):
>    pc: add parser for OVMF reset block
>    sev: update sev-inject-launch-secret to make gpa optional
> 
>   hw/i386/pc_sysfw.c    | 112 ++++++++++++++++++++++++++++++++++++++++++
>   include/hw/i386/pc.h  |   4 ++
>   qapi/misc-target.json |   2 +-
>   target/i386/monitor.c |  23 ++++++++-
>   4 files changed, 139 insertions(+), 2 deletions(-)
> 

Queued, thanks.

Paolo