diff options
| author | Joe Thornber <ejt@redhat.com> | 2015-02-20 13:01:22 +0000 |
|---|---|---|
| committer | Mike Snitzer <snitzer@redhat.com> | 2015-03-31 12:03:48 -0400 |
| commit | c74ffc5c63b0b2753bedd49bdc1196d570f66803 (patch) | |
| tree | b1b07de4c889e69058533d6ed064c920547bf7a5 /drivers/md/dm-cache-policy-mq.c | |
| parent | 75da39bf256c27e25f395b191ead79f323772672 (diff) | |
| download | linux-c74ffc5c63b0b2753bedd49bdc1196d570f66803.tar.bz2 | |
dm cache policy mq: remove queue_shift_down()
queue_shift_down() didn't adjust the hit_counts to the new levels, so it
just had the effect of scrambling levels.
Signed-off-by: Joe Thornber <ejt@redhat.com>
Signed-off-by: Mike Snitzer <snitzer@redhat.com>
Diffstat (limited to 'drivers/md/dm-cache-policy-mq.c')
| -rw-r--r-- | drivers/md/dm-cache-policy-mq.c | 16 |
1 files changed, 0 insertions, 16 deletions
diff --git a/drivers/md/dm-cache-policy-mq.c b/drivers/md/dm-cache-policy-mq.c index ca05d69191e8..3c86b5efe78f 100644 --- a/drivers/md/dm-cache-policy-mq.c +++ b/drivers/md/dm-cache-policy-mq.c @@ -160,18 +160,6 @@ static void queue_remove(struct queue *q, struct list_head *elt) } /* - * Shifts all regions down one level. This has no effect on the order of - * the queue. - */ -static void queue_shift_down(struct queue *q) -{ - unsigned level; - - for (level = 1; level < NR_QUEUE_LEVELS; level++) - list_splice_init(q->qs + level, q->qs + level - 1); -} - -/* * Gives us the oldest entry of the lowest popoulated level. If the first * level is emptied then we shift down one level. */ @@ -193,10 +181,6 @@ static struct list_head *queue_pop(struct queue *q) if (r) { q->nr_elts--; list_del(r); - - /* have we just emptied the bottom level? */ - if (list_empty(q->qs)) - queue_shift_down(q); } return r; |