music-app blocks system suspend even when not playing any song
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| | Canonical System Image |
High
|
Unassigned | ||
| | qtmir (Ubuntu) |
High
|
Gerry Boland | ||
| | qtmir (Ubuntu RTM) |
High
|
Gerry Boland | ||
| | unity8 (Ubuntu RTM) |
Undecided
|
Unassigned | ||
Bug Description
current build number: 241
device name: krillin
channel: ubuntu-
last update: 2015-02-20 00:41:40
version version: 241
version ubuntu: 20150219
version device: 20150216-fe747ac
version custom: 20150216-561-29-186
I know we currently have an exception in the lifecycle for music-app, and it seems that there is a quite critical bug with it at the current image.
Opening up music-app is already enough for unity8 to hold a system state request from powerd, blocking suspend until you close the app (having it in the background doesn't really help here).
Ideally the system state request (wakelock) should just be enabled and active when the music-app is also active, playing songs.
This bug can drastically affect the battery life for the user of the phone.
You can check this by running 'powerd-cli list' after opening music-app.
phablet@
powerd-cli: Running as user is not fully supported.
System State Requests:
Name: active, Owner: :1.77, State: 1
Related branches
- Albert Astals Cid (community): Approve on 2015-03-06
- PS Jenkins bot: Approve (continuous-integration) on 2015-03-06
- Michał Sawicz: Abstain on 2015-03-05
- Gerry Boland: Abstain on 2015-03-05
- Michael Zanetti (community): Approve on 2015-02-25
-
Diff: 1165 lines (+646/-246)13 files modifiedCMakeLists.txt (+2/-0)
debian/control (+2/-0)
src/common/abstractdbusservicemonitor.cpp (+2/-3)
src/common/abstractdbusservicemonitor.h (+1/-7)
src/modules/Unity/Application/application.cpp (+3/-1)
src/modules/Unity/Application/application_manager.cpp (+5/-2)
src/modules/Unity/Application/sharedwakelock.cpp (+120/-57)
src/modules/Unity/Application/sharedwakelock.h (+14/-9)
tests/modules/Application/application_test.cpp (+7/-7)
tests/modules/ApplicationManager/application_manager_test.cpp (+78/-0)
tests/modules/SharedWakelock/CMakeLists.txt (+6/-0)
tests/modules/SharedWakelock/sharedwakelock_test.cpp (+365/-146)
tests/modules/common/mock_shared_wakelock.h (+41/-14)
- Albert Astals Cid (community): Approve on 2015-03-06
- Gerry Boland: Abstain on 2015-03-06
-
Diff: 1184 lines (+653/-249)15 files modifieddebian/control (+2/-0)
src/common/abstractdbusservicemonitor.cpp (+2/-3)
src/common/abstractdbusservicemonitor.h (+1/-7)
src/modules/Unity/Application/Application.pro (+1/-1)
src/modules/Unity/Application/application.cpp (+3/-1)
src/modules/Unity/Application/application_manager.cpp (+5/-2)
src/modules/Unity/Application/sharedwakelock.cpp (+120/-57)
src/modules/Unity/Application/sharedwakelock.h (+14/-9)
tests/modules/Application/application_test.cpp (+7/-7)
tests/modules/ApplicationManager/application_manager_test.cpp (+78/-0)
tests/modules/SharedWakelock/SharedWakelock.pro (+11/-0)
tests/modules/SharedWakelock/sharedwakelock_test.cpp (+365/-146)
tests/modules/common/common.pri (+2/-1)
tests/modules/common/mock_shared_wakelock.h (+41/-14)
tests/modules/modules.pro (+1/-1)
| affects: | unity8 (Ubuntu) → unity8 (Ubuntu RTM) |
| tags: | added: battery |
| Changed in canonical-devices-system-image: | |
| status: | New → Confirmed |
| importance: | Undecided → High |
| Changed in qtmir (Ubuntu RTM): | |
| assignee: | nobody → Gerry Boland (gerboland) |
| importance: | Undecided → High |
| status: | New → In Progress |
| Changed in unity8 (Ubuntu RTM): | |
| status: | New → Invalid |
| Changed in canonical-devices-system-image: | |
| milestone: | none → ww13-ota |
| Changed in canonical-devices-system-image: | |
| status: | Confirmed → In Progress |
| Changed in qtmir (Ubuntu): | |
| status: | Confirmed → In Progress |
| assignee: | nobody → Gerry Boland (gerboland) |
| importance: | Undecided → High |
| Launchpad Janitor (janitor) wrote : | #2 |
This bug was fixed in the package qtmir - 0.4.4+15.
---------------
qtmir (0.4.4+
[ Gerry Boland ]
* Refactor wakelock handling. Lifecycle exempt apps now release
wakelock when shell tries to suspend them (LP: #1423787)
-- CI Train Bot <email address hidden> Fri, 06 Mar 2015 14:20:33 +0000
| Changed in qtmir (Ubuntu RTM): | |
| status: | In Progress → Fix Released |
| Changed in canonical-devices-system-image: | |
| status: | In Progress → Fix Released |
| Changed in qtmir (Ubuntu): | |
| assignee: | Gerry Boland (gerboland) → Michał Sawicz (saviq) |
| Changed in qtmir (Ubuntu): | |
| assignee: | Michał Sawicz (saviq) → Gerry Boland (gerboland) |
| Launchpad Janitor (janitor) wrote : | #3 |
This bug was fixed in the package qtmir - 0.4.4+15.
---------------
qtmir (0.4.4+
[ Daniel d'Andrada ]
* Port to the new MirEvent API and add support for pointer events.
(LP: #1417650, #1392716)
[ Gerry Boland ]
* Refactor wakelock handling. Lifecycle exempt apps now release
wakelock when shell tries to suspend them (LP: #1423787)
[ Robert Carr ]
* Port to the new MirEvent API and add support for pointer events.
(LP: #1417650, #1392716)
-- CI Train Bot <email address hidden> Wed, 18 Mar 2015 10:12:22 +0000
| Changed in qtmir (Ubuntu): | |
| status: | In Progress → Fix Released |
Status changed to 'Confirmed' because the bug affects multiple users.