]> git.ipfire.org Git - thirdparty/kernel/stable.git/commitdiff
drm/amdgpu: Drop 'fence' check in 'to_amdgpu_amdkfd_fence()'
authorSrinivasan Shanmugam <srinivasan.shanmugam@amd.com>
Wed, 27 Dec 2023 07:24:44 +0000 (12:54 +0530)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Fri, 23 Feb 2024 07:54:49 +0000 (08:54 +0100)
[ Upstream commit bf2ad4fb8adca89374b54b225d494e0b1956dbea ]

Return value of container_of(...) can't be null, so null check is not
required for 'fence'. Hence drop its NULL check.

Fixes the below:
drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd_fence.c:93 to_amdgpu_amdkfd_fence() warn: can 'fence' even be NULL?

Cc: Felix Kuehling <Felix.Kuehling@amd.com>
Cc: Christian König <christian.koenig@amd.com>
Cc: Alex Deucher <alexander.deucher@amd.com>
Signed-off-by: Srinivasan Shanmugam <srinivasan.shanmugam@amd.com>
Reviewed-by: Felix Kuehling <felix.kuehling@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
Signed-off-by: Sasha Levin <sashal@kernel.org>
drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd_fence.c

index 1d0dbff87d3f39715aeee7d4cc84d1d0bf7bf69f..fb66e888ab3f30ed5b0ee6ef0ea5ae74ef76a4b2 100644 (file)
@@ -90,7 +90,7 @@ struct amdgpu_amdkfd_fence *to_amdgpu_amdkfd_fence(struct dma_fence *f)
                return NULL;
 
        fence = container_of(f, struct amdgpu_amdkfd_fence, base);
-       if (fence && f->ops == &amdkfd_fence_ops)
+       if (f->ops == &amdkfd_fence_ops)
                return fence;
 
        return NULL;