Re: [PATCH] lib + libdw: Add and use a concurrent version of the dynamic-size hash table.

2019-11-08 Thread Mark Wielaard
Hi, On Thu, 2019-11-07 at 09:24 -0600, Jonathon Anderson wrote: > On Thu, Nov 7, 2019 at 12:07, Mark Wielaard wrote: > > Looking at the difference between the previous version and this one, > > it > > incorporates my simplification of FIND and lookup functions. And fixes > > it by making insert_

Re: [PATCH] lib + libdw: Add and use a concurrent version of the dynamic-size hash table.

2019-11-08 Thread Jonathon Anderson
On Fri, Nov 8, 2019 at 15:07, Mark Wielaard wrote: Hi, On Thu, 2019-11-07 at 09:24 -0600, Jonathon Anderson wrote: On Thu, Nov 7, 2019 at 12:07, Mark Wielaard > wrote: > Looking at the difference between the previous version and this one, > it > incorporates my s

Re: [PATCH 2/2] libdw: Rewrite the memory handler to be more robust.

2019-11-08 Thread Mark Wielaard
On Thu, 2019-11-07 at 12:40 -0600, Jonathon Anderson wrote: > I haven't benchmarked this version, but I did benchmark the equivalent > earlier version (this version is almost quite literally a rebase of the > other). I don't have the exact results on hand, what I remember is that > the pthread_k

Re: [PATCH 2/2] libdw: Rewrite the memory handler to be more robust.

2019-11-08 Thread Mark Wielaard
Hi, On Thu, 2019-11-07 at 12:13 -0600, Jonathon Anderson wrote: > On Thu, Nov 7, 2019 at 18:20, Mark Wielaard wrote: > > Do we really need this? > > We already use __thread unconditionally in the rest of the code. > > The usage of threads.h seems to imply we actually want C11 > > _Thread_local. I

Re: [PATCH 2/2] libdw: Rewrite the memory handler to be more robust.

2019-11-08 Thread Jonathon Anderson
On Fri, Nov 8, 2019 at 17:22, Mark Wielaard wrote: On Thu, 2019-11-07 at 12:40 -0600, Jonathon Anderson wrote: I haven't benchmarked this version, but I did benchmark the equivalent earlier version (this version is almost quite literally a rebase of the other). I don't have the exact res