[Mesa-dev] [PATCH] Make single-buffered GLES representation internally consistent
chad.versace at intel.com
Fri Jul 1 22:42:08 UTC 2016
On Thu 30 Jun 2016, Stéphane Marchesin wrote:
> On Thu, Jun 30, 2016 at 3:20 PM, Gurchetan Singh
> <gurchetansingh at chromium.org> wrote:
> > There are a few places in the code where clearing and reading are done on
> > incorrect buffers for GLES contexts. See comments for details. This
> > fixes 75 GLES3 dEQP tests on the surfaceless platform with no regressions.
> > v2: Corrected unclear comment
> > v3: Make the change in context.c instead of get.c
> > v4: Removed whitespace
> I looked for a better way than initializing from makecurrent, but
> there doesn't seem to be one, so...
> Reviewed-by: <Stéphane Marchesin marcheu at chromium.org>
This looks like a difficult problem to fix. I also looked for a better
way to fix the problem, but I gave up after concluding that any
thoroughly correct fix seemed to require endless yak shaving.
If this fixes dEQP regressions, without regressing other things, then
I approve the patch. But, I'm not convinced yet that it doesn't regress
X11 pixmaps. I want to run some tests and step through the code with gdb
before I give a reviewed-by.
More information about the mesa-dev