llvmpipe/cs: overhaul cs variant key state.

This just realigns it with the fs state, and fixes some issues
where shaders weren't getting cached correctly.

Reviewed-by: Roland Scheidegger <sroland@vmware.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/5049>
This commit is contained in:
Dave Airlie 2020-05-21 13:21:51 +10:00
parent 8735e96c53
commit e639e311a1
2 changed files with 64 additions and 25 deletions

View file

@ -47,6 +47,10 @@
#include "frontend/sw_winsys.h" #include "frontend/sw_winsys.h"
#include "nir/nir_to_tgsi_info.h" #include "nir/nir_to_tgsi_info.h"
#include "nir_serialize.h" #include "nir_serialize.h"
/** Fragment shader number (for debugging) */
static unsigned cs_no = 0;
struct lp_cs_job_info { struct lp_cs_job_info {
unsigned grid_size[3]; unsigned grid_size[3];
unsigned block_size[3]; unsigned block_size[3];
@ -166,8 +170,8 @@ generate_compute(struct llvmpipe_context *lp,
builder = gallivm->builder; builder = gallivm->builder;
assert(builder); assert(builder);
LLVMPositionBuilderAtEnd(builder, block); LLVMPositionBuilderAtEnd(builder, block);
sampler = lp_llvm_sampler_soa_create(key->state); sampler = lp_llvm_sampler_soa_create(key->samplers);
image = lp_llvm_image_soa_create(key->image_state); image = lp_llvm_image_soa_create(lp_cs_variant_key_images(key));
struct lp_build_loop_state loop_state[4]; struct lp_build_loop_state loop_state[4];
LLVMValueRef num_x_loop; LLVMValueRef num_x_loop;
@ -421,10 +425,13 @@ llvmpipe_create_compute_state(struct pipe_context *pipe,
{ {
struct lp_compute_shader *shader; struct lp_compute_shader *shader;
int nr_samplers, nr_sampler_views; int nr_samplers, nr_sampler_views;
shader = CALLOC_STRUCT(lp_compute_shader); shader = CALLOC_STRUCT(lp_compute_shader);
if (!shader) if (!shader)
return NULL; return NULL;
shader->no = cs_no++;
shader->base.type = templ->ir_type; shader->base.type = templ->ir_type;
if (templ->ir_type == PIPE_SHADER_IR_NIR_SERIALIZED) { if (templ->ir_type == PIPE_SHADER_IR_NIR_SERIALIZED) {
struct blob_reader reader; struct blob_reader reader;
@ -453,8 +460,9 @@ llvmpipe_create_compute_state(struct pipe_context *pipe,
nr_samplers = shader->info.base.file_max[TGSI_FILE_SAMPLER] + 1; nr_samplers = shader->info.base.file_max[TGSI_FILE_SAMPLER] + 1;
nr_sampler_views = shader->info.base.file_max[TGSI_FILE_SAMPLER_VIEW] + 1; nr_sampler_views = shader->info.base.file_max[TGSI_FILE_SAMPLER_VIEW] + 1;
shader->variant_key_size = Offset(struct lp_compute_shader_variant_key, int nr_images = shader->info.base.file_max[TGSI_FILE_IMAGE] + 1;
state[MAX2(nr_samplers, nr_sampler_views)]); shader->variant_key_size = lp_cs_variant_key_size(MAX2(nr_samplers, nr_sampler_views), nr_images);
return shader; return shader;
} }
@ -529,22 +537,26 @@ llvmpipe_delete_compute_state(struct pipe_context *pipe,
FREE(shader); FREE(shader);
} }
static void static struct lp_compute_shader_variant_key *
make_variant_key(struct llvmpipe_context *lp, make_variant_key(struct llvmpipe_context *lp,
struct lp_compute_shader *shader, struct lp_compute_shader *shader,
struct lp_compute_shader_variant_key *key) char *store)
{ {
int i; int i;
struct lp_compute_shader_variant_key *key;
memset(key, 0, shader->variant_key_size); key = (struct lp_compute_shader_variant_key *)store;
memset(key, 0, offsetof(struct lp_compute_shader_variant_key, samplers[1]));
/* This value will be the same for all the variants of a given shader: /* This value will be the same for all the variants of a given shader:
*/ */
key->nr_samplers = shader->info.base.file_max[TGSI_FILE_SAMPLER] + 1; key->nr_samplers = shader->info.base.file_max[TGSI_FILE_SAMPLER] + 1;
struct lp_sampler_static_state *cs_sampler;
cs_sampler = key->samplers;
for(i = 0; i < key->nr_samplers; ++i) { for(i = 0; i < key->nr_samplers; ++i) {
if(shader->info.base.file_mask[TGSI_FILE_SAMPLER] & (1 << i)) { if(shader->info.base.file_mask[TGSI_FILE_SAMPLER] & (1 << i)) {
lp_sampler_static_sampler_state(&key->state[i].sampler_state, lp_sampler_static_sampler_state(&cs_sampler[i].sampler_state,
lp->samplers[PIPE_SHADER_COMPUTE][i]); lp->samplers[PIPE_SHADER_COMPUTE][i]);
} }
} }
@ -563,7 +575,7 @@ make_variant_key(struct llvmpipe_context *lp,
* used views may be included in the shader key. * used views may be included in the shader key.
*/ */
if(shader->info.base.file_mask[TGSI_FILE_SAMPLER_VIEW] & (1u << (i & 31))) { if(shader->info.base.file_mask[TGSI_FILE_SAMPLER_VIEW] & (1u << (i & 31))) {
lp_sampler_static_texture_state(&key->state[i].texture_state, lp_sampler_static_texture_state(&cs_sampler[i].texture_state,
lp->sampler_views[PIPE_SHADER_COMPUTE][i]); lp->sampler_views[PIPE_SHADER_COMPUTE][i]);
} }
} }
@ -572,19 +584,22 @@ make_variant_key(struct llvmpipe_context *lp,
key->nr_sampler_views = key->nr_samplers; key->nr_sampler_views = key->nr_samplers;
for(i = 0; i < key->nr_sampler_views; ++i) { for(i = 0; i < key->nr_sampler_views; ++i) {
if(shader->info.base.file_mask[TGSI_FILE_SAMPLER] & (1 << i)) { if(shader->info.base.file_mask[TGSI_FILE_SAMPLER] & (1 << i)) {
lp_sampler_static_texture_state(&key->state[i].texture_state, lp_sampler_static_texture_state(&cs_sampler[i].texture_state,
lp->sampler_views[PIPE_SHADER_COMPUTE][i]); lp->sampler_views[PIPE_SHADER_COMPUTE][i]);
} }
} }
} }
struct lp_image_static_state *lp_image;
lp_image = lp_cs_variant_key_images(key);
key->nr_images = shader->info.base.file_max[TGSI_FILE_IMAGE] + 1; key->nr_images = shader->info.base.file_max[TGSI_FILE_IMAGE] + 1;
for (i = 0; i < key->nr_images; ++i) { for (i = 0; i < key->nr_images; ++i) {
if (shader->info.base.file_mask[TGSI_FILE_IMAGE] & (1 << i)) { if (shader->info.base.file_mask[TGSI_FILE_IMAGE] & (1 << i)) {
lp_sampler_static_texture_state_image(&key->image_state[i].image_state, lp_sampler_static_texture_state_image(&lp_image[i].image_state,
&lp->images[PIPE_SHADER_COMPUTE][i]); &lp->images[PIPE_SHADER_COMPUTE][i]);
} }
} }
return key;
} }
static void static void
@ -594,7 +609,7 @@ dump_cs_variant_key(const struct lp_compute_shader_variant_key *key)
debug_printf("cs variant %p:\n", (void *) key); debug_printf("cs variant %p:\n", (void *) key);
for (i = 0; i < key->nr_samplers; ++i) { for (i = 0; i < key->nr_samplers; ++i) {
const struct lp_static_sampler_state *sampler = &key->state[i].sampler_state; const struct lp_static_sampler_state *sampler = &key->samplers[i].sampler_state;
debug_printf("sampler[%u] = \n", i); debug_printf("sampler[%u] = \n", i);
debug_printf(" .wrap = %s %s %s\n", debug_printf(" .wrap = %s %s %s\n",
util_str_tex_wrap(sampler->wrap_s, TRUE), util_str_tex_wrap(sampler->wrap_s, TRUE),
@ -615,7 +630,7 @@ dump_cs_variant_key(const struct lp_compute_shader_variant_key *key)
debug_printf(" .apply_max_lod = %u\n", sampler->apply_max_lod); debug_printf(" .apply_max_lod = %u\n", sampler->apply_max_lod);
} }
for (i = 0; i < key->nr_sampler_views; ++i) { for (i = 0; i < key->nr_sampler_views; ++i) {
const struct lp_static_texture_state *texture = &key->state[i].texture_state; const struct lp_static_texture_state *texture = &key->samplers[i].texture_state;
debug_printf("texture[%u] = \n", i); debug_printf("texture[%u] = \n", i);
debug_printf(" .format = %s\n", debug_printf(" .format = %s\n",
util_format_name(texture->format)); util_format_name(texture->format));
@ -628,8 +643,9 @@ dump_cs_variant_key(const struct lp_compute_shader_variant_key *key)
texture->pot_height, texture->pot_height,
texture->pot_depth); texture->pot_depth);
} }
struct lp_image_static_state *images = lp_cs_variant_key_images(key);
for (i = 0; i < key->nr_images; ++i) { for (i = 0; i < key->nr_images; ++i) {
const struct lp_static_texture_state *image = &key->image_state[i].image_state; const struct lp_static_texture_state *image = &images[i].image_state;
debug_printf("image[%u] = \n", i); debug_printf("image[%u] = \n", i);
debug_printf(" .format = %s\n", debug_printf(" .format = %s\n",
util_format_name(image->format)); util_format_name(image->format));
@ -664,11 +680,11 @@ generate_variant(struct llvmpipe_context *lp,
{ {
struct lp_compute_shader_variant *variant; struct lp_compute_shader_variant *variant;
char module_name[64]; char module_name[64];
variant = MALLOC(sizeof *variant + shader->variant_key_size - sizeof variant->key);
variant = CALLOC_STRUCT(lp_compute_shader_variant);
if (!variant) if (!variant)
return NULL; return NULL;
memset(variant, 0, sizeof(*variant));
snprintf(module_name, sizeof(module_name), "cs%u_variant%u", snprintf(module_name, sizeof(module_name), "cs%u_variant%u",
shader->no, shader->variants_created); shader->no, shader->variants_created);
@ -715,16 +731,17 @@ llvmpipe_update_cs(struct llvmpipe_context *lp)
{ {
struct lp_compute_shader *shader = lp->cs; struct lp_compute_shader *shader = lp->cs;
struct lp_compute_shader_variant_key key; struct lp_compute_shader_variant_key *key;
struct lp_compute_shader_variant *variant = NULL; struct lp_compute_shader_variant *variant = NULL;
struct lp_cs_variant_list_item *li; struct lp_cs_variant_list_item *li;
char store[LP_CS_MAX_VARIANT_KEY_SIZE];
make_variant_key(lp, shader, &key); key = make_variant_key(lp, shader, store);
/* Search the variants for one which matches the key */ /* Search the variants for one which matches the key */
li = first_elem(&shader->variants); li = first_elem(&shader->variants);
while(!at_end(&shader->variants, li)) { while(!at_end(&shader->variants, li)) {
if(memcmp(&li->base->key, &key, shader->variant_key_size) == 0) { if(memcmp(&li->base->key, key, shader->variant_key_size) == 0) {
variant = li->base; variant = li->base;
break; break;
} }
@ -786,7 +803,7 @@ llvmpipe_update_cs(struct llvmpipe_context *lp)
* Generate the new variant. * Generate the new variant.
*/ */
t0 = os_time_get(); t0 = os_time_get();
variant = generate_variant(lp, shader, &key); variant = generate_variant(lp, shader, key);
t1 = os_time_get(); t1 = os_time_get();
dt = t1 - t0; dt = t1 - t0;
LP_COUNT_ADD(llvm_compile_time, dt); LP_COUNT_ADD(llvm_compile_time, dt);

View file

@ -42,10 +42,31 @@ struct lp_compute_shader_variant_key
unsigned nr_samplers:8; unsigned nr_samplers:8;
unsigned nr_sampler_views:8; unsigned nr_sampler_views:8;
unsigned nr_images:8; unsigned nr_images:8;
struct lp_image_static_state image_state[PIPE_MAX_SHADER_IMAGES]; /* followed by variable number of images */
struct lp_sampler_static_state state[PIPE_MAX_SHADER_SAMPLER_VIEWS]; struct lp_sampler_static_state samplers[1];
}; };
#define LP_CS_MAX_VARIANT_KEY_SIZE \
(sizeof(struct lp_compute_shader_variant_key) + \
PIPE_MAX_SHADER_SAMPLER_VIEWS * sizeof(struct lp_sampler_static_state) +\
PIPE_MAX_SHADER_IMAGES * sizeof(struct lp_image_static_state))
static inline size_t
lp_cs_variant_key_size(unsigned nr_samplers, unsigned nr_images)
{
unsigned samplers = nr_samplers > 1 ? (nr_samplers - 1) : 0;
return (sizeof(struct lp_compute_shader_variant_key) +
samplers * sizeof(struct lp_sampler_static_state) +
nr_images * sizeof(struct lp_image_static_state));
}
static inline struct lp_image_static_state *
lp_cs_variant_key_images(const struct lp_compute_shader_variant_key *key)
{
return (struct lp_image_static_state *)
&key->samplers[key->nr_samplers];
}
struct lp_cs_variant_list_item struct lp_cs_variant_list_item
{ {
struct lp_compute_shader_variant *base; struct lp_compute_shader_variant *base;
@ -54,8 +75,6 @@ struct lp_cs_variant_list_item
struct lp_compute_shader_variant struct lp_compute_shader_variant
{ {
struct lp_compute_shader_variant_key key;
struct gallivm_state *gallivm; struct gallivm_state *gallivm;
LLVMTypeRef jit_cs_context_ptr_type; LLVMTypeRef jit_cs_context_ptr_type;
@ -73,6 +92,9 @@ struct lp_compute_shader_variant
/* For debugging/profiling purposes */ /* For debugging/profiling purposes */
unsigned no; unsigned no;
/* key is variable-sized, must be last */
struct lp_compute_shader_variant_key key;
}; };
struct lp_compute_shader { struct lp_compute_shader {