unity8 consuming excessive cpu in a preview page

Bug #1441822 reported by Pat McGowan on 2015-04-08
18
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Canonical System Image
High
Zoltan Balogh
qtdeclarative-opensource-src (Ubuntu)
High
Timo Jyrinki
ubuntu-ui-toolkit (Ubuntu)
Critical
Loïc Molinari
unity8 (Ubuntu)
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

description: updated
description: updated
description: updated
kevin gunn (kgunn72) wrote :

seeing this on mako #164 also

description: updated
kevin gunn (kgunn72) wrote :

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

kevin gunn (kgunn72) wrote :

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

kevin gunn (kgunn72) wrote :

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

kevin gunn (kgunn72) wrote :

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

kevin gunn (kgunn72) wrote :

happens with vivid devel-proposed #125

kevin gunn (kgunn72) wrote :

doesn't happen with #123

kevin gunn (kgunn72) wrote :

and doesn't happen with #124

Changed in canonical-devices-system-image:
importance: Undecided → High
assignee: nobody → kevin gunn (kgunn72)
milestone: none → ww17-2015
Michał Sawicz (saviq) on 2015-04-09
Changed in unity8 (Ubuntu):
status: New → Triaged
assignee: nobody → Albert Astals Cid (aacid)
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
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) on 2015-04-10
Changed in ubuntu-ui-toolkit (Ubuntu):
assignee: Zsombor Egri (zsombi) → Loïc Molinari (loic.molinari)
Zoltan Balogh (bzoltan) on 2015-04-10
Changed in ubuntu-ui-toolkit (Ubuntu):
importance: Undecided → Critical
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.

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) on 2015-04-10
Changed in ubuntu-ui-toolkit (Ubuntu):
status: New → Confirmed
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/ .

Loïc Molinari (loic.molinari) wrote :

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

Changed in qtdeclarative-opensource-src (Ubuntu):
assignee: nobody → Timo Jyrinki (timo-jyrinki)
status: New → In Progress
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
Changed in qtdeclarative-opensource-src (Ubuntu):
status: Confirmed → In Progress
Timo Jyrinki (timo-jyrinki) wrote :

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

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
Changed in canonical-devices-system-image:
status: New → Fix Released
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  Edit
Everyone can see this information.

Duplicates of this bug

Other bug subscribers