<div dir="ltr"><div class="gmail_quote"><div class="gmail_attr">Do you have a branch with patch 7/8 and 8/8? I'm interested in EGL_EXT_platform_device on radeonsi.</div><div class="gmail_attr"><br></div><div class="gmail_attr">Thanks,<br></div><div class="gmail_attr">Marek<br></div><div dir="ltr" class="gmail_attr"><br></div><div dir="ltr" class="gmail_attr">On Wed, Oct 3, 2018 at 4:36 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">Hi Emil,<br>
<br>
Ok, thanks for picking that up.<br>
<br>
On Tuesday, 2 October 2018 12:23:30 CEST Emil Velikov wrote:<br>
> On Thu, 20 Sep 2018 at 15:13, Mathias Fröhlich<br>
> <<a href="mailto:Mathias.Froehlich@gmx.net" target="_blank">Mathias.Froehlich@gmx.net</a>> wrote:<br>
> <br>
> ><br>
> > If I replace the above with<br>
> ><br>
> >          EGLint surface_type = 0;<br>
> >          /* Only advertise pbuffer configs for non swrast devices */<br>
> >          if (dri2_dpy->image_driver)<br>
> >             surface_type = EGL_PBUFFER_BIT;<br>
> ><br>
> >          dri2_conf = dri2_add_config(dpy, dri2_dpy->driver_configs[i],<br>
> >                config_count + 1, surface_type, NULL,<br>
> >                visuals[j].rgba_masks);<br>
> ><br>
> > then I can easily prohibit the crash that I mentioned when trying to<br>
> > create a pbuffer surface on the swrast device.<br>
> > At least I do no longer get a valid pbuffer config from eglChooseConfig<br>
> > and without that we cannot reach the crashing<br>
> > dri2_dpy->image_driver->createNewDrawable<br>
> > call somewhere from eglCreatePbufferSurface anymore.<br>
> ><br>
> > Still getting a surface less context on swrast should work...<br>
> ><br>
> Issue is that we do not know how to create a "pbuffer only" swrast.<br>
> <br>
> Hence one resolves to hacks like the ones we have in platform_surfaceless.<br>
> Effectively pilling hacks upon hacks - see swrast_loader_extensions<br>
> and "software path w/o DRM.".<br>
> <br>
> That said, I'm working on proper solution but since it will take some<br>
> time to finish/merge I'll drop this and 7/8 for now.<br>
<br>
<br>
That's fine too.<br>
<br>
What I wanted to avoid is that people using the extension correctly fail with a<br>
crash in the application when running on mesa.<br>
Think about it. Thats really bad from an application writers point of view<br>
as you do all right with checking extensions and that and then the opengl<br>
stack suddenly crashes.<br>
<br>
best<br>
<br>
Mathias<br>
<br>
<br>
_______________________________________________<br>
mesa-dev mailing list<br>
<a href="mailto:mesa-dev@lists.freedesktop.org" target="_blank">mesa-dev@lists.freedesktop.org</a><br>
<a href="https://lists.freedesktop.org/mailman/listinfo/mesa-dev" rel="noreferrer" target="_blank">https://lists.freedesktop.org/mailman/listinfo/mesa-dev</a><br>
</blockquote></div></div>