[Mesa-dev] [PATCH 00/11] More patches fixing dEQP test fails

[Iago Toral]

Hi,

there are still a few unreviewed patches in this batch:

PATCH 7/11, i965: Make sure we always mark array surfaces as such

PATCH 9/11, i965: Fix non-AA wide line rendering with fractional line 
widths

PATCH 11/11, i965: Fix textureSize for Lod > 0 with non-mipmap filters

Any takers?

Also there are a couple of other patches that got new versions after
comments. Ian, are the new versions good?

PATCH 1/11, i965: Fix ctx->Texture.CubeMapSeamless (v2)

PATCH 4/11, mesa: Add missing error checks to GetProgramInfoLog,
GetShaderInfoLog and GetProgramiv (v3)

Iago

On Tue, 2015-02-10 at 16:40 +0100, Eduardo Lima Mitev wrote:
> Hi,
> 
> This is yet another batch of fixes for dEQP failing tests. It fixes a total of 249 test cases.
> The test failures were gathered on i965 (gen7) against 10.4.2, but again there are driver and version agnostic fixes.
> 
> A GIT tree with these patches based on git-08a06 is available at: https://github.com/Igalia/mesa/tree/deqp-fixes-batch4-rebased
> 
> There is one known piglit regression caused by patch 10/11: "mesa: Return error if BeginQuery is called with an existing object of different type". Per the specs, OpenGL forbids calling glBeginQuery() with a query object that was already bound to a different target.
> That's what the dEQP test expects, and piglit ignores this constraint. I submitted a patch against piglit this morning [1] to fix the test. With that patch applied, there are no regressions for this series.
> 
> cheers,
> Eduardo
> 
> [1] http://lists.freedesktop.org/archives/mesa-dev/2015-February/076472.html
> 
> 
> Eduardo Lima Mitev (5):
>   mesa: Return INVALID_ENUM in glClearBufferiv() when buffer is not
>     color or stencil
>   mesa: Add missing error checks to GetProgramInfoLog, GetShaderInfoLog
>     and GetProgramiv
>   mesa: Fix error validating args for TexSubImage3D
>   mesa: Return INVALID_OPERATION when querying a never bound Query obj
>   mesa: Return error if BeginQuery is called with an existing object of
>     different type
> 
> Iago Toral Quiroga (5):
>   i965: Fix ctx->Texture.CubeMapSeamless
>   mesa: Add _mesa_is_array_texture helper
>   i965: Make sure we always mark array surfaces as such
>   i965: Fix non-AA wide line rendering with fractional line widths
>   i965: Fix textureSize for Lod > 0 with non-mipmap filters
> 
> Samuel Iglesias Gonsalvez (1):
>   i965/blorp: round to nearest when converting float to integer
> 
>  src/mesa/drivers/dri/i965/brw_blorp_blit.cpp      | 11 +--
>  src/mesa/drivers/dri/i965/brw_context.c           | 10 ++-
>  src/mesa/drivers/dri/i965/gen6_sf_state.c         |  8 ++-
>  src/mesa/drivers/dri/i965/gen7_sf_state.c         |  8 ++-
>  src/mesa/drivers/dri/i965/gen7_wm_surface_state.c |  4 +-
>  src/mesa/drivers/dri/i965/gen8_sf_state.c         |  8 ++-
>  src/mesa/drivers/dri/i965/gen8_surface_state.c    |  4 +-
>  src/mesa/drivers/dri/i965/intel_tex_validate.c    |  6 +-
>  src/mesa/main/clear.c                             | 13 ++++
>  src/mesa/main/queryobj.c                          | 24 +++++--
>  src/mesa/main/shaderapi.c                         | 84 +++++++++++++++++++++--
>  src/mesa/main/teximage.c                          | 26 ++++++-
>  src/mesa/main/teximage.h                          |  3 +
>  src/mesa/main/texstate.c                          |  8 +++
>  14 files changed, 190 insertions(+), 27 deletions(-)
>