On Tue, Mar 24, 2026 at 3:51 AM Sunil Khatri <[email protected]> wrote: > > In function amdgpu_userq_restore_worker we dont need to use > goto as we already in the end of function and it will exit > naturally. > > Signed-off-by: Sunil Khatri <[email protected]>
Reviewed-by: Alex Deucher <[email protected]> > --- > drivers/gpu/drm/amd/amdgpu/amdgpu_userq.c | 4 +--- > 1 file changed, 1 insertion(+), 3 deletions(-) > > diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_userq.c > b/drivers/gpu/drm/amd/amdgpu/amdgpu_userq.c > index eb07ed078bfa..a351383b432b 100644 > --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_userq.c > +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_userq.c > @@ -1227,10 +1227,8 @@ static void amdgpu_userq_restore_worker(struct > work_struct *work) > } > > ret = amdgpu_userq_restore_all(uq_mgr); > - if (ret) { > + if (ret) > drm_file_err(uq_mgr->file, "Failed to restore all queues\n"); > - goto unlock; > - } > > unlock: > mutex_unlock(&uq_mgr->userq_mutex); > -- > 2.34.1 >
