Age | Commit message (Collapse) | Author | Files | Lines |
|
|
|
|
|
|
|
|
|
+ Use level 0 texture coordinates to really verify the Lod fetch works.
It was using scaled coordinates, so textureLodOffset could have been
replaced by textureOffset and the test would pass. Now the test will
fail if drivers attempt to fake it that way.
|
|
|
|
|
|
+ a comment about offsets
|
|
|
|
|
|
|
|
|
|
+ some bug fixes
|
|
|
|
|
|
|
|
|
|
Use the larger of the default window size and the size required by
the test parameters.
Reviewed-by: José Fonseca <jfonseca@vmware.com>
|
|
Reviewed-by: José Fonseca <jfonseca@vmware.com>
|
|
Signed-off-by: Anuj Phogat <anuj.phogat@gmail.com>
|
|
This patch is to verify a bug fix in mesa commit 079bff5. This commit
allowed GL_DEPTH_COMPONENT and GL_DEPTH_STENCIL combinations in
glTexImage{123}D functions.
V2: Add extension check for GL_EXT_texture_array
Signed-off-by: Anuj Phogat <anuj.phogat@gmail.com>
Reviewed-by: Brian Paul <brianp@vmware.com>
|
|
Signed-off-by: Anuj Phogat <anuj.phogat@gmail.com>
Reviewed-by: Brian Paul <brianp@vmware.com>
|
|
As with the previous patch for the tex3d test.
Reviewed-by: José Fonseca <jfonseca@vmware.com>
|
|
Don't call piglit_report_result() - return result from piglit_display().
Use default window size and fix "row wrapping" to avoid drawing off the
edge of the window.
Use bool instead of int.
Use piglit_get_gl_enum_name().
Use GLubyte instead of unsigned char.
Set texture env mode to replace.
Reviewed-by: José Fonseca <jfonseca@vmware.com>
|
|
MSVC 2013 provides all math.h functions that were missing in previous
versions.
Reviewed-by: Brian Paul <brianp@vmware.com>
|
|
Reviewed-by: José Fonseca <jfonseca@vmware.com>
|
|
Reviewed-by: José Fonseca <jfonseca@vmware.com>
|
|
Reviewed-by: José Fonseca <jfonseca@vmware.com>
|
|
Reviewed-by: José Fonseca <jfonseca@vmware.com>
|
|
Reviewed-by: José Fonseca <jfonseca@vmware.com>
|
|
Reviewed-by: José Fonseca <jfonseca@vmware.com>
|
|
Reviewed-by: José Fonseca <jfonseca@vmware.com>
|
|
Reviewed-by: José Fonseca <jfonseca@vmware.com>
|
|
Reviewed-by: José Fonseca <jfonseca@vmware.com>
|
|
Not supported by MSVC.
Reviewed-by: Brian Paul <brianp@vmware.com>
|
|
It didn't test the GL_ONE and GL_ZERO terms and was needlessly
complicated. Replaced by ext_texture_swizzle-swizzle test.
Reviewed-by: Eric Anholt <eric@anholt.net>
|
|
This increases the texture size and uses min/max allowable offsets
instead of just -8/7. (Unfortunately this also slows the test down, as
it will create a 64x64 texture for a -32/31 min/max.)
Signed-off-by: Ilia Mirkin <imirkin@alum.mit.edu>
Reviewed-by: Chris Forbes <chrisf@ijw.co.nz>
|
|
- Take 4 samples rather than 3 horizontally within each test.
- Make EQUAL / NOTEQUAL cases more like the others, so we can
see visually if they are wrong, and to also remove the tendency
for spurious failures.
Signed-off-by: Chris Forbes <chrisf@ijw.co.nz>
Reviewed-by: Brian Paul <brianp@vmware.com>
|
|
Signed-off-by: Chris Forbes <chrisf@ijw.co.nz>
Reviewed-by: Brian Paul <brianp@vmware.com>
|
|
Fixes MSVC build.
Bugzilla: http://bugs.freedesktop.org/show_bug.cgi?id=76849
|
|
Reviewed-by: Chia-I Wu <olv@lunarg.com>
|
|
Reviewed-by: Chia-I Wu <olv@lunarg.com>
|
|
Signed-off-by: Ilia Mirkin <imirkin@alum.mit.edu>
Reviewed-by: Marek Olšák <marek.olsak@amd.com>
|
|
Signed-off-by: Ilia Mirkin <imirkin@alum.mit.edu>
Reviewed-by: Marek Olšák <marek.olsak@amd.com>
|
|
If texInitData() failed, it was with maxSide/2, not maxSide.
Reviewed-by: Ian Romanick <ian.d.romanick@intel.com>
|
|
All the other texture targets test glTexSubImage(), so do it for
this target too.
Reviewed-by: Ian Romanick <ian.d.romanick@intel.com>
|
|
The params argument in glGetTexLevelParameteriv is of type GLint*.
This patch fixes these GCC warnings introduced with commit
4f9d98dbaea90c11a930fd774382ff364892ce3f.
s3tc-errors.c:156:6: warning: passing argument 4 of ‘piglit_dispatch_glGetTexLevelParameteriv’ from incompatible pointer type [enabled by default]
&is_compressed);
^
s3tc-errors.c:156:6: note: expected ‘GLint *’ but argument is of type ‘_Bool *’
s3tc-errors.c:158:6: warning: pointer targets in passing argument 4 of ‘piglit_dispatch_glGetTexLevelParameteriv’ differ in signedness [-Wpointer-sign]
&format);
^
s3tc-errors.c:158:6: note: expected ‘GLint *’ but argument is of type ‘GLenum *’
s3tc-errors.c:161:6: warning: pointer targets in passing argument 4 of ‘piglit_dispatch_glGetTexLevelParameteriv’ differ in signedness [-Wpointer-sign]
&compressed_size);
^
s3tc-errors.c:161:6: note: expected ‘GLint *’ but argument is of type ‘GLuint *’
Signed-off-by: Vinson Lee <vlee@freedesktop.org>
Reviewed-by: Brian Paul <brianp@vmware.com>
Reviewed-by: Daniel Kurtz <djkurtz@chromium.org>
|
|
If the GL does not actually support one of the compression formats in
s3tc_formats, it may silently choose a different actual internalformat.
In such a case, the resulting compressed image size may be larger,
leading to memory corruption when the bigger-than-expected image is read
back with glGetCompressedTexImage() into a buffer that was allocated to
the expected, smaller, size.
Instead, first confirm the texture has really been compressed, that the
format is as expected, and that the size matches our pre-computed
expected size. If any of these fail, set pass to false and spit an
error, but keep going, using the actual format and size returned by the
GL.
Ian Romanick pointed out that "if we're going to all this effort, we
should probably go all the way. A follow-on patch should check that
glGetCompressedTexImage doesn't over-run the buffer."
v2:
* Fixed parameter check block format per Ian Romanick review
* Updated commit message title per Brian Paul review
* Function call continuation lines align with open '(' (like in rest of
file).
Signed-off-by: Daniel Kurtz <djkurtz@chromium.org>
Reviewed-by: Ian Romanick <ian.d.romanick@intel.com>
|
|
Test for checking textureGatherOffsets from ARB_gpu_shader5 works, this
is tested on the nvidia binary driver and passes.
also add to all.py.
v2.1 fix VS bits + whitespace
Reviewed-by: Chris Forbes <chrisf@ijw.co.nz>
Signed-off-by: Dave Airlie <airlied@redhat.com>
|
|
I honestly have no idea why, but the original test calls TexSubImage
with a width and height of *half* the texture size. Yet it allocates
enough temporary pixel data for a full texture. This makes no sense,
and clearly one or the other should change.
Without understanding why the existing test does what it does, I chose
to not change it and simply cut the memory allocation. Cutting memory
usage also may reduce the likelihood of GL_OUT_OF_MEMORY errors.
Signed-off-by: Kenneth Graunke <kenneth@whitecape.org>
Reviewed-by: Matt Turner <mattst88@gmail.com>
|