I haven't run the CTS tests, but both the deqp and the piglit test pass on my skl with Marek's patches applied.
Dylan
Quoting Marek Olšák (2017-10-31 18:16:51)
> This is how I run piglit on i965:
> WAFFLE_GBM_DEVICE=/dev/dri/renderD128 PIGLIT_PLATFORM=gbm
> piglit/bin/glinfo|head
> GL_RENDERER = Mesa DRI Intel(R) Ivybridge Desktop
>
> Of course I have Radeon as my main GPU.
>
> Marek
>
> On Wed, Nov 1, 2017 at 12:52 AM, Mark Janes <mark.a.ja...@intel.com> wrote:
> > No, please don't. We can't have thousands of failures added to CI.
> >
> > Dylan, can you double check this for Marek? I'm surprised that it
> > doesn't reproduce for him.
> >
> > -Mark
> >
> > Marek Olšák <mar...@gmail.com> writes:
> >
> >> I can't reproduce the issue on my IVB. How about I just push this and
> >> you guys fix i965 when you have time?
> >>
> >> Marek
> >>
> >> On Thu, Oct 26, 2017 at 5:46 PM, Mark Janes <mark.a.ja...@intel.com> wrote:
> >>> That fixed many issues, but:
> >>>
> >>> On deqp EGL tests, we get:
> >>>
> >>> dEQP-EGL.functional.create_context_ext.gl_42.rgba8888_no_depth_no_stencil
> >>> deqp-egl: main/api_exec.c:135: _mesa_initialize_exec_table: Assertion
> >>> `ctx->Version > 0' failed.
> >>>
> >>> On ES31 CTS tests:
> >>> ES31-CTS.functional.texture.texture_buffer.render.as_vertex_array_as_vertex_texture_as_fragment_texture.range_size_513
> >>> glGetIntegerv() failed: glGetError() returned GL_INVALID_ENUM at
> >>> gluContextInfo.cpp:229
> >>>
> >>> on GL46 CTS (27 failures):
> >>> src/mesa/main/teximage.c:595: _mesa_tex_target_is_layered: Assertion
> >>> `!"Invalid texture target."' failed.
> >>>
> >>> Piglit fails 3.5k tests with the same `ctx->Version > 0' assertion as
> >>> deqp-egl.
> >>>
> >>> -Mark
> >>>
> >>> Dylan Baker <dy...@pnwbakers.com> writes:
> >>>
> >>>> I've pushed it to our CI, I'll let you know tomorrow what it looks like
> >>>>
> >>>> Dylan
> >>>>
> >>>> Quoting Marek Olšák (2017-10-25 19:52:43)
> >>>>> Hi Dylan,
> >>>>>
> >>>>> Can you please add this and re-test?
> >>>>>
> >>>>> diff --git a/src/mesa/drivers/dri/i965/intel_extensions.c
> >>>>> b/src/mesa/drivers/
> >>>>> dri/i965/intel_extensions.c
> >>>>> index 21cf632..4d17393 100644
> >>>>> --- a/src/mesa/drivers/dri/i965/intel_extensions.c
> >>>>> +++ b/src/mesa/drivers/dri/i965/intel_extensions.c
> >>>>> @@ -178,7 +178,7 @@ intelInitExtensions(struct gl_context *ctx)
> >>>>> ctx->Extensions.ARB_pipeline_statistics_query = true;
> >>>>> ctx->Extensions.ARB_sample_shading = true;
> >>>>> ctx->Extensions.ARB_shading_language_420pack = true;
> >>>>> - if (ctx->API == API_OPENGL_CORE) {
> >>>>> + if (ctx->API != API_OPENGL_COMPAT) {
> >>>>> ctx->Extensions.ARB_texture_buffer_object = true;
> >>>>> ctx->Extensions.ARB_texture_buffer_object_rgb32 = true;
> >>>>> ctx->Extensions.ARB_texture_buffer_range = true;
> >>>>>
> >>>>>
> >>>>> Marek
> >>>>>
> >>>>> On Wed, Oct 25, 2017 at 11:42 PM, Dylan Baker
> >>>>> <dylanx.c.ba...@intel.com> wrote:
> >>>>>
> >>>>> There are a significant number of i965 regressions from
> >>>>> d96c68146a781c79a23f5181d7050174f1070d90, largely related to
> >>>>> texturing (I
> >>>>> can
> >>>>> send you a complete list of regressions if you care, but due to the
> >>>>> large
> >>>>> number
> >>>>> of them I suspect it's something fairly simple).
> >>>>>
> >>>>> For example:
> >>>>> ES31-CTS.functional.texture.format.buffer.r32ui_npot
> >>>>>
> >>>>> glGetIntegerv() failed: glGetError() returned GL_INVALID_ENUM at
> >>>>> gluContextInfo.cpp:229
> >>>>>
> >>>>> dEQP-GLES31.functional.state_query.integer.texture_buffer_
> >>>>> binding_getinteger:
> >>>>>
> >>>>> glGetIntegerv(GL_TEXTURE_BUFFER_BINDING, 0x00007ffee0c43834);
> >>>>> // data = { -555819298 }
> >>>>> glGetError();
> >>>>> // GL_INVALID_ENUM returned
> >>>>> // ERROR: glGetIntegerv: glGetError() returned GL_INVALID_ENUM
> >>>>>
> >>>>> Dylan
> >>>>>
> >>>>> Quoting Emil Velikov (2017-10-24 09:30:03)
> >>>>> > Hi Marek,
> >>>>> >
> >>>>> > On 21 October 2017 at 13:54, Marek Olšák <mar...@gmail.com> wrote:
> >>>>> > > From: Marek Olšák <marek.ol...@amd.com>
> >>>>> > >
> >>>>> > > We already have piglit tests testing alpha, luminance, and
> >>>>> intensity
> >>>>> > > formats. They were skipped by piglit until now.
> >>>>> > >
> >>>>> > > Additionally, I'm enabling one ARB_texture_buffer_range piglit
> >>>>> test to
> >>>>> run
> >>>>> > > with the compat profile.
> >>>>> >
> >>>>> > Can you please mention that ARB_texture_buffer_* on i965 is
> >>>>> unchanged
> >>>>> > - aka still enabled only for core profiles.
> >>>>> > Out of curiosity - can you tried the series with anything more
> >>>>> than
> >>>>> piglit?
> >>>>> >
> >>>>> > The Intel guys can run the lot through CTS, dEQP... admittedly
> >>>>> only on
> >>>>> > Intel hardware.
> >>>>> > Still it should help catch if a piece is missing somewhere.
> >>>>> >
> >>>>> >
> >>>>> > -Emil
> >>>>> > _______________________________________________
> >>>>> > mesa-dev mailing list
> >>>>> > mesa-dev@lists.freedesktop.org
> >>>>> > https://lists.freedesktop.org/mailman/listinfo/mesa-dev
> >>>>>
> >>>>>
signature.asc
Description: signature
_______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev
