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)
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 |
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 .
Comment 1•10 years ago
|
||
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
Comment 3•10 years ago
|
||
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)
Updated•10 years ago
|
Whiteboard: [Vixen]
Updated•10 years ago
|
Whiteboard: [Vixen] → [Vixen][TCP=performance]
Comment 4•6 years ago
|
||
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.
Description
•