mesa/formats: make format testing a gtest

We currently check that our format info table is sane during context
initialization in debug builds. Perform this check during
`make check` instead. This enables format testing in release builds
and removes the requirement of an exhuastive switch for
_mesa_uncompressed_format_to_type_and_comps().

v2. indentation and conditional inclusion fixes (Chad).
    allow tests to continue running if any format fails
    and display the failing format name.

Reviewed-by: Chad Versace <chad.versace@intel.com>
Signed-off-by: Nanley Chery <nanley.g.chery@intel.com>
This commit is contained in:
Nanley Chery 2015-08-18 12:42:57 -07:00
parent 1bec29d04d
commit 8e581747d2
4 changed files with 137 additions and 155 deletions

View file

@ -402,10 +402,6 @@ one_time_init( struct gl_context *ctx )
PACKAGE_VERSION, __DATE__, __TIME__);
}
#endif
#ifdef DEBUG
_mesa_test_formats();
#endif
}
/* per-API one-time init */

View file

@ -83,6 +83,7 @@ static const struct gl_format_info *
_mesa_get_format_info(mesa_format format)
{
const struct gl_format_info *info = &format_info[format];
STATIC_ASSERT(ARRAY_SIZE(format_info) == MESA_FORMAT_COUNT);
assert(info->Name == format);
return info;
}
@ -864,118 +865,6 @@ _mesa_format_row_stride(mesa_format format, GLsizei width)
}
/**
* Debug/test: check that all uncompressed formats are handled in the
* _mesa_uncompressed_format_to_type_and_comps() function. When new pixel
* formats are added to Mesa, that function needs to be updated.
* This is a no-op after the first call.
*/
static void
check_format_to_type_and_comps(void)
{
mesa_format f;
for (f = MESA_FORMAT_NONE + 1; f < MESA_FORMAT_COUNT; f++) {
GLenum datatype = 0;
GLuint comps = 0;
/* This function will emit a problem/warning if the format is
* not handled.
*/
if (!_mesa_is_format_compressed(f))
_mesa_uncompressed_format_to_type_and_comps(f, &datatype, &comps);
}
}
/**
* Do sanity checking of the format info table.
*/
void
_mesa_test_formats(void)
{
GLuint i;
STATIC_ASSERT(ARRAY_SIZE(format_info) == MESA_FORMAT_COUNT);
for (i = 0; i < MESA_FORMAT_COUNT; i++) {
const struct gl_format_info *info = _mesa_get_format_info(i);
assert(info);
assert(info->Name == i);
if (info->Name == MESA_FORMAT_NONE)
continue;
if (info->BlockWidth == 1 && info->BlockHeight == 1) {
if (info->RedBits > 0) {
GLuint t = info->RedBits + info->GreenBits
+ info->BlueBits + info->AlphaBits;
assert(t / 8 <= info->BytesPerBlock);
(void) t;
}
}
assert(info->DataType == GL_UNSIGNED_NORMALIZED ||
info->DataType == GL_SIGNED_NORMALIZED ||
info->DataType == GL_UNSIGNED_INT ||
info->DataType == GL_INT ||
info->DataType == GL_FLOAT ||
/* Z32_FLOAT_X24S8 has DataType of GL_NONE */
info->DataType == GL_NONE);
if (info->BaseFormat == GL_RGB) {
assert(info->RedBits > 0);
assert(info->GreenBits > 0);
assert(info->BlueBits > 0);
assert(info->AlphaBits == 0);
assert(info->LuminanceBits == 0);
assert(info->IntensityBits == 0);
}
else if (info->BaseFormat == GL_RGBA) {
assert(info->RedBits > 0);
assert(info->GreenBits > 0);
assert(info->BlueBits > 0);
assert(info->AlphaBits > 0);
assert(info->LuminanceBits == 0);
assert(info->IntensityBits == 0);
}
else if (info->BaseFormat == GL_RG) {
assert(info->RedBits > 0);
assert(info->GreenBits > 0);
assert(info->BlueBits == 0);
assert(info->AlphaBits == 0);
assert(info->LuminanceBits == 0);
assert(info->IntensityBits == 0);
}
else if (info->BaseFormat == GL_RED) {
assert(info->RedBits > 0);
assert(info->GreenBits == 0);
assert(info->BlueBits == 0);
assert(info->AlphaBits == 0);
assert(info->LuminanceBits == 0);
assert(info->IntensityBits == 0);
}
else if (info->BaseFormat == GL_LUMINANCE) {
assert(info->RedBits == 0);
assert(info->GreenBits == 0);
assert(info->BlueBits == 0);
assert(info->AlphaBits == 0);
assert(info->LuminanceBits > 0);
assert(info->IntensityBits == 0);
}
else if (info->BaseFormat == GL_INTENSITY) {
assert(info->RedBits == 0);
assert(info->GreenBits == 0);
assert(info->BlueBits == 0);
assert(info->AlphaBits == 0);
assert(info->LuminanceBits == 0);
assert(info->IntensityBits > 0);
}
}
check_format_to_type_and_comps();
}
/**
* Return datatype and number of components per texel for the given
@ -1513,48 +1402,12 @@ _mesa_uncompressed_format_to_type_and_comps(mesa_format format,
case MESA_FORMAT_COUNT:
assert(0);
return;
case MESA_FORMAT_RGB_FXT1:
case MESA_FORMAT_RGBA_FXT1:
case MESA_FORMAT_RGB_DXT1:
case MESA_FORMAT_RGBA_DXT1:
case MESA_FORMAT_RGBA_DXT3:
case MESA_FORMAT_RGBA_DXT5:
case MESA_FORMAT_SRGB_DXT1:
case MESA_FORMAT_SRGBA_DXT1:
case MESA_FORMAT_SRGBA_DXT3:
case MESA_FORMAT_SRGBA_DXT5:
case MESA_FORMAT_R_RGTC1_UNORM:
case MESA_FORMAT_R_RGTC1_SNORM:
case MESA_FORMAT_RG_RGTC2_UNORM:
case MESA_FORMAT_RG_RGTC2_SNORM:
case MESA_FORMAT_L_LATC1_UNORM:
case MESA_FORMAT_L_LATC1_SNORM:
case MESA_FORMAT_LA_LATC2_UNORM:
case MESA_FORMAT_LA_LATC2_SNORM:
case MESA_FORMAT_ETC1_RGB8:
case MESA_FORMAT_ETC2_RGB8:
case MESA_FORMAT_ETC2_SRGB8:
case MESA_FORMAT_ETC2_RGBA8_EAC:
case MESA_FORMAT_ETC2_SRGB8_ALPHA8_EAC:
case MESA_FORMAT_ETC2_R11_EAC:
case MESA_FORMAT_ETC2_RG11_EAC:
case MESA_FORMAT_ETC2_SIGNED_R11_EAC:
case MESA_FORMAT_ETC2_SIGNED_RG11_EAC:
case MESA_FORMAT_ETC2_RGB8_PUNCHTHROUGH_ALPHA1:
case MESA_FORMAT_ETC2_SRGB8_PUNCHTHROUGH_ALPHA1:
case MESA_FORMAT_BPTC_RGBA_UNORM:
case MESA_FORMAT_BPTC_SRGB_ALPHA_UNORM:
case MESA_FORMAT_BPTC_RGB_SIGNED_FLOAT:
case MESA_FORMAT_BPTC_RGB_UNSIGNED_FLOAT:
assert(_mesa_is_format_compressed(format));
case MESA_FORMAT_NONE:
/* For debug builds, warn if any formats are not handled */
#ifdef DEBUG
default:
#endif
/* Warn if any formats are not handled */
_mesa_problem(NULL, "bad format %s in _mesa_uncompressed_format_to_type_and_comps",
_mesa_get_format_name(format));
assert(format == MESA_FORMAT_NONE ||
_mesa_is_format_compressed(format));
*datatype = 0;
*comps = 1;
}

