[<prev] [next>] [thread-next>] [day] [month] [year] [list]
Message-ID: <20240523113236.432585-1-adrian.larumbe@collabora.com>
Date: Thu, 23 May 2024 12:32:16 +0100
From: Adrián Larumbe <adrian.larumbe@...labora.com>
To: Boris Brezillon <boris.brezillon@...labora.com>,
Rob Herring <robh@...nel.org>,
Steven Price <steven.price@....com>,
Maarten Lankhorst <maarten.lankhorst@...ux.intel.com>,
Maxime Ripard <mripard@...nel.org>,
Thomas Zimmermann <tzimmermann@...e.de>,
David Airlie <airlied@...il.com>,
Daniel Vetter <daniel@...ll.ch>,
Sumit Semwal <sumit.semwal@...aro.org>,
Christian König <christian.koenig@....com>,
Dmitry Osipenko <dmitry.osipenko@...labora.com>,
Zack Rusin <zack.rusin@...adcom.com>
Cc: kernel@...labora.com,
Adrián Larumbe <adrian.larumbe@...labora.com>,
dri-devel@...ts.freedesktop.org,
linux-kernel@...r.kernel.org,
linux-media@...r.kernel.org,
linaro-mm-sig@...ts.linaro.org
Subject: [PATCH v4 0/3] drm: Fix dma_resv deadlock at drm object pin time
This is v4 of https://lore.kernel.org/lkml/20240521181817.097af5e1@collabora.com/T/
The goal of this patch series is fixing a deadlock upon locking the dma reservation
of a DRM gem object when pinning it, at a prime import operation.
Changelog:
v3:
- Split driver fixes into separate commits for Panfrost and Lima
- Make drivers call drm_gem_shmem_pin_locked instead of drm_gem_shmem_object_pin
- Improved commit message for first patch to explain why dma resv locking in the
pin callback is no longer necessary.
v2:
- Removed comment explaining reason why an already-locked
pin function replaced the locked variant inside Panfrost's
object pin callback.
- Moved already-assigned attachment warning into generic
already-locked gem object pin function
Adrián Larumbe (3):
drm/panfrost: Fix dma_resv deadlock at drm object pin time
drm/lima: Fix dma_resv deadlock at drm object pin time
drm/gem-shmem: Add import attachment warning to locked pin function
drivers/gpu/drm/drm_gem_shmem_helper.c | 2 ++
drivers/gpu/drm/lima/lima_gem.c | 2 +-
drivers/gpu/drm/panfrost/panfrost_gem.c | 2 +-
3 files changed, 4 insertions(+), 2 deletions(-)
base-commit: 7acacca1b157fcb258cfd781603425f73bc7370b
--
2.45.1
Powered by blists - more mailing lists