Closed Bug 996361 Opened 7 years ago Closed 7 years ago

[Tarako][Camera] After taking very large, hi res picture (memory pressure), both edit mode and gallery fail to subsequently reload

Categories

(Firefox OS Graveyard :: Gaia::Gallery, defect, P3)

ARM
Gonk (Firefox OS)
defect

Tracking

(b2g-v1.3 unaffected, b2g-v1.3T affected)

RESOLVED WORKSFORME
1.4 S6 (25apr)
Tracking Status
b2g-v1.3 --- unaffected
b2g-v1.3T --- affected

People

(Reporter: jhammink, Unassigned)

Details

(Keywords: memory-footprint, perf, Whiteboard: [c=memory p= s=2014.04.25 u=tarako] [MemShrink:P3])

Attachments

(3 files)

Attached file dmesg.log
[Device]
  Tarako
---------------------------------------------
Gaia      0e7f21e61625b75a9149480cd5a259211549f020
Gecko     https://hg.mozilla.org/releases/mozilla-b2g28_v1_3t/rev/3b02811c314a
BuildID   20140414004001
Version   28.1
ro.build.version.incremental=215
ro.build.date=Mon Mar 31 10:11:50 CST 2014
---------------------------------------------
[STR]
1. Launch Camera.  Position "Color Test" picture on monitor at center of viewport, focusing on colorful area.  Take several pictures.  Try to edit some of the pictures.

---------------------------------------------
[Expected Result]
Picture can be edited, saved to gallery as edited and gallery app closed gracefully.

---------------------------------------------
[Actual Result]
Gallery mode hangs on edit preview.  Eventually gallery app crashes.  After restarting the gallery, gallery goes into loop and continually shows progress, but never seems to fully load.
Attached image color_test1mb.png
No longer depends on: 982545, 996357
Comment on attachment 8406525 [details]
gallery_in_loading_loop.log

adb logcat of gallery stuck in loop during reload
Blocks: 996367
No longer blocks: 996367
Possibly related to: https://bugzilla.mozilla.org/show_bug.cgi?id=995148
Summary: [Tarako][Keyboard][Camera] After taking very large, hi res picture (memory pressure), both edit mode and gallery fail to subsequently reload → [Tarako][Camera] After taking very large, hi res picture (memory pressure), both edit mode and gallery fail to subsequently reload
Can we confirm we can't repro this on Buri 1.3?
Component: Gaia::Dialer → Gaia::Gallery
Keywords: qawanted
Whiteboard: OOM
It appears that this does not happen on buri given the STRs.  Perhaps with even a larger picture file?
Gaia      03340f9b00343512066ab0dceca1641ef9a5374c
Gecko     https://hg.mozilla.org/releases/mozilla-b2g28_v1_3/rev/0b0822ffdbf8
BuildID   20140414164003
Version   28.0
ro.build.version.incremental=324
ro.build.date=Thu Dec 19 14:04:55 CST 2013
Buri

OOM in John's Log:
<4>0[ 2149.188707] select 3341 (Camera), adj 10, size 7141, to kill
<4>0[ 2149.188773] send sigkill to 3341 (Camera), adj 10, size 7141
Keywords: qawantedfootprint, perf
Whiteboard: [c=memory p= s= u=] [MemShrink]
Can we get reduced STR for this bug explicitly calling out each step that was used?
Keywords: qawanted
Whiteboard: [c=memory p= s= u=] [MemShrink] → [c=memory p= s= u=] [MemShrink:P3]
Priority: -- → P3
Whiteboard: [c=memory p= s= u=] [MemShrink:P3] → [c=memory p= s= u=tarako] [MemShrink:P3]
(In reply to Jason Smith [:jsmith] from comment #7)
> Can we get reduced STR for this bug explicitly calling out each step that
> was used?

I was able to reproduce the gallery crash when saving an edited image, but not the subsequent behavior (gallery goes into loop and continually shows progress, but never seems to fully load). I've been testing specifically on editing images in Gallery today and I've not seen this subsequent behavior.

And I believe the gallery crash is being covered more clearly in bug 998095.

Tested on:
Device: Tarako 1.3T MOZ
BuildID: 20140424014003
Gaia: 706f474230432c8cb01e124aee2c055ec902aa1d
Gecko: 26ef3dded9ff
Version: 28.1
Firmware Version: sp6821a_gonk4.0_user.pac
Keywords: qawanted
Closing as wfm then per comment 8.
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → WORKSFORME
Whiteboard: [c=memory p= s= u=tarako] [MemShrink:P3] → [c=memory p= s=2014.04.25 u=tarako] [MemShrink:P3]
Target Milestone: --- → 1.4 S6 (25apr)
You need to log in before you can comment on or make changes to this bug.