Re: [PATCH 6/6] sysinfo: implement qemu support

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, Feb 07, 2011 at 05:08:50PM -0700, Eric Blake wrote:
> * 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.

  Yeah, the only unfortunate thing is that if people want to reuse
the output they will have to reindent (well not strictly needed, just
nicer). but it's better to generate a nice output. on input formatting
is ignored anyway !

ACK,

Daniel

-- 
Daniel Veillard      | libxml Gnome XML XSLT toolkit  http://xmlsoft.org/
daniel@xxxxxxxxxxxx  | Rpmfind RPM search engine http://rpmfind.net/
http://veillard.com/ | virtualization library  http://libvirt.org/

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


[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]