[libvirt] [PATCH] numa: avoid failure in nodememstats on non-NUMA systems

Viktor Mihajlovski mihajlov at linux.vnet.ibm.com
Fri Nov 10 11:33:22 UTC 2017


On 07.11.2017 21:53, John Ferlan wrote:
[...]
>> +        /* 
>              ^
> A 'git am' told me there was an extra blank space after the * ;-)
> >> +         * Even if built without numactl, libvirt claims
>> +         * to have a one-cells NUMA topology. In such a
>> +         * case return the statistics for the entire host.
>> +         */
>> +        if (!virNumaIsAvailable() && cellNum == 0)
>> +            cellNum = VIR_NODE_MEMORY_STATS_ALL_CELLS;
>> +
> 
> I'm NUMA challenged, but this certainly seems reasonable... and it
> appears you have an ACK from Boris anyway... So I'll give it until
> tomorrow for anyone else to throw up their hands and say this isn't
> right. ;-)... if no one does, then I'll push with the space fix...
thanks for fixing and pushing
[...]

-- 

Mit freundlichen Grüßen/Kind Regards
   Viktor Mihajlovski




More information about the libvir-list mailing list