On 5/13/26 12:47, David Rosca wrote: > Use correct u64 type. > > Signed-off-by: David Rosca <[email protected]>
Good catch, Reviewed-by: Christian König <[email protected]> Vitaly and Sunil do we have an IGT test case for that? If not we should probably create one. Thanks, Christian. > --- > drivers/gpu/drm/amd/amdgpu/amdgpu_userq_fence.c | 9 +++++---- > 1 file changed, 5 insertions(+), 4 deletions(-) > > diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_userq_fence.c > b/drivers/gpu/drm/amd/amdgpu/amdgpu_userq_fence.c > index 008330a0d852..c0d68863fa17 100644 > --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_userq_fence.c > +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_userq_fence.c > @@ -593,7 +593,7 @@ int amdgpu_userq_signal_ioctl(struct drm_device *dev, > void *data, > static int > amdgpu_userq_wait_count_fences(struct drm_file *filp, > struct drm_amdgpu_userq_wait *wait_info, > - u32 *syncobj_handles, u32 *timeline_points, > + u32 *syncobj_handles, u64 *timeline_points, > u32 *timeline_handles, > struct drm_gem_object **gobj_write, > struct drm_gem_object **gobj_read) > @@ -703,7 +703,7 @@ amdgpu_userq_wait_add_fence(struct drm_amdgpu_userq_wait > *wait_info, > static int > amdgpu_userq_wait_return_fence_info(struct drm_file *filp, > struct drm_amdgpu_userq_wait *wait_info, > - u32 *syncobj_handles, u32 *timeline_points, > + u32 *syncobj_handles, u64 *timeline_points, > u32 *timeline_handles, > struct drm_gem_object **gobj_write, > struct drm_gem_object **gobj_read) > @@ -906,7 +906,8 @@ int amdgpu_userq_wait_ioctl(struct drm_device *dev, void > *data, > struct drm_file *filp) > { > int num_points, num_syncobj, num_read_bo_handles, num_write_bo_handles; > - u32 *syncobj_handles, *timeline_points, *timeline_handles; > + u32 *syncobj_handles, *timeline_handles; > + u64 *timeline_points; > struct drm_amdgpu_userq_wait *wait_info = data; > struct drm_gem_object **gobj_write; > struct drm_gem_object **gobj_read; > @@ -935,7 +936,7 @@ int amdgpu_userq_wait_ioctl(struct drm_device *dev, void > *data, > } > > ptr = u64_to_user_ptr(wait_info->syncobj_timeline_points); > - timeline_points = memdup_array_user(ptr, num_points, sizeof(u32)); > + timeline_points = memdup_array_user(ptr, num_points, sizeof(u64)); > if (IS_ERR(timeline_points)) { > r = PTR_ERR(timeline_points); > goto free_timeline_handles;
