[PATCH] drm/amdgpu: Remove useless timeout error message

Zhang, Hawking Hawking.Zhang at amd.com
Mon Jun 30 07:12:31 UTC 2025


[AMD Official Use Only - AMD Internal Distribution Only]

Reviewed-by: Hawking Zhang <Hawking.Zhang at amd.com>

Regards,
Hawking
-----Original Message-----
From: Chai, Thomas <YiPeng.Chai at amd.com>
Sent: Monday, June 30, 2025 14:54
To: amd-gfx at lists.freedesktop.org
Cc: Chai, Thomas <YiPeng.Chai at amd.com>; Zhang, Hawking <Hawking.Zhang at amd.com>; Zhou1, Tao <Tao.Zhou1 at amd.com>; Yang, Stanley <Stanley.Yang at amd.com>; Chai, Thomas <YiPeng.Chai at amd.com>
Subject: [PATCH] drm/amdgpu: Remove useless timeout error message

The timeout is only used to interrupt polling and not need to print a error message.

Signed-off-by: YiPeng Chai <YiPeng.Chai at amd.com>
---
 drivers/gpu/drm/amd/amdgpu/amdgpu_ras.c | 11 +----------
 1 file changed, 1 insertion(+), 10 deletions(-)

diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_ras.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_ras.c
index 514b56e5d8ba..daab4414df50 100644
--- a/drivers/gpu/drm/amd/amdgpu/amdgpu_ras.c
+++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_ras.c
@@ -3304,7 +3304,6 @@ static int amdgpu_ras_poison_creation_handler(struct amdgpu_device *adev,
        uint64_t de_queried_count;
        uint32_t new_detect_count, total_detect_count;
        uint32_t need_query_count = poison_creation_count;
-       bool query_data_timeout = false;
        enum ras_event_type type = RAS_EVENT_TYPE_POISON_CREATION;

        memset(&info, 0, sizeof(info));
@@ -3333,21 +3332,13 @@ static int amdgpu_ras_poison_creation_handler(struct amdgpu_device *adev,
                                timeout = MAX_UMC_POISON_POLLING_TIME_ASYNC;

                        if (timeout) {
-                               if (!--timeout) {
-                                       query_data_timeout = true;
+                               if (!--timeout)
                                        break;
-                               }
                                msleep(1);
                        }
                }
        } while (total_detect_count < need_query_count);

-       if (query_data_timeout) {
-               dev_warn(adev->dev, "Can't find deferred error! count: %u\n",
-                       (need_query_count - total_detect_count));
-               return -ENOENT;
-       }
-
        if (total_detect_count)
                schedule_delayed_work(&ras->page_retirement_dwork, 0);

--
2.34.1



More information about the amd-gfx mailing list