Properly check the return values for function, as done elsewhere.

Signed-off-by: Sunday Clement <[email protected]>
---
 drivers/gpu/drm/amd/amdkfd/kfd_device_queue_manager.c | 8 +++++---
 1 file changed, 5 insertions(+), 3 deletions(-)

diff --git a/drivers/gpu/drm/amd/amdkfd/kfd_device_queue_manager.c 
b/drivers/gpu/drm/amd/amdkfd/kfd_device_queue_manager.c
index 6e7bc983fc0b..3b696c58c530 100644
--- a/drivers/gpu/drm/amd/amdkfd/kfd_device_queue_manager.c
+++ b/drivers/gpu/drm/amd/amdkfd/kfd_device_queue_manager.c
@@ -1897,6 +1897,8 @@ static int start_cpsch(struct device_queue_manager *dqm)
 
 static int stop_cpsch(struct device_queue_manager *dqm)
 {
+       int ret = 0;
+
        dqm_lock(dqm);
        if (!dqm->sched_running) {
                dqm_unlock(dqm);
@@ -1904,9 +1906,9 @@ static int stop_cpsch(struct device_queue_manager *dqm)
        }
 
        if (!dqm->dev->kfd->shared_resources.enable_mes)
-               unmap_queues_cpsch(dqm, KFD_UNMAP_QUEUES_FILTER_ALL_QUEUES, 0, 
USE_DEFAULT_GRACE_PERIOD, false);
+               ret = unmap_queues_cpsch(dqm, 
KFD_UNMAP_QUEUES_FILTER_ALL_QUEUES, 0, USE_DEFAULT_GRACE_PERIOD, false);
        else
-               remove_all_kfd_queues_mes(dqm);
+               ret = remove_all_kfd_queues_mes(dqm);
 
        dqm->sched_running = false;
 
@@ -1920,7 +1922,7 @@ static int stop_cpsch(struct device_queue_manager *dqm)
        dqm->detect_hang_info = NULL;
        dqm_unlock(dqm);
 
-       return 0;
+       return ret;
 }
 
 static int create_kernel_queue_cpsch(struct device_queue_manager *dqm,
-- 
2.43.0

Reply via email to