diff options
| author | Abd-Alrhman Masalkhi <abd.masalkhi@gmail.com> | 2026-04-28 15:05:24 +0200 |
|---|---|---|
| committer | Yu Kuai <yukuai@fygo.io> | 2026-05-31 19:09:17 +0800 |
| commit | abaf4783822851678632e5cea98aa5aead99852f (patch) | |
| tree | a93f46cbe7133c97bdbf5c470eef8c5f369c55d6 /drivers/md | |
| parent | 1d24d4c1ea1d9d3211c1d178e0c8d95be0348975 (diff) | |
| download | linux-next-abaf4783822851678632e5cea98aa5aead99852f.tar.gz linux-next-abaf4783822851678632e5cea98aa5aead99852f.zip | |
md: skip redundant raid_disks update when value is unchanged
Calling update_raid_disks() with the same value as the current one
can trigger unnecessary work. For example, RAID1 will reallocate
resources such as the mempool for r1bio.
Signed-off-by: Abd-Alrhman Masalkhi <abd.masalkhi@gmail.com>
Link: https://patch.msgid.link/20260428130524.448063-1-abd.masalkhi@gmail.com
Signed-off-by: Yu Kuai <yukuai@fygo.io>
Diffstat (limited to 'drivers/md')
| -rw-r--r-- | drivers/md/md.c | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/drivers/md/md.c b/drivers/md/md.c index 8b568eee8743..6cb2c452f963 100644 --- a/drivers/md/md.c +++ b/drivers/md/md.c @@ -4414,9 +4414,10 @@ raid_disks_store(struct mddev *mddev, const char *buf, size_t len) err = mddev_suspend_and_lock(mddev); if (err) return err; - if (mddev->pers) - err = update_raid_disks(mddev, n); - else if (mddev->reshape_position != MaxSector) { + if (mddev->pers) { + if (n != mddev->raid_disks) + err = update_raid_disks(mddev, n); + } else if (mddev->reshape_position != MaxSector) { struct md_rdev *rdev; int olddisks = mddev->raid_disks - mddev->delta_disks; |