Skip to content

drm/msm: fix bind error handling

Johan Hovold dri-devel linux-arm-msm lkml stable 1 version Applied

Status from this message: Applied, thanks!

Get series with b4
b4 am [email protected]

Versions newest first

  1. v1 Latest10 patches 17 repliesApplied

Patches in v1 Applied

#Subject and filesSizeRepliesApplied
1/10Revert "drm/msm: Add missing check and destroy for alloc_ordered_workqueue"drivers/gpu/drm/msm/msm_drv.c+0 -21 reply
2/10Revert "drm/msm: Fix failure paths in msm_drm_init()"drivers/gpu/drm/msm/disp/msm_disp_snapshot.c drivers/gpu/drm/msm/msm_drv.c+4 -101 reply
3/10drm/msm: fix NULL-deref on snapshot tear downdrivers/gpu/drm/msm/msm_drv.c+2 -11 reply
4/10drm/msm: fix NULL-deref on irq uninstalldrivers/gpu/drm/msm/msm_drv.c+5 -31 reply
5/10drm/msm: fix drm device leak on bind errorsdrivers/gpu/drm/msm/msm_drv.c+8 -23 replies
6/10drm/msm: fix vram leak on bind errorsdrivers/gpu/drm/msm/msm_drv.c+19 -71 reply
7/10drm/msm: fix missing wq allocation error handlingdrivers/gpu/drm/msm/msm_drv.c+4 -01 reply
8/10drm/msm: fix workqueue leak on bind errorsdrivers/gpu/drm/msm/msm_drv.c+4 -11 reply
9/10drm/msm: use drmm_mode_config_init()drivers/gpu/drm/msm/msm_drv.c+5 -62 replies
10/10drm/msm: move include directivedrivers/gpu/drm/msm/msm_drv.c+1 -21 reply

Reviews trailers per patch

9 of 10 patches have a Reviewed-by or Acked-by.

PatchDmitry Baryshkov
1/10R-b
2/10R-b
3/10R-b
4/10R-b
5/10R-b
6/10R-b
7/10R-b
8/10R-b
9/10
10/10R-b

Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes.

Read the v1 thread

Cover letter [PATCH 00/10] drm/msm: fix bind error handling

I had reasons to look closer at the MSM DRM driver error handling and realised that it had suffered from a fair amount of bit rot over the years.

Unfortunately, I started fixing this in my 6.2 branch and failed to notice two partial and, as it turned out, broken attempts to address this that are now in 6.3-rc1.

Instead of trying to salvage this incrementally, I'm reverting the two broken commits so that clean and backportable fixes can be added in their place.

Included are also two related cleanups.

Johan

Johan Hovold (10):
Revert "drm/msm: Add missing check and destroy for
alloc_ordered_workqueue"
Revert "drm/msm: Fix failure paths in msm_drm_init()" drm/msm: fix NULL-deref on snapshot tear down drm/msm: fix NULL-deref on irq uninstall drm/msm: fix drm device leak on bind errors drm/msm: fix vram leak on bind errors drm/msm: fix missing wq allocation error handling drm/msm: fix workqueue leak on bind errors drm/msm: use drmm_mode_config_init() drm/msm: move include directive

 drivers/gpu/drm/msm/disp/msm_disp_snapshot.c |  3 -
 drivers/gpu/drm/msm/msm_drv.c                | 67 +++++++++++++-------
 2 files changed, 44 insertions(+), 26 deletions(-)
Signature · 1 line
2.39.2