[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v3 19/22] monitor: Restrict 'info sev' to x86 targets
From: |
Philippe Mathieu-Daudé |
Subject: |
Re: [PATCH v3 19/22] monitor: Restrict 'info sev' to x86 targets |
Date: |
Thu, 7 Oct 2021 17:18:03 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101 Thunderbird/91.1.0 |
On 10/4/21 10:26, Paolo Bonzini wrote:
> On 02/10/21 14:53, Philippe Mathieu-Daudé wrote:
>> Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>
>> ---
>> include/monitor/hmp-target.h | 1 +
>> include/monitor/hmp.h | 1 -
>> target/i386/sev-sysemu-stub.c | 2 +-
>> target/i386/sev.c | 2 +-
>> 4 files changed, 3 insertions(+), 3 deletions(-)
>>
>> diff --git a/include/monitor/hmp-target.h b/include/monitor/hmp-target.h
>> index dc53add7eef..96956d0fc41 100644
>> --- a/include/monitor/hmp-target.h
>> +++ b/include/monitor/hmp-target.h
>> @@ -49,6 +49,7 @@ void hmp_info_tlb(Monitor *mon, const QDict *qdict);
>> void hmp_mce(Monitor *mon, const QDict *qdict);
>> void hmp_info_local_apic(Monitor *mon, const QDict *qdict);
>> void hmp_info_io_apic(Monitor *mon, const QDict *qdict);
>> +void hmp_info_sev(Monitor *mon, const QDict *qdict);
>> void hmp_info_sgx(Monitor *mon, const QDict *qdict);
>> #endif /* MONITOR_HMP_TARGET_H */
>> diff --git a/include/monitor/hmp.h b/include/monitor/hmp.h
>> index 3baa1058e2c..6bc27639e01 100644
>> --- a/include/monitor/hmp.h
>> +++ b/include/monitor/hmp.h
>> @@ -124,7 +124,6 @@ void hmp_info_ramblock(Monitor *mon, const QDict
>> *qdict);
>> void hmp_hotpluggable_cpus(Monitor *mon, const QDict *qdict);
>> void hmp_info_vm_generation_id(Monitor *mon, const QDict *qdict);
>> void hmp_info_memory_size_summary(Monitor *mon, const QDict *qdict);
>> -void hmp_info_sev(Monitor *mon, const QDict *qdict);
>> void hmp_info_replay(Monitor *mon, const QDict *qdict);
>> void hmp_replay_break(Monitor *mon, const QDict *qdict);
>> void hmp_replay_delete_break(Monitor *mon, const QDict *qdict);
>> diff --git a/target/i386/sev-sysemu-stub.c
>> b/target/i386/sev-sysemu-stub.c
>> index 1836b32e4fc..b2a4033a030 100644
>> --- a/target/i386/sev-sysemu-stub.c
>> +++ b/target/i386/sev-sysemu-stub.c
>> @@ -13,7 +13,7 @@
>> #include "qemu/osdep.h"
>> #include "monitor/monitor.h"
>> -#include "monitor/hmp.h"
>> +#include "monitor/hmp-target.h"
>> #include "qapi/qapi-commands-misc-target.h"
>> #include "qapi/qmp/qerror.h"
>> #include "qapi/error.h"
>> diff --git a/target/i386/sev.c b/target/i386/sev.c
>> index 7caaa117ff7..c6d8fc52eb2 100644
>> --- a/target/i386/sev.c
>> +++ b/target/i386/sev.c
>> @@ -32,7 +32,7 @@
>> #include "migration/blocker.h"
>> #include "qom/object.h"
>> #include "monitor/monitor.h"
>> -#include "monitor/hmp.h"
>> +#include "monitor/hmp-target.h"
>> #include "qapi/qapi-commands-misc-target.h"
>> #include "qapi/qmp/qerror.h"
>> #include "exec/confidential-guest-support.h"
>>
>
>
> This is only a cleanup, isn't it? The #ifdef is already in
> hmp-commands-info.hx.
IIUC the prototype is exposed to all targets, while with
this patch, only to the files including "monitor/hmp-target.h".
You are right the command is only added for TARGET_I386 in
hmp-commands-info.hx.
>
> Anyway,
>
> Reviewed-by: Paolo Bonzini <pbonzini@redhat.com>
>
> but please adjust the commit message in v4.
OK, thanks.
- [PATCH v3 15/22] target/i386/sev: Move qmp_sev_inject_launch_secret() to sev.c, (continued)
- [PATCH v3 15/22] target/i386/sev: Move qmp_sev_inject_launch_secret() to sev.c, Philippe Mathieu-Daudé, 2021/10/02
- [PATCH v3 16/22] target/i386/sev: Move qmp_query_sev_capabilities() to sev.c, Philippe Mathieu-Daudé, 2021/10/02
- [PATCH v3 17/22] target/i386/sev: Move qmp_query_sev_launch_measure() to sev.c, Philippe Mathieu-Daudé, 2021/10/02
- [PATCH v3 18/22] target/i386/sev: Move qmp_query_sev() & hmp_info_sev() to sev.c, Philippe Mathieu-Daudé, 2021/10/02
- [PATCH v3 19/22] monitor: Restrict 'info sev' to x86 targets, Philippe Mathieu-Daudé, 2021/10/02
- [PATCH v3 20/22] sev/i386: Introduce sev_add_kernel_loader_hashes for measured linux boot, Philippe Mathieu-Daudé, 2021/10/02
- [PATCH v3 21/22] x86/sev: generate SEV kernel loader hashes in x86_load_linux, Philippe Mathieu-Daudé, 2021/10/02
- [PATCH v3 22/22] MAINTAINERS: Cover AMD SEV files, Philippe Mathieu-Daudé, 2021/10/02
- Re: [PATCH v3 00/22] target/i386/sev: Housekeeping SEV + measured Linux SEV guest, Paolo Bonzini, 2021/10/04