<div dir="ltr">I mostly ran other tests suites. We actually don't run dEQP on VMs since our current swrast-based implementation would segfault. We would need a patch like: <div><br></div><div><a href="https://lists.freedesktop.org/archives/mesa-dev/2017-August/164922.html">https://lists.freedesktop.org/archives/mesa-dev/2017-August/164922.html</a></div><div><br></div><div>for it to work. However, I was able to verify dEQP successfully runs with this patchset, though I just ran a few tests. I expect the full suite to be up and running soon ... </div></div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Oct 3, 2017 at 10:04 AM, Eric Engestrom <span dir="ltr"><<a href="mailto:eric.engestrom@imgtec.com" target="_blank">eric.engestrom@imgtec.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Tuesday, 2017-10-03 16:30:18 +0000, Gurchetan Singh wrote:<br>
> Hi Eric,<br>
><br>
> Yes, you pushing this series would be great.<br>
<br>
</span>Done; since I was editing it anyway, I also moved the _eglLog() string<br>
to avoid breaking it (makes grepping easier).<br>
<span class=""><br>
> I did run our test suite on this patchset as well.<br>
<br>
</span>You mean deqp, or another one?<br>
I assume the results were "100% success"?<br>
<div class="HOEnZb"><div class="h5"><br>
><br>
> Best wishes,<br>
> Gurchetan<br>
><br>
> On Tue, Oct 3, 2017 at 7:02 AM, Eric Engestrom <<a href="mailto:eric.engestrom@imgtec.com">eric.engestrom@imgtec.com</a>><br>
> wrote:<br>
><br>
> > On Monday, 2017-10-02 20:48:24 +0000, <a href="mailto:gurchetansingh@chromium.org">gurchetansingh@chromium.org</a> wrote:<br>
> > > From: Gurchetan Singh <<a href="mailto:gurchetansingh@chromium.org">gurchetansingh@chromium.org</a>><br>
> > ><br>
> > > The kms_swrast extension is an actively developed software fallback,<br>
> > > and platform_surfaceless can use it if there are no available<br>
> > > hardware drivers.<br>
> > ><br>
> > > v2: Split into 2 patches, use booleans, check LIBGL_ALWAYS_SOFTWARE,<br>
> > >     and modify the eglLog level (Emil, Eric, Tomasz).<br>
> ><br>
> > Thanks, this is perfect :)<br>
> ><br>
> > Reviewed-by: Eric Engestrom <<a href="mailto:eric.engestrom@imgtec.com">eric.engestrom@imgtec.com</a>><br>
> ><br>
> > Do you want me to push this for you?<br>
> ><br>
> > It might be worth running deqp and other test suites to make sure<br>
> > everything is wired up correctly.<br>
> > (Given your chromium affiliation, I'm assuming you already know how to<br>
> > use the surfaceless patches for deqp [1])<br>
> ><br>
> > [1] <a href="https://chromium.googlesource.com/chromiumos/overlays/" rel="noreferrer" target="_blank">https://chromium.googlesource.<wbr>com/chromiumos/overlays/</a><br>
> > chromiumos-overlay/+/master/<wbr>media-gfx/deqp<br>
> ><br>
> > > ---<br>
> > >  src/egl/drivers/dri2/platform_<wbr>surfaceless.c | 19 +++++++++++++++----<br>
> > >  1 file changed, 15 insertions(+), 4 deletions(-)<br>
> > ><br>
> > > diff --git a/src/egl/drivers/dri2/<wbr>platform_surfaceless.c<br>
> > b/src/egl/drivers/dri2/<wbr>platform_surfaceless.c<br>
> > > index f6aa217d39..b0a43ac9de 100644<br>
> > > --- a/src/egl/drivers/dri2/<wbr>platform_surfaceless.c<br>
> > > +++ b/src/egl/drivers/dri2/<wbr>platform_surfaceless.c<br>
> > [snip]<br>
> > > @@ -320,7 +324,14 @@ dri2_initialize_surfaceless(_<wbr>EGLDriver *drv,<br>
> > _EGLDisplay *disp)<br>
> > ><br>
> > >     dri2_dpy->fd = -1;<br>
> > >     disp->DriverData = (void *) dri2_dpy;<br>
> ><br>
> > Nit: newline here<br>
> > (I'll add it if/when I commit it, don't send a v3 for that)<br>
> ><br>
> > > -   if (!surfaceless_probe_device(<wbr>disp)) {<br>
> > > +   if (!env_var_as_boolean("LIBGL_<wbr>ALWAYS_SOFTWARE", false)) {<br>
> > > +      driver_loaded = surfaceless_probe_device(disp, false);<br>
> > > +      if (!driver_loaded)<br>
> > > +         _eglLog(_EGL_WARNING, "No hardware driver found, falling back<br>
> > to "<br>
> > > +              "software rendering");<br>
> > > +   }<br>
> > > +<br>
> > > +   if (!driver_loaded && !surfaceless_probe_device(<wbr>disp, true)) {<br>
> > >        err = "DRI2: failed to load driver";<br>
> > >        goto cleanup;<br>
> > >     }<br>
> > > --<br>
> > > 2.13.5<br>
> > ><br>
> ><br>
</div></div></blockquote></div><br></div>