RepoMirrors/mpv
1
0
Fork 0
mirror of https://github.com/mpv-player/mpv synced 2025-02-08 07:57:19 +00:00
Code Issues Releases Wiki Activity

vo_gpu: hwdec_vaapi: Don't probe formats for irrelevant endpoints

Browse Source 
While testing support for the exotic formats used by Intel vaapi for
4:2:2 and 4:4:4 surfaces, I realised that we were enumerating all
endpoints and checking formats for them. The problem with this is
that decoding (VLD) endpoints are only a subset of what vaapi exposes.

All the encoding endpoints are there too, and there is also the
None profile that we don't care about, but which generates ffmpeg
warnings if you try and examine it.

So, let's only look at VLD endpoints. This will speed things up a
little bit and make the logging less noisy.
This commit is contained in:
Philip Langdale 2022-01-31 13:03:10 -08:00 committed by Jan Ekström
parent 83f1c87676
commit 240340d60a
1 changed files with 8 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
video/out/hwdec/hwdec_vaapi.c
View File

@ -362,6 +362,10 @@ static void determine_working_formats(struct ra_hwdec *hw)
for (int n = 0; n < num_profiles; n++) { for (int n = 0; n < num_profiles; n++) {
VAProfile profile = profiles[n]; VAProfile profile = profiles[n];
if (profile == VAProfileNone) {
// We don't use the None profile.
continue;
}
int num_ep = 0; int num_ep = 0;
status = vaQueryConfigEntrypoints(p->display, profile, entrypoints, status = vaQueryConfigEntrypoints(p->display, profile, entrypoints,
&num_ep); &num_ep);
@ -371,6 +375,10 @@ static void determine_working_formats(struct ra_hwdec *hw)
continue; continue;
} }
for (int ep = 0; ep < num_ep; ep++) { for (int ep = 0; ep < num_ep; ep++) {
if (entrypoints[ep] != VAEntrypointVLD) {
// We are only interested in decoding entrypoints.
continue;
}
VAConfigID config = VA_INVALID_ID; VAConfigID config = VA_INVALID_ID;
status = vaCreateConfig(p->display, profile, entrypoints[ep], status = vaCreateConfig(p->display, profile, entrypoints[ep],
NULL, 0, &config); NULL, 0, &config);