Ubuntu
unity8 package

unity8 consuming excessive cpu in a preview page

Bug #1441822 reported by Pat McGowan
18
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Canonical System Image
Fix Released
High
Zoltan Balogh
Canonical System Image ww17-2015
qtdeclarative-opensource-src (Ubuntu)
Fix Released
High
Timo Jyrinki
ubuntu-ui-toolkit (Ubuntu)
Invalid
Critical
Loïc Molinari
unity8 (Ubuntu)
Invalid
High
Unassigned

Bug Description

krillin running vivid 175

Opened a youtube video preview from the Video scope and did not interact with it
Running top I see unity8 between 22 and 32 % and unity-8 dash between 15 and 20
Same with music preview

On mako running rtm 219 its over 80% combined with unity8-dash using the majority
When the screen turns off it continues to use over 20% cpu

The CPU is still used when the scope is in the background

Does not happen in a news or nearby scope preview
Does happen with youtube ad 7digital previews

See original description
Tags: battery
Pat McGowan (pat-mcgowan)
description: updated
description: updated
description: updated
Revision history for this message
kevin gunn (kgunn72) wrote :

seeing this on mako #164 also

Pat McGowan (pat-mcgowan)
description: updated
Revision history for this message
kevin gunn (kgunn72) wrote :

and with ubuntu-touch/ubuntu-rtm/14.09-proposed #260 (mar-30) on krilin (just happened to be laying here)

Revision history for this message
kevin gunn (kgunn72) wrote :

mako vivid devel-proposed #144 (mar-23)

Revision history for this message
kevin gunn (kgunn72) wrote :

doesn't happen on vivid devel-proposed #121 (mar 4)

Revision history for this message
kevin gunn (kgunn72) wrote :

forgot to say it does happen on vivid devel-proposed #130 (mar 12)

Revision history for this message
kevin gunn (kgunn72) wrote :

happens with vivid devel-proposed #125

Revision history for this message
kevin gunn (kgunn72) wrote :

doesn't happen with #123

Revision history for this message
kevin gunn (kgunn72) wrote :

and doesn't happen with #124

Revision history for this message
kevin gunn (kgunn72) wrote :

iiuc, this is what landed in #125
http://people.canonical.com/~lzemczak/landing-team/ubuntu/vivid/125.commitlog

Pat McGowan (pat-mcgowan)
Changed in canonical-devices-system-image:
importance: Undecided → High
assignee: nobody → kevin gunn (kgunn72)
milestone: none → ww17-2015
Michał Sawicz (saviq)
Changed in unity8 (Ubuntu):
status: New → Triaged
assignee: nobody → Albert Astals Cid (aacid)
Revision history for this message
kevin gunn (kgunn72) wrote :

ok, just retested image #125 where this problem appeared, and only downgraded 2 packages....based on the commit logs

Title: The final and ultimat 1.2 version of of the UITK
Owner: bzoltan, zsombi
Included packages:
 qtdeclarative5-ubuntu-ui-toolkit-plugin
   from 1.1.1403+15.04.20150206-0ubuntu2
     to 1.2.1433+15.04.20150306-0ubuntu1
 ubuntu-ui-toolkit-theme
   from 1.1.1403+15.04.20150206-0ubuntu2
     to 1.2.1433+15.04.20150306-0ubuntu1

