@@ -121,12 +121,18 @@ struct btrfs_fs_devices {
/* the device with this id has the most recent copy of the super */
u64 latest_devid;
u64 latest_trans;
+ /* num_devices contains run time count of devices which are part
+ * of this FSID. (the FS devices + replacing devices + ?
+ */
u64 num_devices;
u64 open_devices;
u64 rw_devices;
u64 missing_devices;
u64 total_rw_bytes;
u64 num_can_discard;
+ /* total_devices contains static recorded count of device which
+ * are part of this FSID. (the FS devices + seed devices + ?
+ */
u64 total_devices;
struct block_device *latest_bdev;
The intended usage of total_devices and num_devices should be recorded in the comments so that these two counters can be used correctly to match the original intentions. And there appears to be bugs due to missing clarity, the bugs are mainly in the fact that in the current code the num_devices does not count seed devices where as total_devices does, next, total_devices does not count the replacing devices where as num_devices does. In this situation the ioctl(BTRFS_IOC_DEVICES_READY) will fails when there is seed FS. The applications using ioctl(BTRFS_IOC_FS_INFO) and ioctl(BTRFS_IOC_DEV_INFO) in conjuction to allocate the slots will also be confused as farmer depend on the num_devices where as later would match total_devices (except when replace is running). This patch will help have the clarity on usage of these two counter so that bugs related to this can be fixed. Signed-off-by: Anand Jain <Anand.Jain@oracle.com> --- fs/btrfs/volumes.h | 6 ++++++ 1 file changed, 6 insertions(+)