[PATCH 7/7] Btrfs: end_workqueue_bio use switch case instead of else if

[Timofey Titovets]

If arg to "switch case" is determined and it's a consecutive numbers
(This is enum btrfs_wq_endio_type)
Compiler can create jump table to optimize logic

Signed-off-by: Timofey Titovets <nefelim...@gmail.com>
---
 fs/btrfs/disk-io.c | 17 +++++++++++++----
 1 file changed, 13 insertions(+), 4 deletions(-)

diff --git a/fs/btrfs/disk-io.c b/fs/btrfs/disk-io.c
index 8685d6718..72208826d 100644
--- a/fs/btrfs/disk-io.c
+++ b/fs/btrfs/disk-io.c
@@ -802,18 +802,27 @@ static void end_workqueue_bio(struct bio *bio)
        end_io_wq->error = bio->bi_error;

        if (bio_op(bio) == REQ_OP_WRITE) {
-               if (end_io_wq->metadata == BTRFS_WQ_ENDIO_METADATA) {
+               switch (end_io_wq->metadata) {
+               case BTRFS_WQ_ENDIO_DATA:
+                       wq = fs_info->endio_write_workers;
+                       func = btrfs_endio_write_helper;
+                       break;
+               case BTRFS_WQ_ENDIO_METADATA:
                        wq = fs_info->endio_meta_write_workers;
                        func = btrfs_endio_meta_write_helper;
-               } else if (end_io_wq->metadata == BTRFS_WQ_ENDIO_FREE_SPACE) {
+                       break;
+               case BTRFS_WQ_ENDIO_FREE_SPACE:
                        wq = fs_info->endio_freespace_worker;
                        func = btrfs_freespace_write_helper;
-               } else if (end_io_wq->metadata == BTRFS_WQ_ENDIO_RAID56) {
+                       break;
+               case BTRFS_WQ_ENDIO_RAID56:
                        wq = fs_info->endio_raid56_workers;
                        func = btrfs_endio_raid56_helper;
-               } else {
+                       break;
+               case BTRFS_WQ_ENDIO_DIO_REPAIR:
                        wq = fs_info->endio_write_workers;
                        func = btrfs_endio_write_helper;
+                       break;
                }
        } else {
                if (unlikely(end_io_wq->metadata ==
--
2.13.0
--
To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html