cpuinfo: iowait, irq, softirq, steal

[Int]

Quick patch to make zabbix_agentd known about extended cpu stats
please comment

PS: Linux only

[bbrendon]

Seems like a reasonable patch to include in upstream.
Those stats are very important for performance analysis. Zabbix agents out of the box don't collect enough info to troubleshoot performance.

[SharkCZ]

Yes, these values are really important. I have updated the patch for 1.4.2.

Edit: patch updated 19.9.2007

[SharkCZ]

Yes, these values are really important. I have updated the patch for 1.4.2.

Not only are these numbers important, but the current state can be considered buggy (at least for Linux). For example on a system with IDE disks the iowait state can take tens of percent.

[noxis]

This is a fantastic patch, we run large clusters of servers (ISP) for different uses (web,mail, mysql etc etc). IOWAIT is one of the BIGGEST factors we need to know when things slow down. To date (out of the box) only munin would graph this.

Alexei: Please PLEASE add this into the next version.

[Alexei]

It is already in our TODO list

[noxis]

Is this in 1.4.3/4 or is a new patch required? Cheers

[SharkCZ]

Is this in 1.4.3/4 or is a new patch required? Cheers

The patch is not integrated yet and can be applied without modification.

[bbrendon]

Any update? Has this been included? Whats the holdup??

[maxpower]

Same question here. Obviously the agent still doesn't report cpu idle times in 1.4.5 (at least for Solaris).

[maxpower]

Bug found

Again a "bug" in configure.

While /proc/loadavg and others might exist on one Solaris machine it could not exist on another but the behaviour is fixed in Zabbix cpu.c (ifdef HAVE_PROC_LOADAVG ...). Please determine the existence of this file dynamically and not at compile time !
It will not report any cpu data because of this.

[maxpower]

cpu.c broken for solaris

Again a "bug" in configure.

While /proc/loadavg and others might exist on one ...

You don't even have to look at this file ! Just call getloadavg as you did in earlier versions. The exsistence of /proc/loadavg is no precondition for this function to work !
Please remove all #ifdef HAVE_PROC_LOADAVG calls from cpu.c.

[SharkCZ]

patch updated for current trunk

[maxpower]

Sorry, i tried your patch for zabbix_agent cpustat.[ch] and still i get "not supported" for idle,user,etc.
I didn't apply your Linux patch because i monitor Solaris.

[bbrendon]

Alexei & Crew-
Can you add this to your code?