On Sun, Oct 01, 2017 at 04:56:01PM +0200, Martin Ågren wrote: > A recent series allowed `struct lock_file`s to be freed [1], so I wanted > to get rid of the "simple" leaks of this kind. I found a couple of > lock-related cleanups along the way and it resulted in this series. It > feels a bit scary at eleven patches -- especially as this is about > locking -- but I hope I've managed to put this into understandable and > reviewable patches. Reviews, thoughts, opinions welcome, as always.
Thanks for working on this (and cleaning up several messes that I left). I've read through each and with the exception of patches 10 and 11, they all look good to me. For 10 and 11, I think you've convinced me that the current behavior is buggy. I do wonder if the subtleties might be easier to understand as a single patch, but I'm OK either way. -Peff
