]> git.ipfire.org Git - thirdparty/kernel/linux.git/commitdiff
accel/amdxdna: Fix uninitialized return value
authorLizhi Hou <lizhi.hou@amd.com>
Fri, 24 Oct 2025 16:55:03 +0000 (09:55 -0700)
committerLizhi Hou <lizhi.hou@amd.com>
Fri, 24 Oct 2025 20:06:06 +0000 (13:06 -0700)
In aie2_get_hwctx_status() and aie2_query_ctx_status_array(), the
functions could return an uninitialized value in some cases. Update them
to always return 0. The amount of valid results is indicated by the
returned buffer_size, element_size, and num_element fields.

Fixes: 2f509fe6a42c ("accel/amdxdna: Add ioctl DRM_IOCTL_AMDXDNA_GET_ARRAY")
Reviewed-by: Mario Limonciello (AMD) <superm1@kernel.org>
Signed-off-by: Lizhi Hou <lizhi.hou@amd.com>
Link: https://patch.msgid.link/20251024165503.1548131-1-lizhi.hou@amd.com
drivers/accel/amdxdna/aie2_pci.c

index f48045318dc02406bde118703611583731081ba7..f1a8112b080ffe269c17beffe4dae6d574e22c6a 100644 (file)
@@ -822,7 +822,7 @@ static int aie2_get_hwctx_status(struct amdxdna_client *client,
        }
 
        args->buffer_size -= (u32)(array_args.buffer - args->buffer);
-       return ret;
+       return 0;
 }
 
 static int aie2_get_info(struct amdxdna_client *client, struct amdxdna_drm_get_info *args)
@@ -904,7 +904,7 @@ static int aie2_query_ctx_status_array(struct amdxdna_client *client,
        args->num_element = (u32)((array_args.buffer - args->buffer) /
                                  args->element_size);
 
-       return ret;
+       return 0;
 }
 
 static int aie2_get_array(struct amdxdna_client *client,