CPUMonitor Docklet shows low CPU usage even when CPU heavily used.

Bug #604677 reported by Tim Gilbert
10
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Docky
Fix Released
Low
Robert Dyer
docky (Ubuntu)
Fix Released
Undecided
Unassigned
Lucid
Fix Released
Undecided
Unassigned

Bug Description

Please see attached photographic evidence.

Ubuntu 10.04:
Linux tgilbert 2.6.32-23-generic #37-Ubuntu SMP Fri Jun 11 08:03:28 UTC 2010 x86_64 GNU/Linux
Docky 2.1.0 bzr docky r1568 ppa
xorg. Installed: 1:7.5+5ubuntu1
mono: 2.4.4 svn151842
nvidia 240m (v195)

affects CPUMonitor Docklet (not sure of its version).

Issue:

The green 'eye' doesn't always get bigger when the CPU is being used. When hovering over the eye, it reports a very different CPU usage than System Monitor, top, or the gnome panel app suggests. I believe that the value reported by CPUMonitor is incorrect.

Steps to reproduce.

Setup several programs running high cpu, in my case I have VMWare running a virtual compiling code, I was deleting a couple of snapshot to free up disk space, also I was running Diskspace analyser. I then noticed that CPUMonitor was still green.

Revision history for this message
Tim Gilbert (knaak) wrote :
Revision history for this message
Robert Dyer (psybers) wrote :

Can not confirm.

Changed in docky:
status: New → Incomplete
importance: Undecided → Low
assignee: nobody → Jason Smith (jassmith)
Revision history for this message
Robert Dyer (psybers) wrote :

Please post the output of running 'cat /proc/stat' from a terminal.

Revision history for this message
Tim Gilbert (knaak) wrote :

as requested.

Revision history for this message
Tim Gilbert (knaak) wrote :

I pulled the source code to have a lookin in CPUMonitorDocItem.cs I see you are adding up usage, but you don't include IOWait in the first bit. Then with idle, you subtract IOWait.

When I am running the Virtual, my IOWaits were higher than I usually seem. I added a patch.

Revision history for this message
Tim Gilbert (knaak) wrote :

Not sure if i broke etiquette but I moved from incomplete to confirmed since there is a patch attached to this bug i figured it couldn't be incomplete... hope that is ok.

Changed in docky:
status: Incomplete → Confirmed
Revision history for this message
Robert Dyer (psybers) wrote :

I can confirm that we forgot IOWait in the computation of 'total usage', which of course will make the percentage off.

This is fixed in rev 1616.

Changed in docky:
milestone: none → 2.0.6
status: Confirmed → Fix Committed
assignee: Jason Smith (jassmith) → Robert Dyer (psybers)
Changed in docky:
status: Fix Committed → Fix Released
Revision history for this message
Rico Tzschichholz (ricotz) wrote :

debdiff which fixes this is attached to bug #584094

Revision history for this message
Martin Pitt (pitti) wrote : Please test proposed package

Accepted docky into lucid-proposed, the package will build now and be available in a few hours. Please test and give feedback here. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation how to enable and use -proposed. Thank you in advance!

Changed in docky (Ubuntu Lucid):
status: New → Fix Committed
tags: added: verification-needed
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package docky - 2.0.6-2

---------------
docky (2.0.6-2) unstable; urgency=low

  * Upload to unstable.

docky (2.0.6-1) experimental; urgency=low

  * New upstream release 2.0.6 "Just lovin' it",
    changes include:
    + fix check for existing instance
    + hitting ESC should cancel drags (LP: #460071)
    + make sure plugins enabled by default are added to the dock on first run
      and enable Trash docklet by default
    + update badge drawing so the text is consistently sized
    + fix leak of ThemedIcon/FileIcon in IconFromGIcon
    + fix non-square items arent properly darken (LP: #584094)
    + remove MenuList.Dispose which is not really needed, solves (LP: #609170)
    + shrink config dialog (LP: #591448)

    + RecentDocs: fix crash (LP: #580723)
    + Mounter: use a better MountItem.UniqueID and don't fill up gconf with
      useless keys (temporary fix for LP: #600311)
    + Weather: add weather icon for Smoke (LP: #612437)
    + CPUMonitor: fix CPU utilization percentage (LP: #604677)
    + GMail:
      - add debuging messages to gmail and log web response status when we
        have a web exception
      - fix crash and leak in Gmail
  * debian/patches:
    + drop 01_use_cli_for_wrapper.patch
  * patch build-system and bump dependency for new gio-sharp >= 2.22.2
    + update debian/patches/use-system-gio-sharp.patch
 -- Iain Lane <email address hidden> Tue, 07 Sep 2010 09:03:13 +0100

Changed in docky (Ubuntu):
status: New → Fix Released
Revision history for this message
Jean-Baptiste Lallement (jibel) wrote :

SRU verification for Lucid:
I have reproduced the problem with docky 2.0.5-0ubuntu1 in lucid-updates and have verified that the version of docky 2.0.6-0ubuntu1 in -proposed fixes the issue.

Marking as verification-done

tags: added: verification-done
removed: verification-needed
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package docky - 2.0.6-0ubuntu1

---------------
docky (2.0.6-0ubuntu1) lucid-proposed; urgency=low

  * New upstream release 2.0.6 "Just lovin' it",
    changes include:
    + fix check for existing instance
    + hitting ESC should cancel drags (LP: #460071)
    + make sure plugins enabled by default are added to the dock on first run
      and enable Trash docklet by default
    + update badge drawing so the text is consistently sized
    + fix leak of ThemedIcon/FileIcon in IconFromGIcon
    + fix non-square items arent properly darken (LP: #584094)
    + remove MenuList.Dispose which is not really needed, solves (LP: #609170)
    + shrink config dialog (LP: #591448)

    + RecentDocs: fix crash (LP: #580723)
    + Mounter: use a better MountItem.UniqueID and don't fill up gconf with
      useless keys (temporary fix for LP: #600311)
    + Weather: add weather icon for Smoke (LP: #612437)
    + CPUMonitor: fix CPU utilization percentage (LP: #604677)
    + GMail:
      - add debuging messages to gmail and log web response status when we
        have a web exception
      - fix crash and leak in Gmail
 -- Rico Tzschichholz <email address hidden> Wed, 01 Sep 2010 10:20:15 +0200

Changed in docky (Ubuntu Lucid):
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.