Re: [PATCH] BSD: implement nodeGetCPUStats

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

 



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.

Daniel
-- 
|: http://berrange.com      -o-    http://www.flickr.com/photos/dberrange/ :|
|: http://libvirt.org              -o-             http://virt-manager.org :|
|: http://autobuild.org       -o-         http://search.cpan.org/~danberr/ :|
|: http://entangle-photo.org       -o-       http://live.gnome.org/gtk-vnc :|

--
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]