diff options
author | Shaohua Li <shli@fb.com> | 2017-02-10 19:18:09 -0500 |
---|---|---|
committer | Shaohua Li <shli@fb.com> | 2017-02-13 12:20:05 -0500 |
commit | e33fbb9cc73d6502e69eaf1c178e0c39059763ea (patch) | |
tree | 3a75c9e524eab2facb854a41e970dfb86fc538c1 /drivers/md/raid5.c | |
parent | e8fd52eec2cd25b917983b3f3aa738b722522376 (diff) |
md/raid5-cache: exclude reclaiming stripes in reclaim check
stripes which are being reclaimed are still accounted into cached
stripes. The reclaim takes time. r5c_do_reclaim isn't aware of the
stripes and does unnecessary stripe reclaim. In practice, I saw one
stripe is reclaimed one time. This will cause bad IO pattern. Fixing
this by excluding the reclaing stripes in the check.
Cc: Song Liu <songliubraving@fb.com>
Signed-off-by: Shaohua Li <shli@fb.com>
Diffstat (limited to 'drivers/md/raid5.c')
-rw-r--r-- | drivers/md/raid5.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/drivers/md/raid5.c b/drivers/md/raid5.c index b0d1345c832c..e2fa88fc6a1b 100644 --- a/drivers/md/raid5.c +++ b/drivers/md/raid5.c | |||
@@ -6838,6 +6838,8 @@ static struct r5conf *setup_conf(struct mddev *mddev) | |||
6838 | INIT_LIST_HEAD(&conf->r5c_full_stripe_list); | 6838 | INIT_LIST_HEAD(&conf->r5c_full_stripe_list); |
6839 | atomic_set(&conf->r5c_cached_partial_stripes, 0); | 6839 | atomic_set(&conf->r5c_cached_partial_stripes, 0); |
6840 | INIT_LIST_HEAD(&conf->r5c_partial_stripe_list); | 6840 | INIT_LIST_HEAD(&conf->r5c_partial_stripe_list); |
6841 | atomic_set(&conf->r5c_flushing_full_stripes, 0); | ||
6842 | atomic_set(&conf->r5c_flushing_partial_stripes, 0); | ||
6841 | 6843 | ||
6842 | conf->level = mddev->new_level; | 6844 | conf->level = mddev->new_level; |
6843 | conf->chunk_sectors = mddev->new_chunk_sectors; | 6845 | conf->chunk_sectors = mddev->new_chunk_sectors; |