<div dir="ltr"><div>I'd like to remove swrast from devices. It doesn't work (eglInitialize fails) and I don't think I like swrast there. Any objections?<br></div><div><br></div><div>Marek<br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Apr 17, 2019 at 12:38 AM Mathias Fröhlich <<a href="mailto:Mathias.Froehlich@gmx.net">Mathias.Froehlich@gmx.net</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><br>
Hi,<br>
<br>
On Tuesday, 16 April 2019 17:50:33 CEST Marek Olšák wrote:<br>
> On Wed, Apr 10, 2019 at 5:37 AM Mathias Fröhlich <<a href="mailto:Mathias.Froehlich@gmx.net" target="_blank">Mathias.Froehlich@gmx.net</a>><br>
> wrote:<br>
> <br>
> > Hi Emil,<br>
> ><br>
> > On Monday, 8 April 2019 12:28:55 CEST Emil Velikov wrote:<br>
> > > > Now that I have been putting together a test case for the the actual<br>
> > use<br>
> > > > I do see some issues with the pbuffer code path. Well - still<br>
> > investigating<br>
> > > > if the test is wrong or the result.<br>
> > > ><br>
> > > Actually I do recall some issues with surfaceless (which is<br>
> > > practically identical to this code) and some (with alpha?) formats.<br>
> > > My memory is pretty flaky - I would suggest trying various configs:<br>
> > > with, w/o alpha, 32/24 etc.<br>
> ><br>
> > Ok, yesterday evening I spent some time to look into that.<br>
> > Finally the egl config mechanism puts double buffer visuals<br>
> > to the pbuffer surface which in turn makes mesa think if should look<br>
> > at the back buffer which is in turn not set in the pbuffer gl_framebuffer.<br>
> > That one makes my drivers just throw away the rendering result so a<br>
> > read back does not show anything usable.<br>
> ><br>
> > With that fixed I get in principle correct results.<br>
> ><br>
> > My gitlab repo contains a branch that sketches my quick fix that I use to<br>
> > make<br>
> > at least the new piglit test case work. No further testing done.<br>
> > But may be that helps in getting something reasonable out.<br>
> ><br>
> <br>
> So we can push this series now, right?<br>
<br>
Have you been looking at that branch?<br>
<br>
Its a sketch at best, and under some circumstance it still crashes somewhere<br>
in the back and forth of virtual methods in the loader and driver.<br>
I think it's going the riht direction, but until that is fixed I don't want to have my<br>
name tag on that unfinished stuff.<br>
<br>
If you want to help out, you can take a look at the egl config change in<br>
that branch. One needs to check if that 'double buffering visual' filter there<br>
works as expected. Throw test cases at it. At least punch that code through<br>
piglit and whatever test suite you have access.<br>
<br>
best<br>
<br>
Mathias<br>
<br>
<br>
</blockquote></div></div>