[Mesa-dev] [PATCH] swrast: remove unneeded _glapi_check_multithread() call

[Emil Velikov]

On 18 July 2018 at 19:34, Eric Anholt <eric at anholt.net> wrote:
> Emil Velikov <emil.l.velikov at gmail.com> writes:
>
>> From: Emil Velikov <emil.velikov at collabora.com>
>>
>> With commit c6c0f947142, back in 2006 Brian removed the
>> _glapi_check_multithread() call from core mesa - _mesa_make_current.
>>
>> He moved it to a DRI loader (type of) decision - with equivalents in
>> the monolithic OSMesa and libGL-Xlib.
>>
>> Some time afterwords, swrast (a DRI driver) was introduced as a copy
>> paste of the latter two. Remove the call since the DRI loader handles
>> that for us.
>>
>> Signed-off-by: Emil Velikov <emil.velikov at collabora.com>
>
> Does anything make sure it gets called for classic drivers under EGL?
>
Good call - I cannot spot anything. Which means that all classic
drivers, but swrast is foobar'd.
Gallium itself calls _glapi_check_multithread so those drivers are safe.

> Actually, wouldn't it make sense to move this back to the core, at this
> point?
Since glapi is fiddly, I've opted for the least invasive patch. Moving
the call to core would be better, indeed.

-Emil