Patch "block: don't set GD_NEED_PART_SCAN if scan partition failed" has been added to the 6.1-stable tree

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



This is a note to let you know that I've just added the patch titled

    block: don't set GD_NEED_PART_SCAN if scan partition failed

to the 6.1-stable tree which can be found at:
    http://www.kernel.org/git/?p=linux/kernel/git/stable/stable-queue.git;a=summary

The filename of the patch is:
     block-don-t-set-gd_need_part_scan-if-scan-partition-.patch
and it can be found in the queue-6.1 subdirectory.

If you, or anyone else, feels it should not be added to the stable tree,
please let <stable@xxxxxxxxxxxxxxx> know about it.



commit d20107fd9d71883dae753ff7c886177572624b91
Author: Yu Kuai <yukuai3@xxxxxxxxxx>
Date:   Wed Mar 22 11:59:26 2023 +0800

    block: don't set GD_NEED_PART_SCAN if scan partition failed
    
    [ Upstream commit 3723091ea1884d599cc8b8bf719d6f42e8d4d8b1 ]
    
    Currently if disk_scan_partitions() failed, GD_NEED_PART_SCAN will still
    set, and partition scan will be proceed again when blkdev_get_by_dev()
    is called. However, this will cause a problem that re-assemble partitioned
    raid device will creat partition for underlying disk.
    
    Test procedure:
    
    mdadm -CR /dev/md0 -l 1 -n 2 /dev/sda /dev/sdb -e 1.0
    sgdisk -n 0:0:+100MiB /dev/md0
    blockdev --rereadpt /dev/sda
    blockdev --rereadpt /dev/sdb
    mdadm -S /dev/md0
    mdadm -A /dev/md0 /dev/sda /dev/sdb
    
    Test result: underlying disk partition and raid partition can be
    observed at the same time
    
    Note that this can still happen in come corner cases that
    GD_NEED_PART_SCAN can be set for underlying disk while re-assemble raid
    device.
    
    Fixes: e5cfefa97bcc ("block: fix scan partition for exclusively open device again")
    Reviewed-by: Jan Kara <jack@xxxxxxx>
    Reviewed-by: Ming Lei <ming.lei@xxxxxxxxxx>
    Signed-off-by: Yu Kuai <yukuai3@xxxxxxxxxx>
    Signed-off-by: Jens Axboe <axboe@xxxxxxxxx>
    Signed-off-by: Sasha Levin <sashal@xxxxxxxxxx>

diff --git a/block/genhd.c b/block/genhd.c
index 0b6928e948f31..62a61388e752d 100644
--- a/block/genhd.c
+++ b/block/genhd.c
@@ -368,7 +368,6 @@ int disk_scan_partitions(struct gendisk *disk, fmode_t mode)
 	if (disk->open_partitions)
 		return -EBUSY;
 
-	set_bit(GD_NEED_PART_SCAN, &disk->state);
 	/*
 	 * If the device is opened exclusively by current thread already, it's
 	 * safe to scan partitons, otherwise, use bd_prepare_to_claim() to
@@ -381,12 +380,19 @@ int disk_scan_partitions(struct gendisk *disk, fmode_t mode)
 			return ret;
 	}
 
+	set_bit(GD_NEED_PART_SCAN, &disk->state);
 	bdev = blkdev_get_by_dev(disk_devt(disk), mode & ~FMODE_EXCL, NULL);
 	if (IS_ERR(bdev))
 		ret =  PTR_ERR(bdev);
 	else
 		blkdev_put(bdev, mode & ~FMODE_EXCL);
 
+	/*
+	 * If blkdev_get_by_dev() failed early, GD_NEED_PART_SCAN is still set,
+	 * and this will cause that re-assemble partitioned raid device will
+	 * creat partition for underlying disk.
+	 */
+	clear_bit(GD_NEED_PART_SCAN, &disk->state);
 	if (!(mode & FMODE_EXCL))
 		bd_abort_claiming(disk->part0, disk_scan_partitions);
 	return ret;



[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux