1094676 - [Midori 2.0][Power consumption][Camera] Camera Standby current (screen off) is 170mA.

Status: RESOLVED WONTFIX

(Firefox OS Graveyard :: Gaia::Camera, defect, P1)

Description

[sync-1]

FireFox OS V2.0
 mozilla build id:20141019000201
 
 DEFECT DESCRIPTION:
  Camera Standby current (screen off) is 170mA.
 
  REPRODUCING PROCEDURES:
  1. Enter camera and click power key to make screen off;
  2. Take measure standby current, it is 170mA.--KO
 
  EXPECTED BEHAVIOUR:
  Standby current should <5mA.
 
  ASSOCIATE SPECIFICATION:
 
  TEST PLAN REFERENCE:
 
  TOOLS AND PLATFORMS USED:
 
  USER IMPACT:
 
  REPRODUCING RATE:
 
  For FT PR, Please list reference mobile's behavior:
  Fire E 1.3 is <5mA.

Comment 1

[hanp]

[Blocking Requested - why for this release]:

The current standby is too high because camera isn't notified to be released
It is a serious problem.

Comment 2

[Mike Habicher [:mikeh] (high bugzilla latency)]

Diego, do we release the camera hardware when the screen is turned off?

Comment 3

[Diego Marcos [:dmarcos]]

(In reply to Mike Habicher [:mikeh] from comment #2)
> Diego, do we release the camera hardware when the screen is turned off?

We release the camera when the app goes to background but not when the screen turns off. I don't even know if there's a way to detect the screen going off within gaia. I would have to investigate

Comment 4

[Wesly Huang (EPM)]

Hi [e-mail]:

I would recommend you to since check from lower part to confirm if there is anything unusual to consume such power. As I checked other products (not yet really measure it but ask if similar issue) and answer is no. So doubt it's integration problem in camera SW.

Currently tend to be device-specific thus candidate for de-tag.

Comment 5

[Guoqiang.CHEN]

Dear Wesly,

In this build(20141019000201) when screen off, camera is not switch to back.
you can run "adb shell b2g-ps --oom" command in terminal, check camera OOM_ADJ.
when screen off, OOM_ADJ is 2, it should be 10.

In this build(

Comment 6

[Guoqiang.CHEN]

Dear Wesly,

Can you help to tell us, when the screen is off, which code responsible for switch app to the background?
Thanks.

Comment 7

[Wesly Huang (EPM)]

Hi Diego, do you know the answer for question in comment#6? Or know who might be able to answer? Thank you.

Comment 8

[Justin D'Arcangelo [:justindarc]]

Just checked if `visibilitychange` was being triggered when the screen gets locked and it appears that it is. When `visibilitychange` fires, the app releases the camera hardware, so I'm not sure what's causing the 170mA.

Gaia-Rev        d3e4da377ee448f9c25f908159480e867dfb13f3
Gecko-Rev       https://hg.mozilla.org/releases/mozilla-b2g32_v2_0/rev/7198906837e7
Build-ID        20141110000204
Version         32.0

Comment 9

[Guoqiang.CHEN]

Dear Justin,

Camera did not receive 'visibilitychange' when screen turn off, and check camera OOM_ADJ value, it always 2 when screen turn off, the right value should be 10. I want to know which code responsible for switch app to the background when screen turn off.

Thanks.

Comment 10

[Justin D'Arcangelo [:justindarc]]

NI for Alive for question in Comment 9.

Comment 11

[(inactive after 6/18) Alive Kuo [:alive]]

This should not happen unless -- Camera is having an active in normal audio channel.
Because of bug 828283, any app with normal channel playing will not be sent to background even when screen is off. Platform is trying to fix that but I don't know the progress.

If all app is foreground but not only camera, it might be another platform issue or window-mgmt issue.

Please confirm.

Comment 12

[Randy Lin [:rlin] - Be Mozillian on 2015/01/26]

Hi Reporter, 
Could you. 
1. provide the current when enter the home screen and screen off. 
2. run adb shell dumpsys media.audio_policy
3. open camera
4. run adb shell dumpsys media.audio_policy again. 

We can check if any AudioTracks has been opened. 

If audio keeps alive, the current I think should up to around 3~40ma.

Comment 13

[Guoqiang.CHEN]

Attachment: camera.txt

Comment 14

[Guoqiang.CHEN]

Attachment: homescreen.txt

Comment 15

[Randy Lin [:rlin] - Be Mozillian on 2015/01/26]

Base on c13/c14, audio system isn't active on that time.

Comment 16

[Danny Liang [:dliang]]

I do a quickly test on flame v2.0, cannot reproduce this issue, the standby power consumption is ~1.5mA.

My version as below:
Gaia      d3e4da377ee448f9c25f908159480e867dfb13f3
Gecko     dd3811901c677c1a4c71b7797fb1275c71cc8a66
BuildID   20141110112036
Version   32.0
ro.build.version.incremental=eng.waynechen.20141027.153908
ro.build.date=Mon Oct 27 15:39:41 CST 2014

Comment 17

[Rachelle Yang [:ryang][[e-mail]]]

Per Comment 16 from Danny, the standby power consumption is ~1.5mA. Thus, it should be device specific issue. Denominate it for now.

Comment 18

[Guoqiang.CHEN]

Dear Mozilla,

Can you provide more info which code switch app to background?(gecko or kernel?)
In our device, only camera has this issue.

Comment 19

[Guoqiang.CHEN]

Dear Mozilla,

For other app, when press power key, will call function nsDocShell::SetIsActive(bool aIsActive)( in nsDocShell.cpp), but when open camera and press power key, this function not be called.
Can anyone provide some info where should I continue check ? Thanks.

Comment 20

[Wesley Huang [:wesley_huang] (EPM) (NI me)]

I guess you meant to ping Wesly Huang (TAM).

Comment 21

[Guoqiang.CHEN]

It cause by this patch.
https://bugzilla.mozilla.org/show_bug.cgi?id=1058452
https://hg.mozilla.org/integration/mozilla-inbound/rev/286f6adfb567

Comment 22

[hanp]

need to apply patch in https://bugzilla.mozilla.org/show_bug.cgi?id=1069109
but it's for 2.2 and cannot merge to 2.0

Comment 23

[Wesly Huang (EPM)]

Hi hanp, have you tried the re-based 2.0 patch in https://bugzilla.mozilla.org/show_bug.cgi?id=1069109#c17?

Comment 24

[Wesly Huang (EPM)]

Hi hanp, have you tried the re-based 2.0 patch in https://bugzilla.mozilla.org/show_bug.cgi?id=1069109#c17? I think that's all we can do per current timing of 2.0. Thanks for the understanding.

Comment 25

[hanp]

(In reply to Wesly Huang from comment #24)
> Hi hanp, have you tried the re-based 2.0 patch in
> https://bugzilla.mozilla.org/show_bug.cgi?id=1069109#c17? I think that's all
> we can do per current timing of 2.0. Thanks for the understanding.

Hi, wesly, sotaro had given us the patch in https://bugzilla.mozilla.org/show_bug.cgi?id=1058452#c49
and it's ok. Camera standby current problem resolved. Thanks.

Comment 26

[BMO Automation]

Firefox OS is not being worked on