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. -- dm-devel mailing list [email protected] https://listman.redhat.com/mailman/listinfo/dm-devel
