diff options
| author | Samuel Pitoiset <[email protected]> | 2018-06-19 15:24:39 +0200 |
|---|---|---|
| committer | Samuel Pitoiset <[email protected]> | 2018-06-20 10:08:37 +0200 |
| commit | 916dda5cf7abdcc0f95293758942951b34af7a6d (patch) | |
| tree | 3c78d9d593d7a6e79b5b684bce98627e0c6830f1 /src | |
| parent | 4705a5dfda38f40e4d4827058738490d164f198c (diff) | |
radv: remove unnecessary code around CACHE_FLUSH_AND_INV_TS_EVENT
AMDVLK also always uses CACHE_FLUSH_AND_INV_TS_EVENT. The other
workaround is to flush DB metadata after emitting the framebuffer,
but that seems slower.
Signed-off-by: Samuel Pitoiset <[email protected]>
Reviewed-by: Bas Nieuwenhuizen <[email protected]>
Diffstat (limited to 'src')
| -rw-r--r-- | src/amd/vulkan/si_cmd_buffer.c | 19 |
1 files changed, 1 insertions, 18 deletions
diff --git a/src/amd/vulkan/si_cmd_buffer.c b/src/amd/vulkan/si_cmd_buffer.c index 0692124bf51..a663d2add6d 100644 --- a/src/amd/vulkan/si_cmd_buffer.c +++ b/src/amd/vulkan/si_cmd_buffer.c @@ -834,26 +834,9 @@ si_cs_emit_cache_flush(struct radeon_cmdbuf *cs, if (chip_class >= GFX9 && flush_cb_db) { unsigned cb_db_event, tc_flags; -#if 0 - /* This breaks a bunch of: - dEQP-VK.renderpass.dedicated_allocation.formats.d32_sfloat_s8_uint.input*. - use the big hammer always. - */ /* Set the CB/DB flush event. */ - switch (flush_cb_db) { - case RADV_CMD_FLAG_FLUSH_AND_INV_CB: - cb_db_event = V_028A90_FLUSH_AND_INV_CB_DATA_TS; - break; - case RADV_CMD_FLAG_FLUSH_AND_INV_DB: - cb_db_event = V_028A90_FLUSH_AND_INV_DB_DATA_TS; - break; - default: - /* both CB & DB */ - cb_db_event = V_028A90_CACHE_FLUSH_AND_INV_TS_EVENT; - } -#else cb_db_event = V_028A90_CACHE_FLUSH_AND_INV_TS_EVENT; -#endif + /* These are the only allowed combinations. If you need to * do multiple operations at once, do them separately. * All operations that invalidate L2 also seem to invalidate |