drbd: Enforce limits of disk_conf members; centralized these checks

Signed-off-by: Philipp Reisner <philipp.reisner@linbit.com>
Signed-off-by: Lars Ellenberg <lars.ellenberg@linbit.com>

diff --git a/drivers/block/drbd/drbd_nl.c b/drivers/block/drbd/drbd_nl.c
index 7fda3d9445ca96a042d09e49cf8e1c18eb2eae9d..812e91f1b6d43bb3b7316b06f3523c57c5499d1e 100644 (file)
--- a/drivers/block/drbd/drbd_nl.c
+++ b/drivers/block/drbd/drbd_nl.c
@@ -937,9 +937,6 @@ static int drbd_check_al_size(struct drbd_conf *mdev, struct disk_conf *dc)
        unsigned int in_use;
        int i;
 
-       if (!expect(dc->al_extents >= DRBD_AL_EXTENTS_MIN))
-               dc->al_extents = DRBD_AL_EXTENTS_MIN;
-
        if (mdev->act_log &&
            mdev->act_log->nr_elements == dc->al_extents)
                return 0;
@@ -1102,6 +1099,17 @@ static bool should_set_defaults(struct genl_info *info)
        return 0 != (flags & DRBD_GENL_F_SET_DEFAULTS);
 }
 
+static void enforce_disk_conf_limits(struct disk_conf *dc)
+{
+       if (dc->al_extents < DRBD_AL_EXTENTS_MIN)
+               dc->al_extents = DRBD_AL_EXTENTS_MIN;
+       if (dc->al_extents > DRBD_AL_EXTENTS_MAX)
+               dc->al_extents = DRBD_AL_EXTENTS_MAX;
+
+       if (dc->c_plan_ahead > DRBD_C_PLAN_AHEAD_MAX)
+               dc->c_plan_ahead = DRBD_C_PLAN_AHEAD_MAX;
+}
+
 int drbd_adm_disk_opts(struct sk_buff *skb, struct genl_info *info)
 {
        enum drbd_ret_code retcode;
@@ -1146,11 +1154,7 @@ int drbd_adm_disk_opts(struct sk_buff *skb, struct genl_info *info)
        if (!expect(new_disk_conf->resync_rate >= 1))
                new_disk_conf->resync_rate = 1;
 
-       /* clip to allowed range */
-       if (!expect(new_disk_conf->al_extents >= DRBD_AL_EXTENTS_MIN))
-               new_disk_conf->al_extents = DRBD_AL_EXTENTS_MIN;
-       if (!expect(new_disk_conf->al_extents <= DRBD_AL_EXTENTS_MAX))
-               new_disk_conf->al_extents = DRBD_AL_EXTENTS_MAX;
+       enforce_disk_conf_limits(new_disk_conf);
 
        fifo_size = (new_disk_conf->c_plan_ahead * 10 * SLEEP_TIME) / HZ;
        if (fifo_size != mdev->rs_plan_s->size) {
@@ -1273,6 +1277,8 @@ int drbd_adm_attach(struct sk_buff *skb, struct genl_info *info)
                goto fail;
        }
 
+       enforce_disk_conf_limits(new_disk_conf);
+
        new_plan = fifo_alloc((new_disk_conf->c_plan_ahead * 10 * SLEEP_TIME) / HZ);
        if (!new_plan) {
                retcode = ERR_NOMEM;
@@ -1450,8 +1456,6 @@ int drbd_adm_attach(struct sk_buff *skb, struct genl_info *info)
        else
                clear_bit(MD_NO_FUA, &mdev->flags);
 
-       /* FIXME Missing stuff: clip al range */
-
        /* Point of no return reached.
         * Devices and memory are no longer released by error cleanup below.
         * now mdev takes over responsibility, and the state engine should