diff options
author | Eric Anholt <eric@anholt.net> | 2013-09-26 12:01:56 -0700 |
---|---|---|
committer | Eric Anholt <eric@anholt.net> | 2013-10-10 16:34:30 -0700 |
commit | 083f66fdd6451648fe355b64b02b29a6a4389f0d (patch) | |
tree | 7965c2cd9f5b2f96ea1268ef6640d6e39ed55bff /src/mesa/drivers/dri/i915/intel_screen.c | |
parent | d81632fb1e809a0b1ee9310ae3a4733a1c0651b7 (diff) | |
download | external_mesa3d-083f66fdd6451648fe355b64b02b29a6a4389f0d.zip external_mesa3d-083f66fdd6451648fe355b64b02b29a6a4389f0d.tar.gz external_mesa3d-083f66fdd6451648fe355b64b02b29a6a4389f0d.tar.bz2 |
dri: Move API version validation into dri/common.
i965, i915, radeon, r200, swrast, and nouveau were mostly trying to do the
same logic, except where they failed to. Notably, swrast had code that
appeared to try to enable GLES1/2 but forgot to set api_mask (thus
preventing any gles context from being created), and the non-intel drivers
didn't support MESA_GL_VERSION_OVERRIDE.
nouveau still relies on _mesa_compute_version(), because I don't know what
its limits actually are, and gallium drivers don't declare limits up front
at all. I think I've heard talk about doing so, though.
v2: Compat max version should be 30 (noted by Ken)
Drop r100's custom max version check, too (noted by Emil Velikov)
Reviewed-by: Kenneth Graunke <kenneth@whitecape.org>
Diffstat (limited to 'src/mesa/drivers/dri/i915/intel_screen.c')
-rw-r--r-- | src/mesa/drivers/dri/i915/intel_screen.c | 36 |
1 files changed, 9 insertions, 27 deletions
diff --git a/src/mesa/drivers/dri/i915/intel_screen.c b/src/mesa/drivers/dri/i915/intel_screen.c index e3e6ac2..4f8c342 100644 --- a/src/mesa/drivers/dri/i915/intel_screen.c +++ b/src/mesa/drivers/dri/i915/intel_screen.c @@ -1062,33 +1062,25 @@ intel_screen_make_configs(__DRIscreen *dri_screen) static void set_max_gl_versions(struct intel_screen *screen) { - int gl_version_override = _mesa_get_gl_version_override(); + __DRIscreen *psp = screen->driScrnPriv; switch (screen->gen) { case 3: - screen->max_gl_core_version = 0; - screen->max_gl_es1_version = 11; - screen->max_gl_compat_version = 21; - screen->max_gl_es2_version = 20; + psp->max_gl_core_version = 0; + psp->max_gl_es1_version = 11; + psp->max_gl_compat_version = 21; + psp->max_gl_es2_version = 20; break; case 2: - screen->max_gl_core_version = 0; - screen->max_gl_compat_version = 13; - screen->max_gl_es1_version = 11; - screen->max_gl_es2_version = 0; + psp->max_gl_core_version = 0; + psp->max_gl_compat_version = 13; + psp->max_gl_es1_version = 11; + psp->max_gl_es2_version = 0; break; default: assert(!"unrecognized intel_screen::gen"); break; } - - if (gl_version_override >= 31) { - screen->max_gl_core_version = MAX2(screen->max_gl_core_version, - gl_version_override); - } else { - screen->max_gl_compat_version = MAX2(screen->max_gl_compat_version, - gl_version_override); - } } /** @@ -1137,16 +1129,6 @@ __DRIconfig **intelInitScreen2(__DRIscreen *psp) set_max_gl_versions(intelScreen); - psp->api_mask = (1 << __DRI_API_OPENGL); - if (intelScreen->max_gl_core_version > 0) - psp->api_mask |= (1 << __DRI_API_OPENGL_CORE); - if (intelScreen->max_gl_es1_version > 0) - psp->api_mask |= (1 << __DRI_API_GLES); - if (intelScreen->max_gl_es2_version > 0) - psp->api_mask |= (1 << __DRI_API_GLES2); - if (intelScreen->max_gl_es2_version >= 30) - psp->api_mask |= (1 << __DRI_API_GLES3); - psp->extensions = intelScreenExtensions; return (const __DRIconfig**) intel_screen_make_configs(psp); |