hildon-desktop causes 6+ wakeups /second on a CB

Bug #158377 reported by Inuka Gunawardana
6
Affects Status Importance Assigned to Milestone
Moblin UI Framework
Fix Released
High
Horace

Bug Description

When powertop is run after bootup, the hildon-desktop is shown to cause 6+ wakeups /second.

Changed in moblin-ui-framework:
assignee: nobody → rob-rhoads
Revision history for this message
rob rhoads (rob-rhoads) wrote :

This should be down to 1 or less times per second.

Changed in moblin-ui-framework:
assignee: rob-rhoads → bob-spencer
Revision history for this message
Inuka Gunawardana (inuka) wrote :

More notes:
I tried investigating this earlier. I preformed the following investigations:
1. Disabled hildon desktop plugin, and the other menus to make sure that this is not caused by the moz embedded browser or any other components. The wakeups remained almost the same.
2. Changed g_timeout_add to g_timeout_add_seconds , and also attempted to change the timeouts in g_timeout_add calls in the code to try and root cause this issue. Was still unable to reduce wakeup frequency.

Horace (horace-li)
Changed in moblin-ui-framework:
assignee: bob-spencer → horace-li
status: New → Triaged
yanglei (lei-a-yang)
Changed in moblin-ui-framework:
importance: Undecided → High
status: Triaged → New
Revision history for this message
Inuka Gunawardana (inuka) wrote :

Some more investigation notes:

-Took out all applets, taskbar icons until I got a blank screen to determine the issue. When this is done there are no interrupts reported by powertop
The following packages were removed:
mobile-basic-flash
moblin-keyboard-manager
hildon-theme-mobile-basic
marquee-plugins.

-Once I was able to get hildon-desktop process to zero interrupts I installed each package and executed powertop to determine the increase in wakeups. By doing so I was able to root cause the interrupts to the following.
1. moblin-keyboard-manager causes. 4 wakeups /second.
2. marquee-plugins 1 wakeup /second. When ltrace is executed on hildon-desktop with the marquee-plugins installed, calls to the function hildon_desktop_from_drawable in libhildondesktop can be seen. This seems to be caused by screen redraws due to the expose_event (caused by overlapping widgets?) . This seems to be a hildon-desktop related bug caused by the marquee features.

Revision history for this message
Inuka Gunawardana (inuka) wrote :

Adding to the #2. The package matchbox-keyboard package is installed as a part of moblin-keyboard-manager

Changed in moblin-ui-framework:
status: New → Triaged
Revision history for this message
Horace (horace-li) wrote :

Thanks, Inuka, that really helps to narrow down the root cuase. Could you tell me what is the version of moblin-keyboard-manager you are testing.

Revision history for this message
Horace (horace-li) wrote :

Latest update: I tried to install latest UNRELEASED moblin-keyboard-manager 0.51 in target, and with it installed, hildon-desktop will drop down 1 wakup/second when idle, and 1.1~1.2 wake up/second when keyboard statusbar plugin button is pressed/released.

moblin-keyboard-manager 0.51 mainly changed the method to spawn virtual keyboard, and remove some unnecessary code. fyi!

Revision history for this message
Inuka Gunawardana (inuka) wrote :

Verified Horace's unreleased changes.
The changes get rid of the wake ups caused by the keyboard manger. We are down to 1 wakeup /second caused by the marquee plugin.

Revision history for this message
Inuka Gunawardana (inuka) wrote :

Root caused the 1 wakeup issue to the clock update. Have a simple fix for it which is attached.

Instead of waiting one second, the time function waits one minute if in sync with seconds. If the clock is out of sync it does a wakeup one second at a time.

Also changed g_timeout_add to g_timeout_add_seconds.

If this change is acceptable, and once the keyboard manager change gets into the build hildon-desktop will be down to about 1 wakeup a minute.

Revision history for this message
Horace (horace-li) wrote :

new moblin-keyboard-manager is released, this issue should be fixed. fyi!

Changed in moblin-ui-framework:
status: Triaged → Fix Committed
Revision history for this message
yanglei (lei-a-yang) wrote :

Inuka, if the symptoms are disappeared in latest image, please help to verify the bug and change the status to "Fix Released".
Thanks.

Yang,Ting (ting-yang)
Changed in moblin-ui-framework:
milestone: none → m9
Zhao Fan (fan-zhao)
Changed in moblin-ui-framework:
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

Bug attachments

Remote bug watches

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