Re: [PATCH] drm/amdgpu:guarantee 128dws between vm flush and IB(v3)

2017-01-20 Thread Christian König
, Monk Sent: Thursday, January 19, 2017 10:39 PM To: 'Christian König' <deathsim...@vodafone.de>; amd-gfx@lists.freedesktop.org Subject: RE: [PATCH] drm/amdgpu:guarantee 128dws between vm flush and IB(v3) Christian I just found one issue in your previous patch ( the one you remove inserting

Re: [PATCH] drm/amdgpu:guarantee 128dws between vm flush and IB(v3)

2017-01-19 Thread Grazvydas Ignotas
On Thu, Jan 19, 2017 at 4:32 PM, Christian König wrote: > Am 19.01.2017 um 14:51 schrieb Grazvydas Ignotas: >> >> On Thu, Jan 19, 2017 at 11:10 AM, Christian König >> wrote: >>> >>> Am 18.01.2017 um 12:42 schrieb Monk Liu: @@ -6743,6

Re: [PATCH] drm/amdgpu:guarantee 128dws between vm flush and IB(v3)

2017-01-19 Thread Christian König
Am 19.01.2017 um 14:51 schrieb Grazvydas Ignotas: On Thu, Jan 19, 2017 at 11:10 AM, Christian König wrote: Am 18.01.2017 um 12:42 schrieb Monk Liu: @@ -6743,6 +6741,15 @@ static void gfx_v8_ring_emit_cntxcntl(struct amdgpu_ring *ring, uint32_t flags) if

RE: [PATCH] drm/amdgpu:guarantee 128dws between vm flush and IB(v3)

2017-01-19 Thread Liu, Monk
, January 19, 2017 10:39 PM To: 'Christian König' <deathsim...@vodafone.de>; amd-gfx@lists.freedesktop.org Subject: RE: [PATCH] drm/amdgpu:guarantee 128dws between vm flush and IB(v3) Christian I just found one issue in your previous patch ( the one you remove inserting 128nop before v

RE: [PATCH] drm/amdgpu:guarantee 128dws between vm flush and IB(v3)

2017-01-19 Thread Liu, Monk
mit BR Monk -Original Message- From: amd-gfx [mailto:amd-gfx-boun...@lists.freedesktop.org] On Behalf Of Christian K?nig Sent: Thursday, January 19, 2017 5:11 PM To: Liu, Monk <monk@amd.com>; amd-gfx@lists.freedesktop.org Subject: Re: [PATCH] drm/amdgpu:guarantee 128dws between

Re: [PATCH] drm/amdgpu:guarantee 128dws between vm flush and IB(v3)

2017-01-19 Thread Grazvydas Ignotas
On Thu, Jan 19, 2017 at 11:10 AM, Christian König wrote: > Am 18.01.2017 um 12:42 schrieb Monk Liu: >> @@ -6743,6 +6741,15 @@ static void gfx_v8_ring_emit_cntxcntl(struct >> amdgpu_ring *ring, uint32_t flags) >> if (amdgpu_sriov_vf(ring->adev)) >>

Re: [PATCH] drm/amdgpu:guarantee 128dws between vm flush and IB(v3)

2017-01-19 Thread Christian König
Am 18.01.2017 um 12:42 schrieb Monk Liu: previously we always insert 128nops behind vm_flush, which may lead to DAMframe size above 256 dw and automatially aligned to 512 dw. now we calculate how many DWs already inserted after vm_flush and make up for the reset to pad up to 128dws before