Commit 3d8d3287 authored by Yu Kuai's avatar Yu Kuai Committed by Song Liu
Browse files

md: factor out a helper from mddev_put() 



There are no functional changes, prepare to simplify md_seq_ops in next
patch.

Signed-off-by: default avatarYu Kuai <yukuai3@huawei.com>
Signed-off-by: default avatarSong Liu <song@kernel.org>
Link: https://lore.kernel.org/r/20230927061241.1552837-2-yukuai1@huaweicloud.com
parent ceb04163

drivers/md/md.c

+17 −12
Original line number Diff line number Diff line
@@ -616,23 +616,28 @@ static inline struct mddev *mddev_get(struct mddev *mddev) 


static void mddev_delayed_delete(struct work_struct *ws);



void mddev_put(struct mddev *mddev)

static void __mddev_put(struct mddev *mddev)

{

	if (!atomic_dec_and_lock(&mddev->active, &all_mddevs_lock))

	if (mddev->raid_disks || !list_empty(&mddev->disks) ||

	    mddev->ctime || mddev->hold_active)

		return;

	if (!mddev->raid_disks && list_empty(&mddev->disks) &&

	    mddev->ctime == 0 && !mddev->hold_active) {

		/* Array is not configured at all, and not held active,

		 * so destroy it */



	/* Array is not configured at all, and not held active, so destroy it */

	set_bit(MD_DELETED, &mddev->flags);



	/*

		 * Call queue_work inside the spinlock so that

		 * flush_workqueue() after mddev_find will succeed in waiting

		 * for the work to be done.

	 * Call queue_work inside the spinlock so that flush_workqueue() after

	 * mddev_find will succeed in waiting for the work to be done.

	 */

	queue_work(md_misc_wq, &mddev->del_work);

}



void mddev_put(struct mddev *mddev)

{

	if (!atomic_dec_and_lock(&mddev->active, &all_mddevs_lock))

		return;



	__mddev_put(mddev);

	spin_unlock(&all_mddevs_lock);

}