Awn Extras

System monitor applet: incorrect sort order of entries when size > 2048M

Bug #221345 reported by Fedor Isakov
2
Affects Status Importance Assigned to Milestone
Awn Extras
Fix Released
Undecided
moonbeam
Awn Extras 0.2.8

Bug Description

The system monitor applet displays processes with resident/virtual memory size > 2048 at the bottom of the list. Apparently this is due to signed int used to represent memory size in bytes, which turns negative as soon as the size grows over 2^31.

Hardy/64, awn-core-applets-bzr 0.3.1.bzr439.1~hardy from http://ppa.launchpad.net/reacocard-awn/ubuntu/.

moonbeam (rcryderman)
Changed in awn-extras:
assignee: nobody → rcryderman
Revision history for this message
moonbeam (rcryderman) wrote :

I thought I had this patched a few months ago... I guess the silence I received when I asked for it to be tested was not an indication of it being fixed. I'll have a look.

Revision history for this message
moonbeam (rcryderman) wrote :

Committed rev 447. It'll probably show up in the PPA and reacocard's repo sometime soon.

If this doesn't fix it then I'll have a more detailed look at the code again... I had previously went in and, I believe, hunted down all the signed int s used and replaced with guint64. However I did make a couple structure members guint. I've switched them to guint64. I'm not sure that is the cause though as I wouldn't expect that to cause the behavior you're seeing... instead I would expect an issue at >4096. Anyway, please test.

Changed in awn-extras:
status: New → In Progress
Revision history for this message
Fedor Isakov (fisakov) wrote :

The bug that was fixed dealt with how size was displayed, and that was indeed fixed.

Revision history for this message
moonbeam (rcryderman) wrote :

Additional commit. rev 448. Fixed some comparison routines to account for the change to guint64.

Revision history for this message
moonbeam (rcryderman) wrote :

Fedor,

Thanks for the clarification - I'm glad the previous issue was resolved. I _suspect_ rev 448 will fix this issue until such time as process sizes begin to exceed 2^32.

Revision history for this message
moonbeam (rcryderman) wrote :

Fedor,

  Are you still being affected by this bug?

Revision history for this message
Fedor Isakov (fisakov) wrote :

Not anymore, as of awn-core-applets-bzr version 0.3.1.bzr473.1~hardy.

Thanks.

moonbeam (rcryderman)
Changed in awn-extras:
status: In Progress → Fix Committed
Mark Lee (malept)
Changed in awn-extras:
milestone: none → 0.2.8
moonbeam (rcryderman)
Changed in awn-extras:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

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