On Wed, May 16, 2018 at 01:52:02AM +0800, Liu Bo wrote: > These're the cleanups I made for btrfs_search_slot() and its callees. All the patches have very terse changelog and I have no idea why you think the code change is correct so I can compare my review against that. It's touching code around locking that's never bad to be more verbose and introduce a bit of the context, the simple sentence is maybe clear to you when you go through the code, but imagine you read the changelog in a week or month or a year. It still needs to make sense or give enough pointers to find out from the code. Please update and resend, thanks. -- To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
