Re: [PATCH v5 01/40] drm/gpuvm: Don't require obj lock in destructor path

On Tue, May 20, 2025 at 08:45:24AM -0700, Rob Clark wrote: > On Tue, May 20, 2025 at 8:21 AM Danilo Krummrich wrote: > > > > On Tue, May 20, 2025 at 07:57:36AM -0700, Rob Clark wrote: > > > On Tue, May 20, 2025 at 12:23 AM Danilo Krummrich wrote: > > > > On Mon, May 19, 2025 at 10:51:24AM -0700,

Re: [PATCH v5 01/40] drm/gpuvm: Don't require obj lock in destructor path

On Tue, May 20, 2025 at 8:21 AM Danilo Krummrich wrote: > > On Tue, May 20, 2025 at 07:57:36AM -0700, Rob Clark wrote: > > On Tue, May 20, 2025 at 12:23 AM Danilo Krummrich wrote: > > > On Mon, May 19, 2025 at 10:51:24AM -0700, Rob Clark wrote: > > > > diff --git a/drivers/gpu/drm/drm_gpuvm.c b/d

Re: [PATCH v5 01/40] drm/gpuvm: Don't require obj lock in destructor path

On Tue, May 20, 2025 at 07:57:36AM -0700, Rob Clark wrote: > On Tue, May 20, 2025 at 12:23 AM Danilo Krummrich wrote: > > On Mon, May 19, 2025 at 10:51:24AM -0700, Rob Clark wrote: > > > diff --git a/drivers/gpu/drm/drm_gpuvm.c b/drivers/gpu/drm/drm_gpuvm.c > > > index f9eb56f24bef..1e89a98caad4 1

Re: [PATCH v5 01/40] drm/gpuvm: Don't require obj lock in destructor path

On Tue, May 20, 2025 at 12:23 AM Danilo Krummrich wrote: > > On Mon, May 19, 2025 at 10:51:24AM -0700, Rob Clark wrote: > > From: Rob Clark > > > > See commit a414fe3a2129 ("drm/msm/gem: Drop obj lock in > > msm_gem_free_object()") for justification. > > I asked for a proper commit message in v4.

Re: [PATCH v5 01/40] drm/gpuvm: Don't require obj lock in destructor path

On Mon, May 19, 2025 at 10:51:24AM -0700, Rob Clark wrote: > From: Rob Clark > > See commit a414fe3a2129 ("drm/msm/gem: Drop obj lock in > msm_gem_free_object()") for justification. I asked for a proper commit message in v4. Only referring to a driver commit and let the people figure out how th

[PATCH v5 01/40] drm/gpuvm: Don't require obj lock in destructor path

From: Rob Clark See commit a414fe3a2129 ("drm/msm/gem: Drop obj lock in msm_gem_free_object()") for justification. Cc: Danilo Krummrich Signed-off-by: Rob Clark --- drivers/gpu/drm/drm_gpuvm.c | 7 +-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/drivers/gpu/drm/drm_gpuvm