-
Notifications
You must be signed in to change notification settings - Fork 348
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
[Feature]: VC-1 decoding support for DG2 #1658
Comments
I also want to see the same WVC1 decoding using Linux - Windows is doing fine with all different profiles I tried (VC1- Advanced L4 - up to 2048x1536 and VC1-Advanced L3 - 1920x1080 progressive and interlaced) - please enable that for open source/Linux, too @XinfengZhang - Intel hardware can do it, linux software stack is missing some features, again |
Auto Created VSMGWL-66369 for further analysis. |
suppose that is issue in document. On all xe platforms, media driver has no support for vc1 decode anymore which is not our POR feature. If you anyone still want to use it, you may need to enable it by self, but it is not commitment without any potential issues. |
How do I do this? Is this some compiler flag that needs to be added? |
The feature is disabled through ddi report. |
It fails when calling
But from the header file, it seems the cm rt currently only supports up to TGLx/DG1. @Jexu Is it possible to fix this by adding headers just like 2bb733b? Or is it no-trivial and requires more changes in cmrtlib? |
cm rt is also abandoned from dg2 which is used to init olp kernel. |
May I ask if there is anything new on this one? @Jexu |
@Jexu If you can point me in the right direction for fixing vc1 decode, I can try to fix it and submit a PR. I'm not sure what the exact issue is, so any background information you can provide me will be helpful. |
@bobby3605 Currently, vc1 caps is disabled in media sku and cm rt is abandoned as well. Both are what you need to fix. First one is easy to enable in media sku mentioned in previous comment, but cm rt is not available on DG2. It is only an interface to load binary kernel, you probobaly could refer to the code logic for tgl e.g to use other method to load kernel or try to enable cm rt again?. (btw, is it failed in https://github.com/intel/media-driver/blob/master/media_driver/agnostic/Xe_M/Xe_HPM/codec/hal/codechal_kernel_olp_mdf_xe_hpm.cpp#L47 ?) But for a note, even you could submit a PR finally, I think it is not easy to be accepted and merged into media driver again since it is an abandoned feature. You may need to maintain it as a private change/ feature etc. |
I submitted a simple fix for this. #1846 If any of you are interested you can give it a try. @mitzsch @bavdevc @bobby3605 @Jexu |
What Feature?
Currently, VC-1 decoding support is not available on Linux on DG2 devices even though they support it and decoding of vc1 works with Windows. Intel reference document also shows VC-1 support for DG2 cards. => https://www.intel.com/content/www/us/en/docs/onevpl/developer-reference-media-intel-hardware/1-1/overview.html#DECODE-OVERVIEW-DISCRETE
What's the usage scenario would be benifited?
Transcode for media delivery, Playback
What impacted?
Having VC-1 decoding support is necessary for playback and also transcoding scenarios.
Accelerating playback is nice, but more importantly for transcoding purposes. The sw decoder for vc-1 is limited to a single thread and therefore limiting transcodes quite a bit - having a GPU that could accelerate this is a must. As it already works on Windows I don´t really understand why it's not supported on Linux (yet).
Would be awesome if it could be added in a future release. Thanks.
Do you want to contribute a patch to develop this feature?
None
The text was updated successfully, but these errors were encountered: