On Thu, Dec 06, 2018 at 06:10:47PM +0000, Robin Murphy wrote: > AFAICS the tmp list wasn't about locking as much as meaning that if > kzalloc() failed at any point, we can free the partial allocation and back > out without disturbing free_entries at all - that still makes sense to me > up until patch #8 where we embrace the "never free anything" paradigm and > rip out the final traces. > > That said, maybe I should just drop the refactoring of > dma_debug_resize_entries() now that I'm deleting it as part of the same > series anyway - then I guess I squash what's left of this patch into #4 and > bring forward some of the simplification from #8 to start with. Would that > be more agreeable?
Yes, I just noticed all this goes away toward the end anyway. We can either keep it as is, or just drop the intermediate step if that is easy enough for you.
