Bug 107634

Summary: [ICL] AVC encoder causes GPU hang on ICL
Product: DRI Reporter: Tony Ye <tony.ye>
Component: DRM/IntelAssignee: Tvrtko Ursulin <tvrtko.ursulin>
Status: CLOSED FIXED QA Contact: Intel GFX Bugs mailing list <intel-gfx-bugs>
Severity: normal    
Priority: high CC: dmitry.v.rogozhkin, intel-gfx-bugs, lakshminarayana.vudum, salwan.searty
Version: DRI git   
Hardware: Other   
OS: All   
Whiteboard: Triaged, ReadyForDev
i915 platform: ICL i915 features: GEM/Other

Description Tony Ye 2018-08-21 03:01:43 UTC
The issue can be reproduced with the media-driver on ICL LP by following the steps described in https://github.com/intel/media-driver/issues/267.

It can be fixed with https://github.com/intel/media-driver/pull/271, but this media-driver patch depends on the i915 "Per context dynamic (sub)slice power-gating" series -- https://patchwork.freedesktop.org/series/48194/.
Comment 1 Francesco Balestrieri 2018-08-21 08:15:50 UTC
Assigned to Tvrtko.
Comment 2 Lakshmi 2018-08-30 05:50:23 UTC
Priority changed to High as the issue hangs GPU.
Comment 3 Tvrtko Ursulin 2018-09-12 10:29:20 UTC
Tony, could you help me creating a simplest possible batch buffer which hangs on ICL if dynamic SSEU is not working or not present? I need to add a more proper functional test to the existing IGT (https://patchwork.freedesktop.org/patch/248544/) since currently all we have is the register read-back.
Comment 4 Tony Ye 2018-09-13 08:28:35 UTC
Tvrtko, the VME sampler cannot be accessed in a simple batch. You need to prepare some states, surfaces, and then launch a kernel to do something with VME. It's similar to https://cgit.freedesktop.org/xorg/app/intel-gpu-tools/tree/lib/media_fill.c, but more efforts are required. Unfortunately, I do not have something which is already working to share with you.

The simplest way to reproduce the issue to use the media driver to encode a stream.
Comment 5 Tvrtko Ursulin 2018-09-13 11:22:48 UTC
Would you or someone from your team be able to estimate the amount of work needed, and then maybe add this test case as follow-up?

We could also look at the feasibility of integrating any test suites you have into the i915 continuous integration testing. That would ensure i915 changes do not break the media driver ever.
Comment 6 Francesco Balestrieri 2019-01-08 09:51:34 UTC
Tvrtko, I think we now have a testcase for this, is that right?
Comment 7 Tvrtko Ursulin 2019-01-08 10:24:42 UTC
Correct, as far as I understand it is all mostly there apart from some process bits and a final rebase pending.
Comment 8 Francesco Balestrieri 2019-02-06 07:19:08 UTC
I believe the required patches are merged. Tony Ye, can you verify whether this is resolved with latest drm-tip?
Comment 9 Tony Ye 2019-02-11 08:09:19 UTC
Yes, the issue is fixed with latest drm-tip. Thank you, @Tvrtko, @Francesco.

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.