Commit 5aacf891 authored by Frank Min's avatar Frank Min Committed by Alex Deucher
Browse files

drm/amdgpu: change non-dcc buffer copy configuration 



Without setting cpv bit and 7th ib dw, non-dcc buffer copy will have
random corruption

So set the cpv bit and clear the 7th ib dw for copy non-dcc buffers

Signed-off-by: default avatarFrank Min <Frank.Min@amd.com>
Acked-by: default avatarChristian König <christian.koenig@amd.com>
Reviewed-by: default avatarHawking Zhang <Hawking.Zhang@amd.com>
Signed-off-by: default avatarAlex Deucher <alexander.deucher@amd.com>
parent b3a3c9a6

drivers/gpu/drm/amd/amdgpu/sdma_v7_0.c

+3 −2
Original line number Diff line number Diff line
@@ -1688,8 +1688,7 @@ static void sdma_v7_0_emit_copy_buffer(struct amdgpu_ib *ib, 
	ib->ptr[ib->length_dw++] = SDMA_PKT_COPY_LINEAR_HEADER_OP(SDMA_OP_COPY) |

		SDMA_PKT_COPY_LINEAR_HEADER_SUB_OP(SDMA_SUBOP_COPY_LINEAR) |

		SDMA_PKT_COPY_LINEAR_HEADER_TMZ((copy_flags & AMDGPU_COPY_FLAGS_TMZ) ? 1 : 0) |

		SDMA_PKT_COPY_LINEAR_HEADER_CPV((copy_flags &

			(AMDGPU_COPY_FLAGS_READ_DECOMPRESSED | AMDGPU_COPY_FLAGS_WRITE_COMPRESSED)) ? 1 : 0);

		SDMA_PKT_COPY_LINEAR_HEADER_CPV(1);



	ib->ptr[ib->length_dw++] = byte_count - 1;

	ib->ptr[ib->length_dw++] = 0; /* src/dst endian swap */
@@ -1703,6 +1702,8 @@ static void sdma_v7_0_emit_copy_buffer(struct amdgpu_ib *ib, 
			((copy_flags & AMDGPU_COPY_FLAGS_READ_DECOMPRESSED) ? SDMA_DCC_READ_CM(2) : 0) |

			((copy_flags & AMDGPU_COPY_FLAGS_WRITE_COMPRESSED) ? SDMA_DCC_WRITE_CM(1) : 0) |

			SDMA_DCC_MAX_COM(max_com) | SDMA_DCC_MAX_UCOM(1);

	else

		ib->ptr[ib->length_dw++] = 0;

}



/**