Mesa (main): radeonsi: rename RADEON_FLAG_UNCACHED -> RADEON_FLAG_GL2_BYPASS
GitLab Mirror
gitlab-mirror at kemper.freedesktop.org
Mon May 16 11:32:13 UTC 2022
Module: Mesa
Branch: main
Commit: ceddd7d49a6b2fe4ae2d6c9599ace8beac64b1ed
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=ceddd7d49a6b2fe4ae2d6c9599ace8beac64b1ed
Author: Marek Olšák <marek.olsak at amd.com>
Date: Tue May 10 17:00:07 2022 -0400
radeonsi: rename RADEON_FLAG_UNCACHED -> RADEON_FLAG_GL2_BYPASS
Reviewed-by: Pierre-Eric Pelloux-Prayer <pierre-eric.pelloux-prayer at amd.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/16466>
---
src/gallium/drivers/radeonsi/si_buffer.c | 8 ++--
src/gallium/drivers/radeonsi/si_pipe.h | 2 +-
src/gallium/drivers/radeonsi/si_test_dma_perf.c | 4 +-
src/gallium/drivers/radeonsi/si_texture.c | 2 +-
src/gallium/include/winsys/radeon_winsys.h | 58 ++++++++++++-------------
src/gallium/winsys/amdgpu/drm/amdgpu_bo.c | 2 +-
6 files changed, 38 insertions(+), 38 deletions(-)
diff --git a/src/gallium/drivers/radeonsi/si_buffer.c b/src/gallium/drivers/radeonsi/si_buffer.c
index 682967aa345..46ba6144782 100644
--- a/src/gallium/drivers/radeonsi/si_buffer.c
+++ b/src/gallium/drivers/radeonsi/si_buffer.c
@@ -141,11 +141,11 @@ void si_init_resource_fields(struct si_screen *sscreen, struct si_resource *res,
/* For higher throughput and lower latency over PCIe assuming sequential access.
* Only CP DMA and optimized compute benefit from this.
- * GFX8 and older don't support RADEON_FLAG_UNCACHED.
+ * GFX8 and older don't support RADEON_FLAG_GL2_BYPASS.
*/
if (sscreen->info.gfx_level >= GFX9 &&
- res->b.b.flags & SI_RESOURCE_FLAG_UNCACHED)
- res->flags |= RADEON_FLAG_UNCACHED;
+ res->b.b.flags & SI_RESOURCE_FLAG_GL2_BYPASS)
+ res->flags |= RADEON_FLAG_GL2_BYPASS;
/* Set expected VRAM and GART usage for the buffer. */
res->memory_usage_kb = MAX2(1, size / 1024);
@@ -446,7 +446,7 @@ static void *si_buffer_transfer_map(struct pipe_context *ctx, struct pipe_resour
assert(!(usage & (TC_TRANSFER_MAP_THREADED_UNSYNC | PIPE_MAP_THREAD_SAFE)));
staging = si_aligned_buffer_create(ctx->screen,
- SI_RESOURCE_FLAG_UNCACHED | SI_RESOURCE_FLAG_DRIVER_INTERNAL,
+ SI_RESOURCE_FLAG_GL2_BYPASS | SI_RESOURCE_FLAG_DRIVER_INTERNAL,
PIPE_USAGE_STAGING,
box->width + (box->x % SI_MAP_BUFFER_ALIGNMENT), 256);
if (staging) {
diff --git a/src/gallium/drivers/radeonsi/si_pipe.h b/src/gallium/drivers/radeonsi/si_pipe.h
index bf107bfaffa..645cbb57089 100644
--- a/src/gallium/drivers/radeonsi/si_pipe.h
+++ b/src/gallium/drivers/radeonsi/si_pipe.h
@@ -145,7 +145,7 @@ extern "C" {
(((x)&0x3) << SI_RESOURCE_FLAG_MICRO_TILE_MODE_SHIFT)
#define SI_RESOURCE_FLAG_MICRO_TILE_MODE_GET(x) \
(((x) >> SI_RESOURCE_FLAG_MICRO_TILE_MODE_SHIFT) & 0x3)
-#define SI_RESOURCE_FLAG_UNCACHED (PIPE_RESOURCE_FLAG_DRV_PRIV << 12)
+#define SI_RESOURCE_FLAG_GL2_BYPASS (PIPE_RESOURCE_FLAG_DRV_PRIV << 12)
enum si_has_gs {
GS_OFF,
diff --git a/src/gallium/drivers/radeonsi/si_test_dma_perf.c b/src/gallium/drivers/radeonsi/si_test_dma_perf.c
index 9bb07c7ec62..e3d220b9033 100644
--- a/src/gallium/drivers/radeonsi/si_test_dma_perf.c
+++ b/src/gallium/drivers/radeonsi/si_test_dma_perf.c
@@ -117,7 +117,7 @@ void si_test_dma_perf(struct si_screen *sscreen)
continue;
}
- /* SI_RESOURCE_FLAG_UNCACHED setting RADEON_FLAG_UNCACHED doesn't affect
+ /* SI_RESOURCE_FLAG_GL2_BYPASS setting RADEON_FLAG_GL2_BYPASS doesn't affect
* chips before gfx9.
*/
if (test_cs && cache_policy && sctx->gfx_level < GFX9)
@@ -153,7 +153,7 @@ void si_test_dma_perf(struct si_screen *sscreen)
enum pipe_resource_usage dst_usage, src_usage;
struct pipe_resource *dst, *src;
unsigned query_type = PIPE_QUERY_TIME_ELAPSED;
- unsigned flags = cache_policy == L2_BYPASS ? SI_RESOURCE_FLAG_UNCACHED : 0;
+ unsigned flags = cache_policy == L2_BYPASS ? SI_RESOURCE_FLAG_GL2_BYPASS : 0;
if (placement == 0 || placement == 2 || placement == 4)
dst_usage = PIPE_USAGE_DEFAULT;
diff --git a/src/gallium/drivers/radeonsi/si_texture.c b/src/gallium/drivers/radeonsi/si_texture.c
index d3bf3ad96fa..a736221ea40 100644
--- a/src/gallium/drivers/radeonsi/si_texture.c
+++ b/src/gallium/drivers/radeonsi/si_texture.c
@@ -1025,7 +1025,7 @@ static struct si_texture *si_texture_create_object(struct pipe_screen *screen,
if (base->flags & PIPE_RESOURCE_FLAG_SPARSE)
resource->b.b.flags |= PIPE_RESOURCE_FLAG_UNMAPPABLE;
if (base->bind & PIPE_BIND_PRIME_BLIT_DST)
- resource->b.b.flags |= SI_RESOURCE_FLAG_UNCACHED;
+ resource->b.b.flags |= SI_RESOURCE_FLAG_GL2_BYPASS;
/* Create the backing buffer. */
si_init_resource_fields(sscreen, resource, alloc_size, alignment);
diff --git a/src/gallium/include/winsys/radeon_winsys.h b/src/gallium/include/winsys/radeon_winsys.h
index 06652e35e63..78ee0b49d34 100644
--- a/src/gallium/include/winsys/radeon_winsys.h
+++ b/src/gallium/include/winsys/radeon_winsys.h
@@ -74,7 +74,7 @@ enum radeon_bo_flag
RADEON_FLAG_READ_ONLY = (1 << 5),
RADEON_FLAG_32BIT = (1 << 6),
RADEON_FLAG_ENCRYPTED = (1 << 7),
- RADEON_FLAG_UNCACHED = (1 << 8), /* only gfx9 and newer */
+ RADEON_FLAG_GL2_BYPASS = (1 << 8), /* only gfx9 and newer */
RADEON_FLAG_DRIVER_INTERNAL = (1 << 9),
};
@@ -720,11 +720,11 @@ enum radeon_heap
RADEON_HEAP_GTT_WC_READ_ONLY_32BIT,
RADEON_HEAP_GTT_WC_32BIT,
RADEON_HEAP_GTT,
- RADEON_HEAP_GTT_UNCACHED_WC,
- RADEON_HEAP_GTT_UNCACHED_WC_READ_ONLY,
- RADEON_HEAP_GTT_UNCACHED_WC_READ_ONLY_32BIT,
- RADEON_HEAP_GTT_UNCACHED_WC_32BIT,
- RADEON_HEAP_GTT_UNCACHED,
+ RADEON_HEAP_GTT_GL2_BYPASS_WC,
+ RADEON_HEAP_GTT_GL2_BYPASS_WC_READ_ONLY,
+ RADEON_HEAP_GTT_GL2_BYPASS_WC_READ_ONLY_32BIT,
+ RADEON_HEAP_GTT_GL2_BYPASS_WC_32BIT,
+ RADEON_HEAP_GTT_GL2_BYPASS,
RADEON_MAX_SLAB_HEAPS,
RADEON_MAX_CACHED_HEAPS = RADEON_MAX_SLAB_HEAPS,
};
@@ -743,11 +743,11 @@ static inline enum radeon_bo_domain radeon_domain_from_heap(enum radeon_heap hea
case RADEON_HEAP_GTT_WC_READ_ONLY_32BIT:
case RADEON_HEAP_GTT_WC_32BIT:
case RADEON_HEAP_GTT:
- case RADEON_HEAP_GTT_UNCACHED_WC:
- case RADEON_HEAP_GTT_UNCACHED_WC_READ_ONLY:
- case RADEON_HEAP_GTT_UNCACHED_WC_READ_ONLY_32BIT:
- case RADEON_HEAP_GTT_UNCACHED_WC_32BIT:
- case RADEON_HEAP_GTT_UNCACHED:
+ case RADEON_HEAP_GTT_GL2_BYPASS_WC:
+ case RADEON_HEAP_GTT_GL2_BYPASS_WC_READ_ONLY:
+ case RADEON_HEAP_GTT_GL2_BYPASS_WC_READ_ONLY_32BIT:
+ case RADEON_HEAP_GTT_GL2_BYPASS_WC_32BIT:
+ case RADEON_HEAP_GTT_GL2_BYPASS:
return RADEON_DOMAIN_GTT;
default:
assert(0);
@@ -761,19 +761,19 @@ static inline unsigned radeon_flags_from_heap(enum radeon_heap heap)
switch (heap) {
case RADEON_HEAP_GTT:
- case RADEON_HEAP_GTT_UNCACHED:
+ case RADEON_HEAP_GTT_GL2_BYPASS:
break;
default:
flags |= RADEON_FLAG_GTT_WC;
}
switch (heap) {
- case RADEON_HEAP_GTT_UNCACHED_WC:
- case RADEON_HEAP_GTT_UNCACHED_WC_READ_ONLY:
- case RADEON_HEAP_GTT_UNCACHED_WC_READ_ONLY_32BIT:
- case RADEON_HEAP_GTT_UNCACHED_WC_32BIT:
- case RADEON_HEAP_GTT_UNCACHED:
- flags |= RADEON_FLAG_UNCACHED;
+ case RADEON_HEAP_GTT_GL2_BYPASS_WC:
+ case RADEON_HEAP_GTT_GL2_BYPASS_WC_READ_ONLY:
+ case RADEON_HEAP_GTT_GL2_BYPASS_WC_READ_ONLY_32BIT:
+ case RADEON_HEAP_GTT_GL2_BYPASS_WC_32BIT:
+ case RADEON_HEAP_GTT_GL2_BYPASS:
+ flags |= RADEON_FLAG_GL2_BYPASS;
break;
default:
break;
@@ -784,8 +784,8 @@ static inline unsigned radeon_flags_from_heap(enum radeon_heap heap)
case RADEON_HEAP_VRAM_READ_ONLY_32BIT:
case RADEON_HEAP_GTT_WC_READ_ONLY:
case RADEON_HEAP_GTT_WC_READ_ONLY_32BIT:
- case RADEON_HEAP_GTT_UNCACHED_WC_READ_ONLY:
- case RADEON_HEAP_GTT_UNCACHED_WC_READ_ONLY_32BIT:
+ case RADEON_HEAP_GTT_GL2_BYPASS_WC_READ_ONLY:
+ case RADEON_HEAP_GTT_GL2_BYPASS_WC_READ_ONLY_32BIT:
flags |= RADEON_FLAG_READ_ONLY;
break;
default:
@@ -797,8 +797,8 @@ static inline unsigned radeon_flags_from_heap(enum radeon_heap heap)
case RADEON_HEAP_VRAM_32BIT:
case RADEON_HEAP_GTT_WC_READ_ONLY_32BIT:
case RADEON_HEAP_GTT_WC_32BIT:
- case RADEON_HEAP_GTT_UNCACHED_WC_READ_ONLY_32BIT:
- case RADEON_HEAP_GTT_UNCACHED_WC_32BIT:
+ case RADEON_HEAP_GTT_GL2_BYPASS_WC_READ_ONLY_32BIT:
+ case RADEON_HEAP_GTT_GL2_BYPASS_WC_32BIT:
flags |= RADEON_FLAG_32BIT;
FALLTHROUGH;
default:
@@ -831,7 +831,7 @@ static inline int radeon_get_heap_index(enum radeon_bo_domain domain, enum radeo
return -1;
/* Unsupported flags: NO_SUBALLOC, SPARSE. */
- if (flags & ~(RADEON_FLAG_GTT_WC | RADEON_FLAG_NO_CPU_ACCESS | RADEON_FLAG_UNCACHED |
+ if (flags & ~(RADEON_FLAG_GTT_WC | RADEON_FLAG_NO_CPU_ACCESS | RADEON_FLAG_GL2_BYPASS |
RADEON_FLAG_NO_INTERPROCESS_SHARING | RADEON_FLAG_READ_ONLY | RADEON_FLAG_32BIT |
RADEON_FLAG_DRIVER_INTERNAL))
return -1;
@@ -859,20 +859,20 @@ static inline int radeon_get_heap_index(enum radeon_bo_domain domain, enum radeo
}
break;
case RADEON_DOMAIN_GTT:
- uncached = flags & RADEON_FLAG_UNCACHED;
+ uncached = flags & RADEON_FLAG_GL2_BYPASS;
switch (flags & (RADEON_FLAG_GTT_WC | RADEON_FLAG_READ_ONLY | RADEON_FLAG_32BIT)) {
case RADEON_FLAG_GTT_WC | RADEON_FLAG_READ_ONLY | RADEON_FLAG_32BIT:
- return uncached ? RADEON_HEAP_GTT_UNCACHED_WC_READ_ONLY_32BIT
+ return uncached ? RADEON_HEAP_GTT_GL2_BYPASS_WC_READ_ONLY_32BIT
: RADEON_HEAP_GTT_WC_READ_ONLY_32BIT;
case RADEON_FLAG_GTT_WC | RADEON_FLAG_READ_ONLY:
- return uncached ? RADEON_HEAP_GTT_UNCACHED_WC_READ_ONLY
+ return uncached ? RADEON_HEAP_GTT_GL2_BYPASS_WC_READ_ONLY
: RADEON_HEAP_GTT_WC_READ_ONLY;
case RADEON_FLAG_GTT_WC | RADEON_FLAG_32BIT:
- return uncached ? RADEON_HEAP_GTT_UNCACHED_WC_32BIT
+ return uncached ? RADEON_HEAP_GTT_GL2_BYPASS_WC_32BIT
: RADEON_HEAP_GTT_WC_32BIT;
case RADEON_FLAG_GTT_WC:
- return uncached ? RADEON_HEAP_GTT_UNCACHED_WC : RADEON_HEAP_GTT_WC;
+ return uncached ? RADEON_HEAP_GTT_GL2_BYPASS_WC : RADEON_HEAP_GTT_WC;
case RADEON_FLAG_READ_ONLY | RADEON_FLAG_32BIT:
case RADEON_FLAG_READ_ONLY:
assert(!"READ_ONLY without WC is disallowed");
@@ -881,7 +881,7 @@ static inline int radeon_get_heap_index(enum radeon_bo_domain domain, enum radeo
assert(!"32BIT without WC is disallowed");
return -1;
case 0:
- return uncached ? RADEON_HEAP_GTT_UNCACHED : RADEON_HEAP_GTT;
+ return uncached ? RADEON_HEAP_GTT_GL2_BYPASS : RADEON_HEAP_GTT;
}
break;
default:
diff --git a/src/gallium/winsys/amdgpu/drm/amdgpu_bo.c b/src/gallium/winsys/amdgpu/drm/amdgpu_bo.c
index 955d95124e2..54e32d1be4a 100644
--- a/src/gallium/winsys/amdgpu/drm/amdgpu_bo.c
+++ b/src/gallium/winsys/amdgpu/drm/amdgpu_bo.c
@@ -574,7 +574,7 @@ static struct amdgpu_winsys_bo *amdgpu_create_bo(struct amdgpu_winsys *ws,
if (!(flags & RADEON_FLAG_READ_ONLY))
vm_flags |= AMDGPU_VM_PAGE_WRITEABLE;
- if (flags & RADEON_FLAG_UNCACHED)
+ if (flags & RADEON_FLAG_GL2_BYPASS)
vm_flags |= AMDGPU_VM_MTYPE_UC;
r = amdgpu_bo_va_op_raw(ws->dev, buf_handle, 0, size, va, vm_flags,
More information about the mesa-commit
mailing list