mirror of
https://github.com/libvirt/libvirt.git
synced 2025-02-25 18:55:26 -06:00
Add a qemuMonitorGetObjectProps() method for QMP device-list-properties command
Add a new qemuMonitorGetObjectProps() method to support invocation of the 'device-list-properties' JSON monitor command. No HMP equivalent is required, since this will only be present for QEMU >= 1.2 Signed-off-by: Daniel P. Berrange <berrange@redhat.com>
This commit is contained in:
parent
2ccaa8e6ba
commit
b785cec0dc
@ -3163,3 +3163,26 @@ int qemuMonitorGetObjectTypes(qemuMonitorPtr mon,
|
|||||||
|
|
||||||
return qemuMonitorJSONGetObjectTypes(mon, types);
|
return qemuMonitorJSONGetObjectTypes(mon, types);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
||||||
|
int qemuMonitorGetObjectProps(qemuMonitorPtr mon,
|
||||||
|
const char *type,
|
||||||
|
char ***props)
|
||||||
|
{
|
||||||
|
VIR_DEBUG("mon=%p type=%s props=%p",
|
||||||
|
mon, type, props);
|
||||||
|
|
||||||
|
if (!mon) {
|
||||||
|
virReportError(VIR_ERR_INVALID_ARG, "%s",
|
||||||
|
_("monitor must not be NULL"));
|
||||||
|
return -1;
|
||||||
|
}
|
||||||
|
|
||||||
|
if (!mon->json) {
|
||||||
|
virReportError(VIR_ERR_CONFIG_UNSUPPORTED, "%s",
|
||||||
|
_("JSON monitor is required"));
|
||||||
|
return -1;
|
||||||
|
}
|
||||||
|
|
||||||
|
return qemuMonitorJSONGetObjectProps(mon, type, props);
|
||||||
|
}
|
||||||
|
@ -603,6 +603,9 @@ int qemuMonitorGetEvents(qemuMonitorPtr mon,
|
|||||||
|
|
||||||
int qemuMonitorGetObjectTypes(qemuMonitorPtr mon,
|
int qemuMonitorGetObjectTypes(qemuMonitorPtr mon,
|
||||||
char ***types);
|
char ***types);
|
||||||
|
int qemuMonitorGetObjectProps(qemuMonitorPtr mon,
|
||||||
|
const char *type,
|
||||||
|
char ***props);
|
||||||
|
|
||||||
|
|
||||||
/**
|
/**
|
||||||
|
@ -4292,3 +4292,85 @@ cleanup:
|
|||||||
virJSONValueFree(reply);
|
virJSONValueFree(reply);
|
||||||
return ret;
|
return ret;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
||||||
|
int qemuMonitorJSONGetObjectProps(qemuMonitorPtr mon,
|
||||||
|
const char *type,
|
||||||
|
char ***props)
|
||||||
|
{
|
||||||
|
int ret;
|
||||||
|
virJSONValuePtr cmd;
|
||||||
|
virJSONValuePtr reply = NULL;
|
||||||
|
virJSONValuePtr data;
|
||||||
|
char **proplist = NULL;
|
||||||
|
int n = 0;
|
||||||
|
size_t i;
|
||||||
|
|
||||||
|
*props = NULL;
|
||||||
|
|
||||||
|
if (!(cmd = qemuMonitorJSONMakeCommand("device-list-properties",
|
||||||
|
"s:typename", type,
|
||||||
|
NULL)))
|
||||||
|
return -1;
|
||||||
|
|
||||||
|
ret = qemuMonitorJSONCommand(mon, cmd, &reply);
|
||||||
|
|
||||||
|
if (ret == 0 &&
|
||||||
|
qemuMonitorJSONHasError(reply, "DeviceNotFound")) {
|
||||||
|
goto cleanup;
|
||||||
|
}
|
||||||
|
|
||||||
|
if (ret == 0)
|
||||||
|
ret = qemuMonitorJSONCheckError(cmd, reply);
|
||||||
|
|
||||||
|
if (ret < 0)
|
||||||
|
goto cleanup;
|
||||||
|
|
||||||
|
ret = -1;
|
||||||
|
|
||||||
|
if (!(data = virJSONValueObjectGet(reply, "return"))) {
|
||||||
|
virReportError(VIR_ERR_INTERNAL_ERROR, "%s",
|
||||||
|
_("device-list-properties reply was missing return data"));
|
||||||
|
goto cleanup;
|
||||||
|
}
|
||||||
|
|
||||||
|
if ((n = virJSONValueArraySize(data)) < 0) {
|
||||||
|
virReportError(VIR_ERR_INTERNAL_ERROR, "%s",
|
||||||
|
_("device-list-properties reply data was not an array"));
|
||||||
|
goto cleanup;
|
||||||
|
}
|
||||||
|
|
||||||
|
if (VIR_ALLOC_N(proplist, n) < 0) {
|
||||||
|
virReportOOMError();
|
||||||
|
goto cleanup;
|
||||||
|
}
|
||||||
|
|
||||||
|
for (i = 0 ; i < n ; i++) {
|
||||||
|
virJSONValuePtr child = virJSONValueArrayGet(data, i);
|
||||||
|
const char *tmp;
|
||||||
|
|
||||||
|
if (!(tmp = virJSONValueObjectGetString(child, "name"))) {
|
||||||
|
virReportError(VIR_ERR_INTERNAL_ERROR, "%s",
|
||||||
|
_("device-list-properties reply data was missing 'name'"));
|
||||||
|
goto cleanup;
|
||||||
|
}
|
||||||
|
|
||||||
|
if (!(proplist[i] = strdup(tmp))) {
|
||||||
|
virReportOOMError();
|
||||||
|
goto cleanup;
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
ret = n;
|
||||||
|
*props = proplist;
|
||||||
|
|
||||||
|
cleanup:
|
||||||
|
if (ret < 0 && proplist) {
|
||||||
|
for (i = 0 ; i < n ; i++)
|
||||||
|
VIR_FREE(proplist[i]);
|
||||||
|
VIR_FREE(proplist);
|
||||||
|
}
|
||||||
|
virJSONValueFree(cmd);
|
||||||
|
virJSONValueFree(reply);
|
||||||
|
return ret;
|
||||||
|
}
|
||||||
|
@ -308,5 +308,9 @@ int qemuMonitorJSONGetEvents(qemuMonitorPtr mon,
|
|||||||
int qemuMonitorJSONGetObjectTypes(qemuMonitorPtr mon,
|
int qemuMonitorJSONGetObjectTypes(qemuMonitorPtr mon,
|
||||||
char ***types)
|
char ***types)
|
||||||
ATTRIBUTE_NONNULL(2);
|
ATTRIBUTE_NONNULL(2);
|
||||||
|
int qemuMonitorJSONGetObjectProps(qemuMonitorPtr mon,
|
||||||
|
const char *type,
|
||||||
|
char ***props)
|
||||||
|
ATTRIBUTE_NONNULL(2) ATTRIBUTE_NONNULL(3);
|
||||||
|
|
||||||
#endif /* QEMU_MONITOR_JSON_H */
|
#endif /* QEMU_MONITOR_JSON_H */
|
||||||
|
Loading…
Reference in New Issue
Block a user