[Intel-xe] [PATCH v9 4/8] drm/xe: tweak lock ordering for freq_lock

Matthew Auld matthew.auld at intel.com
Tue May 23 09:47:23 UTC 2023


Lockdep spits out:

Possible unsafe locking scenario:
   CPU0				CPU1
   ----				----
   lock(&xe->mem_access.lock);
				lock(&pc->freq_lock);
				lock(&xe->mem_access.lock);
   lock(&pc->freq_lock);

It looks like we already mostly take care to ensure that the mem_access
ref is taken outside of taking freq_lock, since that is also grabbed
from the runtime_pm callbacks, except for a couple of spots in
xe_guc_pc and guc_ct_send_recv(). But tt looks like all the callers are
already holding the mem_access.ref, so just switch over to use
guc_ct_send_locked().

v2:
 - There are few more cases it seems.

Signed-off-by: Matthew Auld <matthew.auld at intel.com>
Cc: Rodrigo Vivi <rodrigo.vivi at intel.com>
---
 drivers/gpu/drm/xe/xe_guc_ct.c | 4 +++-
 drivers/gpu/drm/xe/xe_guc_pc.c | 8 ++++++--
 2 files changed, 9 insertions(+), 3 deletions(-)

diff --git a/drivers/gpu/drm/xe/xe_guc_ct.c b/drivers/gpu/drm/xe/xe_guc_ct.c
index 9dc906f2651a..7a8c9d6a03f3 100644
--- a/drivers/gpu/drm/xe/xe_guc_ct.c
+++ b/drivers/gpu/drm/xe/xe_guc_ct.c
@@ -689,7 +689,9 @@ static int guc_ct_send_recv(struct xe_guc_ct *ct, const u32 *action, u32 len,
 retry:
 	g2h_fence_init(&g2h_fence, response_buffer);
 retry_same_fence:
-	ret = guc_ct_send(ct, action, len, 0, 0, &g2h_fence);
+	mutex_lock(&ct->lock);
+	ret = guc_ct_send_locked(ct, action, len, 0, 0, &g2h_fence);
+	mutex_unlock(&ct->lock);
 	if (unlikely(ret == -ENOMEM)) {
 		void *ptr;
 
diff --git a/drivers/gpu/drm/xe/xe_guc_pc.c b/drivers/gpu/drm/xe/xe_guc_pc.c
index e799faa1c6b8..3b56e57d29af 100644
--- a/drivers/gpu/drm/xe/xe_guc_pc.c
+++ b/drivers/gpu/drm/xe/xe_guc_pc.c
@@ -225,7 +225,9 @@ static int pc_action_set_param(struct xe_guc_pc *pc, u8 id, u32 value)
 	if (wait_for_pc_state(pc, SLPC_GLOBAL_STATE_RUNNING))
 		return -EAGAIN;
 
-	ret = xe_guc_ct_send(ct, action, ARRAY_SIZE(action), 0, 0);
+	mutex_lock(&ct->lock);
+	ret = xe_guc_ct_send_locked(ct, action, ARRAY_SIZE(action), 0, 0);
+	mutex_unlock(&ct->lock);
 	if (ret)
 		drm_err(&pc_to_xe(pc)->drm, "GuC PC set param failed: %pe",
 			ERR_PTR(ret));
@@ -242,7 +244,9 @@ static int pc_action_setup_gucrc(struct xe_guc_pc *pc, u32 mode)
 	};
 	int ret;
 
-	ret = xe_guc_ct_send(ct, action, ARRAY_SIZE(action), 0, 0);
+	mutex_lock(&ct->lock);
+	ret = xe_guc_ct_send_locked(ct, action, ARRAY_SIZE(action), 0, 0);
+	mutex_unlock(&ct->lock);
 	if (ret)
 		drm_err(&pc_to_xe(pc)->drm, "GuC RC enable failed: %pe",
 			ERR_PTR(ret));
-- 
2.40.1



More information about the Intel-xe mailing list