fdo-mirrors/mesa
1
0
Fork 0
mirror of https://gitlab.freedesktop.org/mesa/mesa.git synced 2025-12-22 02:40:11 +01:00
Code Issues Projects Releases Packages Wiki Activity Actions 1

pan/mdg: Identify barrier out-of-order field

Browse source 
All barrier types are implied regardless, this is an optimization for
out-of-order.

Signed-off-by: Alyssa Rosenzweig <alyssa.rosenzweig@collabora.com>
Reviewed-by: Tomeu Vizoso <tomeu.vizoso@collabora.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/6321>
This commit is contained in:
Alyssa Rosenzweig 2020-08-13 17:14:58 -04:00 • committed by Marge Bot
parent af97331aba
commit 14e774929e
3 changed files with 6 additions and 28 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

14
src/panfrost/midgard/disassemble.c
View file

@ -1391,18 +1391,8 @@ print_texture_barrier(FILE *fp, uint32_t *word)
if (barrier->zero5)
fprintf(fp, "/* zero4 = 0x%" PRIx64 " */ ", barrier->zero5);
/* Control barriers are always implied, so include for obviousness */
fprintf(fp, " control");
if (barrier->buffer)
fprintf(fp, " | buffer");
if (barrier->shared)
fprintf(fp, " | shared");
if (barrier->stack)
fprintf(fp, " | stack");
if (barrier->out_of_order)
fprintf(fp, ".ooo%u", barrier->out_of_order);
fprintf(fp, "\n");
}

15
src/panfrost/midgard/midgard.h
View file

@ -657,9 +657,6 @@ enum mali_sampler_type {
MALI_SAMPLER_SIGNED = 0x3, /* isampler */
};
#define MIDGARD_BARRIER_BUFFER (1 << 0)
#define MIDGARD_BARRIER_SHARED (1 << 1)
typedef struct
__attribute__((__packed__))
{
@ -722,10 +719,8 @@ __attribute__((__packed__))
unsigned swizzle : 8;
/* For barriers, control barriers are implied regardless, but these
* bits also enable memory barriers of various types. For regular
* textures, these indicate how many bundles after this texture op may
* be executed in parallel with this op. We may execute only ALU and
/* These indicate how many bundles after this texture op may be
* executed in parallel with this op. We may execute only ALU and
* ld/st in parallel (not other textures), and obviously there cannot
* be any dependency (the blob appears to forbid even accessing other
* channels of a given texture register). */
@ -801,10 +796,8 @@ __attribute__((__packed__))
unsigned zero2 : 14;
unsigned zero3 : 24;
unsigned buffer : 1;
unsigned shared : 1;
unsigned stack : 1;
unsigned zero4 : 5;
unsigned out_of_order : 4;
unsigned zero4 : 4;
uint64_t zero5;
} midgard_texture_barrier_word;

5
src/panfrost/midgard/midgard_compile.c
View file

@ -1570,11 +1570,6 @@ emit_control_barrier(compiler_context *ctx)
.dest = ~0,
.src = { ~0, ~0, ~0, ~0 },
.op = TEXTURE_OP_BARRIER,
.texture = {
/* TODO: optimize */
.out_of_order = MIDGARD_BARRIER_BUFFER |
MIDGARD_BARRIER_SHARED ,
}
};
emit_mir_instruction(ctx, ins);