Bug 107399

Setting the highest priority since it blocks the execution of the rest of the self tests.

Is it truly ready for dev when the debug logs are absent?

Fair point... not sure what to do here...

Paulo was already looking at this internally, so assigning this over to him

Created attachment 141015 [details]
dmesg

The bug seems to happen during others-priority. If I comment out the tests that include TEST_PRIORITY then live_hangcheck succeeds (although it gives some messages about ring heads not parking randomly).

I added a DRM_DEBUG_KMS to print the values of the GEM_BUG_ON condition.

It may be worth investigating why the CI is not able to capture this output. I'm using the serial console and never get the unprintable bytes we can see on the CI logs.

RFC patch submitted: https://patchwork.freedesktop.org/series/47976/

commit ee435831ec83344dba5ccddd4ffcc6ca95d1cf77 (HEAD -> drm-intel-next-queued, hsw/hsw, drm-intel/drm-intel-next-queued)
Author: Paulo Zanoni <paulo.r.zanoni@intel.com>
Date:   Thu Aug 9 16:58:52 2018 -0700

    drm/i915/icl: account for context save/restore removed bits
    
    The RS_CTX_ENABLE and CTX_SAVE_INHIBIT bits are not present on ICL
    anymore, but we still try to set them and then check them with
    GEM_BUG_ON, resulting in a BUG() call. The bug can be reproduced by
    igt/drv_selftest/live_hangcheck/others-priority and our CI was able
    to catch it.
    
    It is worth noticing that commit 05f0addd9b10 ("drm/i915/icl: Enhanced
    execution list support") already tried to avoid the save bits
    on ICL, but only inside populate_lr_context().
    
    Cc: Chris Wilson <chris@chris-wilson.co.uk>
    Cc: Mika Kuoppala <mika.kuoppala@linux.intel.com>
    Testcase: igt/drv_selftest/live_hangcheck/others-priority
    Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=107399
    References: 05f0addd9b10 ("drm/i915/icl: Enhanced execution list support")
    Signed-off-by: Paulo Zanoni <paulo.r.zanoni@intel.com>
    Link: https://patchwork.freedesktop.org/patch/msgid/20180809235852.24516-1-paulo.r.zanoni@intel.com
    Reviewed-by: Chris Wilson <chris@chris-wilson.co.uk>
    Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>

Test is now green on CI when run (or dmesg-warn for unrelated issues), so closing