diff options
| author |   Joe Thornber <ejt@redhat.com> | 2017-11-08 06:09:38 -0500 |
|---|---|---|
| committer |   Mike Snitzer <snitzer@redhat.com> | 2017-11-10 15:45:02 -0500 |
| commit | deb71918ae29e23ff7f4537f5ff654f8b6580af2 (patch) | |
| tree | 8c43506f892804b4ebc3cf004ae2b9e19ea29b1b /drivers/md | |
| parent | dm cache policy smq: handle races with queuing background_work (diff) | |
| download | linux-dev-deb71918ae29e23ff7f4537f5ff654f8b6580af2.tar.xz linux-dev-deb71918ae29e23ff7f4537f5ff654f8b6580af2.zip | |
dm cache policy smq: take origin idle status into account when queuing writebacks
If the origin device is idle try and writeback more data.
Signed-off-by: Joe Thornber <ejt@redhat.com>
Signed-off-by: Mike Snitzer <snitzer@redhat.com>
Diffstat (limited to 'drivers/md')
| -rw-r--r-- | drivers/md/dm-cache-policy-smq.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/drivers/md/dm-cache-policy-smq.c b/drivers/md/dm-cache-policy-smq.c index 42e5c4b59889..99fae819a0e7 100644 --- a/drivers/md/dm-cache-policy-smq.c +++ b/drivers/md/dm-cache-policy-smq.c @@ -1158,13 +1158,13 @@ static void clear_pending(struct smq_policy *mq, struct entry *e) e->pending_work = false; } -static void queue_writeback(struct smq_policy *mq) +static void queue_writeback(struct smq_policy *mq, bool idle) { int r; struct policy_work work; struct entry *e; - e = q_peek(&mq->dirty, mq->dirty.nr_levels, !mq->migrations_allowed); + e = q_peek(&mq->dirty, mq->dirty.nr_levels, idle); if (e) { mark_pending(mq, e); q_del(&mq->dirty, e); @@ -1193,7 +1193,7 @@ static void queue_demotion(struct smq_policy *mq) e = q_peek(&mq->clean, mq->clean.nr_levels / 2, true); if (!e) { if (!clean_target_met(mq, true)) - queue_writeback(mq); + queue_writeback(mq, false); return; } @@ -1429,7 +1429,7 @@ static int smq_get_background_work(struct dm_cache_policy *p, bool idle, r = btracker_issue(mq->bg_work, result); if (r == -ENODATA) { if (!clean_target_met(mq, idle)) { - queue_writeback(mq); + queue_writeback(mq, idle); r = btracker_issue(mq->bg_work, result); } } |