lkml.org 
[lkml]   [2015]   [Sep]   [26]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH 4.1 087/159] md/raid10: always set reshape_safe when initializing reshape_position.
    Date
    4.1-stable review patch.  If anyone has any objections, please let me know.

    ------------------

    From: NeilBrown <neilb@suse.com>

    commit 299b0685e31c9f3dcc2d58ee3beca761a40b44b3 upstream.

    'reshape_position' tracks where in the reshape we have reached.
    'reshape_safe' tracks where in the reshape we have safely recorded
    in the metadata.

    These are compared to determine when to update the metadata.
    So it is important that reshape_safe is initialised properly.
    Currently it isn't. When starting a reshape from the beginning
    it usually has the correct value by luck. But when reducing the
    number of devices in a RAID10, it has the wrong value and this leads
    to the metadata not being updated correctly.
    This can lead to corruption if the reshape is not allowed to complete.

    This patch is suitable for any -stable kernel which supports RAID10
    reshape, which is 3.5 and later.

    Fixes: 3ea7daa5d7fd ("md/raid10: add reshape support")
    Signed-off-by: NeilBrown <neilb@suse.com>
    Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

    ---
    drivers/md/raid10.c | 5 ++++-
    1 file changed, 4 insertions(+), 1 deletion(-)

    --- a/drivers/md/raid10.c
    +++ b/drivers/md/raid10.c
    @@ -3566,6 +3566,7 @@ static struct r10conf *setup_conf(struct
    /* far_copies must be 1 */
    conf->prev.stride = conf->dev_sectors;
    }
    + conf->reshape_safe = conf->reshape_progress;
    spin_lock_init(&conf->device_lock);
    INIT_LIST_HEAD(&conf->retry_list);

    @@ -3770,7 +3771,6 @@ static int run(struct mddev *mddev)
    }
    conf->offset_diff = min_offset_diff;

    - conf->reshape_safe = conf->reshape_progress;
    clear_bit(MD_RECOVERY_SYNC, &mddev->recovery);
    clear_bit(MD_RECOVERY_CHECK, &mddev->recovery);
    set_bit(MD_RECOVERY_RESHAPE, &mddev->recovery);
    @@ -4113,6 +4113,7 @@ static int raid10_start_reshape(struct m
    conf->reshape_progress = size;
    } else
    conf->reshape_progress = 0;
    + conf->reshape_safe = conf->reshape_progress;
    spin_unlock_irq(&conf->device_lock);

    if (mddev->delta_disks && mddev->bitmap) {
    @@ -4180,6 +4181,7 @@ abort:
    rdev->new_data_offset = rdev->data_offset;
    smp_wmb();
    conf->reshape_progress = MaxSector;
    + conf->reshape_safe = MaxSector;
    mddev->reshape_position = MaxSector;
    spin_unlock_irq(&conf->device_lock);
    return ret;
    @@ -4534,6 +4536,7 @@ static void end_reshape(struct r10conf *
    md_finish_reshape(conf->mddev);
    smp_wmb();
    conf->reshape_progress = MaxSector;
    + conf->reshape_safe = MaxSector;
    spin_unlock_irq(&conf->device_lock);

    /* read-ahead size must cover two whole stripes, which is



    \
     
     \ /
      Last update: 2015-09-27 00:01    [W:4.072 / U:0.008 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site