]> git.proxmox.com Git - mirror_ubuntu-eoan-kernel.git/commitdiff
dm cache policy smq: cleanup free_target_met() and clean_target_met()
authorMike Snitzer <snitzer@redhat.com>
Thu, 4 May 2017 14:04:18 +0000 (10:04 -0400)
committerMike Snitzer <snitzer@redhat.com>
Thu, 4 May 2017 14:27:47 +0000 (10:27 -0400)
Depending on the passed @idle arg, there may be no need to calculate
'nr_free' or 'nr_clean' respectively in free_target_met() and
clean_target_met().

Signed-off-by: Mike Snitzer <snitzer@redhat.com>
drivers/md/dm-cache-policy-smq.c

index d13d9edf8dfe4dc55d88065df4417c8f9ba94942..72479bd61e118e51c4bada036901ebe1bc4b1091 100644 (file)
@@ -1120,28 +1120,30 @@ static bool clean_target_met(struct smq_policy *mq, bool idle)
         * Cache entries may not be populated.  So we cannot rely on the
         * size of the clean queue.
         */
-       unsigned nr_clean = from_cblock(mq->cache_size) - q_size(&mq->dirty);
+       unsigned nr_clean;
 
-       if (idle)
+       if (idle) {
                /*
                 * We'd like to clean everything.
                 */
                return q_size(&mq->dirty) == 0u;
-       else
-               return (nr_clean + btracker_nr_writebacks_queued(mq->bg_work)) >=
-                      percent_to_target(mq, CLEAN_TARGET);
+       }
+
+       nr_clean = from_cblock(mq->cache_size) - q_size(&mq->dirty);
+       return (nr_clean + btracker_nr_writebacks_queued(mq->bg_work)) >=
+               percent_to_target(mq, CLEAN_TARGET);
 }
 
 static bool free_target_met(struct smq_policy *mq, bool idle)
 {
-       unsigned nr_free = from_cblock(mq->cache_size) -
-                          mq->cache_alloc.nr_allocated;
+       unsigned nr_free;
 
-       if (idle)
-               return (nr_free + btracker_nr_demotions_queued(mq->bg_work)) >=
-                      percent_to_target(mq, FREE_TARGET);
-       else
+       if (!idle)
                return true;
+
+       nr_free = from_cblock(mq->cache_size) - mq->cache_alloc.nr_allocated;
+       return (nr_free + btracker_nr_demotions_queued(mq->bg_work)) >=
+               percent_to_target(mq, FREE_TARGET);
 }
 
 /*----------------------------------------------------------------*/