1098685 - [Midori 2.0][Power consumption][Music] Standby current is 34mA (screen off) after pause music.

Status: RESOLVED WONTFIX

(Firefox OS Graveyard :: Gaia::Music, defect, P2)

Description

[sync-1]

FireFox OS V2.0
 
 mozilla build id:20141019000201
 
 DEFECT DESCRIPTION:
  Standby current is 34mA (screen off) after pause music.
 
  REPRODUCING PROCEDURES:
  1. Enter "Music"-> select a song to play;
  2. Pause it, make screen off, take measure current, it is 34mA.--KO
 
 
  EXPECTED BEHAVIOUR:
  Standby current should be<5mA.
 
  ASSOCIATE SPECIFICATION:
 
  TEST PLAN REFERENCE:
 
  TOOLS AND PLATFORMS USED:
 
  USER IMPACT:
 
  REPRODUCING RATE:

Comment 1

[hanp]

[Blocking Requested - why for this release]:
Power consumption problem is a serious problem, please support. thanks

Comment 2

[Shine(Leave from Mozilla)]

Please add logcat and the information of “this is reproducible in 1.3 SW or not” for this issue.

Comment 3

[Wesly Huang (EPM)]

de-tag candidate as rend to be device specific, as this is tested in Flame in other issue and not repro.

Comment 4

[Wesly Huang (EPM)]

[Triage] de-tag as tends to be device-specific.

Comment 5

[hanp]

(In reply to Wesly Huang from comment #4)
> [Triage] de-tag as tends to be device-specific.

hi, wesly, I think that this bug may be having the same issue with bug 1098682
as talked in that bug, blake said should not use pause to save power consumption, and should apply dormant mechanism.
As we debugged on this bug, we got to the same result that while music paused, some audio resources is not released, that's the reason why standby power consumption raised. It's obviously not device-specific.

Comment 6

[Jack Liu]

Hi Wesly,

It was also reproduced on Flame2.0, so I think it's a common issue, at least TCL 8210 platform issue.

We are investigating internally,  need mozilla to check on Flame2.0 at the same time.

Thanks for your understanding.

Comment 7

[Sotaro Ikeda [:sotaro]]

At fist, this problem should be handle as a hw codec's problem. On caf, OMXCocec::pause() should reduce power consumption.

Comment 8

[Sotaro Ikeda [:sotaro]]

diego, can you comment to this bug?

Comment 9

[Diego Wilson [:diego]]

Bhargav,

We test for this, don't we?

Comment 10

[bhargavg1]

See Bug 994881 for more info on how the pause is handled

Comment 11

[bhargavg1]

(In reply to bhargavg1 from comment #10)
> See Bug 994881 for more info on how the pause is handled

Also the following final change, https://www.codeaurora.org/cgit/quic/la/platform/frameworks/av/commit/services/audioflinger/Threads.cpp?h=LNX.LF.3.5.1_RB1.2&id=0e950093daca4e7faa92df31d6fb9e011e77692b

Comment 12

[hanp]

It seems in gecko/content/media/omx/AudioOffloadPlayer.cpp##AudioOffloadPlayer::Pause()
that the power consumption will reduce after a OFFLOAD_PAUSE_MAX_MSECS timer
that is 60s.

So I think it's designed to be so.

Comment 13

[Justin D'Arcangelo [:justindarc]]

Setting NI? for myself to see if this is still happening.

Comment 14

[Justin D'Arcangelo [:justindarc]]

Clearing old NI? requests. Please re-flag me if necessary. Thanks

Comment 15

[BMO Automation]

Firefox OS is not being worked on