returning qtdeclarative5-ubuntu-ui-toolkit-plugin to 1.1.1403+15.04.20150206-0ubuntu2
& ubuntu-ui-toolkit-theme to 1.1.1403+15.04.20150206-0ubuntu2
(used these debs https://launchpad.net/~ci-train-ppa-service/+archive/ubuntu/landing-005/+build/6975630 )

Changed in unity8 (Ubuntu):
status: Triaged → Invalid
assignee: Albert Astals Cid (aacid) → nobody
Revision history for this message
Pat McGowan (pat-mcgowan) wrote :

Can you take a look to see if we regressed something here, remember happens on rtm and vivid

Changed in ubuntu-ui-toolkit (Ubuntu):
assignee: nobody → Zsombor Egri (zsombi)
Changed in canonical-devices-system-image:
assignee: kevin gunn (kgunn72) → Zoltan Balogh (bzoltan)
Zsombor Egri (zsombi)
Changed in ubuntu-ui-toolkit (Ubuntu):
assignee: Zsombor Egri (zsombi) → Loïc Molinari (loic.molinari)
Zoltan Balogh (bzoltan)
Changed in ubuntu-ui-toolkit (Ubuntu):
importance: Undecided → Critical
Revision history for this message
Zsombor Egri (zsombi) wrote :

Loic, could you take a look pls? I suspect the UbuntuShape, being the only component in 1.2 which may have relation with the CPU peak, Videos are embedded in a shape.

Revision history for this message
Michał Sawicz (saviq) wrote :

The root cause of this is the same as bug #1431957. See description there for some more info.

Michał Sawicz (saviq)
Changed in ubuntu-ui-toolkit (Ubuntu):
status: New → Confirmed
Revision history for this message
Loïc Molinari (loic.molinari) wrote :

There's a race condition in QtDeclarative that prevents rendering thread animations to be stopped. It's been patched recently in QtDeclarative's trunk. See https://codereview.qt-project.org/#/c/109953/ .

Revision history for this message
Loïc Molinari (loic.molinari) wrote :

Bug #1431957 is not a consequence of this though, so that one still needs more investigations.

Timo Jyrinki (timo-jyrinki)
Changed in qtdeclarative-opensource-src (Ubuntu):
assignee: nobody → Timo Jyrinki (timo-jyrinki)
status: New → In Progress
Loïc Molinari (loic.molinari)
Changed in qtdeclarative-opensource-src (Ubuntu):
status: In Progress → Invalid
status: Invalid → Confirmed
Changed in ubuntu-ui-toolkit (Ubuntu):
status: Confirmed → Invalid
Changed in qtdeclarative-opensource-src (Ubuntu):
importance: Undecided → High
Timo Jyrinki (timo-jyrinki)
Changed in qtdeclarative-opensource-src (Ubuntu):
status: Confirmed → In Progress
Revision history for this message
Timo Jyrinki (timo-jyrinki) wrote :

Fix in landing-015 and seems to be working + no regressions spotted in AP testing.

Revision history for this message
Timo Jyrinki (timo-jyrinki) wrote :

Landed to https://launchpad.net/~ci-train-ppa-service/+archive/ubuntu/stable-phone-overlay/+packages

qtdeclarative-opensource-src (5.4.1-1ubuntu6) vivid; urgency=medium

  * debian/patches/Make-sure-we-stop-animators-if-they-are-yet-to-be-st.patch
    - Pick from upstream (LP: #1441822)
 -- Timo Jyrinki <email address hidden> Mon, 20 Apr 2015 08:39:58 +0000

Changed in qtdeclarative-opensource-src (Ubuntu):
status: In Progress → Fix Committed
Pat McGowan (pat-mcgowan)
Changed in canonical-devices-system-image:
status: New → Fix Released
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package qtdeclarative-opensource-src - 5.4.1-1ubuntu7

---------------
qtdeclarative-opensource-src (5.4.1-1ubuntu7) vivid; urgency=medium

  * debian/patches/Add-QQuickAsyncImageProvider.patch:
    - Backport QQuickAsyncImageProvider (LP: #1439263) (LP: #1425043)
  * debian/patches/Always-remove-reply-from-replies-when-processing-can.patch:
    - Drop, replaced by the new patch.

 -- Timo Jyrinki <email address hidden> Tue, 28 Apr 2015 08:50:56 +0000

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

Duplicates of this bug

Subscribing...

Other bug subscribers

Remote bug watches

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