probably. The original changeset passes NULL(which is then unconditionally accessed) at least in:
src/gallium/auxiliary/vl/vl_winsys_dri.c src/gallium/auxiliary/vl/vl_winsys_dri3.c src/gallium/auxiliary/vl/vl_winsys_drm.c src/gallium/state_trackers/xa/xa_tracker.c src/gallium/targets/d3dadapter9/drm.c src/gallium/tests/trivial/compute.c src/gallium/tests/trivial/quad-tex.c src/gallium/tests/trivial/tri.c Not sure if the original idea was to handle NULL gracefully, but it looks like the change broke almost everything other than GL. Jan On Wed, 2017-08-02 at 21:28 +0100, Andy Furniss wrote: > Hmm, been AFK, but now have actually installed as well as just testing > build. > > Testing with this and the patch that avoids segfaults for cl. > > I am getting segfaults when using vdpau for video, so maybe that needs > something similar. > > Jan Vesely wrote: > > The patch still causes segfaults because you pass NULL at > > core/device.cpp:45 > > yet unconditionally access the pointer at pipe_loader.c:130 > > > > Jan > > > > On Wed, 2017-08-02 at 18:44 +0200, Nicolai Hähnle wrote: > > > Thanks! I'll take that as a Tested-by. > > > > > > On 02.08.2017 18:43, Andy Furniss wrote: > > > > Building OK with this.
signature.asc
Description: This is a digitally signed message part
_______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev
