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

Mark Wielaard mark@klomp.org
Tue Nov 12 21:45:00 GMT 2019


Hi Jonathon,

On Mon, 2019-11-11 at 17:38 -0600, Jonathon Anderson wrote:
> > Since the memblocks are per thread, it seems we could easily back out
> > an allocation we don't need as long as the thread hasn't done any 
> > other
> > allocation from the memblock. What do you think of the attached patch?
> 
> Looks good to me.
> 
> > It is a bit hard to test without a massively parallel testcase where
> > things collide a lot. Since you have one, could you try this out?
> 
> Everything seems to work fine on this end.
> 
> (Plus, if you replace the `Abbrev_find == NULL` with `1`, you can make 
> everything collide. Which I did too, and it still works fine.)

O nice trick. Just build and ran a distcheck with this to be sure
things still work.

Thanks for your review and testing. Pushed to master.

Cheers,

Mark



More information about the Elfutils-devel mailing list