Closed
Bug 986753
Opened 10 years ago
Closed 10 years ago
[Tarako] Alarm Clock can get killed in the background and never ring
Categories
(Firefox OS Graveyard :: Gaia::Clock, defect)
Tracking
(blocking-b2g:1.3+, b2g-v1.3 fixed, b2g-v1.3T fixed, b2g-v1.4 fixed, b2g-v2.0 fixed)
People
(Reporter: nhirata, Assigned: mcav)
Details
(Keywords: perf, Whiteboard: [p=3])
Attachments
(1 file)
46 bytes,
text/x-github-pull-request
|
jrburke
:
review+
fabrice
:
approval-gaia-v1.3+
|
Details | Review |
1. launch alarm clock 2. set alarm for 2 minutes into the future with a ring 3. launch camera 4. take a picture 5. switch to gallery 6. open the picture 7. edit the picture, crop, sepia, effects 8. save the picture 9. repeat 6 to 8 until past the time Expected: the alarm rings on time Actual: the alarm doesn't ring Gaia 63a7f183788218871c95a887328d091d58519edf Gecko https://hg.mozilla.org/releases/mozilla-b2g28_v1_3t/rev/818e38fe5f4d BuildID 20140321101803 Version 28.0 ro.build.version.incremental=86 ro.build.date=Thu Mar 20 11:25:47 CST 2014 Tarako Note: 1. sometimes after you reboot the alarm might go off with the time that you had set it to.
Reporter | ||
Comment 1•10 years ago
|
||
noming because the purpose of the alarm fails to function.
Comment 2•10 years ago
|
||
traige: 1.3T+ for failing functinoality ni? Dylan, do you mind having someone to look at it? Thanks
blocking-b2g: 1.3T? → 1.3T+
Flags: needinfo?(doliver)
Comment 3•10 years ago
|
||
cc: bkelly Marcus, can you work with Ben on this one to learn some of the perf team's tricks of the trade on handling Tarako low memory situations?
Assignee | ||
Comment 4•10 years ago
|
||
I can ping :bkelly, though this particular bug looks more like a clock alarm-firing bug than anything else (unless Clock is somehow getting killed when it's trying to fire the alarm, though that seems unlikely). Will look into it.
Status: NEW → ASSIGNED
Comment 5•10 years ago
|
||
Yea, I'd say figuring out if the alarm is trying to fire or not would be the first step. Do we use a hardware clock or something to get notified about the alarm? I wonder if the firmware bits are there or not. Please let me know if I can help with any general memory or perf investigation questions.
Assignee | ||
Comment 6•10 years ago
|
||
In the voice of Monk, here's what happened: In super-high-memory-pressure situations, it may take a long time for the pressure to be relieved enough for Clock's alarm screen to boot up (load modules, etc). The module loader was set to timeout after 7 seconds; if it took longer than that to load the modules, the loader would timeout and the alarm would never display. In manual testing, upping the timeout value caused the alarm to properly fire when the phone caught up with memory use. In this patch, I set the limit at 3 minutes -- but it really could be arbitrarily high. I'm having :jrburke review this, in case he has any thoughts on setting an even higher value.
Attachment #8396458 -
Flags: review?(jrburke)
Assignee | ||
Updated•10 years ago
|
Target Milestone: --- → 1.4 S4 (28mar)
Comment 7•10 years ago
|
||
Comment on attachment 8396458 [details] [review] Link to Github pull-request: https://github.com/mozilla-b2g/gaia/pull/17587 I added some info to the pull request, feel free to comment or update the pull request an flip it back to me for review.
Attachment #8396458 -
Flags: review?(jrburke)
Assignee | ||
Comment 8•10 years ago
|
||
Comment on attachment 8396458 [details] [review] Link to Github pull-request: https://github.com/mozilla-b2g/gaia/pull/17587 Yes, your suggestion is much better. I've revised it accordingly and verified that waitSeconds actually does get set to 0 in the app zip after `make`.
Attachment #8396458 -
Flags: review?(jrburke)
Updated•10 years ago
|
Attachment #8396458 -
Flags: review?(jrburke) → review+
Assignee | ||
Comment 9•10 years ago
|
||
Landed in master: https://github.com/mozilla-b2g/gaia/commit/ede2b2488f467a0b8bddf5a741e07464cd15f735
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
status-b2g-v1.3:
--- → affected
status-b2g-v1.3T:
--- → affected
status-b2g-v1.4:
--- → affected
Resolution: --- → FIXED
Assignee | ||
Comment 10•10 years ago
|
||
Comment on attachment 8396458 [details] [review] Link to Github pull-request: https://github.com/mozilla-b2g/gaia/pull/17587 [Approval Request Comment] [Bug caused by] (feature/regressing bug #): n/a [User impact] if declined: Alarms may fail to fire if the phone is under severe memory pressure at alarm time. [Testing completed]: Verification on Tarako. [Risk to taking this patch] (and alternatives if risky): Low risk. [String changes made]: None.
Attachment #8396458 -
Flags: approval-gaia-v1.4?
Attachment #8396458 -
Flags: approval-gaia-v1.3?(fabrice)
Updated•10 years ago
|
Attachment #8396458 -
Flags: approval-gaia-v1.3?(fabrice) → approval-gaia-v1.3+
Comment 11•10 years ago
|
||
Hi, sorry had to revert this change since i think this might have caused test failures like https://tbpl.mozilla.org/php/getParsedLog.php?id=36703800&tree=B2g-Inbound
Updated•10 years ago
|
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Assignee | ||
Comment 12•10 years ago
|
||
It seems highly unlikely that this is the cause of those failures. Did backing it out solve the TBPL failures you saw, Carsten?
Flags: needinfo?(cbook)
Comment 13•10 years ago
|
||
yeah the failures cleared up after i backed this out, which might also be not really depend on this backout but on the other reverts and issues like bug 971585 we had. I guess if its unlikey that this one caused the test failure you could land this again :)
Flags: needinfo?(cbook)
Assignee | ||
Comment 14•10 years ago
|
||
Thanks for the quick reply, Carsten. Since Travis was happy and this ought to have nothing to do with the failures you saw, I relanded. If you see further problems, of course, let me know. master: 65c1f1385a00a897a62f35886bb03ec350b3b79c
Status: REOPENED → RESOLVED
Closed: 10 years ago → 10 years ago
Resolution: --- → FIXED
Updated•10 years ago
|
blocking-b2g: 1.3T+ → 1.3+
Assignee | ||
Updated•10 years ago
|
Attachment #8396458 -
Flags: approval-gaia-v1.4?
Comment 15•10 years ago
|
||
v1.4: https://github.com/mozilla-b2g/gaia/commit/7d716de0c186416b5b123baa1f3242e23d50529b v1.3: https://github.com/mozilla-b2g/gaia/commit/8d159066289fdb4651e13e838fb91fb9226a2cd5
Updated•10 years ago
|
Assignee | ||
Updated•10 years ago
|
Whiteboard: [p=3]
Comment 16•10 years ago
|
||
verified and fixed with v1.3t Gaia 98ce1340a6c27694f3b32a36b772e8da57caf19a Gecko https://hg.mozilla.org/releases/mozilla-b2g28_v1_3t/rev/bdf9a55b4553 BuildID 20140409164001 Version 28.1
Updated•10 years ago
|
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•