On Tue, Nov 27, 2018 at 04:50:57PM +0800, Qu Wenruo wrote:
> >> -static int fixup_devices(struct btrfs_fs_info *fs_info,
> >> - struct mdrestore_struct *mdres, off_t dev_size)
> >> +static int fixup_device_size(struct btrfs_trans_handle *trans,
> >> + struct mdrestore_struct *mdres,
> >> + off_t dev_size)
> >> {
> >> - struct btrfs_trans_handle *trans;
> >> + struct btrfs_fs_info *fs_info = trans->fs_info;
> >> struct btrfs_dev_item *dev_item;
> >> struct btrfs_path path;
> >> - struct extent_buffer *leaf;
> >> struct btrfs_root *root = fs_info->chunk_root;
> >> struct btrfs_key key;
> >> + struct extent_buffer *leaf;
> >
> > nit: Unnecessary change
>
> Doesn't it look better when all btrfs_ prefix get batched together? :)
Please don't do unrelated changes like that.