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

improve 10-bit hevc format handling #45

Open
airlied opened this issue May 30, 2023 · 2 comments
Open

improve 10-bit hevc format handling #45

airlied opened this issue May 30, 2023 · 2 comments

Comments

@airlied
Copy link

airlied commented May 30, 2023

Attempting a 10-bit HEVC decode on radv results in some issues, I haven't solved then all yet.

The first one though is the format picking, I think there is one bug in radv, but there's a second bug.

With AMD hw the DPB and decode are distinct, for 10-bit the DPB needs to be in 10-bit, but the output can be 8-bit and the hardware has the ability to handle that.

So radv returns both an 8-bit and 10-bit format for the decode output, but you always pick the 8-bit, and I think you should probably be smarter and scan all returned formats and pick the optimal one?

@charlie-ht
Copy link
Contributor

That's due to this

referencePicturesFormat = supportedDpbFormats[0];

I changed this for my WIP CTS tests for 10-bit here: KhronosGroup/VK-GL-CTS@94425ce#diff-d73087a9725d59f7a76bdaf9f683fa4c773a10122bbce3b419b81fceda71ebb2R698

We need something similar (and probably smarter) for the samples app.

@zlatinski
Copy link
Contributor

Aha, okay. Thank you, charlie-ht and airlied. Indeed, we'll need a fix for that in the sample app. I'll look into fixing this soon.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants