[Bug 790752] msdk: supports bufferpool

[GStreamer (GNOME Bugzilla)]

https://bugzilla.gnome.org/show_bug.cgi?id=790752

--- Comment #50 from sreerenj <bsreerenj at gmail.com> ---
@Hynjun: I just want to clarify a few things.

IIUC, you are trying to get all buffer pools stuffs implemented with system
memory first, right? I am a bit afraid that there could be rewrite needed in
the way we treating memory create/map/unmap once there is VIDEO_MEMORY support.
But honestly, I am not a pro in msdk, so could be mistaken too. :)

We need to consider a number of things for designing the buffer pool.
Mainly,

1: It requires vasurface & vaimage handling (gstreamer-vaapi code in a
simplified way if possible) for VIDEO_MEMORY support.

2: Tasks can be submitted together if they share contexts (joined sessions).

3. There is a third memory type called "opaque memory". We might not need this
for our use cases though.

Just wondering what could be the best way to proceed here.

Option 1: Just focus on system memory in this bug and implement all pool
sharing between decoder+vpp+encoder. Later we can do/discuss(rewrite needed
most probably) all video memory stuffs in a separate bug.

Option2: Discuss all features (videomory, task session sharing, and dmabuf
support too) as a part of this bug itself and do all development in a separate
branch before pushing to master.

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.