aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorNeilBrown <neilb@suse.de>2009-12-13 20:49:54 -0500
committerNeilBrown <neilb@suse.de>2009-12-13 20:51:41 -0500
commit9cd30fdc33cde9ae4ac55a1ccbbb89f3f7b9b2f2 (patch)
treeb6a4e0303fe3261892f24f26d7ecebf06a6be1a9
parent1b04be96f6910ee415287bf0d5309c7d4c94bd2b (diff)
md: remove needless setting of thread->timeout in raid10_quiesce
As bitmap_create and bitmap_destroy already set thread->timeout as appropriate, there is no need to do it in raid10_quiesce. There is a possible need to wake the thread after the timeout has been set low, but it is better to do that where the timeout is actually set low, in bitmap_create. Signed-off-by: NeilBrown <neilb@suse.de>
-rw-r--r--drivers/md/bitmap.c1
-rw-r--r--drivers/md/raid10.c7
2 files changed, 1 insertions, 7 deletions
diff --git a/drivers/md/bitmap.c b/drivers/md/bitmap.c
index c1f28a87ea59..b1bcd36ca963 100644
--- a/drivers/md/bitmap.c
+++ b/drivers/md/bitmap.c
@@ -1695,6 +1695,7 @@ int bitmap_create(mddev_t *mddev)
1695 mddev->bitmap = bitmap; 1695 mddev->bitmap = bitmap;
1696 1696
1697 mddev->thread->timeout = mddev->bitmap_info.daemon_sleep; 1697 mddev->thread->timeout = mddev->bitmap_info.daemon_sleep;
1698 md_wakeup_thread(mddev->thread);
1698 1699
1699 bitmap_update_sb(bitmap); 1700 bitmap_update_sb(bitmap);
1700 1701
diff --git a/drivers/md/raid10.c b/drivers/md/raid10.c
index 064c2bb5ee63..d9e28a65af58 100644
--- a/drivers/md/raid10.c
+++ b/drivers/md/raid10.c
@@ -2275,13 +2275,6 @@ static void raid10_quiesce(mddev_t *mddev, int state)
2275 lower_barrier(conf); 2275 lower_barrier(conf);
2276 break; 2276 break;
2277 } 2277 }
2278 if (mddev->thread) {
2279 if (mddev->bitmap)
2280 mddev->thread->timeout = mddev->bitmap_info.daemon_sleep;
2281 else
2282 mddev->thread->timeout = MAX_SCHEDULE_TIMEOUT;
2283 md_wakeup_thread(mddev->thread);
2284 }
2285} 2278}
2286 2279
2287static struct mdk_personality raid10_personality = 2280static struct mdk_personality raid10_personality =