[PATCH 0/3] drm/vc4: use drm managed resources

Danilo Krummrich posted 3 patches 3 years, 9 months ago
drivers/gpu/drm/vc4/vc4_crtc.c  |  2 +-
drivers/gpu/drm/vc4/vc4_dpi.c   | 11 ++++++-----
drivers/gpu/drm/vc4/vc4_dsi.c   | 10 +++++-----
drivers/gpu/drm/vc4/vc4_hdmi.c  | 10 ++++++----
drivers/gpu/drm/vc4/vc4_plane.c | 19 +++++++------------
drivers/gpu/drm/vc4/vc4_txp.c   |  2 +-
drivers/gpu/drm/vc4/vc4_vec.c   | 11 ++++++-----
7 files changed, 32 insertions(+), 33 deletions(-)
[PATCH 0/3] drm/vc4: use drm managed resources
Posted by Danilo Krummrich 3 years, 9 months ago
This patch series converts DRM modeset object allocations from devm_*()
to drmm_*() memory allocators, or their corresponding convenience
wrappers, respectively, in order to tie the release action to the
underlaying struct drm_device.

This can prevent potential use-after free issues on driver unload or
EPROBE_DEFERRED backoff.

Danilo Krummrich (3):
  drm/vc4: plane: use drm managed resources
  drm/vc4: crtc: use drm managed resources
  drm/vc4: encoder: use drm managed resources

 drivers/gpu/drm/vc4/vc4_crtc.c  |  2 +-
 drivers/gpu/drm/vc4/vc4_dpi.c   | 11 ++++++-----
 drivers/gpu/drm/vc4/vc4_dsi.c   | 10 +++++-----
 drivers/gpu/drm/vc4/vc4_hdmi.c  | 10 ++++++----
 drivers/gpu/drm/vc4/vc4_plane.c | 19 +++++++------------
 drivers/gpu/drm/vc4/vc4_txp.c   |  2 +-
 drivers/gpu/drm/vc4/vc4_vec.c   | 11 ++++++-----
 7 files changed, 32 insertions(+), 33 deletions(-)

-- 
2.36.1
Re: [PATCH 0/3] drm/vc4: use drm managed resources
Posted by Maxime Ripard 3 years, 9 months ago
Hi Danilo,

On Wed, Jul 13, 2022 at 10:54:57AM +0200, Danilo Krummrich wrote:
> This patch series converts DRM modeset object allocations from devm_*()
> to drmm_*() memory allocators, or their corresponding convenience
> wrappers, respectively, in order to tie the release action to the
> underlaying struct drm_device.
> 
> This can prevent potential use-after free issues on driver unload or
> EPROBE_DEFERRED backoff.

Yeah, the driver had a lot of this kind of issues.

As it turns out, at the moment you sent it, I was applying a larger
series (hopefully) addressing all of them:
https://lore.kernel.org/all/20220711173939.1132294-1-maxime@cerno.tech/

Maxime
Re: [PATCH 0/3] drm/vc4: use drm managed resources
Posted by Danilo Krummrich 3 years, 9 months ago
Hi Maxime,

On 7/13/22 11:10, Maxime Ripard wrote:
> Hi Danilo,
> 
> On Wed, Jul 13, 2022 at 10:54:57AM +0200, Danilo Krummrich wrote:
>> This patch series converts DRM modeset object allocations from devm_*()
>> to drmm_*() memory allocators, or their corresponding convenience
>> wrappers, respectively, in order to tie the release action to the
>> underlaying struct drm_device.
>>
>> This can prevent potential use-after free issues on driver unload or
>> EPROBE_DEFERRED backoff.
> 
> Yeah, the driver had a lot of this kind of issues.
> 
> As it turns out, at the moment you sent it, I was applying a larger
> series (hopefully) addressing all of them:
> https://lore.kernel.org/all/20220711173939.1132294-1-maxime@cerno.tech/
Ah, great! That's covering even more than the series I sent.
> 
> Maxime

- Danilo