[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v12 23/28] qmp: add query-sev-launch-measure com
From: |
Daniel P . Berrangé |
Subject: |
Re: [Qemu-devel] [PATCH v12 23/28] qmp: add query-sev-launch-measure command |
Date: |
Thu, 8 Mar 2018 17:03:34 +0000 |
User-agent: |
Mutt/1.9.2 (2017-12-15) |
On Thu, Mar 08, 2018 at 06:48:56AM -0600, Brijesh Singh wrote:
> The command can be used by libvirt to retrieve the measurement of SEV guest.
> This measurement is a signature of the memory contents that was encrypted
> through the LAUNCH_UPDATE_DATA.
>
> Cc: "Daniel P. Berrangé" <address@hidden>
> Cc: "Dr. David Alan Gilbert" <address@hidden>
> Cc: Markus Armbruster <address@hidden>
> Reviewed-by: Eric Blake <address@hidden>
> Signed-off-by: Brijesh Singh <address@hidden>
> ---
> monitor.c | 7 +++++++
> qapi/misc.json | 29 +++++++++++++++++++++++++++++
> target/i386/monitor.c | 17 +++++++++++++++++
> 3 files changed, 53 insertions(+)
>
> diff --git a/monitor.c b/monitor.c
> index 2225cf5030dc..d53ecc5ddab3 100644
> --- a/monitor.c
> +++ b/monitor.c
> @@ -984,6 +984,7 @@ static void qmp_unregister_commands_hack(void)
> #ifndef TARGET_I386
> qmp_unregister_command(&qmp_commands, "rtc-reset-reinjection");
> qmp_unregister_command(&qmp_commands, "query-sev");
> + qmp_unregister_command(&qmp_commands, "query-sev-launch-measure");
> #endif
> #ifndef TARGET_S390X
> qmp_unregister_command(&qmp_commands, "dump-skeys");
> @@ -4110,6 +4111,12 @@ SevInfo *qmp_query_sev(Error **errp)
> error_setg(errp, QERR_FEATURE_DISABLED, "query-sev");
> return NULL;
> }
> +
> +SevLaunchMeasureInfo *qmp_query_sev_launch_measure(Error **errp)
> +{
> + error_setg(errp, QERR_FEATURE_DISABLED, "query-sev-launch-measure");
> + return NULL;
> +}
> #endif
>
> #ifndef TARGET_S390X
> diff --git a/qapi/misc.json b/qapi/misc.json
> index 14681729f8fc..a39c43aa64b1 100644
> --- a/qapi/misc.json
> +++ b/qapi/misc.json
> @@ -3277,3 +3277,32 @@
> #
> ##
> { 'command': 'query-sev', 'returns': 'SevInfo' }
> +
> +##
> +# @SevLaunchMeasureInfo:
> +#
> +# SEV Guest Launch measurement information
> +#
> +# @data: the measurement value encoded in base64
> +#
> +# Since: 2.12
> +#
> +##
> +{ 'struct': 'SevLaunchMeasureInfo', 'data': {'data': 'str'} }
> +
> +##
> +# @query-sev-launch-measure:
> +#
> +# Query the SEV guest launch information.
> +#
> +# Returns: The @SevLaunchMeasureInfo for the guest
> +#
> +# Since: 2.12
> +#
> +# Example:
> +#
> +# -> { "execute": "query-sev-launch-measure" }
> +# <- { "return": { "data": "4l8LXeNlSPUDlXPJG5966/8%YZ" } }
> +#
> +##
> +{ 'command': 'query-sev-launch-measure', 'returns': 'SevLaunchMeasureInfo' }
> diff --git a/target/i386/monitor.c b/target/i386/monitor.c
> index 7df31c3cdf1b..e5596bbc0fc2 100644
> --- a/target/i386/monitor.c
> +++ b/target/i386/monitor.c
> @@ -720,3 +720,20 @@ void hmp_info_sev(Monitor *mon, const QDict *qdict)
> monitor_printf(mon, "SEV is not enabled\n");
> }
> }
> +
> +SevLaunchMeasureInfo *qmp_query_sev_launch_measure(Error **errp)
> +{
> + char *data;
> + SevLaunchMeasureInfo *info;
> +
> + data = sev_get_launch_measurement();
> + if (!data) {
> + error_setg(errp, "Measurement is not available");
> + return NULL;
> + }
> +
> + info = g_malloc0(sizeof(*info));
Minor preference for g_new0(SevLaunchMeasureInfo, 1);
> + info->data = data;
> +
> + return info;
> +}
Regardless though
Reviewed-by: Daniel P. Berrangé <address@hidden>
Regards,
Daniel
--
|: https://berrange.com -o- https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org -o- https://fstop138.berrange.com :|
|: https://entangle-photo.org -o- https://www.instagram.com/dberrange :|
- [Qemu-devel] [PATCH v12 13/28] kvm: introduce memory encryption APIs, (continued)
- [Qemu-devel] [PATCH v12 13/28] kvm: introduce memory encryption APIs, Brijesh Singh, 2018/03/08
- [Qemu-devel] [PATCH v12 14/28] hmp: add 'info sev' command, Brijesh Singh, 2018/03/08
- [Qemu-devel] [PATCH v12 15/28] sev/i386: add command to create launch memory encryption context, Brijesh Singh, 2018/03/08
- [Qemu-devel] [PATCH v12 17/28] target/i386: encrypt bios rom, Brijesh Singh, 2018/03/08
- [Qemu-devel] [PATCH v12 18/28] sev/i386: add support to LAUNCH_MEASURE command, Brijesh Singh, 2018/03/08
- [Qemu-devel] [PATCH v12 20/28] hw/i386: set ram_debug_ops when memory encryption is enabled, Brijesh Singh, 2018/03/08
- [Qemu-devel] [PATCH v12 16/28] sev/i386: add command to encrypt guest memory region, Brijesh Singh, 2018/03/08
- [Qemu-devel] [PATCH v12 19/28] sev/i386: finalize the SEV guest launch flow, Brijesh Singh, 2018/03/08
- [Qemu-devel] [PATCH v12 21/28] sev/i386: add debug encrypt and decrypt commands, Brijesh Singh, 2018/03/08
- [Qemu-devel] [PATCH v12 23/28] qmp: add query-sev-launch-measure command, Brijesh Singh, 2018/03/08
- Re: [Qemu-devel] [PATCH v12 23/28] qmp: add query-sev-launch-measure command,
Daniel P . Berrangé <=
- [Qemu-devel] [PATCH v12 22/28] target/i386: clear C-bit when walking SEV guest page table, Brijesh Singh, 2018/03/08
- [Qemu-devel] [PATCH v12 24/28] sev/i386: add migration blocker, Brijesh Singh, 2018/03/08
- [Qemu-devel] [PATCH v12 25/28] cpu/i386: populate CPUID 0x8000_001F when SEV is active, Brijesh Singh, 2018/03/08
- [Qemu-devel] [PATCH v12 27/28] sev/i386: add sev_get_capabilities(), Brijesh Singh, 2018/03/08
- [Qemu-devel] [PATCH v12 26/28] qmp: add query-sev-capabilities command, Brijesh Singh, 2018/03/08