radv: use a define for the transition point between cp and compute shader
For certain buffer meta ops we can use the CP or a compute shader, we should use a define to rather than hardcoding 4096, allows for easier testing and more consistency. Reviewed-by: Samuel Pitoiset <samuel.pitoiset@gmail.com> Reviewed-by: Bas Nieuwenhuizen <bas@basnieuwenhuizen.nl> Signed-off-by: Dave Airlie <airlied@redhat.com>
This commit is contained in:
parent
b704538b00
commit
3cd3035ace
|
@ -4,6 +4,12 @@
|
|||
#include "sid.h"
|
||||
#include "radv_cs.h"
|
||||
|
||||
/*
|
||||
* This is the point we switch from using CP to compute shader
|
||||
* for certain buffer operations.
|
||||
*/
|
||||
#define RADV_BUFFER_OPS_CS_THRESHOLD 4096
|
||||
|
||||
static nir_shader *
|
||||
build_buffer_fill_shader(struct radv_device *dev)
|
||||
{
|
||||
|
@ -405,7 +411,7 @@ void radv_fill_buffer(struct radv_cmd_buffer *cmd_buffer,
|
|||
assert(!(offset & 3));
|
||||
assert(!(size & 3));
|
||||
|
||||
if (size >= 4096)
|
||||
if (size >= RADV_BUFFER_OPS_CS_THRESHOLD)
|
||||
fill_buffer_shader(cmd_buffer, bo, offset, size, value);
|
||||
else if (size) {
|
||||
uint64_t va = radv_buffer_get_va(bo);
|
||||
|
@ -422,7 +428,7 @@ void radv_copy_buffer(struct radv_cmd_buffer *cmd_buffer,
|
|||
uint64_t src_offset, uint64_t dst_offset,
|
||||
uint64_t size)
|
||||
{
|
||||
if (size >= 4096 && !(size & 3) && !(src_offset & 3) && !(dst_offset & 3))
|
||||
if (size >= RADV_BUFFER_OPS_CS_THRESHOLD && !(size & 3) && !(src_offset & 3) && !(dst_offset & 3))
|
||||
copy_buffer_shader(cmd_buffer, src_bo, dst_bo,
|
||||
src_offset, dst_offset, size);
|
||||
else if (size) {
|
||||
|
@ -496,7 +502,7 @@ void radv_CmdUpdateBuffer(
|
|||
if (!dataSize)
|
||||
return;
|
||||
|
||||
if (dataSize < 4096) {
|
||||
if (dataSize < RADV_BUFFER_OPS_CS_THRESHOLD) {
|
||||
si_emit_cache_flush(cmd_buffer);
|
||||
|
||||
cmd_buffer->device->ws->cs_add_buffer(cmd_buffer->cs, dst_buffer->bo, 8);
|
||||
|
|
Loading…
Reference in New Issue