[libvirt] [PATCH] Bug fix: Allow sysinfo to display processor information

Eric Blake eblake at redhat.com
Tue Feb 7 21:48:12 UTC 2012

On 02/07/2012 04:43 AM, Prerna Saxena wrote:
> From: Prerna Saxena <prerna at linux.vnet.ibm.com>
> Date: Tue, 7 Feb 2012 17:05:37 +0530
> Subject: [PATCH] On systems with dmidecode version 2.10 or older,
>  dmidecode displays processor information, followed by BIOS, system and
>  memory-DIMM details.
>  Calls to virSysinfoParseBIOS(), virSysinfoParseSystem() would update
>  the buffer pointer 'base', so the processor information would be lost
>  before virSysinfoParseProcessor() was called. Sysinfo would therefore
>  not be able to display processor details -- It only described <bios>,
>  <system> and <memory_device> details.
>  This patch attempts to insulate sysinfo from ordering of dmidecode
>  output.

I was thinking of going one step further - since we no longer care about
the result, there's no longer a need to return a char *, and we can
instead return an int (0 for success, -1 for failure).  But that can be
a separate cleanup; your patch is a minimal fix for the bug at hand, so
ACK and pushed.

> -    base = outbuf;
> -
> -    if ((base = virSysinfoParseBIOS(base, ret)) == NULL)
> +    if ((virSysinfoParseBIOS(outbuf, ret)) == NULL)
>          goto no_memory;

Eric Blake   eblake at redhat.com    +1-919-301-3266
Libvirt virtualization library http://libvirt.org

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 620 bytes
Desc: OpenPGP digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20120207/5e719f40/attachment-0001.sig>

More information about the libvir-list mailing list