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
