On 2019/9/10 下午5:07, Anand Jain wrote:
>
>
>
>
>> diff --git a/fs/btrfs/tree-checker.c b/fs/btrfs/tree-checker.c
>> index ccd5706199d7..15d1aa7cef1f 100644
>> --- a/fs/btrfs/tree-checker.c
>> +++ b/fs/btrfs/tree-checker.c
>> @@ -686,9 +686,7 @@ static void dev_item_err(const struct
>> extent_buffer *eb, int slot,
>> static int check_dev_item(struct extent_buffer *leaf,
>> struct btrfs_key *key, int slot)
>> {
>> - struct btrfs_fs_info *fs_info = leaf->fs_info;
>> struct btrfs_dev_item *ditem;
>> - u64 max_devid = max(BTRFS_MAX_DEVS(fs_info),
>> BTRFS_MAX_DEVS_SYS_CHUNK);
>
> As I commented in v2.
> I see that BTRFS_MAX_DEVS_SYS_CHUNK is not being used anywhere
> else after this being removed. So good to delete the define.
> I am bit surprised as well if I am missing?
Please check the first patch.
It adds back the reference to it as an early exit for btrfs_alloc_chunk().
Thanks,
Qu
>
> Thanks, Anand
>
