Re: Question about e8569813849b5da394a195c7e76b4faa452b12d1

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



2008/10/9 Josef Bacik <jbacik@xxxxxxxxxx>:
> Hello,
>
> Is there a reason
>
> -       if (search_end == (u64)-1)
> -               search_end = btrfs_super_total_bytes(&info->super_copy);
>
> happened in the commit from $SUBJECT?  If its just a mistake thats fine I'll add
> it back with my current patch, but if there is a reason it needed to go then I
> need to know that too so I can figure out a different way to do what I want to
> do.  Thanks,
>

btrfs_super_total_bytes is total size of all devices. we can't rely on it as
search_end for two reasons:

1) If metadata mirror is enable, the value can be larger than the last byte of
    last chunk.

2) For a FS has been balanced, there are some 'holes' in the chunk address
    space, so the value can be smaller than the last byte of last chunk.

Regards
Yan Zheng
--
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

[Index of Archives]     [Linux Filesystem Development]     [Linux NFS]     [Linux NILFS]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux