brl/mutter
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

progend-glsl: Fix handling of the builtin uniforms on non-GLES2

Browse Source 
The ‘builtin uniforms’ are added to the GLSL code generated for the
GLES2 driver to implement missing fixed functionality such as the
builtin point sprite size and the alpha test reference. Previously the
code that accessed these was #ifdef'd to be compiled only when GLES2
was enabled. However since 2701b93f part of this code is now always
used even for non-GLES2 drivers. The code that accessed the builtin
uniforms array was however no longer #ifdef'd which meant that it
wouldn't compile any more if GLES2 was not enabled. This was further
broken becase the GL3 driver actually should be using the alpha test
uniform because that also does not provide any fixed functionality for
alpha testing.

To fix this the builtin uniform array is now always compiled in and
the code to access it is always used. A new member has been added to
the array to mark which private feature the uniform is used to
replace. That is checked before updating the uniform so that under
GLES2 it will update both uniforms but under GL3 it will only update
the alpha test reference.

Reviewed-by: Robert Bragg <robert@linux.intel.com>

(cherry picked from commit 5469a25413883080df75a80153accf5d9124f716)
This commit is contained in:
Neil Roberts 2012-10-01 15:01:05 +01:00 committed by Robert Bragg
parent 9b59588c53
commit bf8a1b1eda
1 changed files with 25 additions and 30 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

55
cogl/driver/gl/cogl-pipeline-progend-glsl.c
View File

@ -51,10 +51,9 @@
#include "cogl-framebuffer-private.h"
#include "cogl-pipeline-progend-glsl-private.h"
#ifdef HAVE_COGL_GLES2
/* These are used to generalise updating some uniforms that are
required when building for GLES2 */
required when building for drivers missing some fixed function
state that we use */
typedef void (* UpdateUniformFunc) (CoglPipeline *pipeline,
int uniform_location,
@ -70,20 +69,24 @@ typedef struct
void *getter_func;
UpdateUniformFunc update_func;
CoglPipelineState change;
/* This builtin is only necessary if the following private feature
* is not implemented in the driver */
CoglPrivateFeatureFlags feature_replacement;
} BuiltinUniformData;
static BuiltinUniformData builtin_uniforms[] =
{
{ "cogl_point_size_in",
cogl_pipeline_get_point_size, update_float_uniform,
COGL_PIPELINE_STATE_POINT_SIZE },
COGL_PIPELINE_STATE_POINT_SIZE,
COGL_PRIVATE_FEATURE_BUILTIN_POINT_SIZE_UNIFORM },
{ "_cogl_alpha_test_ref",
cogl_pipeline_get_alpha_test_reference, update_float_uniform,
COGL_PIPELINE_STATE_ALPHA_FUNC_REFERENCE }
COGL_PIPELINE_STATE_ALPHA_FUNC_REFERENCE,
COGL_PRIVATE_FEATURE_ALPHA_TEST }
};
#endif /* HAVE_COGL_GLES2 */
const CoglPipelineProgend _cogl_pipeline_glsl_progend;
typedef struct _UnitState
@ -115,10 +118,8 @@ typedef struct
* if this changes. */
int n_tex_coord_attribs;
#ifdef HAVE_COGL_GLES2
unsigned long dirty_builtin_uniforms;
GLint builtin_uniform_locations[G_N_ELEMENTS (builtin_uniforms)];
#endif
GLint modelview_uniform;
GLint projection_uniform;
@ -438,10 +439,9 @@ update_constants_cb (CoglPipeline *pipeline,
return TRUE;
}
#ifdef HAVE_COGL_GLES2
static void
update_builtin_uniforms (CoglPipeline *pipeline,
update_builtin_uniforms (CoglContext *context,
CoglPipeline *pipeline,
GLuint gl_program,
CoglPipelineProgramState *program_state)
{
@ -451,7 +451,9 @@ update_builtin_uniforms (CoglPipeline *pipeline,
return;
for (i = 0; i < G_N_ELEMENTS (builtin_uniforms); i++)
if ((program_state->dirty_builtin_uniforms & (1 << i)) &&
if ((context->private_feature_flags &
builtin_uniforms[i].feature_replacement) == 0 &&
(program_state->dirty_builtin_uniforms & (1 << i)) &&
program_state->builtin_uniform_locations[i] != -1)
builtin_uniforms[i].update_func (pipeline,
program_state
@ -461,8 +463,6 @@ update_builtin_uniforms (CoglPipeline *pipeline,
program_state->dirty_builtin_uniforms = 0;
}
#endif /* HAVE_COGL_GLES2 */
typedef struct
{
CoglPipelineProgramState *program_state;
@ -801,9 +801,11 @@ _cogl_pipeline_progend_glsl_end (CoglPipeline *pipeline,
clear_flushed_matrix_stacks (program_state);
for (i = 0; i < G_N_ELEMENTS (builtin_uniforms); i++)
GE_RET( program_state->builtin_uniform_locations[i], ctx,
glGetUniformLocation (gl_program,
builtin_uniforms[i].uniform_name) );
if ((ctx->private_feature_flags &
builtin_uniforms[i].feature_replacement) == 0)
GE_RET( program_state->builtin_uniform_locations[i], ctx,
glGetUniformLocation (gl_program,
builtin_uniforms[i].uniform_name) );
GE_RET( program_state->modelview_uniform, ctx,
glGetUniformLocation (gl_program,
@ -818,13 +820,11 @@ _cogl_pipeline_progend_glsl_end (CoglPipeline *pipeline,
"cogl_modelview_projection_matrix") );
}
#ifdef HAVE_COGL_GLES2
if (program_changed ||
program_state->last_used_for_pipeline != pipeline)
program_state->dirty_builtin_uniforms = ~(unsigned long) 0;
update_builtin_uniforms (pipeline, gl_program, program_state);
#endif
update_builtin_uniforms (ctx, pipeline, gl_program, program_state);
_cogl_pipeline_progend_glsl_flush_uniforms (pipeline,
program_state,
@ -850,14 +850,14 @@ _cogl_pipeline_progend_glsl_pre_change_notify (CoglPipeline *pipeline,
if ((change & _cogl_pipeline_get_state_for_fragment_codegen (ctx)))
dirty_program_state (pipeline);
#ifdef HAVE_COGL_GLES2
else if (ctx->driver == COGL_DRIVER_GLES2)
else
{
int i;
for (i = 0; i < G_N_ELEMENTS (builtin_uniforms); i++)
if ((change & builtin_uniforms[i].change))
if ((ctx->private_feature_flags &
builtin_uniforms[i].feature_replacement) == 0 &&
(change & builtin_uniforms[i].change))
{
CoglPipelineProgramState *program_state
= get_program_state (pipeline);
@ -866,7 +866,6 @@ _cogl_pipeline_progend_glsl_pre_change_notify (CoglPipeline *pipeline,
return;
}
}
#endif /* HAVE_COGL_GLES2 */
}
/* NB: layers are considered immutable once they have any dependants
@ -1035,8 +1034,6 @@ _cogl_pipeline_progend_glsl_pre_paint (CoglPipeline *pipeline,
}
}
#ifdef HAVE_COGL_GLES2
static void
update_float_uniform (CoglPipeline *pipeline,
int uniform_location,
@ -1051,8 +1048,6 @@ update_float_uniform (CoglPipeline *pipeline,
GE( ctx, glUniform1f (uniform_location, value) );
}
#endif
const CoglPipelineProgend _cogl_pipeline_glsl_progend =
{
COGL_PIPELINE_VERTEND_GLSL,