Re: [libvirt] [PATCH v5 11/15] qemu_monitor: implement query-cpu-model-comparison

2019-10-02 Thread Jiri Denemark
On Thu, Sep 19, 2019 at 16:25:02 -0400, Collin Walling wrote:
> Interfaces with QEMU to compare CPU models. The command takes two CPU
> models, A and B, that are given a model name and an optional list of
> CPU features. Through the query-cpu-model-comparison command issued
> via QMP, a result is produced that contains the comparison evaluation
> string (identical, superset, subset, incompatible).
> 
> The list of properties (aka CPU features) that is returned from the QMP
> response is ignored.
> 
> Signed-off-by: Collin Walling 
> Reviewed-by: Daniel Henrique Barboza 
> ---
>  src/qemu/qemu_monitor.c  | 14 ++
>  src/qemu/qemu_monitor.h  |  5 +
>  src/qemu/qemu_monitor_json.c | 42 ++
>  src/qemu/qemu_monitor_json.h |  6 ++
>  4 files changed, 67 insertions(+)

Reviewed-by: Jiri Denemark 

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list


[libvirt] [PATCH v5 11/15] qemu_monitor: implement query-cpu-model-comparison

2019-09-19 Thread Collin Walling
Interfaces with QEMU to compare CPU models. The command takes two CPU
models, A and B, that are given a model name and an optional list of
CPU features. Through the query-cpu-model-comparison command issued
via QMP, a result is produced that contains the comparison evaluation
string (identical, superset, subset, incompatible).

The list of properties (aka CPU features) that is returned from the QMP
response is ignored.

Signed-off-by: Collin Walling 
Reviewed-by: Daniel Henrique Barboza 
---
 src/qemu/qemu_monitor.c  | 14 ++
 src/qemu/qemu_monitor.h  |  5 +
 src/qemu/qemu_monitor_json.c | 42 ++
 src/qemu/qemu_monitor_json.h |  6 ++
 4 files changed, 67 insertions(+)

diff --git a/src/qemu/qemu_monitor.c b/src/qemu/qemu_monitor.c
index 834eaf2..76b5547 100644
--- a/src/qemu/qemu_monitor.c
+++ b/src/qemu/qemu_monitor.c
@@ -3745,6 +3745,20 @@ qemuMonitorGetCPUModelBaseline(qemuMonitorPtr mon,
 }
 
 
+int
+qemuMonitorGetCPUModelComparison(qemuMonitorPtr mon,
+ virCPUDefPtr cpu_a,
+ virCPUDefPtr cpu_b,
+ char **result)
+{
+VIR_DEBUG("cpu_a=%p cpu_b=%p", cpu_a, cpu_b);
+
+QEMU_CHECK_MONITOR(mon);
+
+return qemuMonitorJSONGetCPUModelComparison(mon, cpu_a, cpu_b, result);
+}
+
+
 void
 qemuMonitorCPUModelInfoFree(qemuMonitorCPUModelInfoPtr model_info)
 {
diff --git a/src/qemu/qemu_monitor.h b/src/qemu/qemu_monitor.h
index 8974ba8..6f46931 100644
--- a/src/qemu/qemu_monitor.h
+++ b/src/qemu/qemu_monitor.h
@@ -1169,6 +1169,11 @@ int qemuMonitorGetCPUModelBaseline(qemuMonitorPtr mon,
virCPUDefPtr cpu_b,
qemuMonitorCPUModelInfoPtr *baseline);
 
+int qemuMonitorGetCPUModelComparison(qemuMonitorPtr mon,
+ virCPUDefPtr cpu_a,
+ virCPUDefPtr cpu_b,
+ char **result);
+
 qemuMonitorCPUModelInfoPtr
 qemuMonitorCPUModelInfoCopy(const qemuMonitorCPUModelInfo *orig);
 
diff --git a/src/qemu/qemu_monitor_json.c b/src/qemu/qemu_monitor_json.c
index 679531d..d8f7668 100644
--- a/src/qemu/qemu_monitor_json.c
+++ b/src/qemu/qemu_monitor_json.c
@@ -6016,6 +6016,48 @@ qemuMonitorJSONGetCPUModelBaseline(qemuMonitorPtr mon,
 }
 
 
+int
+qemuMonitorJSONGetCPUModelComparison(qemuMonitorPtr mon,
+ virCPUDefPtr cpu_a,
+ virCPUDefPtr cpu_b,
+ char **result)
+{
+VIR_AUTOPTR(virJSONValue) model_a = NULL;
+VIR_AUTOPTR(virJSONValue) model_b = NULL;
+VIR_AUTOPTR(virJSONValue) cmd = NULL;
+VIR_AUTOPTR(virJSONValue) reply = NULL;
+const char *data_result;
+virJSONValuePtr data;
+
+if (!(model_a = qemuMonitorJSONMakeCPUModel(cpu_a, true)) ||
+!(model_b = qemuMonitorJSONMakeCPUModel(cpu_b, true)))
+return -1;
+
+if (!(cmd = qemuMonitorJSONMakeCommand("query-cpu-model-comparison",
+   "a:modela", _a,
+   "a:modelb", _b,
+   NULL)))
+return -1;
+
+if (qemuMonitorJSONCommand(mon, cmd, ) < 0)
+return -1;
+
+if (qemuMonitorJSONCheckError(cmd, reply) < 0)
+return -1;
+
+data = virJSONValueObjectGetObject(reply, "return");
+
+if (!(data_result = virJSONValueObjectGetString(data, "result"))) {
+virReportError(VIR_ERR_INTERNAL_ERROR, "%s",
+   _("query-cpu-model-comparison reply data was missing "
+ "'result'"));
+return -1;
+}
+
+return VIR_STRDUP(*result, data_result);
+}
+
+
 int qemuMonitorJSONGetCommands(qemuMonitorPtr mon,
char ***commands)
 {
diff --git a/src/qemu/qemu_monitor_json.h b/src/qemu/qemu_monitor_json.h
index 77ea41b..52b24f0 100644
--- a/src/qemu/qemu_monitor_json.h
+++ b/src/qemu/qemu_monitor_json.h
@@ -402,6 +402,12 @@ int qemuMonitorJSONGetCPUModelBaseline(qemuMonitorPtr mon,
qemuMonitorCPUModelInfoPtr *baseline)
 ATTRIBUTE_NONNULL(2) ATTRIBUTE_NONNULL(3) ATTRIBUTE_NONNULL(4);
 
+int qemuMonitorJSONGetCPUModelComparison(qemuMonitorPtr mon,
+ virCPUDefPtr cpu_a,
+ virCPUDefPtr cpu_b,
+ char **result)
+ATTRIBUTE_NONNULL(2) ATTRIBUTE_NONNULL(3);
+
 int qemuMonitorJSONGetCommands(qemuMonitorPtr mon,
char ***commands)
 ATTRIBUTE_NONNULL(2);
-- 
2.7.4

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list