dm raid: be prepared to accept arbitrary '- -' tuples

During raid set resize checks and setting up the recovery offset in case a raid
set grows, calculated rd->md.dev_sectors is compared to rs->dev[0].rdev.sectors.

Device 0 may not be defined in case userspace passes in '- -' for it
(lvm2 doesn't do that so far), thus it's device sectors can't be taken
authoritatively in this comparison and another valid device must be used
to retrieve the device size.

Use mddev->dev_sectors in checking for ongoing recovery for the same reason.

Signed-off-by: Heinz Mauelshagen <heinzm@redhat.com>
Signed-off-by: Mike Snitzer <snitzer@redhat.com>

diff --git a/drivers/md/dm-raid.c b/drivers/md/dm-raid.c
index b40a088a2d92cb25e5e466a41204f51ab117ee66..d7e652a22a66d1a609c97f1a7b3614227cb10d98 100644 (file)
--- a/drivers/md/dm-raid.c
+++ b/drivers/md/dm-raid.c
@@ -370,7 +370,7 @@ static bool rs_is_reshapable(struct raid_set *rs)
 /* Return true, if raid set in @rs is recovering */
 static bool rs_is_recovering(struct raid_set *rs)
 {
-       return rs->md.recovery_cp < rs->dev[0].rdev.sectors;
+       return rs->md.recovery_cp < rs->md.dev_sectors;
 }
 
 /* Return true, if raid set in @rs is reshaping */
@@ -1425,6 +1425,24 @@ static unsigned int rs_data_stripes(struct raid_set *rs)
        return rs->raid_disks - rs->raid_type->parity_devs;
 }
 
+/*
+ * Retrieve rdev->sectors from any valid raid device of @rs
+ * to allow userpace to pass in arbitray "- -" device tupples.
+ */
+static sector_t __rdev_sectors(struct raid_set *rs)
+{
+       int i;
+
+       for (i = 0; i < rs->md.raid_disks; i++) {
+               struct md_rdev *rdev = &rs->dev[i].rdev;
+
+               if (rdev->bdev && rdev->sectors)
+                       return rdev->sectors;
+       }
+
+       BUG(); /* Constructor ensures we got some. */
+}
+
 /* Calculate the sectors per device and per array used for @rs */
 static int rs_set_dev_and_array_sectors(struct raid_set *rs, bool use_mddev)
 {
@@ -1510,9 +1528,9 @@ static void rs_setup_recovery(struct raid_set *rs, sector_t dev_sectors)
        else if (dev_sectors == MaxSector)
                /* Prevent recovery */
                __rs_setup_recovery(rs, MaxSector);
-       else if (rs->dev[0].rdev.sectors < dev_sectors)
+       else if (__rdev_sectors(rs) < dev_sectors)
                /* Grown raid set */
-               __rs_setup_recovery(rs, rs->dev[0].rdev.sectors);
+               __rs_setup_recovery(rs, __rdev_sectors(rs));
        else
                __rs_setup_recovery(rs, MaxSector);
 }
@@ -2828,7 +2846,7 @@ static int raid_ctr(struct dm_target *ti, unsigned int argc, char **argv)
        if (r)
                goto bad;
 
-       calculated_dev_sectors = rs->dev[0].rdev.sectors;
+       calculated_dev_sectors = rs->md.dev_sectors;
 
        /*
         * Backup any new raid set level, layout, ...
@@ -2841,7 +2859,7 @@ static int raid_ctr(struct dm_target *ti, unsigned int argc, char **argv)
        if (r)
                goto bad;
 
-       resize = calculated_dev_sectors != rs->dev[0].rdev.sectors;
+       resize = calculated_dev_sectors != __rdev_sectors(rs);
 
        INIT_WORK(&rs->md.event_work, do_table_event);
        ti->private = rs;