[Mesa-dev] [PATCH 5/9] r300: Enable extensions by just setting the flags
Ian Romanick
idr at freedesktop.org
Tue Sep 6 14:54:34 PDT 2011
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1
On 09/06/2011 02:45 PM, Corbin Simpson wrote:
> So much win! I have a couple nits; replying inline.
>
> On Tue, Sep 6, 2011 at 1:13 PM, Ian Romanick <idr at freedesktop.org> wrote:
>> From: Ian Romanick <ian.d.romanick at intel.com>
>>
>> Core Mesa already does the dispatch offset remapping for every
>> function that could possibly ever be supported. There's no need to
>> continue using that cruft in the driver.
>>
>> Since the call to _mesa_enable_imaging_extensions (via
>> driInitExtensions) is removed, EXT_blend_color and EXT_blend_logic_op
>> are explicitly added to the list.
>>
>> Cc: Alex Deucher <alexdeucher at gmail.com>
>> Cc: Dave Airlie <airlied at redhat.com>
>> ---
>> src/mesa/drivers/dri/r300/r300_context.c | 162 ++++++++++++------------------
>> 1 files changed, 63 insertions(+), 99 deletions(-)
>>
>> diff --git a/src/mesa/drivers/dri/r300/r300_context.c b/src/mesa/drivers/dri/r300/r300_context.c
>> index 213d3c0..984aff5 100644
>> --- a/src/mesa/drivers/dri/r300/r300_context.c
>> +++ b/src/mesa/drivers/dri/r300/r300_context.c
>> @@ -35,6 +35,7 @@ WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.
>> * \author Nicolai Haehnle <prefect_ at gmx.net>
>> */
>>
>> +#include <stdbool.h>
>> #include "main/glheader.h"
>> #include "main/api_arrayelt.h"
>> #include "main/context.h"
>> @@ -71,91 +72,6 @@ WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.
>> #include "utils.h"
>> #include "xmlpool.h" /* for symbolic values of enum-type options */
>>
>> -#define need_GL_VERSION_2_0
>> -#define need_GL_ARB_occlusion_query
>> -#define need_GL_ARB_point_parameters
>> -#define need_GL_ARB_vertex_program
>> -#define need_GL_EXT_blend_equation_separate
>> -#define need_GL_EXT_blend_func_separate
>> -#define need_GL_EXT_blend_minmax
>> -#define need_GL_EXT_framebuffer_blit
>> -#define need_GL_EXT_framebuffer_object
>> -#define need_GL_EXT_fog_coord
>> -#define need_GL_EXT_gpu_program_parameters
>> -#define need_GL_EXT_provoking_vertex
>> -#define need_GL_EXT_secondary_color
>> -#define need_GL_EXT_stencil_two_side
>> -#define need_GL_ATI_separate_stencil
>> -#define need_GL_NV_vertex_program
>> -#define need_GL_OES_EGL_image
>> -
>> -#include "main/remap_helper.h"
>> -
>> -static const struct dri_extension card_extensions[] = {
>> - /* *INDENT-OFF* */
>> - {"GL_ARB_depth_texture", NULL},
>> - {"GL_ARB_fragment_program", NULL},
>> - {"GL_ARB_occlusion_query", GL_ARB_occlusion_query_functions},
>> - {"GL_ARB_multitexture", NULL},
>> - {"GL_ARB_point_parameters", GL_ARB_point_parameters_functions},
>> - {"GL_ARB_shadow", NULL},
>> - {"GL_ARB_shadow_ambient", NULL},
>> - {"GL_ARB_texture_border_clamp", NULL},
>> - {"GL_ARB_texture_cube_map", NULL},
>> - {"GL_ARB_texture_env_add", NULL},
>> - {"GL_ARB_texture_env_combine", NULL},
>> - {"GL_ARB_texture_env_crossbar", NULL},
>> - {"GL_ARB_texture_env_dot3", NULL},
>> - {"GL_ARB_texture_mirrored_repeat", NULL},
>> - {"GL_ARB_vertex_program", GL_ARB_vertex_program_functions},
>> - {"GL_EXT_blend_equation_separate", GL_EXT_blend_equation_separate_functions},
>> - {"GL_EXT_blend_func_separate", GL_EXT_blend_func_separate_functions},
>> - {"GL_EXT_blend_minmax", GL_EXT_blend_minmax_functions},
>> - {"GL_EXT_blend_subtract", NULL},
>> - {"GL_EXT_fog_coord", GL_EXT_fog_coord_functions },
>> - {"GL_EXT_gpu_program_parameters", GL_EXT_gpu_program_parameters_functions},
>> - {"GL_EXT_provoking_vertex", GL_EXT_provoking_vertex_functions },
>> - {"GL_EXT_secondary_color", GL_EXT_secondary_color_functions},
>> - {"GL_EXT_shadow_funcs", NULL},
>> - {"GL_EXT_stencil_two_side", GL_EXT_stencil_two_side_functions},
>> - {"GL_EXT_stencil_wrap", NULL},
>> - {"GL_EXT_texture_edge_clamp", NULL},
>> - {"GL_EXT_texture_env_combine", NULL},
>> - {"GL_EXT_texture_env_dot3", NULL},
>> - {"GL_EXT_texture_filter_anisotropic", NULL},
>> - {"GL_EXT_texture_lod_bias", NULL},
>> - {"GL_EXT_texture_mirror_clamp", NULL},
>> - {"GL_EXT_texture_rectangle", NULL},
>> - {"GL_EXT_texture_sRGB", NULL},
>> - {"GL_EXT_vertex_array_bgra", NULL},
>> - {"GL_ATI_separate_stencil", GL_ATI_separate_stencil_functions},
>> - {"GL_ATI_texture_env_combine3", NULL},
>> - {"GL_ATI_texture_mirror_once", NULL},
>> - {"GL_MESA_pack_invert", NULL},
>> - {"GL_MESA_ycbcr_texture", NULL},
>> - {"GL_NV_blend_square", NULL},
>> - {"GL_NV_vertex_program", GL_NV_vertex_program_functions},
>> -#if FEATURE_OES_EGL_image
>> - {"GL_OES_EGL_image", GL_OES_EGL_image_functions },
>> -#endif
>> - {NULL, NULL}
>> - /* *INDENT-ON* */
>> -};
>> -
>> -
>> -static const struct dri_extension mm_extensions[] = {
>> - { "GL_EXT_framebuffer_blit", GL_EXT_framebuffer_blit_functions },
>> - { "GL_EXT_framebuffer_object", GL_EXT_framebuffer_object_functions },
>> - { NULL, NULL }
>> -};
>> -
>> -/**
>> - * The GL 2.0 functions are needed to make display lists work with
>> - * functions added by GL_ATI_separate_stencil.
>> - */
>> -static const struct dri_extension gl_20_extension[] = {
>> - {"GL_VERSION_2_0", GL_VERSION_2_0_functions },
>> -};
>>
>> static const struct tnl_pipeline_stage *r300_pipeline[] = {
>> /* Catch any t&l fallbacks
>> @@ -447,28 +363,76 @@ static void r300InitGLExtensions(struct gl_context *ctx)
>> {
>> r300ContextPtr r300 = R300_CONTEXT(ctx);
>>
>> - driInitExtensions(ctx, card_extensions, GL_TRUE);
>> - if (r300->radeon.radeonScreen->kernel_mm)
>> - driInitExtensions(ctx, mm_extensions, GL_FALSE);
>> + ctx->Extensions.ARB_depth_texture = true;
>> + ctx->Extensions.ARB_fragment_program = true;
>> + ctx->Extensions.ARB_multitexture = true;
>> + ctx->Extensions.ARB_shadow = true;
>> + ctx->Extensions.ARB_shadow_ambient = true;
>> + ctx->Extensions.ARB_texture_border_clamp = true;
>> + ctx->Extensions.ARB_texture_cube_map = true;
>> + ctx->Extensions.ARB_texture_env_combine = true;
>> + ctx->Extensions.ARB_texture_env_crossbar = true;
>> + ctx->Extensions.ARB_texture_env_dot3 = true;
>> + ctx->Extensions.ARB_texture_mirrored_repeat = true;
>> + ctx->Extensions.ARB_vertex_program = true;
>> + ctx->Extensions.EXT_blend_color = true;
>> + ctx->Extensions.EXT_blend_equation_separate = true;
>> + ctx->Extensions.EXT_blend_func_separate = true;
>> + ctx->Extensions.EXT_blend_logic_op = true;
>> + ctx->Extensions.EXT_blend_minmax = true;
>> + ctx->Extensions.EXT_blend_subtract = true;
>> + ctx->Extensions.EXT_fog_coord = true;
>> + ctx->Extensions.EXT_gpu_program_parameters = true;
>> + ctx->Extensions.EXT_point_parameters = true;
>> + ctx->Extensions.EXT_provoking_vertex = true;
>> + ctx->Extensions.EXT_secondary_color = true;
>> + ctx->Extensions.EXT_shadow_funcs = true;
>> + ctx->Extensions.EXT_stencil_two_side = true;
>> + ctx->Extensions.EXT_stencil_wrap = true;
>> + ctx->Extensions.EXT_texture_env_add = true;
>> + ctx->Extensions.EXT_texture_env_combine = true;
>> + ctx->Extensions.EXT_texture_env_dot3 = true;
>> + ctx->Extensions.EXT_texture_filter_anisotropic = true;
>> + ctx->Extensions.EXT_texture_lod_bias = true;
>> + ctx->Extensions.EXT_texture_mirror_clamp = true;
>> + ctx->Extensions.EXT_texture_sRGB = true;
>> + ctx->Extensions.EXT_vertex_array_bgra = true;
>> + ctx->Extensions.ATI_separate_stencil = true;
>> + ctx->Extensions.ATI_texture_env_combine3 = true;
>> + ctx->Extensions.ATI_texture_mirror_once = true;
>> + ctx->Extensions.MESA_pack_invert = true;
>> + ctx->Extensions.MESA_ycbcr_texture = true;
>> + ctx->Extensions.NV_blend_square = true;
>> + ctx->Extensions.NV_texture_rectangle = true;
>> + ctx->Extensions.NV_vertex_program = true;
>> +#if FEATURE_OES_EGL_image
>> + ctx->Extensions.OES_EGL_image = true;
>> +#endif
>> + ctx->Extensions.SGIS_texture_edge_clamp = true;
>> +
>> + if (r300->radeon.radeonScreen->kernel_mm) {
>> + ctx->Extensions.EXT_framebuffer_blit = true;
>> + ctx->Extensions.EXT_framebuffer_object = true;
>> + }
>>
>> if (r300->options.stencil_two_side_disabled)
>> - _mesa_disable_extension(ctx, "GL_EXT_stencil_two_side");
>> + ctx->Extensions.EXT_stencil_two_side = true;
>
> This is wrong; it should be = false. Even better, up above in the big
> list, we could do:
Ah, good catch.
> ctx->Extensions.EXT_stencil_two_side =
> !r300->options.stencil_two_side_disabled;
>
> Or whatever equivalent idiom is accepted in Mesa.
I tried not to make changes like this because I wanted accidental
functional changes (like this one!) to be more noticeable.
>>
>> if (r300->options.s3tc_force_disabled) {
>> - _mesa_disable_extension(ctx, "GL_EXT_texture_compression_s3tc");
>> + ctx->Extensions.EXT_texture_compression_s3tc = false;
>> } else if (ctx->Mesa_DXTn || r300->options.s3tc_force_enabled) {
>> - _mesa_enable_extension(ctx, "GL_EXT_texture_compression_s3tc");
>> - _mesa_enable_extension(ctx, "GL_S3_s3tc");
>> + ctx->Extensions.EXT_texture_compression_s3tc = true;
>> + ctx->Extensions.S3_s3tc = true;
>> }
>
> Should we force off S3_s3tc when EXT_tc_s3tc is forced off?
Perhaps. Since almost every DRI driver has some code like this, I was
thinking of refactoring it out to a function in
src/mesa/drivers/dri/common/utils.c.
>>
>> - if (!r300->radeon.radeonScreen->drmSupportsOcclusionQueries) {
>> - _mesa_disable_extension(ctx, "GL_ARB_occlusion_query");
>> - }
>> - if (r300->radeon.radeonScreen->chip_family >= CHIP_FAMILY_R420)
>> - _mesa_enable_extension(ctx, "GL_ARB_half_float_vertex");
>> + ctx->Extensions.ARB_occlusion_query =
>> + r300->radeon.radeonScreen->drmSupportsOcclusionQueries;
>> +
>> + ctx->Extensions.ARB_half_float_vertex =
>> + (r300->radeon.radeonScreen->chip_family >= CHIP_FAMILY_R420);
>>
>> - if (r300->radeon.radeonScreen->chip_family >= CHIP_FAMILY_RV515)
>> - _mesa_enable_extension(ctx, "GL_EXT_packed_depth_stencil");
>> + ctx->Extensions.EXT_packed_depth_stencil =
>> + (r300->radeon.radeonScreen->chip_family >= CHIP_FAMILY_RV515);
>> }
>>
>> static void r300InitIoctlFuncs(struct dd_function_table *functions)
>> --
>> 1.7.4.4
>
> FWIW,
>
> Reviewed-by: Corbin Simpson <MostAwesomeDude at gmail.com>
>
> Thanks for this! I'm so happy!
> ~ C.
>
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.11 (GNU/Linux)
Comment: Using GnuPG with Fedora - http://enigmail.mozdev.org/
iEYEARECAAYFAk5mlpoACgkQX1gOwKyEAw+4ZgCeLTVl9VaD8+Zbmncr/A99LkCB
MdwAoITv5mBdsVR5WnEHPnthTv2ea/Ae
=MQ3i
-----END PGP SIGNATURE-----
More information about the mesa-dev
mailing list