On Wed, Mar 17, 2021 at 8:18 AM Thomas Munro <thomas.mu...@gmail.com> wrote: > > On Wed, Mar 17, 2021 at 6:58 PM Thomas Munro <thomas.mu...@gmail.com> wrote: > > According to BF animal elver there is something wrong with this > > commit. Looking into it. > > Assertion failure reproduced here and understood, but unfortunately > it'll take some more time to fix this. I've reverted the commit for > now to unbreak the ~5 machines that are currently showing red in the > build farm.
Also, silly question: why couldn't we just set the pstate->batches pointer to InvalidDsaPointer before doing dsa_free() (of course saving the pointer so that we can actually do the freeing with it)? Is there still a race?