Emil, this patch has been stalled in the -stable ML for quite some time without update.
Unless you say otherwise, I will just ignore it at this point and trust that you will also Cc -stable in the future, in case you come with another version. On Wed, 2017-09-27 at 17:36 +0200, Juan A. Suarez Romero wrote: > On Wed, 2017-09-27 at 16:15 +0100, Emil Velikov wrote: > > On 26 September 2017 at 18:08, Juan A. Suarez Romero > > <jasua...@igalia.com> wrote: > > > On Wed, 2017-09-06 at 15:07 +0100, Emil Velikov wrote: > > > > On 5 August 2017 at 00:25, Emil Velikov <emil.l.veli...@gmail.com> > > > > wrote: > > > > > From: Emil Velikov <emil.veli...@collabora.com> > > > > > > > > > > As mentioned in previous commit the negative tests in dEQP expect the > > > > > arguments to be evaluated in particular order. > > > > > > > > > > Namely - first the dpy, then the config, followed by the > > > > > surface/window. > > > > > > > > > > Move the check further down or executing the test below will produce > > > > > the following error. > > > > > > > > > > dEQP-EGL.functional.negative_api.create_pbuffer_surface > > > > > > > > > > > > > > > <Section Name="Test2" Description="EGL_BAD_CONFIG is generated if > > > > > config is not an EGL frame buffer configuration"> > > > > > <Text>eglCreateWindowSurface(0x9bfff0f150, 0xffffffffffffffff, > > > > > 0x0000000000000000, { EGL_NONE });</Text> > > > > > <Text>// 0x0000000000000000 returned</Text> > > > > > <Text>// ERROR expected: EGL_BAD_CONFIG, Got: > > > > > EGL_BAD_NATIVE_WINDOW</Text> > > > > > </Section> > > > > > > > > > > > > > FTR dEQP has been "fixed" (by removing the test all together) to not > > > > generate the above error. At the same the Pixman equivalent is still > > > > buggy, hence the v2 of commit > > > > df8efd5b74d45e2bfb977a92dcd3db86abd6b143. > > > > > > > > > > Emil, does it mean this patch is superseded? I'm interesting in knowing > > > the situation as this was tagged to be included in stable release. > > > > > > > I'd like to keep this open and eventually merge it. > > The dEQP patches need 'a bit' of polishing. > > > > Sure. I'll skip it for next release. Thanks > > J.A. > > > -Emil > > _______________________________________________ > > mesa-stable mailing list > > mesa-sta...@lists.freedesktop.org > > https://lists.freedesktop.org/mailman/listinfo/mesa-stable > > _______________________________________________ > mesa-stable mailing list > mesa-sta...@lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/mesa-stable -- Br, Andres _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev