Daniel P. Berrange wrote:
On Tue, Jan 14, 2014 at 12:47:20PM +0400, Roman Bogorodskiy wrote:
I had doubts how to implement that. Looks like the current implementation is tied to Linux CPU metrics:
user nice system idle iowait
That list is hardcoded into virsh-host.c. FreeBSD has a slightly different set of metrics:
user nice system intr idle
I.e. it's interrupt time instead of i/o time.
That's not a problem. Just define a new API constant for INTR and don't use the IOWAIT constant in your impl. No impl is required to support every single possible value.
I decided to go without virsh-host.c modification and used VIR_NODE_CPU_STATS_UTILIZATION.
I'd avoid using that. It only exists for virt which is incapable of providing the raw values. It is much better to let the apps calculate the utilization themselves since they can choose the time period over which to perform the calculation better.
Thanks for advise, I'll provide an updated patch. Roman Bogorodskiy