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

[Bug]: VP9 CBR video is not decoded in correct order #337

Open
Shao-Feng opened this issue Jun 28, 2024 · 1 comment
Open

[Bug]: VP9 CBR video is not decoded in correct order #337

Shao-Feng opened this issue Jun 28, 2024 · 1 comment
Assignees
Labels
Decode Video Decoding VP9 VP9 Codec

Comments

@Shao-Feng
Copy link

Which component impacted?

Decode

Is it regression? Good in old configuration?

Yes, it's good in old version

What happened?

1/ On Android U.
2/ with below media components 2024Q1 release:
VPL GPU Runtime: https://github.com/intel/vpl-gpu-rt/releases/tag/intel-onevpl-24.1.5
VPL Dispatcher and Samples: https://github.com/intel/libvpl/releases/tag/v2.10.2
Driver: https://github.com/intel/media-driver/releases/tag/intel-media-24.1.5
Gmmlib: https://github.com/intel/gmmlib/releases/tag/intel-gmmlib-22.3.18
libva: https://github.com/intel/libva/releases/tag/2.21.0
libva-utils: https://github.com/intel/libva-utils/releases/tag/2.21.0

3/ VP9 CBR video is not decoded in correct order. From video player, some frames are shown in unexcepted time.

What's the usage scenario when you are seeing the problem?

Playback

What impacted?

No response

Debug Information

commit c40736a
Author: gfxVPLsdm [email protected]
Date: Tue Feb 27 14:25:48 2024 +0800

[Decode] Complete cur frame in child thread to unlock res (#6509)

Currently, main thread is responsiable to complete finished frames to
unlock resource, but all available surfaces maybe locked by multi-frame
submission that causes main thread/ application unable to find a free
surface to continue even previous frames decoding are done, so
application has no chance to call async decode api to complete them.

Co-authored-by: Xu, Zhengguo [[email protected]](mailto:[email protected])

After reverting above patch, this issue is not reproduced any more.
Do we have an official fix for it?

Do you want to contribute a patch to fix the issue?

None

@Jexu Jexu self-assigned this Jun 28, 2024
@Jexu Jexu added Decode Video Decoding VP9 VP9 Codec labels Jun 28, 2024
@intel-mediadev
Copy link
Contributor

Auto Created VSMGWL-74567 for further analysis.

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

No branches or pull requests

3 participants