987102 - [Camera] Helix fails to record video in 480p or 720p

Status: RESOLVED WONTFIX

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

Description

[Wilson Page [:wilsonpage]]

We're exposing higher resolution video profiles 480p and 720p, but Helix errors when trying to record at these levels, despite exposing them as supported recorder profiles in `mozCamera.capabilities.recorderProfiles`.

STEPS:

1. In apps/camera/js/config/app.js show recorder profiles in settings menu by uncommenting `videoRecorderProfiles*` from `settingsMenu` and uncomment '720p' and '450p' in recorderProfilesBack `options`.
2. Launch camera app
3. Set video resolution to 720p or 480p via settings menu
4. Start recording
5. Observe recording error overlay.
6. Change to video resolution to 'CIF' via settings menu
7. Observe video records successfully

Comment 1

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

Wilson, can get capture/attach a logcat? I'm guessing we'll see the same pmem/gralloc failures from the other bug.

mwu, do you know how much pmem we're configuring on our devices? I seem to recall (way back when) that we pruned some of the higher-resolution recording modes from Unagi because of insufficient pmem, which was limited by a configuration choice we had made to give more regular memory to Gecko.

Comment 2

[Wilson Page [:wilsonpage]]

bug 900327 could be related...?

Comment 3

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

(In reply to Wilson Page [:wilsonpage] from comment #2)
>
> bug 900327 could be related...?

Yes, it does. Maybe ping Lecky in the other bug to see if there's any news?

Comment 4

[Michael Wu [:mwu]]

(In reply to Mike Habicher [:mikeh] from comment #1)
> mwu, do you know how much pmem we're configuring on our devices? I seem to
> recall (way back when) that we pruned some of the higher-resolution
> recording modes from Unagi because of insufficient pmem, which was limited
> by a configuration choice we had made to give more regular memory to Gecko.


I don't know how it's configured, but you can check your pmem configuration if you have a device:

adb shell cat /sys/kernel/pmem_regions/pmem*/size

Unfortunately I don't have a device so I can't check it. I believe pmem_adsp is the one you should be looking at for camera issues.

Comment 5

[Wilson Page [:wilsonpage]]

bug 987058 just landed on 'master' which means that video recording will be broken on Helix devices. This is because, we now select the highest quality recorder profile by default; on the Helix this is 720p.

Two options:

1. We leave it up to partners to use a build time configuration for Helix devices that excludes the 480p and 720p profiles from the recorderProfile options.
2. We fix Helix devices at the Gecko level to not expose 480p or 720p options via `camera.capabilities.recorderProfiles`

Passing to mikeh to decide :)

Comment 6

[Wilson Page [:wilsonpage]]

STEPS (updated):

1. Launch camara app on Helix device
2. Switch to 'video' mode
3. Attempt to record a video
4. Observe video record error overlay

Comment 7

[Hema Koka [:hema]]

We do not support those resolutions on helix today (removing 1.4?) ...

Wilson what does the video record overlay error message say?

Comment 8

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

(In reply to Wilson Page [:wilsonpage] from comment #5)
> bug 987058 just landed on 'master' which means that video recording will be
> broken on Helix devices. This is because, we now select the highest quality
> recorder profile by default; on the Helix this is 720p.
> 
> Two options:
> 
> 1. We leave it up to partners to use a build time configuration for Helix
> devices that excludes the 480p and 720p profiles from the recorderProfile
> options.
> 2. We fix Helix devices at the Gecko level to not expose 480p or 720p
> options via `camera.capabilities.recorderProfiles`
> 
> Passing to mikeh to decide :)

I think this will be up to the partners.

mwu, is it possible to change the pmem allocation on Helix/other devices, to try higher resolution recordings?

Comment 9

[Michael Wu [:mwu]]

Yup - there are command line parameters that let you control pmem allocation. pmem_adsp_size is probably what you need. You'll need to repack the boot image with the kernel command line you want.

Comment 10

[BMO Automation]

Firefox OS is not being worked on