Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[linux] allow using VAAPI with DRM PRIME #19142

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

lrusak
Copy link
Contributor

@lrusak lrusak commented Jan 28, 2021

This is in conjunction with #19140 as it won't work on it's own. I'm PRing it separate to keep review smaller (and because I'm able to as the decoder is separate from the renderer).

This allow using ffmpegs av_hwframe_map to convert AV_PIX_FMT_VAAPI to AV_PIX_FMT_DRM_PRIME. The difference is basically a struct and doesn't require any copying.

This allow us to use the direct-to-plane rendering and EGL render methods for DRM PRIME.

Currently this PR doesn't offer any reason to use DRM PRIME with vaapi over our normal vaapi renderer. However in the future we will be able to activate HDR with VAAPI.

@lrusak lrusak added Type: Feature non-breaking change which adds functionality Component: Video Platform: Linux v20 Nexus labels Jan 28, 2021
@lrusak lrusak added this to the N* 20.0 Alpha 1 milestone Jan 28, 2021
@lrusak
Copy link
Contributor Author

lrusak commented Sep 14, 2021

@jernejsk I added a commit to check for m_pFrame->buf[0]. Please check 😃

@jenkins4kodi jenkins4kodi added the Rebase needed PR that does not apply/merge cleanly to current base branch label Nov 25, 2021
@jenkins4kodi
Copy link
Contributor

@lrusak this needs a rebase

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Component: Video Platform: Linux Rebase needed PR that does not apply/merge cleanly to current base branch Type: Feature non-breaking change which adds functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants