Bug 112118 - [CI][SHARDS]igt@gem_exec_schedule@preempt-hang-blt - skip - Test requirement: has_gpu_reset(fd), SKIP
Summary: [CI][SHARDS]igt@gem_exec_schedule@preempt-hang-blt - skip - Test requirement:...
Status: RESOLVED FIXED
Alias: None
Product: DRI
Classification: Unclassified
Component: DRM/Intel (show other bugs)
Version: DRI git
Hardware: Other All
: not set not set
Assignee: Intel GFX Bugs mailing list
QA Contact: Intel GFX Bugs mailing list
URL:
Whiteboard:
Keywords:
: 112128 112129 112130 112141 (view as bug list)
Depends on:
Blocks:
 
Reported: 2019-10-24 05:56 UTC by Lakshmi
Modified: 2019-10-29 23:55 UTC (History)
1 user (show)

See Also:
i915 platform: ICL
i915 features: GEM/Other


Attachments

Description Lakshmi 2019-10-24 05:56:51 UTC
https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7159/shard-iclb3/igt@gem_exec_schedule@preempt-hang-blt.html
IGT-Version: 1.24-g8153b95b5 (x86_64) (Linux: 5.4.0-rc4-CI-CI_DRM_7159+ x86_64)
Using Execlists submission
Has kernel scheduler
 - With priority sorting
 - With preemption enabled
 - With HW semaphores enabled
Test requirement not met in function igt_allow_hang, file ../lib/igt_gt.c:179:
Test requirement: has_gpu_reset(fd)
Last errno: 22, Invalid argument
Test requirement not met in function igt_allow_hang, file ../lib/igt_gt.c:179:
Test requirement: has_gpu_reset(fd)
Last errno: 2, No such file or directory
Test requirement not met in function __real_main1680, file ../tests/i915/gem_exec_schedule.c:1745:
Test requirement: gem_ring_has_physical_engine(fd, e->exec_id | e->flags)
Last errno: 2, No such file or directory
Test requirement not met in function __real_main1680, file ../tests/i915/gem_exec_schedule.c:1745:
Test requirement: gem_ring_has_physical_engine(fd, e->exec_id | e->flags)
Last errno: 2, No such file or directory
Test requirement not met in function igt_allow_hang, file ../lib/igt_gt.c:179:
Test requirement: has_gpu_reset(fd)
Last errno: 2, No such file or directory
Subtest preempt-hang-blt: SKIP
Test requirement not met in function igt_allow_hang, file ../lib/igt_gt.c:179:
Test requirement: has_gpu_reset(fd)
Last errno: 2, No such file or directory
Test requirement not met in function __real_main1680, file ../tests/i915/gem_exec_schedule.c:1838:
Test requirement: gem_ring_has_physical_engine(fd, e->exec_id | e->flags)
Test requirement not met in function __real_main1680, file ../tests/i915/gem_exec_schedule.c:1838:
Test requirement: gem_ring_has_physical_engine(fd, e->exec_id | e->flags)
Comment 2 Chris Wilson 2019-10-24 15:35:15 UTC
has_gpu_reset() ->

        case I915_PARAM_HAS_GPU_RESET:
                value = i915_modparams.enable_hangcheck &&
                        intel_has_gpu_reset(&i915->gt);
                if (value && intel_has_reset_engine(&i915->gt))
                        value = 2;

has_gpu_reset() is INTEL_INFO() & i915.reset

So either somebody has left i915.enable_hangcheck as 0 or i915.reset as 0. We can at least move the restoration of i915.reset earlier so that it is not implicated here -- my suspicion is that an earlier test leaked i915.enable_hangcheck=0
Comment 3 Chris Wilson 2019-10-25 08:08:39 UTC
commit fb817cd37bcd5192d5749d464129a31c7bb852e5 (upstream/master, origin/master, origin/HEAD)
Author: Chris Wilson <chris@chris-wilson.co.uk>
Date:   Fri Oct 25 00:25:25 2019 +0100

    i915/gem_ctx_exec: Restore i915.enable_hangcheck before failing
    
    If the test fails, don't abort leaving the global i915.enable_hangcheck
    still disabled as that may affect later tests.
    
    Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=112118
    Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
    Reviewed-by: Joonas Lahtinen <joonas.lahtinen@linux.intel.com>
Comment 4 Chris Wilson 2019-10-25 08:11:17 UTC
*** Bug 112130 has been marked as a duplicate of this bug. ***
Comment 5 Chris Wilson 2019-10-25 08:11:41 UTC
*** Bug 112129 has been marked as a duplicate of this bug. ***
Comment 6 Chris Wilson 2019-10-25 08:12:03 UTC
*** Bug 112128 has been marked as a duplicate of this bug. ***
Comment 7 Chris Wilson 2019-10-26 21:02:34 UTC
*** Bug 112141 has been marked as a duplicate of this bug. ***
Comment 9 CI Bug Log 2019-10-28 09:52:29 UTC
The CI Bug Log issue associated to this bug has been updated.

### New filters associated

* KBL: igt@gem_exec_schedule@out-order-* - fail - Failed assertion: result == ctx[0]
  - https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7178/shard-kbl3/igt@gem_exec_schedule@out-order-vebox.html
  - https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7179/shard-kbl3/igt@gem_exec_schedule@out-order-bsd2.html
Comment 10 CI Bug Log 2019-10-28 10:26:57 UTC
A CI Bug Log filter associated to this bug has been updated:

{- SKL: igt@gem_exec_schedule@promotion-blt - fail - Failed assertion: result_read == ctx[2] -}
{+ SKL: igt@gem_exec_schedule@promotion-* - fail - Failed assertion: result_read == ctx[2] +}

New failures caught by the filter:

  * https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7178/shard-skl3/igt@gem_exec_schedule@promotion-vebox.html
Comment 12 Lakshmi 2019-10-29 16:38:59 UTC
Still happening 
https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7170/shard-iclb1/igt@gem_exec_schedule@preempt-hang-blt.html
IGT-Version: 1.24-g8153b95b5 (x86_64) (Linux: 5.4.0-rc4-CI-CI_DRM_7170+ x86_64)
Using Execlists submission
Has kernel scheduler
 - With priority sorting
 - With preemption enabled
 - With HW semaphores enabled
Test requirement not met in function igt_allow_hang, file ../lib/igt_gt.c:179:
Test requirement: has_gpu_reset(fd)
Last errno: 22, Invalid argument
Test requirement not met in function __real_main1680, file ../tests/i915/gem_exec_schedule.c:1745:
Test requirement: gem_ring_has_physical_engine(fd, e->exec_id | e->flags)
Test requirement not met in function igt_allow_hang, file ../lib/igt_gt.c:179:
Test requirement: has_gpu_reset(fd)
Last errno: 2, No such file or directory
Test requirement not met in function igt_allow_hang, file ../lib/igt_gt.c:179:
Test requirement: has_gpu_reset(fd)
Last errno: 2, No such file or directory
Test requirement not met in function igt_allow_hang, file ../lib/igt_gt.c:179:
Test requirement: has_gpu_reset(fd)
Last errno: 2, No such file or directory
Subtest preempt-hang-blt: SKIP
Test requirement not met in function igt_allow_hang, file ../lib/igt_gt.c:179:
Test requirement: has_gpu_reset(fd)
Last errno: 2, No such file or directory
Test requirement not met in function __real_main1680, file ../tests/i915/gem_exec_schedule.c:1838:
Test requirement: gem_ring_has_physical_engine(fd, e->exec_id | e->flags)


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.