View file

@ -27,6 +27,7 @@ AM_CPPFLAGS += -DHAVE_SHARED_GLAPI
main_test_SOURCES += \
dispatch_sanity.cpp \
mesa_formats.cpp \
program_state_string.cpp
main_test_LDADD += \

View file

@ -0,0 +1,132 @@
/*
* Copyright © 2015 Intel Corporation
*
* Permission is hereby granted, free of charge, to any person obtaining a
* copy of this software and associated documentation files (the "Software"),
* to deal in the Software without restriction, including without limitation
* the rights to use, copy, modify, merge, publish, distribute, sublicense,
* and/or sell copies of the Software, and to permit persons to whom the
* Software is furnished to do so, subject to the following conditions:
*
* The above copyright notice and this permission notice (including the next
* paragraph) shall be included in all copies or substantial portions of the
* Software.
*
* THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
* IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
* FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL
* THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
* LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING
* FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER
* DEALINGS IN THE SOFTWARE.
*/
/**
* \name mesa_formats.cpp
*
* Verify that all mesa formats are handled in certain functions and that
* the format info table is sane.
*
*/
#include <gtest/gtest.h>
#include "main/formats.h"
#include "main/glformats.h"
/**
* Debug/test: check that all uncompressed formats are handled in the
* _mesa_uncompressed_format_to_type_and_comps() function. When new pixel
* formats are added to Mesa, that function needs to be updated.
*/
TEST(MesaFormatsTest, FormatTypeAndComps)
{
for (int fi = MESA_FORMAT_NONE + 1; fi < MESA_FORMAT_COUNT; ++fi) {
mesa_format f = (mesa_format) fi;
SCOPED_TRACE(_mesa_get_format_name(f));
/* This function will emit a problem/warning if the format is
* not handled.
*/
if (!_mesa_is_format_compressed(f)) {
GLenum datatype = 0;
GLuint comps = 0;
_mesa_uncompressed_format_to_type_and_comps(f, &datatype, &comps);
/* If the datatype is zero, the format was not handled */
EXPECT_NE(datatype, (GLenum)0);
}
}
}
/**
* Do sanity checking of the format info table.
*/
TEST(MesaFormatsTest, FormatSanity)
{
for (int fi = 0; fi < MESA_FORMAT_COUNT; ++fi) {
mesa_format f = (mesa_format) fi;
SCOPED_TRACE(_mesa_get_format_name(f));
GLenum datatype = _mesa_get_format_datatype(f);
GLint r = _mesa_get_format_bits(f, GL_RED_BITS);
GLint g = _mesa_get_format_bits(f, GL_GREEN_BITS);
GLint b = _mesa_get_format_bits(f, GL_BLUE_BITS);
GLint a = _mesa_get_format_bits(f, GL_ALPHA_BITS);
GLint l = _mesa_get_format_bits(f, GL_TEXTURE_LUMINANCE_SIZE);
GLint i = _mesa_get_format_bits(f, GL_TEXTURE_INTENSITY_SIZE);
/* Note: Z32_FLOAT_X24S8 has datatype of GL_NONE */
EXPECT_TRUE(datatype == GL_NONE ||
datatype == GL_UNSIGNED_NORMALIZED ||
datatype == GL_SIGNED_NORMALIZED ||
datatype == GL_UNSIGNED_INT ||
datatype == GL_INT ||
datatype == GL_FLOAT);
if (r > 0 && !_mesa_is_format_compressed(f)) {
GLint bytes = _mesa_get_format_bytes(f);
EXPECT_LE((r+g+b+a) / 8, bytes);
}
/* Determines if the base format has a channel [rgba] or property [li].
* > indicates existance
* == indicates non-existance
*/
#define HAS_PROP(rop,gop,bop,aop,lop,iop) \
do { \
EXPECT_TRUE(r rop 0); \
EXPECT_TRUE(g gop 0); \
EXPECT_TRUE(b bop 0); \
EXPECT_TRUE(a aop 0); \
EXPECT_TRUE(l lop 0); \
EXPECT_TRUE(i iop 0); \
} while(0)
switch (_mesa_get_format_base_format(f)) {
case GL_RGBA:
HAS_PROP(>,>,>,>,==,==);
break;
case GL_RGB:
HAS_PROP(>,>,>,==,==,==);
break;
case GL_RG:
HAS_PROP(>,>,==,==,==,==);
break;
case GL_RED:
HAS_PROP(>,==,==,==,==,==);
break;
case GL_LUMINANCE:
HAS_PROP(==,==,==,==,>,==);
break;
case GL_INTENSITY:
HAS_PROP(==,==,==,==,==,>);
break;
default:
break;
}
#undef HAS_PROP
}
}