Applied. thanks! On Wed, Feb 20, 2019 at 3:05 AM YueHaibing <yuehaibing@xxxxxxxxxx> wrote: > > Fixes gcc '-Wunused-but-set-variable' warning: > > drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd_gpuvm.c: In function 'update_gpuvm_pte': > drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd_gpuvm.c:840:20: warning: > variable 'bo' set but not used [-Wunused-but-set-variable] > > drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd_gpuvm.c:838:20: warning: > variable 'vm' set but not used [-Wunused-but-set-variable] > > They're never used since introduction in a46a2cd103a8 ("drm/amdgpu: Add GPUVM > memory management functions for KFD") > > Signed-off-by: YueHaibing <yuehaibing@xxxxxxxxxx> > --- > drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd_gpuvm.c | 8 +------- > 1 file changed, 1 insertion(+), 7 deletions(-) > > diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd_gpuvm.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd_gpuvm.c > index d7b10d79f1de..63daf758cd03 100644 > --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd_gpuvm.c > +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd_gpuvm.c > @@ -837,13 +837,7 @@ static int update_gpuvm_pte(struct amdgpu_device *adev, > struct amdgpu_sync *sync) > { > int ret; > - struct amdgpu_vm *vm; > - struct amdgpu_bo_va *bo_va; > - struct amdgpu_bo *bo; > - > - bo_va = entry->bo_va; > - vm = bo_va->base.vm; > - bo = bo_va->base.bo; > + struct amdgpu_bo_va *bo_va = entry->bo_va; > > /* Update the page tables */ > ret = amdgpu_vm_bo_update(adev, bo_va, false); > > > > > > _______________________________________________ > dri-devel mailing list > dri-devel@xxxxxxxxxxxxxxxxxxxxx > https://lists.freedesktop.org/mailman/listinfo/dri-devel