[Mesa-dev] [PATCH 14/14] i965: Allow creation of OpenGL 3.1 contexts

[Eric Anholt]

Ian Romanick <idr at freedesktop.org> writes:

> From: Ian Romanick <ian.d.romanick at intel.com>
>
> Signed-off-by: Ian Romanick <ian.d.romanick at intel.com>
> ---
>  src/mesa/drivers/dri/i965/brw_context.c |   19 +++++++++++++++++++
>  1 files changed, 19 insertions(+), 0 deletions(-)
>
> diff --git a/src/mesa/drivers/dri/i965/brw_context.c b/src/mesa/drivers/dri/i965/brw_context.c
> index 013ec14..990b467 100644
> --- a/src/mesa/drivers/dri/i965/brw_context.c
> +++ b/src/mesa/drivers/dri/i965/brw_context.c
> @@ -108,6 +108,25 @@ brwCreateContext(int api,
>     case API_OPENGLES:
>     case API_OPENGLES2:
>        break;
> +   case API_OPENGL_CORE: {
> +#ifdef TEXTURE_FLOAT_ENABLED
> +      const unsigned max_version =
> +         (screen->gen == 6 ||
> +          (screen->gen == 7 && screen->kernel_has_gen7_sol_reset))
> +         ? 0x301 : 0;
> +#else
> +      const unsigned max_version = 0;
> +#endif
> +      const unsigned req_version = (major_version << 8) | minor_version;
> +
> +      if (req_version > max_version) {
> +         *error = (max_version == 0)
> +            ? __DRI_CTX_ERROR_BAD_API : __DRI_CTX_ERROR_BAD_VERSION;
> +         return false;
> +      }

I'd slightly prefer seeing this version handling use major*10+minor like
ctx->Version.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 197 bytes
Desc: not available
URL: <http://lists.freedesktop.org/archives/mesa-dev/attachments/20120808/a6d141a2/attachment.pgp>