@@ -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;
(Ignore previous, this is not a final patch, but to seek comments). Thanks. 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 conjunction to allocate the slots will also be confused as former depend on the num_devices where as latter 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(+) -- 1.8.5.3 -- To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html