tree 3e767e1a0ab1fb550159a970e6c10206b131e8ba
parent 65d6f258c5242ac26e17fc803818d76e62c40b83
author Erik Faye-Lund <erik.faye-lund@collabora.com> 1603383667 +0200
committer Marge Bot <eric+marge@anholt.net> 1603715721 +0000

mapi: do not call thread-unsafe dispatch getter

When not using the USE_ELF_TLS code-path, this function is
thread-unsafe, because it returns u_current_table if set without
consulting the ThreadSafe variable in u_current.c.

There's a short period where this can cause problems, if a program uses
multiple threads, but only have made a single context current so far. If
the program issues OpenGL commands from the initialized thread while a
new thread is setting u_current_table to __glapi_noop_table, we will
return the wrong table here.

It doesn't seem right to have two versions of the code that does the
same anyway, so let's use the version that doesn't have this problem
instead.

Reviewed-by: Ian Romanick <ian.d.romanick@intel.com>
Reviewed-by: Louis-Francis Ratté-Boulianne <lfrb@collabora.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/7280>