From 07ea447609cabd487133d00532400a1aabb044bf Mon Sep 17 00:00:00 2001 From: Tvrtko Ursulin Date: Tue, 10 Mar 2026 15:47:25 +0000 Subject: [PATCH] plugins/amdgpu: Do not eat the errno in kmtIoctl Error logging for just a single error condition will eat the errno so lets just remove it. All callers use pr_perror() and rely on the helper not munching it up so no information is lost by removing the internal log. Signed-off-by: Tvrtko Ursulin Reviewed-By: David Francis --- plugins/amdgpu/amdgpu_plugin.c | 5 ----- 1 file changed, 5 deletions(-) diff --git a/plugins/amdgpu/amdgpu_plugin.c b/plugins/amdgpu/amdgpu_plugin.c index 6633b8d71..51a1f694f 100644 --- a/plugins/amdgpu/amdgpu_plugin.c +++ b/plugins/amdgpu/amdgpu_plugin.c @@ -102,11 +102,6 @@ int kmtIoctl(int fd, unsigned long request, void *arg) ret = ioctl(fd, request, arg); } while (ret == -1 && max_retries-- > 0 && (errno == EINTR || errno == EAGAIN)); - if (ret == -1 && errno == EBADF) - /* In case pthread_atfork didn't catch it, this will - * make any subsequent hsaKmt calls fail in CHECK_KFD_OPEN. - */ - pr_perror("KFD file descriptor not valid in this process"); return ret; }