All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH 00/14] drm/msm: de-struct_mutex-ification
@ 2020-10-04 19:21 ` Rob Clark
  0 siblings, 0 replies; 75+ messages in thread
From: Rob Clark @ 2020-10-04 19:21 UTC (permalink / raw)
  To: dri-devel
  Cc: Rob Clark, AngeloGioacchino Del Regno, Ben Dooks, Emil Velikov,
	Eric Anholt, open list:DRM DRIVER FOR MSM ADRENO GPU,
	Jonathan Marek, Jordan Crouse,
	moderated list:DMA BUFFER SHARING FRAMEWORK,
	open list:DRM DRIVER FOR MSM ADRENO GPU, open list,
	open list:DMA BUFFER SHARING FRAMEWORK, Sam Ravnborg,
	Sharat Masetty

From: Rob Clark <robdclark@chromium.org>

This doesn't remove *all* the struct_mutex, but it covers the worst
of it, ie. shrinker/madvise/free/retire.  The submit path still uses
struct_mutex, but it still needs *something* serialize a portion of
the submit path, and lock_stat mostly just shows the lock contention
there being with other submits.  And there are a few other bits of
struct_mutex usage in less critical paths (debugfs, etc).  But this
seems like a reasonable step in the right direction.

Rob Clark (14):
  drm/msm: Use correct drm_gem_object_put() in fail case
  drm/msm: Drop chatty trace
  drm/msm: Move update_fences()
  drm/msm: Add priv->mm_lock to protect active/inactive lists
  drm/msm: Document and rename preempt_lock
  drm/msm: Protect ring->submits with it's own lock
  drm/msm: Refcount submits
  drm/msm: Remove obj->gpu
  drm/msm: Drop struct_mutex from the retire path
  drm/msm: Drop struct_mutex in free_object() path
  drm/msm: remove msm_gem_free_work
  drm/msm: drop struct_mutex in madvise path
  drm/msm: Drop struct_mutex in shrinker path
  drm/msm: Don't implicit-sync if only a single ring

 drivers/gpu/drm/msm/adreno/a5xx_gpu.c     |  4 +-
 drivers/gpu/drm/msm/adreno/a5xx_preempt.c | 12 +--
 drivers/gpu/drm/msm/adreno/a6xx_gpu.c     |  4 +-
 drivers/gpu/drm/msm/msm_debugfs.c         |  7 ++
 drivers/gpu/drm/msm/msm_drv.c             | 15 +---
 drivers/gpu/drm/msm/msm_drv.h             | 19 +++--
 drivers/gpu/drm/msm/msm_gem.c             | 76 ++++++------------
 drivers/gpu/drm/msm/msm_gem.h             | 53 +++++++++----
 drivers/gpu/drm/msm/msm_gem_shrinker.c    | 58 ++------------
 drivers/gpu/drm/msm/msm_gem_submit.c      | 17 ++--
 drivers/gpu/drm/msm/msm_gpu.c             | 96 ++++++++++++++---------
 drivers/gpu/drm/msm/msm_gpu.h             |  5 +-
 drivers/gpu/drm/msm/msm_ringbuffer.c      |  3 +-
 drivers/gpu/drm/msm/msm_ringbuffer.h      | 13 ++-
 14 files changed, 188 insertions(+), 194 deletions(-)

-- 
2.26.2


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

end of thread, other threads:[~2020-10-07  7:22 UTC | newest]

Thread overview: 75+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-10-04 19:21 [PATCH 00/14] drm/msm: de-struct_mutex-ification Rob Clark
2020-10-04 19:21 ` Rob Clark
2020-10-04 19:21 ` [PATCH 01/14] drm/msm: Use correct drm_gem_object_put() in fail case Rob Clark
2020-10-04 19:21   ` Rob Clark
2020-10-04 19:21 ` [PATCH 02/14] drm/msm: Drop chatty trace Rob Clark
2020-10-04 19:21   ` Rob Clark
2020-10-05 14:15   ` [Freedreno] " Jordan Crouse
2020-10-05 14:15     ` Jordan Crouse
2020-10-04 19:21 ` [PATCH 03/14] drm/msm: Move update_fences() Rob Clark
2020-10-04 19:21   ` Rob Clark
2020-10-05 14:16   ` [Freedreno] " Jordan Crouse
2020-10-05 14:16     ` Jordan Crouse
2020-10-04 19:21 ` [PATCH 04/14] drm/msm: Add priv->mm_lock to protect active/inactive lists Rob Clark
2020-10-04 19:21   ` Rob Clark
2020-10-04 22:15   ` Daniel Vetter
2020-10-04 22:15     ` Daniel Vetter
2020-10-05  0:10     ` Rob Clark
2020-10-05  0:10       ` Rob Clark
2020-10-05 14:19   ` [Freedreno] " Jordan Crouse
2020-10-05 14:19     ` Jordan Crouse
2020-10-04 19:21 ` [PATCH 05/14] drm/msm: Document and rename preempt_lock Rob Clark
2020-10-04 19:21   ` Rob Clark
2020-10-05 14:22   ` Jordan Crouse
2020-10-05 14:22     ` Jordan Crouse
2020-10-04 19:21 ` [PATCH 06/14] drm/msm: Protect ring->submits with it's own lock Rob Clark
2020-10-04 19:21   ` Rob Clark
2020-10-05 14:23   ` [Freedreno] " Jordan Crouse
2020-10-05 14:23     ` Jordan Crouse
2020-10-04 19:21 ` [PATCH 07/14] drm/msm: Refcount submits Rob Clark
2020-10-04 19:21   ` Rob Clark
2020-10-05 13:56   ` Daniel Vetter
2020-10-05 13:56     ` Daniel Vetter
2020-10-05 16:24     ` Rob Clark
2020-10-05 16:24       ` Rob Clark
2020-10-05 14:27   ` [Freedreno] " Jordan Crouse
2020-10-05 14:27     ` Jordan Crouse
2020-10-04 19:21 ` [PATCH 08/14] drm/msm: Remove obj->gpu Rob Clark
2020-10-04 19:21   ` Rob Clark
2020-10-05 14:28   ` [Freedreno] " Jordan Crouse
2020-10-05 14:28     ` Jordan Crouse
2020-10-04 19:21 ` [PATCH 09/14] drm/msm: Drop struct_mutex from the retire path Rob Clark
2020-10-04 19:21   ` Rob Clark
2020-10-05 14:29   ` [Freedreno] " Jordan Crouse
2020-10-05 14:29     ` Jordan Crouse
2020-10-04 19:21 ` [PATCH 10/14] drm/msm: Drop struct_mutex in free_object() path Rob Clark
2020-10-04 19:21   ` Rob Clark
2020-10-04 19:21 ` [PATCH 11/14] drm/msm: remove msm_gem_free_work Rob Clark
2020-10-04 19:21   ` Rob Clark
2020-10-04 19:21 ` [PATCH 12/14] drm/msm: drop struct_mutex in madvise path Rob Clark
2020-10-04 19:21   ` Rob Clark
2020-10-04 19:21 ` [PATCH 13/14] drm/msm: Drop struct_mutex in shrinker path Rob Clark
2020-10-04 19:21   ` Rob Clark
2020-10-05  9:24   ` Hillf Danton
2020-10-05 14:02     ` Daniel Vetter
2020-10-05 14:02       ` Daniel Vetter
2020-10-05 16:17       ` Kristian Høgsberg
2020-10-05 16:17         ` Kristian Høgsberg
2020-10-06  0:44         ` Hillf Danton
2020-10-06  3:40           ` Rob Clark
2020-10-06  3:40             ` Rob Clark
2020-10-06  8:24             ` Hillf Danton
2020-10-06  9:35             ` Daniel Vetter
2020-10-06  9:35               ` Daniel Vetter
2020-10-05 16:49       ` Rob Clark
2020-10-05 16:49         ` Rob Clark
2020-10-05 18:18         ` Daniel Vetter
2020-10-05 18:18           ` Daniel Vetter
2020-10-04 19:21 ` [PATCH 14/14] drm/msm: Don't implicit-sync if only a single ring Rob Clark
2020-10-04 19:21   ` Rob Clark
2020-10-05 16:24 ` [Freedreno] [PATCH 00/14] drm/msm: de-struct_mutex-ification Kristian Høgsberg
2020-10-05 16:24   ` Kristian Høgsberg
2020-10-05 18:20   ` Daniel Vetter
2020-10-05 18:20     ` Daniel Vetter
2020-10-06  3:25     ` Rob Clark
2020-10-06  3:25       ` Rob Clark

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.