On Tue, May 05, 2020 at 07:58:23AM +0800, Qu Wenruo wrote: > The name of this function contains the word "cache", which is left from > the era where btrfs_block_group is called btrfs_block_group_cache. > > Now this "cache" doesn't match any thing, and we have better namings for > functions like read/insert/remove_block_group_item(). > > So rename this function to update_block_group_item(). > > Since we're here, also rename the local variables to be more like a > Chrismas tree, and rename @extent_root to @root for later reuse. Please don't do that. The reverse chrismas tree is Somebody Else's Coding Style and you introduce unrelated changes where only function rename is expected. All reverted.
