blk-cgroup: consolidate error handling in blkcg_init_queue
authorChristoph Hellwig <hch@lst.de>
Wed, 14 Nov 2018 16:02:12 +0000 (17:02 +0100)
committerJens Axboe <axboe@kernel.dk>
Thu, 15 Nov 2018 19:13:26 +0000 (12:13 -0700)
Use a goto label to merge two identical pieces of error handling code.

Reviewed-by: Hannes Reinecke <hare@suse.com>
Signed-off-by: Christoph Hellwig <hch@lst.de>
Signed-off-by: Jens Axboe <axboe@kernel.dk>
block/blk-cgroup.c

index 3296c0b..717ab38 100644 (file)
@@ -1173,21 +1173,19 @@ int blkcg_init_queue(struct request_queue *q)
                radix_tree_preload_end();
 
        ret = blk_iolatency_init(q);
-       if (ret) {
-               spin_lock_irq(q->queue_lock);
-               blkg_destroy_all(q);
-               spin_unlock_irq(q->queue_lock);
-               return ret;
-       }
+       if (ret)
+               goto err_destroy_all;
 
        ret = blk_throtl_init(q);
-       if (ret) {
-               spin_lock_irq(q->queue_lock);
-               blkg_destroy_all(q);
-               spin_unlock_irq(q->queue_lock);
-       }
-       return ret;
+       if (ret)
+               goto err_destroy_all;
+       return 0;
 
+err_destroy_all:
+       spin_lock_irq(q->queue_lock);
+       blkg_destroy_all(q);
+       spin_unlock_irq(q->queue_lock);
+       return ret;
 err_unlock:
        spin_unlock_irq(q->queue_lock);
        rcu_read_unlock();