mirror of
https://gitlab.freedesktop.org/mesa/mesa.git
synced 2025-12-23 04:40:09 +01:00
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:
parent
8735e96c53
commit
e639e311a1
2 changed files with 64 additions and 25 deletions
|
|
@ -47,6 +47,10 @@
|
|||
#include "frontend/sw_winsys.h"
|
||||
#include "nir/nir_to_tgsi_info.h"
|
||||
#include "nir_serialize.h"
|
||||
|
||||
/** Fragment shader number (for debugging) */
|
||||
static unsigned cs_no = 0;
|
||||
|
||||
struct lp_cs_job_info {
|
||||
unsigned grid_size[3];
|
||||
unsigned block_size[3];
|
||||
|
|
@ -166,8 +170,8 @@ generate_compute(struct llvmpipe_context *lp,
|
|||
builder = gallivm->builder;
|
||||
assert(builder);
|
||||
LLVMPositionBuilderAtEnd(builder, block);
|
||||
sampler = lp_llvm_sampler_soa_create(key->state);
|
||||
image = lp_llvm_image_soa_create(key->image_state);
|
||||
sampler = lp_llvm_sampler_soa_create(key->samplers);
|
||||
image = lp_llvm_image_soa_create(lp_cs_variant_key_images(key));
|
||||
|
||||
struct lp_build_loop_state loop_state[4];
|
||||
LLVMValueRef num_x_loop;
|
||||
|
|
@ -421,10 +425,13 @@ llvmpipe_create_compute_state(struct pipe_context *pipe,
|
|||
{
|
||||
struct lp_compute_shader *shader;
|
||||
int nr_samplers, nr_sampler_views;
|
||||
|
||||
shader = CALLOC_STRUCT(lp_compute_shader);
|
||||
if (!shader)
|
||||
return NULL;
|
||||
|
||||
shader->no = cs_no++;
|
||||
|
||||
shader->base.type = templ->ir_type;
|
||||
if (templ->ir_type == PIPE_SHADER_IR_NIR_SERIALIZED) {
|
||||
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_sampler_views = shader->info.base.file_max[TGSI_FILE_SAMPLER_VIEW] + 1;
|
||||
shader->variant_key_size = Offset(struct lp_compute_shader_variant_key,
|
||||
state[MAX2(nr_samplers, nr_sampler_views)]);
|
||||
int nr_images = shader->info.base.file_max[TGSI_FILE_IMAGE] + 1;
|
||||
shader->variant_key_size = lp_cs_variant_key_size(MAX2(nr_samplers, nr_sampler_views), nr_images);
|
||||
|
||||
return shader;
|
||||
}
|
||||
|
||||
|
|
@ -529,22 +537,26 @@ llvmpipe_delete_compute_state(struct pipe_context *pipe,
|
|||
FREE(shader);
|
||||
}
|
||||
|
||||
static void
|
||||
static struct lp_compute_shader_variant_key *
|
||||
make_variant_key(struct llvmpipe_context *lp,
|
||||
struct lp_compute_shader *shader,
|
||||
struct lp_compute_shader_variant_key *key)
|
||||
char *store)
|
||||
{
|
||||
int i;
|
||||
|
||||
memset(key, 0, shader->variant_key_size);
|
||||
struct lp_compute_shader_variant_key *key;
|
||||
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:
|
||||
*/
|
||||
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) {
|
||||
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]);
|
||||
}
|
||||
}
|
||||
|
|
@ -563,7 +575,7 @@ make_variant_key(struct llvmpipe_context *lp,
|
|||
* used views may be included in the shader key.
|
||||
*/
|
||||
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]);
|
||||
}
|
||||
}
|
||||
|
|
@ -572,19 +584,22 @@ make_variant_key(struct llvmpipe_context *lp,
|
|||
key->nr_sampler_views = key->nr_samplers;
|
||||
for(i = 0; i < key->nr_sampler_views; ++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]);
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
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;
|
||||
for (i = 0; i < key->nr_images; ++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]);
|
||||
}
|
||||
}
|
||||
return key;
|
||||
}
|
||||
|
||||
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);
|
||||
|
||||
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(" .wrap = %s %s %s\n",
|
||||
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);
|
||||
}
|
||||
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(" .format = %s\n",
|
||||
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_depth);
|
||||
}
|
||||
struct lp_image_static_state *images = lp_cs_variant_key_images(key);
|
||||
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(" .format = %s\n",
|
||||
util_format_name(image->format));
|
||||
|
|
@ -664,11 +680,11 @@ generate_variant(struct llvmpipe_context *lp,
|
|||
{
|
||||
struct lp_compute_shader_variant *variant;
|
||||
char module_name[64];
|
||||
|
||||
variant = CALLOC_STRUCT(lp_compute_shader_variant);
|
||||
variant = MALLOC(sizeof *variant + shader->variant_key_size - sizeof variant->key);
|
||||
if (!variant)
|
||||
return NULL;
|
||||
|
||||
memset(variant, 0, sizeof(*variant));
|
||||
snprintf(module_name, sizeof(module_name), "cs%u_variant%u",
|
||||
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_variant_key key;
|
||||
struct lp_compute_shader_variant_key *key;
|
||||
struct lp_compute_shader_variant *variant = NULL;
|
||||
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 */
|
||||
li = first_elem(&shader->variants);
|
||||
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;
|
||||
break;
|
||||
}
|
||||
|
|
@ -786,7 +803,7 @@ llvmpipe_update_cs(struct llvmpipe_context *lp)
|
|||
* Generate the new variant.
|
||||
*/
|
||||
t0 = os_time_get();
|
||||
variant = generate_variant(lp, shader, &key);
|
||||
variant = generate_variant(lp, shader, key);
|
||||
t1 = os_time_get();
|
||||
dt = t1 - t0;
|
||||
LP_COUNT_ADD(llvm_compile_time, dt);
|
||||
|
|
|
|||
|
|
@ -42,10 +42,31 @@ struct lp_compute_shader_variant_key
|
|||
unsigned nr_samplers:8;
|
||||
unsigned nr_sampler_views:8;
|
||||
unsigned nr_images:8;
|
||||
struct lp_image_static_state image_state[PIPE_MAX_SHADER_IMAGES];
|
||||
struct lp_sampler_static_state state[PIPE_MAX_SHADER_SAMPLER_VIEWS];
|
||||
/* followed by variable number of images */
|
||||
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_compute_shader_variant *base;
|
||||
|
|
@ -54,8 +75,6 @@ struct lp_cs_variant_list_item
|
|||
|
||||
struct lp_compute_shader_variant
|
||||
{
|
||||
struct lp_compute_shader_variant_key key;
|
||||
|
||||
struct gallivm_state *gallivm;
|
||||
|
||||
LLVMTypeRef jit_cs_context_ptr_type;
|
||||
|
|
@ -73,6 +92,9 @@ struct lp_compute_shader_variant
|
|||
|
||||
/* For debugging/profiling purposes */
|
||||
unsigned no;
|
||||
|
||||
/* key is variable-sized, must be last */
|
||||
struct lp_compute_shader_variant_key key;
|
||||
};
|
||||
|
||||
struct lp_compute_shader {
|
||||
|
|
|
|||
Loading…
Add table
Reference in a new issue