On Thu, Feb 10, 2022 at 05:38:27PM -0500, Mike Snitzer wrote: > Rename dm_io struct's 'endio_lock' member to 'lock' to reuse spinlock > to protect new member used to flag if IO accounting has been started. > > Also move kicking of the suspend queue out to dm_io_dec_pending (the > only caller) since end_io_acct will soon only be called if IO > accounting was started. > > Some comment tweaks and removal of local variables. No functional > change. The changes looks fine to me, but for the reader it would be much nicer if the lock rename, kicking changes and local variable tweaks were split into separate patches.