Re: [PATCH] libdw: add thread-safety to dwarf_getabbrev()

[Mark Wielaard <mark@klomp.org>]

Hi Jonathan,

On Mon, 2019-08-26 at 22:52 -0500, Jonathon Anderson wrote:
> Just finished some modifications to the patch series, git request-
> pull output below. This rebases onto the latest master and does a
> little diff cleaning, the major change is that I swapped out the
> memory management to use the pthread_key_* alternative mentioned
> before. I did some performance testing and found it to be notably
> faster in a high-stress microbenchmark (and no measurable difference
> in an application), and it makes the result easier to read overall.
> I've also removed the Valgrind configuration options patch since it
> doesn't add anything at the moment, if it becomes useful I'll submit
> it separately.

Thanks for the pull request. Since I find it nice to also see all
patches on the mailinglist, I am reposting them for easier review.
As replies to this message.

> The following changes since commit 
> 4bcc641d362de4236ae8f0f5bc933c6d84b6f105:
> 
>   libdw: fix latent bug in dwarf_getcfi.c not setting 
> default_same_value. (2019-08-26 15:15:34 +0200)
> 
> are available in the Git repository at:
> 
>   https://github.com/blue42u/elfutils.git parallel-pr-v2
> 
> for you to fetch changes up to
> 1191d9ed292b508d732973a318a01051053e0f61:
> 
>   lib + libdw: Add and use a concurrent version of the dynamic-size 
> hash table. (2019-08-26 22:29:45 -0500)
> 
> ----------------------------------------------------------------
> Jonathon Anderson (2):
>       Add some supporting framework for C11-style atomics.
>       libdw: Rewrite the memory handler to be thread-safe.
> 
> Srđan Milaković (1):
>       lib + libdw: Add and use a concurrent version of the dynamic-
> size hash table.

[PATCH 1/3] Add some supporting framework for C11-style atomics.
[PATCH 2/3] libdw: Rewrite the memory handler to be thread-safe.
[PATCH 3/3] lib + libdw: Add and use a concurrent version of the

Cheers,

Mark