On Mon, Feb 07, 2011 at 05:08:45PM -0700, Eric Blake wrote: Agreed looks fine to me, it's a generally useful API, people don't like (rightfully) to write XML and we need a convenient way to extract this. The flags may be useful in the future to limit the amount of informations extracted, for example if we don't want the UUID exported or something similar, Daniel -- Daniel Veillard | libxml Gnome XML XSLT toolkit http://xmlsoft.org/ daniel(a)veillard.com | Rpmfind RPM search engine http://rpmfind.net/ http://veillard.com/ | virtualization library http://libvirt.org/

* src/libvirt.c (virConnectGetSysinfo): New function. * docs/formatdomain.html.in: Mention it. --- Is my cross-page link in formatdomain correct? I didn't see any other examples of XML documentation linking to an API call. docs/formatdomain.html.in | 4 +++- src/libvirt.c | 42 +++++++++++++++++++++++++++++++++++++++++- 2 files changed, 44 insertions(+), 2 deletions(-) diff --git a/docs/formatdomain.html.in b/docs/formatdomain.html.in index 43c78fc..dd91de7 100644 --- a/docs/formatdomain.html.in +++ b/docs/formatdomain.html.in @@ -125,7 +125,9 @@ The <code>mode</code> attribute must be specified, and is either "emulate" (let the hypervisor generate all values), "host" (copy all of Block 0 and Block 1, except for the UUID, from the host's - SMBIOS values), or "sysinfo" (use the values in + SMBIOS values; the <a href="libvirt-libvirt.html#virConnectGetSysinfo"> + <code>virConnectGetSysinfo</code></a> call can be + used to see what values are copied), or "sysinfo" (use the values in the <a href="#elementsSysinfo">sysinfo</a> element). If not specified, the hypervisor default is used. <span class="since"> Since 0.8.7</span> diff --git a/src/libvirt.c b/src/libvirt.c index c65b1e5..479a9b5 100644 --- a/src/libvirt.c +++ b/src/libvirt.c @@ -776,7 +776,7 @@ virRegisterStateDriver(virStateDriverPtr driver) /** * virStateInitialize: - * @privileged: set to 1 if running with root priviledge, 0 otherwise + * @privileged: set to 1 if running with root privilege, 0 otherwise * * Initialize all virtualization drivers. * @@ -1594,6 +1594,46 @@ error: } /** + * virConnectGetSysinfo: + * @conn: pointer to a hypervisor connection + * @flags: callers should always pass 0 + * + * This returns the XML description of the sysinfo details for the + * host on which the hypervisor is running, in the same format as the + * <sysinfo> element of a domain XML. This information is generally + * available only for hypervisors running with root privileges. + * + * Returns the XML string which must be freed by the caller, or + * NULL if there was an error. + */ +char * +virConnectGetSysinfo (virConnectPtr conn, unsigned int flags) +{ + DEBUG("conn=%p", conn); + + virResetLastError(); + + if (!VIR_IS_CONNECT(conn)) { + virLibConnError(VIR_ERR_INVALID_CONN, __FUNCTION__); + virDispatchError(NULL); + return NULL; + } + + if (conn->driver->getSysinfo) { + char *ret = conn->driver->getSysinfo (conn, flags); + if (!ret) + goto error; + return ret; + } + + virLibConnError(VIR_ERR_NO_SUPPORT, __FUNCTION__); + +error: + virDispatchError(conn); + return NULL; +} + +/** * virConnectGetMaxVcpus: * @conn: pointer to the hypervisor connection * @type: value of the 'type' attribute in the <domain> element -- 1.7.4

Done by editing the first three files, then running 'make -C src rpcgen', then editing src/remote_protocol-structs to match. * daemon/remote.c (remoteDispatchGetSysinfo): New function. * src/remote/remote_driver.c (remoteGetSysinfo, remote_driver): Client side serialization. * src/remote/remote_protocol.x (remote_get_sysinfo_args) (remote_get_sysinfo_ret): New types. (REMOTE_PROC_GET_SYSINFO): New enum value. * daemon/remote_dispatch_args.h: Regenerate. * daemon/remote_dispatch_prototypes.h: Likewise. * daemon/remote_dispatch_ret.h: Likewise. * daemon/remote_dispatch_table.h: Likewise. * src/remote/remote_protocol.c: Likewise. * src/remote/remote_protocol.h: Likewise. * src/remote_protocol-structs: Likewise. --- daemon/remote.c | 25 ++++++++++++++++++++++++- daemon/remote_dispatch_args.h | 1 + daemon/remote_dispatch_prototypes.h | 8 ++++++++ daemon/remote_dispatch_ret.h | 1 + daemon/remote_dispatch_table.h | 5 +++++ src/remote/remote_driver.c | 27 ++++++++++++++++++++++++++- src/remote/remote_protocol.c | 18 ++++++++++++++++++ src/remote/remote_protocol.h | 15 +++++++++++++++ src/remote/remote_protocol.x | 13 +++++++++++-- src/remote_protocol-structs | 6 ++++++ 10 files changed, 115 insertions(+), 4 deletions(-) diff --git a/daemon/remote.c b/daemon/remote.c index aa6a1a4..de45ff9 100644 --- a/daemon/remote.c +++ b/daemon/remote.c @@ -1,7 +1,7 @@ /* * remote.c: handlers for RPC method calls * - * Copyright (C) 2007-2010 Red Hat, Inc. + * Copyright (C) 2007-2011 Red Hat, Inc. * * This library is free software; you can redistribute it and/or * modify it under the terms of the GNU Lesser General Public @@ -585,6 +585,29 @@ remoteDispatchGetUri (struct qemud_server *server ATTRIBUTE_UNUSED, } static int +remoteDispatchGetSysinfo (struct qemud_server *server ATTRIBUTE_UNUSED, + struct qemud_client *client ATTRIBUTE_UNUSED, + virConnectPtr conn, + remote_message_header *hdr ATTRIBUTE_UNUSED, + remote_error *rerr, + remote_get_sysinfo_args *args, + remote_get_sysinfo_ret *ret) +{ + unsigned int flags; + char *sysinfo; + + flags = args->flags; + sysinfo = virConnectGetSysinfo (conn, flags); + if (sysinfo == NULL) { + remoteDispatchConnError(rerr, conn); + return -1; + } + + ret->sysinfo = sysinfo; + return 0; +} + +static int remoteDispatchGetMaxVcpus (struct qemud_server *server ATTRIBUTE_UNUSED, struct qemud_client *client ATTRIBUTE_UNUSED, virConnectPtr conn, diff --git a/daemon/remote_dispatch_args.h b/daemon/remote_dispatch_args.h index 221af88..57962d1 100644 --- a/daemon/remote_dispatch_args.h +++ b/daemon/remote_dispatch_args.h @@ -171,3 +171,4 @@ remote_domain_get_vcpus_flags_args val_remote_domain_get_vcpus_flags_args; remote_domain_open_console_args val_remote_domain_open_console_args; remote_domain_is_updated_args val_remote_domain_is_updated_args; + remote_get_sysinfo_args val_remote_get_sysinfo_args; diff --git a/daemon/remote_dispatch_prototypes.h b/daemon/remote_dispatch_prototypes.h index 4a5246f..e59701a 100644 --- a/daemon/remote_dispatch_prototypes.h +++ b/daemon/remote_dispatch_prototypes.h @@ -730,6 +730,14 @@ static int remoteDispatchGetMaxVcpus( remote_error *err, remote_get_max_vcpus_args *args, remote_get_max_vcpus_ret *ret); +static int remoteDispatchGetSysinfo( + struct qemud_server *server, + struct qemud_client *client, + virConnectPtr conn, + remote_message_header *hdr, + remote_error *err, + remote_get_sysinfo_args *args, + remote_get_sysinfo_ret *ret); static int remoteDispatchGetType( struct qemud_server *server, struct qemud_client *client, diff --git a/daemon/remote_dispatch_ret.h b/daemon/remote_dispatch_ret.h index c01f3ba..78e5469 100644 --- a/daemon/remote_dispatch_ret.h +++ b/daemon/remote_dispatch_ret.h @@ -138,3 +138,4 @@ remote_domain_get_memory_parameters_ret val_remote_domain_get_memory_parameters_ret; remote_domain_get_vcpus_flags_ret val_remote_domain_get_vcpus_flags_ret; remote_domain_is_updated_ret val_remote_domain_is_updated_ret; + remote_get_sysinfo_ret val_remote_get_sysinfo_ret; diff --git a/daemon/remote_dispatch_table.h b/daemon/remote_dispatch_table.h index 3e55424..5d27390 100644 --- a/daemon/remote_dispatch_table.h +++ b/daemon/remote_dispatch_table.h @@ -1017,3 +1017,8 @@ .args_filter = (xdrproc_t) xdr_remote_domain_is_updated_args, .ret_filter = (xdrproc_t) xdr_remote_domain_is_updated_ret, }, +{ /* GetSysinfo => 203 */ + .fn = (dispatch_fn) remoteDispatchGetSysinfo, + .args_filter = (xdrproc_t) xdr_remote_get_sysinfo_args, + .ret_filter = (xdrproc_t) xdr_remote_get_sysinfo_ret, +}, diff --git a/src/remote/remote_driver.c b/src/remote/remote_driver.c index 1d9209d..c33a930 100644 --- a/src/remote/remote_driver.c +++ b/src/remote/remote_driver.c @@ -1748,6 +1748,31 @@ done: return rv; } +static char * +remoteGetSysinfo (virConnectPtr conn, unsigned int flags) +{ + char *rv = NULL; + remote_get_sysinfo_args args; + remote_get_sysinfo_ret ret; + struct private_data *priv = conn->privateData; + + remoteDriverLock(priv); + + memset (&ret, 0, sizeof ret); + args.flags = flags; + if (call (conn, priv, 0, REMOTE_PROC_GET_SYSINFO, + (xdrproc_t) xdr_remote_get_sysinfo_args, (char *) &args, + (xdrproc_t) xdr_remote_get_sysinfo_ret, (char *) &ret) == -1) + goto done; + + /* Caller frees this. */ + rv = ret.sysinfo; + +done: + remoteDriverUnlock(priv); + return rv; +} + static int remoteIsSecure(virConnectPtr conn) { int rv = -1; @@ -10823,7 +10848,7 @@ static virDriver remote_driver = { remoteGetVersion, /* version */ remoteGetLibVersion, /* libvirtVersion */ remoteGetHostname, /* getHostname */ - NULL, /* getSysinfo */ + remoteGetSysinfo, /* getSysinfo */ remoteGetMaxVcpus, /* getMaxVcpus */ remoteNodeGetInfo, /* nodeGetInfo */ remoteGetCapabilities, /* getCapabilities */ diff --git a/src/remote/remote_protocol.c b/src/remote/remote_protocol.c index 6e07b70..bae92ca 100644 --- a/src/remote/remote_protocol.c +++ b/src/remote/remote_protocol.c @@ -420,6 +420,24 @@ xdr_remote_get_hostname_ret (XDR *xdrs, remote_get_hostname_ret *objp) } bool_t +xdr_remote_get_sysinfo_args (XDR *xdrs, remote_get_sysinfo_args *objp) +{ + + if (!xdr_u_int (xdrs, &objp->flags)) + return FALSE; + return TRUE; +} + +bool_t +xdr_remote_get_sysinfo_ret (XDR *xdrs, remote_get_sysinfo_ret *objp) +{ + + if (!xdr_remote_nonnull_string (xdrs, &objp->sysinfo)) + return FALSE; + return TRUE; +} + +bool_t xdr_remote_get_uri_ret (XDR *xdrs, remote_get_uri_ret *objp) { diff --git a/src/remote/remote_protocol.h b/src/remote/remote_protocol.h index 4240963..46d526a 100644 --- a/src/remote/remote_protocol.h +++ b/src/remote/remote_protocol.h @@ -242,6 +242,16 @@ struct remote_get_hostname_ret { }; typedef struct remote_get_hostname_ret remote_get_hostname_ret; +struct remote_get_sysinfo_args { + u_int flags; +}; +typedef struct remote_get_sysinfo_args remote_get_sysinfo_args; + +struct remote_get_sysinfo_ret { + remote_nonnull_string sysinfo; +}; +typedef struct remote_get_sysinfo_ret remote_get_sysinfo_ret; + struct remote_get_uri_ret { remote_nonnull_string uri; }; @@ -2320,6 +2330,7 @@ enum remote_procedure { REMOTE_PROC_DOMAIN_GET_VCPUS_FLAGS = 200, REMOTE_PROC_DOMAIN_OPEN_CONSOLE = 201, REMOTE_PROC_DOMAIN_IS_UPDATED = 202, + REMOTE_PROC_GET_SYSINFO = 203, }; typedef enum remote_procedure remote_procedure; @@ -2384,6 +2395,8 @@ extern bool_t xdr_remote_get_type_ret (XDR *, remote_get_type_ret*); extern bool_t xdr_remote_get_version_ret (XDR *, remote_get_version_ret*); extern bool_t xdr_remote_get_lib_version_ret (XDR *, remote_get_lib_version_ret*); extern bool_t xdr_remote_get_hostname_ret (XDR *, remote_get_hostname_ret*); +extern bool_t xdr_remote_get_sysinfo_args (XDR *, remote_get_sysinfo_args*); +extern bool_t xdr_remote_get_sysinfo_ret (XDR *, remote_get_sysinfo_ret*); extern bool_t xdr_remote_get_uri_ret (XDR *, remote_get_uri_ret*); extern bool_t xdr_remote_get_max_vcpus_args (XDR *, remote_get_max_vcpus_args*); extern bool_t xdr_remote_get_max_vcpus_ret (XDR *, remote_get_max_vcpus_ret*); @@ -2730,6 +2743,8 @@ extern bool_t xdr_remote_get_type_ret (); extern bool_t xdr_remote_get_version_ret (); extern bool_t xdr_remote_get_lib_version_ret (); extern bool_t xdr_remote_get_hostname_ret (); +extern bool_t xdr_remote_get_sysinfo_args (); +extern bool_t xdr_remote_get_sysinfo_ret (); extern bool_t xdr_remote_get_uri_ret (); extern bool_t xdr_remote_get_max_vcpus_args (); extern bool_t xdr_remote_get_max_vcpus_ret (); diff --git a/src/remote/remote_protocol.x b/src/remote/remote_protocol.x index e1981fd..e77aca1 100644 --- a/src/remote/remote_protocol.x +++ b/src/remote/remote_protocol.x @@ -3,7 +3,7 @@ * remote_internal driver and libvirtd. This protocol is * internal and may change at any time. * - * Copyright (C) 2006-2010 Red Hat, Inc. + * Copyright (C) 2006-2011 Red Hat, Inc. * * This library is free software; you can redistribute it and/or * modify it under the terms of the GNU Lesser General Public @@ -379,6 +379,14 @@ struct remote_get_hostname_ret { remote_nonnull_string hostname; }; +struct remote_get_sysinfo_args { + unsigned int flags; +}; + +struct remote_get_sysinfo_ret { + remote_nonnull_string sysinfo; +}; + struct remote_get_uri_ret { remote_nonnull_string uri; }; @@ -2094,7 +2102,8 @@ enum remote_procedure { REMOTE_PROC_DOMAIN_GET_VCPUS_FLAGS = 200, REMOTE_PROC_DOMAIN_OPEN_CONSOLE = 201, - REMOTE_PROC_DOMAIN_IS_UPDATED = 202 + REMOTE_PROC_DOMAIN_IS_UPDATED = 202, + REMOTE_PROC_GET_SYSINFO = 203 /* * Notice how the entries are grouped in sets of 10 ? diff --git a/src/remote_protocol-structs b/src/remote_protocol-structs index 7ca968f..6a7f03d 100644 --- a/src/remote_protocol-structs +++ b/src/remote_protocol-structs @@ -107,6 +107,12 @@ struct remote_get_lib_version_ret { struct remote_get_hostname_ret { remote_nonnull_string hostname; }; +struct remote_get_sysinfo_args { + u_int flags; +}; +struct remote_get_sysinfo_ret { + remote_nonnull_string sysinfo; +}; struct remote_get_uri_ret { remote_nonnull_string uri; }; -- 1.7.4

* src/util/sysinfo.h (virSysinfoFormat): New prototype. * src/conf/domain_conf.c (virDomainSysinfoDefFormat): Move guts... * src/util/sysinfo.c (virSysinfoFormat): ...into new function. * src/libvirt_private.syms: Export it. * src/qemu/qemu_driver.c (qemuGetSysinfo): New function. (qemuDriver): Install it. --- Tested both with qemu:///system (works!) and qemu:///session (error: unsupported configuration: Host SMBIOS information is not available) The prefix hack is necessary so that I can use "" or " " as the leading indentation, depending on host sysinfo vs. domain, so it's not quite straight code motion. src/conf/domain_conf.c | 71 ++------------------------ src/libvirt_private.syms | 1 + src/qemu/qemu_driver.c | 18 ++++++- src/util/sysinfo.c | 125 ++++++++++++++++++++++++++++++++++++++++++++- src/util/sysinfo.h | 5 ++- 5 files changed, 148 insertions(+), 72 deletions(-) diff --git a/src/conf/domain_conf.c b/src/conf/domain_conf.c index 9369ed4..401aee7 100644 --- a/src/conf/domain_conf.c +++ b/src/conf/domain_conf.c @@ -7143,75 +7143,12 @@ static int virDomainSysinfoDefFormat(virBufferPtr buf, virSysinfoDefPtr def) { - const char *type = virDomainSysinfoTypeToString(def->type); + char *format = virSysinfoFormat(def, " "); - if (!type) { - virDomainReportError(VIR_ERR_INTERNAL_ERROR, - _("unexpected sysinfo type model %d"), - def->type); + if (!format) return -1; - } - - virBufferVSprintf(buf, " <sysinfo type='%s'>\n", type); - if ((def->bios_vendor != NULL) || (def->bios_version != NULL) || - (def->bios_date != NULL) || (def->bios_release != NULL)) { - virBufferAddLit(buf, " <bios>\n"); - if (def->bios_vendor != NULL) - virBufferEscapeString(buf, - " <entry name='vendor'>%s</entry>\n", - def->bios_vendor); - if (def->bios_version != NULL) - virBufferEscapeString(buf, - " <entry name='version'>%s</entry>\n", - def->bios_version); - if (def->bios_date != NULL) - virBufferEscapeString(buf, - " <entry name='date'>%s</entry>\n", - def->bios_date); - if (def->bios_release != NULL) - virBufferEscapeString(buf, - " <entry name='release'>%s</entry>\n", - def->bios_release); - virBufferAddLit(buf, " </bios>\n"); - } - if ((def->system_manufacturer != NULL) || (def->system_product != NULL) || - (def->system_version != NULL) || (def->system_serial != NULL) || - (def->system_uuid != NULL) || (def->system_sku != NULL) || - (def->system_family != NULL)) { - virBufferAddLit(buf, " <system>\n"); - if (def->system_manufacturer != NULL) - virBufferEscapeString(buf, - " <entry name='manufacturer'>%s</entry>\n", - def->system_manufacturer); - if (def->system_product != NULL) - virBufferEscapeString(buf, - " <entry name='product'>%s</entry>\n", - def->system_product); - if (def->system_version != NULL) - virBufferEscapeString(buf, - " <entry name='version'>%s</entry>\n", - def->system_version); - if (def->system_serial != NULL) - virBufferEscapeString(buf, - " <entry name='serial'>%s</entry>\n", - def->system_serial); - if (def->system_uuid != NULL) - virBufferEscapeString(buf, - " <entry name='uuid'>%s</entry>\n", - def->system_uuid); - if (def->system_sku != NULL) - virBufferEscapeString(buf, - " <entry name='sku'>%s</entry>\n", - def->system_sku); - if (def->system_family != NULL) - virBufferEscapeString(buf, - " <entry name='family'>%s</entry>\n", - def->system_family); - virBufferAddLit(buf, " </system>\n"); - } - - virBufferAddLit(buf, " </sysinfo>\n"); - + virBufferAdd(buf, format, strlen(format)); + VIR_FREE(format); return 0; } diff --git a/src/libvirt_private.syms b/src/libvirt_private.syms index 88e270c..6348e0d 100644 --- a/src/libvirt_private.syms +++ b/src/libvirt_private.syms @@ -798,6 +798,7 @@ virStorageFileProbeFormatFromFD; # sysinfo.h virSysinfoDefFree; +virSysinfoFormat; virSysinfoRead; diff --git a/src/qemu/qemu_driver.c b/src/qemu/qemu_driver.c index 87d228b..52ea98e 100644 --- a/src/qemu/qemu_driver.c +++ b/src/qemu/qemu_driver.c @@ -3275,6 +3275,22 @@ static int kvmGetMaxVCPUs(void) { } +static char * +qemuGetSysinfo(virConnectPtr conn, unsigned int flags) +{ + struct qemud_driver *driver = conn->privateData; + + virCheckFlags(0, NULL); + + if (!driver->hostsysinfo) { + qemuReportError(VIR_ERR_CONFIG_UNSUPPORTED, "%s", + _("Host SMBIOS information is not available")); + return NULL; + } + + return virSysinfoFormat(driver->hostsysinfo, ""); +} + static int qemudGetMaxVCPUs(virConnectPtr conn ATTRIBUTE_UNUSED, const char *type) { if (!type) return 16; @@ -10375,7 +10391,7 @@ static virDriver qemuDriver = { qemudGetVersion, /* version */ NULL, /* libvirtVersion (impl. in libvirt.c) */ virGetHostname, /* getHostname */ - NULL, /* getSysinfo */ + qemuGetSysinfo, /* getSysinfo */ qemudGetMaxVCPUs, /* getMaxVcpus */ nodeGetInfo, /* nodeGetInfo */ qemudGetCapabilities, /* getCapabilities */ diff --git a/src/util/sysinfo.c b/src/util/sysinfo.c index 48ab267..2b764ae 100644 --- a/src/util/sysinfo.c +++ b/src/util/sysinfo.c @@ -1,7 +1,7 @@ /* * sysinfo.c: get SMBIOS/sysinfo information from the host * - * Copyright (C) 2010 Red Hat, Inc. + * Copyright (C) 2010-2011 Red Hat, Inc. * Copyright (C) 2010 Daniel Veillard * * This library is free software; you can redistribute it and/or @@ -91,7 +91,18 @@ virSysinfoRead(void) { _("Host sysinfo extraction not supported on this platform")); return NULL; } -#else + +char * +virSysinfoFormat(virSysinfoDefPtr def ATTRIBUTE_UNUSED, + const char *prefix ATTRIBUTE_UNUSED) +{ + virReportSystemError(ENOSYS, "%s", + _("Host sysinfo extraction not supported on this platform")); + return NULL; +} + +#else /* !WIN32 */ + virSysinfoDefPtr virSysinfoRead(void) { char *path, *cur, *eol, *base; @@ -207,4 +218,112 @@ no_memory: ret = NULL; goto cleanup; } -#endif + +/** + * virSysinfoFormat: + * @def: structure to convert to xml string + * @prefix: string to prefix before each line of xml + * + * This returns the XML description of the sysinfo, or NULL after + * generating an error message. + */ +char * +virSysinfoFormat(virSysinfoDefPtr def, const char *prefix) +{ + const char *type = virDomainSysinfoTypeToString(def->type); + virBuffer buf = VIR_BUFFER_INITIALIZER; + size_t len = strlen(prefix); + + if (!type) { + virSmbiosReportError(VIR_ERR_INTERNAL_ERROR, + _("unexpected sysinfo type model %d"), + def->type); + return NULL; + } + + virBufferVSprintf(&buf, "%s<sysinfo type='%s'>\n", prefix, type); + if ((def->bios_vendor != NULL) || (def->bios_version != NULL) || + (def->bios_date != NULL) || (def->bios_release != NULL)) { + virBufferVSprintf(&buf, "%s <bios>\n", prefix); + if (def->bios_vendor != NULL) { + virBufferAdd(&buf, prefix, len); + virBufferEscapeString(&buf, + " <entry name='vendor'>%s</entry>\n", + def->bios_vendor); + } + if (def->bios_version != NULL) { + virBufferAdd(&buf, prefix, len); + virBufferEscapeString(&buf, + " <entry name='version'>%s</entry>\n", + def->bios_version); + } + if (def->bios_date != NULL) { + virBufferAdd(&buf, prefix, len); + virBufferEscapeString(&buf, + " <entry name='date'>%s</entry>\n", + def->bios_date); + } + if (def->bios_release != NULL) { + virBufferAdd(&buf, prefix, len); + virBufferEscapeString(&buf, + " <entry name='release'>%s</entry>\n", + def->bios_release); + } + virBufferVSprintf(&buf, "%s </bios>\n", prefix); + } + if ((def->system_manufacturer != NULL) || (def->system_product != NULL) || + (def->system_version != NULL) || (def->system_serial != NULL) || + (def->system_uuid != NULL) || (def->system_sku != NULL) || + (def->system_family != NULL)) { + virBufferVSprintf(&buf, "%s <system>\n", prefix); + if (def->system_manufacturer != NULL) { + virBufferAdd(&buf, prefix, len); + virBufferEscapeString(&buf, + " <entry name='manufacturer'>%s</entry>\n", + def->system_manufacturer); + } + if (def->system_product != NULL) { + virBufferAdd(&buf, prefix, len); + virBufferEscapeString(&buf, + " <entry name='product'>%s</entry>\n", + def->system_product); + } + if (def->system_version != NULL) { + virBufferAdd(&buf, prefix, len); + virBufferEscapeString(&buf, + " <entry name='version'>%s</entry>\n", + def->system_version); + } + if (def->system_serial != NULL) { + virBufferAdd(&buf, prefix, len); + virBufferEscapeString(&buf, + " <entry name='serial'>%s</entry>\n", + def->system_serial); + } + if (def->system_uuid != NULL) { + virBufferAdd(&buf, prefix, len); + virBufferEscapeString(&buf, + " <entry name='uuid'>%s</entry>\n", + def->system_uuid); + } + if (def->system_sku != NULL) { + virBufferAdd(&buf, prefix, len); + virBufferEscapeString(&buf, + " <entry name='sku'>%s</entry>\n", + def->system_sku); + } + if (def->system_family != NULL) { + virBufferAdd(&buf, prefix, len); + virBufferEscapeString(&buf, + " <entry name='family'>%s</entry>\n", + def->system_family); + } + virBufferVSprintf(&buf, "%s </system>\n", prefix); + } + + virBufferVSprintf(&buf, "%s</sysinfo>\n", prefix); + + return virBufferContentAndReset(&buf); +} + +#endif /* !WIN32 */ diff --git a/src/util/sysinfo.h b/src/util/sysinfo.h index 1af7ef6..66a59db 100644 --- a/src/util/sysinfo.h +++ b/src/util/sysinfo.h @@ -1,7 +1,7 @@ /* * sysinfo.h: structure and entry points for sysinfo support * - * Copyright (C) 2010 Red Hat, Inc. + * Copyright (C) 2010-2011 Red Hat, Inc. * Copyright (C) 2010 Daniel Veillard * * This library is free software; you can redistribute it and/or @@ -56,4 +56,7 @@ virSysinfoDefPtr virSysinfoRead(void); void virSysinfoDefFree(virSysinfoDefPtr def); +char *virSysinfoFormat(virSysinfoDefPtr def, const char *prefix) + ATTRIBUTE_NONNULL(1) ATTRIBUTE_NONNULL(2); + #endif /* __VIR_SYSINFOS_H__ */ -- 1.7.4