[Piglit] [PATCH] ext_texture_norm16-render: read SNORM formats if EXT_render_snorm

Lionel Landwerlin lionel.g.landwerlin at intel.com
Tue Jul 31 12:57:04 UTC 2018


On 31/07/18 12:21, Tapani Pälli wrote:
> This was missed from commit a440e4390.
>
> Signed-off-by: Tapani Pälli <tapani.palli at intel.com>

Reviewed-by: Lionel Landwerlin <lionel.g.landwerlin at intel.com>

> ---
>   tests/spec/ext_texture_norm16/render.c | 5 +++--
>   1 file changed, 3 insertions(+), 2 deletions(-)
>
> diff --git a/tests/spec/ext_texture_norm16/render.c b/tests/spec/ext_texture_norm16/render.c
> index 46cc7c91b..08a84501c 100644
> --- a/tests/spec/ext_texture_norm16/render.c
> +++ b/tests/spec/ext_texture_norm16/render.c
> @@ -239,8 +239,8 @@ verify_contents(const struct fmt_test *test)
>   	bool result = true;
>   	unsigned amount = piglit_width * piglit_height;
>   	unsigned short *pix = malloc (amount * 8);
> -	glReadPixels(0, 0, piglit_width, piglit_height, GL_RGBA,
> -		     GL_UNSIGNED_SHORT, pix);
> +	glReadPixels(0, 0, piglit_width, piglit_height, GL_RGBA, test->type,
> +		     pix);
>   
>   	/* Setup expected value, alpha is always max in the test. */
>   	unsigned short value[4] = { 0 };
> @@ -476,6 +476,7 @@ piglit_display(void)
>   			case GL_RG16_SNORM_EXT:
>   			case GL_RGBA16_SNORM_EXT:
>   				test->req_render = true;
> +				test->can_read = true;
>   			}
>   		}
>   	}




More information about the Piglit mailing list