All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] drm/i915: Release connector iterator on a digital port conflict.
@ 2018-02-15  9:14 Maarten Lankhorst
  2018-02-15  9:23 ` ✗ Fi.CI.CHECKPATCH: warning for " Patchwork
                   ` (7 more replies)
  0 siblings, 8 replies; 10+ messages in thread
From: Maarten Lankhorst @ 2018-02-15  9:14 UTC (permalink / raw)
  To: intel-gfx

<4>[   30.702366] ================================================
<4>[   30.702383] WARNING: lock held when returning to user space!
<4>[   30.702404] 4.16.0-rc1-CI-kasan_1+ #1 Tainted: G        W
<4>[   30.702422] ------------------------------------------------
<4>[   30.702440] kms_3d/1439 is leaving the kernel with locks still held!
<4>[   30.702459] 1 lock held by kms_3d/1439:
<4>[   30.702472]  #0:  (drm_connector_list_iter){.+.+}, at: [<000000003745d183>] intel_atomic_check+0x1d9d/0x3ff0 [i915]

Signed-off-by: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
Reported-by: Chris Wilson <chris@chris-wilson.co.uk>
---
 drivers/gpu/drm/i915/intel_display.c | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/drivers/gpu/drm/i915/intel_display.c b/drivers/gpu/drm/i915/intel_display.c
index 3eb4c4b2e0e0..9f9656c9cfbe 100644
--- a/drivers/gpu/drm/i915/intel_display.c
+++ b/drivers/gpu/drm/i915/intel_display.c
@@ -10635,6 +10635,7 @@ static bool check_digital_port_conflicts(struct drm_atomic_state *state)
 	struct drm_connector_list_iter conn_iter;
 	unsigned int used_ports = 0;
 	unsigned int used_mst_ports = 0;
+	bool ret = true;
 
 	/*
 	 * Walk the connector list instead of the encoder
@@ -10669,7 +10670,7 @@ static bool check_digital_port_conflicts(struct drm_atomic_state *state)
 
 			/* the same port mustn't appear more than once */
 			if (used_ports & port_mask)
-				return false;
+				ret = false;
 
 			used_ports |= port_mask;
 			break;
@@ -10687,7 +10688,7 @@ static bool check_digital_port_conflicts(struct drm_atomic_state *state)
 	if (used_ports & used_mst_ports)
 		return false;
 
-	return true;
+	return ret;
 }
 
 static void
-- 
2.16.1

_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

^ permalink raw reply related	[flat|nested] 10+ messages in thread

end of thread, other threads:[~2018-02-19 10:00 UTC | newest]

Thread overview: 10+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2018-02-15  9:14 [PATCH] drm/i915: Release connector iterator on a digital port conflict Maarten Lankhorst
2018-02-15  9:23 ` ✗ Fi.CI.CHECKPATCH: warning for " Patchwork
2018-02-15  9:23 ` [PATCH] " Chris Wilson
2018-02-15  9:39 ` ✗ Fi.CI.BAT: failure for " Patchwork
2018-02-15 10:20 ` ✗ Fi.CI.CHECKPATCH: warning " Patchwork
2018-02-15 10:34 ` ✓ Fi.CI.BAT: success " Patchwork
2018-02-15 15:31 ` ✗ Fi.CI.IGT: warning " Patchwork
2018-02-15 16:50 ` ✓ Fi.CI.IGT: success " Patchwork
2018-02-15 23:38 ` [PATCH] " Rodrigo Vivi
2018-02-19 10:00   ` Maarten Lankhorst

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.