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.

Reply via email to