Bug 105997 - [CI] igt@kms_plane_scaling@* - fail - Test assertion failure function do_display_commit, Last errno: 13, Permission denied
Summary: [CI] igt@kms_plane_scaling@* - fail - Test assertion failure function do_disp...
Status: CLOSED FIXED
Alias: None
Product: DRI
Classification: Unclassified
Component: DRM/Intel (show other bugs)
Version: XOrg git
Hardware: Other All
: medium normal
Assignee: Intel GFX Bugs mailing list
QA Contact: Intel GFX Bugs mailing list
URL:
Whiteboard: ReadyForDev
Keywords:
Depends on:
Blocks:
 
Reported: 2018-04-12 08:26 UTC by Martin Peres
Modified: 2018-04-21 11:15 UTC (History)
1 user (show)

See Also:
i915 platform: GLK
i915 features: display/Other

Attachments

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.