Closed Bug 989896 Opened 10 years ago Closed 6 years ago

[Vixen][Gfx]:optimize 1080p video playback for mali gpu

Categories

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

ARM
Gonk (Firefox OS)
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED WONTFIX

People

(Reporter: SteveXu, Unassigned)

References

Details

(Whiteboard: [Vixen][TCP=performance])

Attachments

(1 file)

3.90 KB, application/x-7z-compressed
Details
Attached file gfx_patch.7z
After decode a video frame,gecko need copy video frame from output buffer to mali gpu,this is not efficient. I think the best way is to share the  video buffer.So I add a patch to do this under our platform .
Hi Peter,

Can you please have a look for this issue? If you need device, you can got it from me. Thanks
Flags: needinfo?(pchang)
Summary: optimize 1080p video playback for mali gpu → [Vixen][Gfx]:optimize 1080p video playback for mali gpu
chiajung, please help to check.
Flags: needinfo?(pchang) → needinfo?(chung)
The patch seems very weird for me:
1. Normal video playback on FxOS device does not call GrallocImage::GetAsSurface, so patch the function does not enhance playback performance (normal video playback does not copy decoded frame!)
2. Direct mmap the GraphicBuffer handle should 100% equal to GraphicBuffer::lock. And to make sure memory clean, GraphicBuffer::lock usually involves Fence sync/cache maintain where mmap directly should cause tearing.
3. such low level operation at content process should be forbidden.

And I highly doubt such change would enhance performance much. Since the converter should be the bottleneck here. Can you provide performance log(By GeckoProfiler/PERF) or at least FPS comparison result to see the difference?

Based on the change, I suspect you are trying to enhance video to canvas performance, if so, please check bug 814524/880114 and see if the patch there helps your case.
Flags: needinfo?(chung)
Blocks: vixen
Whiteboard: [Vixen]
Whiteboard: [Vixen] → [Vixen][TCP=performance]
Firefox OS is not being worked on
Status: UNCONFIRMED → RESOLVED
Closed: 6 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: