From f5e7f12e4a1d2ee98ced36c232842d60181fc01e Mon Sep 17 00:00:00 2001 From: Ian Romanick Date: Sat, 1 Dec 2012 11:05:00 -0800 Subject: mesa: Add driver method to determine the possible sample counts Use this method in _mesa_GetInternalformativ for both GL_SAMPLES and GL_NUM_SAMPLE_COUNTS. v2: internalFormat may not be color renderable by the driver, so zero can be returned as a sample count. Require that drivers supporting the extension provide a QuerySamplesForFormat function. The later was suggested by Eric Anholt. Signed-off-by: Ian Romanick Reviewed-by: Eric Anholt Reviewed-by: Jordan Justen --- src/mesa/main/formatquery.c | 31 +++++++++++++++++++++++++++---- 1 file changed, 27 insertions(+), 4 deletions(-) (limited to 'src/mesa/main/formatquery.c') diff --git a/src/mesa/main/formatquery.c b/src/mesa/main/formatquery.c index 64f10cd..f08ab66 100644 --- a/src/mesa/main/formatquery.c +++ b/src/mesa/main/formatquery.c @@ -42,6 +42,8 @@ _mesa_GetInternalformativ(GLenum target, GLenum internalformat, GLenum pname, return; } + assert(ctx->Driver.QuerySamplesForFormat != NULL); + /* The ARB_internalformat_query spec says: * * "If the parameter to GetInternalformativ is not one of @@ -91,13 +93,34 @@ _mesa_GetInternalformativ(GLenum target, GLenum internalformat, GLenum pname, switch (pname) { case GL_SAMPLES: - buffer[0] = ctx->Const.MaxSamples; - count = 1; + count = ctx->Driver.QuerySamplesForFormat(ctx, internalformat, buffer); break; - case GL_NUM_SAMPLE_COUNTS: - buffer[0] = 1; + case GL_NUM_SAMPLE_COUNTS: { + /* The driver can return 0, and we should pass that along to the + * application. The ARB decided that ARB_internalformat_query should + * behave as ARB_internalformat_query2 in this situation. + * + * The ARB_internalformat_query2 spec says: + * + * "- NUM_SAMPLE_COUNTS: The number of sample counts that would be + * returned by querying SAMPLES is returned in . + * * If is not color-renderable, + * depth-renderable, or stencil-renderable (as defined in + * section 4.4.4), or if does not support multiple + * samples (ie other than TEXTURE_2D_MULTISAMPLE, + * TEXTURE_2D_MULTISAMPLE_ARRAY, or RENDERBUFFER), 0 is + * returned." + */ + const size_t num_samples = + ctx->Driver.QuerySamplesForFormat(ctx, internalformat, buffer); + + /* QuerySamplesForFormat writes some stuff to buffer, so we have to + * separately over-write it with the requested value. + */ + buffer[0] = (GLint) num_samples; count = 1; break; + } default: _mesa_error(ctx, GL_INVALID_ENUM, "glGetInternalformativ(pname=%s)", -- cgit v1.1