[PATCH 0/6] move more drivers to gem_free_object_unlocked

Daniel Vetter daniel.vetter at ffwll.ch
Thu Mar 22 10:51:27 UTC 2018


Hi all,

Random drive-by crusade against dev->struct_mutex usage. The only two
leftover drivers still relying on gem_free_object and the magic lifetime
rule that hodling dev->struct_mutex will prevent gem object destruction
are msm and gma500. Both actually need this, and don't look like they can
be fixed with an easy patch.

But the less this pattern is in the tree the more we can hopefully avoid
copypasta-spreading it more.

Cheers, Daniel

Daniel Vetter (6):
  drm/tinydrm: Use gem_free_object_unlocked
  staging/vboxvideo: Use gem_free_object_unlocked
  drm/rockchip: fixup comment for gem_free_object_unlocked
  drm/udl: Get rid of dev->struct_mutex usage
  drm/omapdrm: Fix mm_list locking
  drm/omapdrm: Switch to gem_free_object_unlocked

 drivers/gpu/drm/omapdrm/omap_debugfs.c      |  2 ++
 drivers/gpu/drm/omapdrm/omap_drv.c          |  4 ++--
 drivers/gpu/drm/omapdrm/omap_drv.h          |  2 +-
 drivers/gpu/drm/omapdrm/omap_gem.c          | 11 +++++++----
 drivers/gpu/drm/rockchip/rockchip_drm_gem.c |  4 ++--
 drivers/gpu/drm/tinydrm/core/tinydrm-core.c |  2 +-
 drivers/gpu/drm/udl/udl_dmabuf.c            |  5 +++--
 drivers/gpu/drm/udl/udl_drv.c               |  2 +-
 drivers/gpu/drm/udl/udl_drv.h               |  2 ++
 drivers/gpu/drm/udl/udl_gem.c               |  5 +++--
 drivers/gpu/drm/udl/udl_main.c              |  2 ++
 drivers/staging/vboxvideo/vbox_drv.c        |  2 +-
 include/drm/tinydrm/tinydrm.h               |  2 +-
 13 files changed, 28 insertions(+), 17 deletions(-)

-- 
2.16.2



More information about the dri-devel mailing list