md: remove 'go_faster' option from ->sync_request()

This option is not well justified and testing suggests that
it hardly ever makes any difference.

The comment suggests there might be a need to wait for non-resync
activity indicated by ->nr_waiting, however raise_barrier()
already waits for all of that.

So just remove it to simplify reasoning about speed limiting.

This allows us to remove a 'FIXME' comment from raid5.c as that
never used the flag.

Signed-off-by: NeilBrown <neilb@suse.de>

5 files changed, 6 insertions, 21 deletions

diff --git a/drivers/md/md.c b/drivers/md/md.c
index 3724a29eaf0e..3b9b032aa006 100644
--- a/drivers/md/md.c
+++ b/drivers/md/md.c
@@ -7820,8 +7820,7 @@ void md_do_sync(struct md_thread *thread)
                 if (test_bit(MD_RECOVERY_INTR, &mddev->recovery))
                         break;
 
-                sectors = mddev->pers->sync_request(mddev, j, &skipped,
-                                                  currspeed < speed_min(mddev));
+                sectors = mddev->pers->sync_request(mddev, j, &skipped);
                 if (sectors == 0) {
                         set_bit(MD_RECOVERY_INTR, &mddev->recovery);
                         break;
@@ -7898,7 +7897,7 @@ void md_do_sync(struct md_thread *thread)
         wait_event(mddev->recovery_wait, !atomic_read(&mddev->recovery_active));
 
         /* tell personality that we are finished */
-        mddev->pers->sync_request(mddev, max_sectors, &skipped, 1);
+        mddev->pers->sync_request(mddev, max_sectors, &skipped);
 
         if (mddev_is_clustered(mddev))
                 md_cluster_ops->resync_finish(mddev);
diff --git a/drivers/md/md.h b/drivers/md/md.h
index ecdce36ec6b8..4046a6c6f223 100644
--- a/drivers/md/md.h
+++ b/drivers/md/md.h
@@ -506,7 +506,7 @@ struct md_personality
         int (*hot_add_disk) (struct mddev *mddev, struct md_rdev *rdev);
         int (*hot_remove_disk) (struct mddev *mddev, struct md_rdev *rdev);
         int (*spare_active) (struct mddev *mddev);
-        sector_t (*sync_request)(struct mddev *mddev, sector_t sector_nr, int *skipped, int go_faster);
+        sector_t (*sync_request)(struct mddev *mddev, sector_t sector_nr, int *skipped);
         int (*resize) (struct mddev *mddev, sector_t sectors);
         sector_t (*size) (struct mddev *mddev, sector_t sectors, int raid_disks);
         int (*check_reshape) (struct mddev *mddev);
diff --git a/drivers/md/raid1.c b/drivers/md/raid1.c
index 4efa50186a2a..9157a29c8dbf 100644
--- a/drivers/md/raid1.c
+++ b/drivers/md/raid1.c
@@ -2480,7 +2480,7 @@ static int init_resync(struct r1conf *conf)
  * that can be installed to exclude normal IO requests.
  */
 
-static sector_t sync_request(struct mddev *mddev, sector_t sector_nr, int *skipped, int go_faster)
+static sector_t sync_request(struct mddev *mddev, sector_t sector_nr, int *skipped)
 {
         struct r1conf *conf = mddev->private;
         struct r1bio *r1_bio;
@@ -2533,13 +2533,6 @@ static sector_t sync_request(struct mddev *mddev, sector_t sector_nr, int *skipp
                 *skipped = 1;
                 return sync_blocks;
         }
-        /*
-         * If there is non-resync activity waiting for a turn,
-         * and resync is going fast enough,
-         * then let it though before starting on this new sync request.
-         */
-        if (!go_faster && conf->nr_waiting)
-                msleep_interruptible(1000);
 
         bitmap_cond_end_sync(mddev->bitmap, sector_nr);
         r1_bio = mempool_alloc(conf->r1buf_pool, GFP_NOIO);
diff --git a/drivers/md/raid10.c b/drivers/md/raid10.c
index a7196c49d15d..e793ab6b3570 100644
--- a/drivers/md/raid10.c
+++ b/drivers/md/raid10.c
@@ -2889,7 +2889,7 @@ static int init_resync(struct r10conf *conf)
  */
 
 static sector_t sync_request(struct mddev *mddev, sector_t sector_nr,
-                             int *skipped, int go_faster)
+                             int *skipped)
 {
         struct r10conf *conf = mddev->private;
         struct r10bio *r10_bio;
@@ -2994,12 +2994,6 @@ static sector_t sync_request(struct mddev *mddev, sector_t sector_nr,
         if (conf->geo.near_copies < conf->geo.raid_disks &&
             max_sector > (sector_nr | chunk_mask))
                 max_sector = (sector_nr | chunk_mask) + 1;
-        /*
-         * If there is non-resync activity waiting for us then
-         * put in a delay to throttle resync.
-         */
-        if (!go_faster && conf->nr_waiting)
-                msleep_interruptible(1000);
 
         /* Again, very different code for resync and recovery.
          * Both must result in an r10bio with a list of bios that
diff --git a/drivers/md/raid5.c b/drivers/md/raid5.c
index cd2f96b2c572..022a0d99e110 100644
--- a/drivers/md/raid5.c
+++ b/drivers/md/raid5.c
@@ -5050,8 +5050,7 @@ ret:
         return reshape_sectors;
 }
 
-/* FIXME go_faster isn't used */
-static inline sector_t sync_request(struct mddev *mddev, sector_t sector_nr, int *skipped, int go_faster)
+static inline sector_t sync_request(struct mddev *mddev, sector_t sector_nr, int *skipped)
 {
         struct r5conf *conf = mddev->private;
         struct stripe_head *sh;