[libvirt] [PATCH] Document that virNodeGetInfo can return mhz == 0.

[Martin Kletzander]

On Fri, Jul 10, 2015 at 10:58:35AM +0200, Martin Kletzander wrote:
>On Mon, May 11, 2015 at 09:30:06PM +0100, Richard W.M. Jones wrote:
>>On the s/390x architecture, libvirt may already return 0 in the
>>node_info->mhz field (see src/nodeinfo.c:linuxNodeInfoCPUPopulate).
>>
>>We may also want to return this on aarch64 in future, because
>>calculating the proper value requires SMBIOS, which is not available
>>on non-server-class systems (specifically on systems which don't
>>adhere to the SBSA standard).
>>
>>Therefore this change documents the existing behaviour and provides a
>>valid path for aarch64.
>>
>>Signed-off-by: Richard W.M. Jones <rjones at redhat.com>
>>Bug-URL: https://bugzilla.redhat.com/1206353
>>---
>>include/libvirt/libvirt-host.h | 3 ++-
>>1 file changed, 2 insertions(+), 1 deletion(-)
>>
>>diff --git a/include/libvirt/libvirt-host.h b/include/libvirt/libvirt-host.h
>>index 953366b..070550b 100644
>>--- a/include/libvirt/libvirt-host.h
>>+++ b/include/libvirt/libvirt-host.h
>>@@ -354,7 +354,8 @@ struct _virNodeInfo {
>>    char model[32];       /* string indicating the CPU model */
>>    unsigned long memory; /* memory size in kilobytes */
>>    unsigned int cpus;    /* the number of active CPUs */
>>-    unsigned int mhz;     /* expected CPU frequency */
>>+    unsigned int mhz;     /* expected CPU frequency, 0 if not known or
>>+                             on unusual architectures */
>
>I feel like if this is documented (which is fine), then clients (virsh
>in particular) should be modified to work with such behaviour
>(e.g. not print that info).
>

Big apologies for reviving old thread about trivial ACK'd patch, my
workflow got spoiled -> I jumped on this mail in my MUA and thought
that's a new patch, so I reviewed it...  I'll send a patch for the
virsh part myself, sorry again.

>>    unsigned int nodes;   /* the number of NUMA cell, 1 for unusual NUMA
>>                             topologies or uniform memory access; check
>>                             capabilities XML for the actual NUMA topology */
>>--
>>2.3.1
>>
>>--
>>libvir-list mailing list
>>libvir-list at redhat.com
>>https://www.redhat.com/mailman/listinfo/libvir-list



>--
>libvir-list mailing list
>libvir-list at redhat.com
>https://www.redhat.com/mailman/listinfo/libvir-list
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20150710/e936d163/attachment-0001.sig>