intel / vpl-gpu-rt

MIT License
115 stars 92 forks source link

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

Open Shao-Feng opened 5 months ago

Shao-Feng commented 5 months ago

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 c40736a486913e4e604dbdbae042fde3d2226387 Author: gfxVPLsdm gfxvplsdm@intel.com 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 [zhengguo.xu@intel.com](mailto:zhengguo.xu@intel.com)

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

intel-mediadev commented 5 months ago

Auto Created VSMGWL-74567 for further analysis.