On 16/08/16 04:11 PM, Tapani Pälli wrote: > On 08/16/2016 09:49 AM, Michel Dänzer wrote: >> From: Michel Dänzer <[email protected]> >> >> There's no guarantee that there is one, and we don't need one anyway. >> >> Fixes piglit tests: >> >> glx@glx-fbconfig-bad > > this one passes for me with and without the patch
Does your Mesa build have assertions enabled? (May require --enable-debug) >> glx@glx_ext_import_context@import context, multi process >> glx@glx_ext_import_context@import context, single process > > these 2 fail for me with and without the patch (and have been failing > already before 2e3f067458e4) ... do these all pass for you before that > change? Yes, they did. (Using the radeonsi driver with DRI3) Anyway, forgetting about piglit for a second, can you agree that the code change makes sense per se? :) -- Earthling Michel Dänzer | http://www.amd.com Libre software enthusiast | Mesa and X developer _______________________________________________ mesa-dev mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/mesa-dev
