Open
Bug 778947
Opened 12 years ago
Updated 2 years ago
Decode video frames into memory allocated by gecko
Categories
(Core :: Audio/Video: Playback, defect, P2)
Core
Audio/Video: Playback
Tracking
()
NEW
People
(Reporter: nical, Unassigned)
References
(Blocks 1 open bug)
Details
Attachments
(4 files)
(deleted),
patch
|
Details | Diff | Splinter Review | |
(deleted),
patch
|
Details | Diff | Splinter Review | |
(deleted),
patch
|
Details | Diff | Splinter Review | |
(deleted),
patch
|
Details | Diff | Splinter Review |
The current video pipeline contains many video frame copies, which is a waste of CPU cycles and memory.
To be optimal in the off-main-thread compositing context, we need to decode video frames directly into some memory that can be accessed by the compositor.
This memory can be gralloc on platforms that support it, or shmem, or whatever we have available.
To do this we need an API in the decoder libs with callback functions called by the decoder so that gecko can allocate the appropriate buffer type, and callbacks to keep track of the reference counting of these buffers.
Comment 1•12 years ago
|
||
Comment 2•12 years ago
|
||
Comment 3•12 years ago
|
||
Comment 4•12 years ago
|
||
Here is an initial implementation of a modified version of the API proposed in bug 643454. This version of the API has been updated to work with Android's gralloc (which requires rectangular textures storing memory in a 2D-cacheable layout, and which can be remapped by the hardware), and to handle some of libtheora's post-processing cases, in which the luma plane and chroma planes are taken from a separate set of buffers (because post-processing on the chroma planes is skipped, so using the same set of buffers would require a copy).
Reporter | ||
Updated•11 years ago
|
Assignee: nobody → luis
Comment 5•11 years ago
|
||
Thanks Nicolas! Will start working on this soon :)
Comment 6•11 years ago
|
||
I have been very busy at work lately. Will work on this bug soon.
Sorry about that.
Reporter | ||
Comment 7•11 years ago
|
||
(In reply to Luis de Bethencourt [:luisbg] from comment #6)
> I have been very busy at work lately. Will work on this bug soon.
>
> Sorry about that.
Don't worry Luis, I myself have been meaning to give you some pointers and provide whatever you need to get started on this, and I haven't got around doing it because I am busy as well. Don't hesitate to ping me for guidance by email and/or irc about this bug.
Comment 8•11 years ago
|
||
Will do Nicolas! Thanks :)
Updated•9 years ago
|
Component: Audio/Video → Audio/Video: Playback
Updated•9 years ago
|
Blocks: unaccel-video
Updated•6 years ago
|
Rank: 15
Priority: -- → P2
Comment 9•3 years ago
|
||
The bug assignee didn't login in Bugzilla in the last 7 months.
:jimm, could you have a look please?
For more information, please visit auto_nag documentation.
Assignee: luis → nobody
Flags: needinfo?(jmathies)
Updated•2 years ago
|
Flags: needinfo?(jmathies)
Updated•2 years ago
|
Severity: normal → S3
You need to log in
before you can comment on or make changes to this bug.
Description
•