diff options
author | Christian König <christian.koenig@amd.com> | 2016-10-05 06:59:20 -0400 |
---|---|---|
committer | Alex Deucher <alexander.deucher@amd.com> | 2016-10-25 14:38:35 -0400 |
commit | e08c90a7746041d0816e32614fd161c2194c7ded (patch) | |
tree | ed354ed157e8b01b19b364ca0c06c01a93a1c0b3 /drivers/gpu/drm/amd/amdgpu | |
parent | c81b07e6bcbc3a29270b4eaf5c0696837212d468 (diff) |
drm/amdgpu: remove 128 NOP hack from vm_flush v2
With the padding raised to 256 DW that shouldn't be needed any more.
v2: reduce estimation as well
Signed-off-by: Christian König <christian.koenig@amd.com>
Reviewed-by: Alex Deucher <alexander.deucher@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
Diffstat (limited to 'drivers/gpu/drm/amd/amdgpu')
-rw-r--r-- | drivers/gpu/drm/amd/amdgpu/gfx_v8_0.c | 6 |
1 files changed, 1 insertions, 5 deletions
diff --git a/drivers/gpu/drm/amd/amdgpu/gfx_v8_0.c b/drivers/gpu/drm/amd/amdgpu/gfx_v8_0.c index 8efcc8d2ad60..bb17538ff109 100644 --- a/drivers/gpu/drm/amd/amdgpu/gfx_v8_0.c +++ b/drivers/gpu/drm/amd/amdgpu/gfx_v8_0.c | |||
@@ -6249,10 +6249,6 @@ static void gfx_v8_0_ring_emit_vm_flush(struct amdgpu_ring *ring, | |||
6249 | { | 6249 | { |
6250 | int usepfp = (ring->type == AMDGPU_RING_TYPE_GFX); | 6250 | int usepfp = (ring->type == AMDGPU_RING_TYPE_GFX); |
6251 | 6251 | ||
6252 | /* GFX8 emits 128 dw nop to prevent DE do vm_flush before CE finish CEIB */ | ||
6253 | if (usepfp) | ||
6254 | amdgpu_ring_insert_nop(ring, 128); | ||
6255 | |||
6256 | amdgpu_ring_write(ring, PACKET3(PACKET3_WRITE_DATA, 3)); | 6252 | amdgpu_ring_write(ring, PACKET3(PACKET3_WRITE_DATA, 3)); |
6257 | amdgpu_ring_write(ring, (WRITE_DATA_ENGINE_SEL(usepfp) | | 6253 | amdgpu_ring_write(ring, (WRITE_DATA_ENGINE_SEL(usepfp) | |
6258 | WRITE_DATA_DST_SEL(0)) | | 6254 | WRITE_DATA_DST_SEL(0)) | |
@@ -6381,7 +6377,7 @@ static unsigned gfx_v8_0_ring_get_dma_frame_size_gfx(struct amdgpu_ring *ring) | |||
6381 | 5 + /* gfx_v8_0_ring_emit_hdp_invalidate */ | 6377 | 5 + /* gfx_v8_0_ring_emit_hdp_invalidate */ |
6382 | 6 + 6 + 6 +/* gfx_v8_0_ring_emit_fence_gfx x3 for user fence, vm fence */ | 6378 | 6 + 6 + 6 +/* gfx_v8_0_ring_emit_fence_gfx x3 for user fence, vm fence */ |
6383 | 7 + /* gfx_v8_0_ring_emit_pipeline_sync */ | 6379 | 7 + /* gfx_v8_0_ring_emit_pipeline_sync */ |
6384 | 256 + 19 + /* gfx_v8_0_ring_emit_vm_flush */ | 6380 | 128 + 19 + /* gfx_v8_0_ring_emit_vm_flush */ |
6385 | 2 + /* gfx_v8_ring_emit_sb */ | 6381 | 2 + /* gfx_v8_ring_emit_sb */ |
6386 | 3; /* gfx_v8_ring_emit_cntxcntl */ | 6382 | 3; /* gfx_v8_ring_emit_cntxcntl */ |
6387 | } | 6383 | } |