elf: Fix slow tls access after dlopen [BZ #19924]
commitd2123d68275acc0f061e73d5f86ca504e0d5a344
authorSzabolcs Nagy <szabolcs.nagy@arm.com>
Tue, 16 Feb 2021 12:55:13 +0000 (16 12:55 +0000)
committerSzabolcs Nagy <szabolcs.nagy@arm.com>
Fri, 1 Sep 2023 07:21:37 +0000 (1 08:21 +0100)
tree76013866674b9b5d8fd3df9d1f16f9dd5f6dacdb
parent1493622f4f9048ffede3fbedb64695efa49d662a
elf: Fix slow tls access after dlopen [BZ #19924]

In short: __tls_get_addr checks the global generation counter and if
the current dtv is older then _dl_update_slotinfo updates dtv up to the
generation of the accessed module. So if the global generation is newer
than generation of the module then __tls_get_addr keeps hitting the
slow dtv update path. The dtv update path includes a number of checks
to see if any update is needed and this already causes measurable tls
access slow down after dlopen.

It may be possible to detect up-to-date dtv faster.  But if there are
many modules loaded (> TLS_SLOTINFO_SURPLUS) then this requires at
least walking the slotinfo list.

This patch tries to update the dtv to the global generation instead, so
after a dlopen the tls access slow path is only hit once.  The modules
with larger generation than the accessed one were not necessarily
synchronized before, so additional synchronization is needed.

This patch uses acquire/release synchronization when accessing the
generation counter.

Note: in the x86_64 version of dl-tls.c the generation is only loaded
once, since relaxed mo is not faster than acquire mo load.

I have not benchmarked this. Tested by Adhemerval Zanella on aarch64,
powerpc, sparc, x86 who reported that it fixes the performance issue
of bug 19924.

Reviewed-by: Adhemerval Zanella <adhemerval.zanella@linaro.org>
elf/dl-close.c
elf/dl-open.c
elf/dl-reloc.c
elf/dl-tls.c
sysdeps/generic/ldsodefs.h
sysdeps/x86_64/dl-tls.c