On 8/26/26 19:58, Sunil Khatri wrote: > rptr is a confusing variable name as it is a fence > value written by the cp in the fence address and no > where related to the rptr. > > Signed-off-by: Sunil Khatri <[email protected]>
Reviewed-by: Christian König <[email protected]> for the series. > --- > drivers/gpu/drm/amd/amdgpu/amdgpu_userq_fence.c | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) > > diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_userq_fence.c > b/drivers/gpu/drm/amd/amdgpu/amdgpu_userq_fence.c > index 09e2079cfdbd..e862b65f5366 100644 > --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_userq_fence.c > +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_userq_fence.c > @@ -148,13 +148,13 @@ amdgpu_userq_fence_driver_process(struct > amdgpu_userq_fence_driver *fence_drv) > LIST_HEAD(to_be_signaled); > struct dma_fence *fence; > unsigned long flags; > - u64 rptr; > + u64 fence_val; > > spin_lock_irqsave(&fence_drv->fence_list_lock, flags); > - rptr = amdgpu_userq_fence_read(fence_drv); > + fence_val = amdgpu_userq_fence_read(fence_drv); > > list_for_each_entry(userq_fence, &fence_drv->fences, link) { > - if (rptr < userq_fence->base.seqno) > + if (fence_val < userq_fence->base.seqno) > break; > } >
