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]: media-driver on kernel 6.2-rc6 and DG2 hardware fails execbuffer ioctl #1602

Closed
airlied opened this issue Feb 2, 2023 · 4 comments
Closed
Assignees
Labels
Common memory, surface, ddi Need Info Need more information from submitter

Comments

@airlied
Copy link

airlied commented Feb 2, 2023

Which component impacted?

Decode

Is it regression? Good in old configuration?

No, this issue exist a long time

What happened?

With kernel 6.2.0-rc6 and latest media-driver on a DG2 system (tested on a380 and a770), av1 decode fails, digging in I found execbuffer was failing due to

i915_gem_execbuffer.c:eb_capture_stage is where it fails

                if (i915_gem_context_is_recoverable(eb->gem_context) &&
                    (IS_DGFX(eb->i915) || GRAPHICS_VER_FULL(eb->i915) > IP_VER(12, 0)))
                        return -EINVAL;

I assume this is media-driver doing something it shouldn't.

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

Playback

What impacted?

No response

Debug Information

No response

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

None

@Jexu
Copy link
Contributor

Jexu commented Feb 2, 2023

This should be incompatible issue of i915 error capture and recoverable ctx involved by i915 change. Suppose media driver has disabled i915 error capure temporarily.
@Dylan-debug can you comment and share the commit.

@Jexu Jexu added Common memory, surface, ddi Need Info Need more information from submitter labels Feb 2, 2023
@nyanmisaka
Copy link
Contributor

This was fixed in the latest tag is intel-media-23.1.0 6b210dd.

22.6.5 and 22.6.6 are two broken versions for using DG2 with upstream kernel.

@intel-mediadev
Copy link
Contributor

Auto Created VSMGWL-61132 for further analysis.

@airlied
Copy link
Author

airlied commented Feb 6, 2023

yes indeed updating to master seems to have fixed it, not sure why I dropped back to an earlier release, I was debugging something else.

@airlied airlied closed this as completed Feb 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Common memory, surface, ddi Need Info Need more information from submitter
Projects
None yet
Development

No branches or pull requests

6 participants