From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga12.intel.com (mga12.intel.com [192.55.52.136]) by gabe.freedesktop.org (Postfix) with ESMTPS id 6411C6E2C8 for ; Wed, 18 Dec 2019 11:36:06 +0000 (UTC) Date: Wed, 18 Dec 2019 13:36:02 +0200 From: Petri Latvala Message-ID: <20191218113602.GH25209@platvala-desk.ger.corp.intel.com> References: <20191218112012.3539911-1-chris@chris-wilson.co.uk> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20191218112012.3539911-1-chris@chris-wilson.co.uk> Subject: Re: [igt-dev] [PATCH i-g-t] tests: Open DRM_MASTER for display operations List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Errors-To: igt-dev-bounces@lists.freedesktop.org Sender: "igt-dev" To: Chris Wilson Cc: igt-dev@lists.freedesktop.org List-ID: On Wed, Dec 18, 2019 at 11:20:12AM +0000, Chris Wilson wrote: > To configure the display, one is required to be the DRM_MASTER. > > Closes: https://gitlab.freedesktop.org/drm/intel/issues/855 > Signed-off-by: Chris Wilson > --- > tests/kms_content_protection.c | 2 +- > tests/kms_panel_fitting.c | 2 +- > tests/kms_plane_alpha_blend.c | 2 +- > 3 files changed, 3 insertions(+), 3 deletions(-) > > diff --git a/tests/kms_content_protection.c b/tests/kms_content_protection.c > index 8b361fdef..3b9cedcb4 100644 > --- a/tests/kms_content_protection.c > +++ b/tests/kms_content_protection.c > @@ -654,7 +654,7 @@ static void test_content_protection_cleanup(void) > igt_main > { > igt_fixture { > - data.drm_fd = drm_open_driver(DRIVER_ANY); > + data.drm_fd = drm_open_driver_master(DRIVER_ANY); > > igt_display_require(&data.display, data.drm_fd); > } > diff --git a/tests/kms_panel_fitting.c b/tests/kms_panel_fitting.c > index ac24730f8..065fc2df8 100644 > --- a/tests/kms_panel_fitting.c > +++ b/tests/kms_panel_fitting.c > @@ -252,7 +252,7 @@ igt_main > data_t data = {}; > > igt_fixture { > - data.drm_fd = drm_open_driver(DRIVER_ANY); > + data.drm_fd = drm_open_driver_master(DRIVER_ANY); > igt_display_require(&data.display, data.drm_fd); > igt_display_require_output(&data.display); > } > diff --git a/tests/kms_plane_alpha_blend.c b/tests/kms_plane_alpha_blend.c > index d7dc44ab9..085099f81 100644 > --- a/tests/kms_plane_alpha_blend.c > +++ b/tests/kms_plane_alpha_blend.c > @@ -565,7 +565,7 @@ igt_main > enum pipe pipe; > > igt_fixture { > - data.gfx_fd = drm_open_driver(DRIVER_ANY); > + data.gfx_fd = drm_open_driver_master(DRIVER_ANY); > igt_require_pipe_crc(data.gfx_fd); > igt_display_require(&data.display, data.gfx_fd); > igt_require(data.display.is_atomic); Don't they implicitly become masters if that's the first-and-only open() on the dev? Did that change? -- Petri Latvala _______________________________________________ igt-dev mailing list igt-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/igt-dev