dm cache policy mq: a few small fixes
authorJoe Thornber <ejt@redhat.com>
Thu, 24 Oct 2013 18:10:28 +0000 (14:10 -0400)
committerMike Snitzer <snitzer@redhat.com>
Sat, 9 Nov 2013 23:20:17 +0000 (18:20 -0500)
Rename takeout_queue to concat_queue.

Fix a harmless bug in mq policies pop() function.  Currently pop()
always succeeds, with up coming changes this wont be the case.

Fix typo in comment above pre_cache_to_cache prototype.

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

index db490f74c7f8b3ac98d428c620bece1a7c8e5ddc..a9a25de5b0119462e3998e4cd58965b6d4e70bc9 100644 (file)
@@ -311,7 +311,7 @@ struct mq_policy {
 
 /*----------------------------------------------------------------*/
 /* Free/alloc mq cache entry structures. */
-static void takeout_queue(struct list_head *lh, struct queue *q)
+static void concat_queue(struct list_head *lh, struct queue *q)
 {
        unsigned level;
 
@@ -323,8 +323,8 @@ static void free_entries(struct mq_policy *mq)
 {
        struct entry *e, *tmp;
 
-       takeout_queue(&mq->free, &mq->pre_cache);
-       takeout_queue(&mq->free, &mq->cache);
+       concat_queue(&mq->free, &mq->pre_cache);
+       concat_queue(&mq->free, &mq->cache);
 
        list_for_each_entry_safe(e, tmp, &mq->free, list)
                kmem_cache_free(mq_entry_cache, e);
@@ -531,14 +531,16 @@ static void del(struct mq_policy *mq, struct entry *e)
  */
 static struct entry *pop(struct mq_policy *mq, struct queue *q)
 {
-       struct entry *e = container_of(queue_pop(q), struct entry, list);
+       struct entry *e;
+       struct list_head *h = queue_pop(q);
 
-       if (e) {
-               hash_remove(e);
+       if (!h)
+               return NULL;
 
-               if (e->in_cache)
-                       free_cblock(mq, e->cblock);
-       }
+       e = container_of(h, struct entry, list);
+       hash_remove(e);
+       if (e->in_cache)
+               free_cblock(mq, e->cblock);
 
        return e;
 }
@@ -697,7 +699,7 @@ static int cache_entry_found(struct mq_policy *mq,
 }
 
 /*
- * Moves and entry from the pre_cache to the cache.  The main work is
+ * Moves an entry from the pre_cache to the cache.  The main work is
  * finding which cache block to use.
  */
 static int pre_cache_to_cache(struct mq_policy *mq, struct entry *e,