Bug 105997

Summary: [CI] igt@kms_plane_scaling@* - fail - Test assertion failure function do_display_commit, Last errno: 13, Permission denied
Product: DRI Reporter: Martin Peres <martin.peres>
Component: DRM/IntelAssignee: Intel GFX Bugs mailing list <intel-gfx-bugs>
Status: CLOSED FIXED QA Contact: Intel GFX Bugs mailing list <intel-gfx-bugs>
Severity: normal    
Priority: medium CC: intel-gfx-bugs
Version: XOrg git   
Hardware: Other   
OS: All   
Whiteboard: ReadyForDev
i915 platform: GLK i915 features: display/Other

Description Martin Peres 2018-04-12 08:26:30 UTC 
https://intel-gfx-ci.01.org/tree/drm-tip/IGT_4423/shard-glkb3/igt@kms_plane_scaling@pipe-a-scaler-with-clipping-clamping.html

https://intel-gfx-ci.01.org/tree/drm-tip/IGT_4424/shard-glkb1/igt@kms_plane_scaling@pipe-a-plane-scaling.html

(kms_plane_scaling:1510) igt_kms-CRITICAL: Test assertion failure function do_display_commit, file ../lib/igt_kms.c:3165:
(kms_plane_scaling:1510) igt_kms-CRITICAL: Failed assertion: ret == 0
(kms_plane_scaling:1510) igt_kms-CRITICAL: Last errno: 13, Permission denied
(kms_plane_scaling:1510) igt_kms-CRITICAL: error: -13 != 0
Subtest pipe-A-scaler-with-clipping-clamping failed.
Comment 1 Chris Wilson 2018-04-12 08:41:03 UTC 
More !master.
Comment 2 Chris Wilson 2018-04-12 16:00:11 UTC 
Doesn't address the underlying cause, but should catch it much earlier

commit aa330c72a74c071eddb35300e4dd1957eb8d30ff (HEAD)
Author: Chris Wilson <chris@chris-wilson.co.uk>
Date:   Thu Apr 12 09:41:10 2018 +0100

    igt/kms_plane_scaling: Open DRM_MASTER
    
    Modesetting requires DRM_MASTER privileges, so use
    drm_open_driver_master() to assert that we do acquire them.
    
    References: https://bugs.freedesktop.org/show_bug.cgi?id=105997
    Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
    Acked-by: Antonio Argenziano <antonio.argenziano@intel.com>

It will however dump debugfs/clients if the call to setmaster fails, so debugging should be easier if it strikes again.
Comment 3 Jani Saarinen 2018-04-20 19:18:27 UTC 
Do we wait more info here with this change?
Comment 4 Martin Peres 2018-04-21 11:15:48 UTC 
(In reply to Jani Saarinen from comment #3)
> Do we wait more info here with this change?

Hmm. Since this bug contains very little information and cibuglog's filter is not gonna match anymore, we could close it and re-open another bug when this happens.

Marking it as fixed because it will never fail in this way anymore.

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.