> I would rather avoid calling the function in the first place. I already did it in patch 08, and you also rejected this patch
So I'll consider patch 08 is still valid, and drop this one -----Original Message----- From: Christian König [mailto:ckoenig.leichtzumer...@gmail.com] Sent: 2018年2月26日 18:09 To: Liu, Monk <monk....@amd.com>; amd-gfx@lists.freedesktop.org Subject: Re: [PATCH 10/22] drm/amdgpu: no need to count emitted for SRIOV I would rather avoid calling the function in the first place. Christian. Am 26.02.2018 um 06:18 schrieb Monk Liu: > Change-Id: I370966acd0f1925a99dfde888678e6e0fd093b15 > Signed-off-by: Monk Liu <monk....@amd.com> > --- > drivers/gpu/drm/amd/amdgpu/amdgpu_uvd.c | 4 +++- > 1 file changed, 3 insertions(+), 1 deletion(-) > > diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_uvd.c > b/drivers/gpu/drm/amd/amdgpu/amdgpu_uvd.c > index 337db57..5fb4357 100644 > --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_uvd.c > +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_uvd.c > @@ -1117,11 +1117,13 @@ static void amdgpu_uvd_idle_work_handler(struct > work_struct *work) > { > struct amdgpu_device *adev = > container_of(work, struct amdgpu_device, uvd.idle_work.work); > - unsigned fences = amdgpu_fence_count_emitted(&adev->uvd.ring); > + unsigned fences; > > if (amdgpu_sriov_vf(adev)) > BUG(); > > + fences = amdgpu_fence_count_emitted(&adev->uvd.ring); > + > if (fences == 0) { > if (adev->pm.dpm_enabled) { > amdgpu_dpm_enable_uvd(adev, false); _______________________________________________ amd-gfx mailing list amd-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/amd-gfx
