Closed Bug 970198 Opened 11 years ago Closed 6 years ago

[b2g] poppit uses excessive CPU due to auto-orientation sensor polling

Categories

(Core :: Hardware Abstraction Layer (HAL), defect, P3)

Product:
Core
Component:
Hardware Abstraction Layer (HAL)
Platform:
ARM
Gonk (Firefox OS)
Type:
defect

Tracking

()

Status:
RESOLVED INVALID

People

(Reporter: bkelly, Assigned: jmunck)

References

Details

(Keywords: perf, Whiteboard: [c=effect p= s= u=])

When using an app with auto-orientation the device will end up using excessive CPU even while effectively idle. This occurs because we poll the orientation sensor every 50ms in order to perform the automatic orientation change. On my buri this causes ~75% CPU while the app is completely idle. It seems like we should perhaps support variable polling rates. Something like auto-orientation probably needs much lower polling rates compared to a game using 'devicemotion' events. It would also be nice to suppress sensor results that represent sensor changes below a certain threshold. User 63%, System 24%, IOW 11%, IRQ 0% User 26 + Nice 173 + Sys 76 + Idle 0 + IOW 37 + IRQ 0 + SIRQ 0 = 312 PID PR CPU% S #THR VSS RSS PCY UID Name 7937 0 75% R 340 489460K 80056K fg app_7937 /system/b2g/plugin-container 7106 0 8% S 40 182476K 56684K fg root /system/b2g/b2g 7932 0 1% R 1 1032K 420K fg root top 3281 0 0% S 1 0K 0K fg root kworker/0:1 6843 0 0% S 1 0K 0K fg root kworker/0:3
The CPU issue seems to only effect poppit on my buri. Perhaps the IPC is triggering GC due to memory pressure.
Blocks: b2g-poppit
Summary: [b2g] auto-orientation uses excessive CPU due to sensor polling → [b2g] poppit app uses excessive CPU due to auto-orientation sensor polling
I believe I may be seeing this on facebook on the tarako. Need to investigate further.
Assignee: nobody → bkelly
Status: NEW → ASSIGNED
blocking-b2g: --- → 1.3T?
Summary: [b2g] poppit app uses excessive CPU due to auto-orientation sensor polling → [b2g] large memory apps use excessive CPU due to auto-orientation sensor polling
On further investigation, the CPU I was seeing in FB on tarako was not related to this. Sorry for the churn.
Assignee: bkelly → nobody
Status: ASSIGNED → NEW
blocking-b2g: 1.3T? → ---
Summary: [b2g] large memory apps use excessive CPU due to auto-orientation sensor polling → [b2g] poppit uses excessive CPU due to auto-orientation sensor polling
Please review and provide a potential fix date. Thank you
Assignee: nobody → lstowasser
Priority: -- → P3
Status: NEW → ASSIGNED
Please add Julia Munck (jmunck@ea.com) to the cc-list for this bug.
Whiteboard: [c= p= s= u=] → [c=effect p= s= u=]
@tcorbett@ea.com The e-mail address for Julia Munck is invalid.
Flags: needinfo?(tcorbett)
@Nicole: jmunck@ea.com should be the correct address. She is listed as one of the submission owners. (Maybe there is a case issues? try JMunck@ea.com). If this still doesn't work please let me know.
Flags: needinfo?(tcorbett) → needinfo?(nfong)
Julia, please review and provide fix date. Thank you
Assignee: lstowasser → jmunck
Flags: needinfo?(jmunck)
Flags: needinfo?(nfong)
Julia - Are there any updates regarding this bug? Is there a potential fix date?
Flags: needinfo?(jmunck)
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → INVALID
You need to log in before you can comment on or make changes to this bug.