IB/iser: Micro-optimize iser logging

And fix a checkpatch warning.

Signed-off-by: Sagi Grimberg <sagig@mellanox.com>
Signed-off-by: Or Gerlitz <ogerlitz@mellanox.com>
Signed-off-by: Roland Dreier <roland@purestorage.com>

1 files changed, 5 insertions, 8 deletions

diff --git a/drivers/infiniband/ulp/iser/iscsi_iser.h b/drivers/infiniband/ulp/iser/iscsi_iser.h
index 4234a9c0ebfd..cf704da3c71e 100644
--- a/drivers/infiniband/ulp/iser/iscsi_iser.h
+++ b/drivers/infiniband/ulp/iser/iscsi_iser.h
@@ -73,30 +73,27 @@
 
 #define iser_dbg(fmt, arg...)                            \
         do {                                             \
-                if (iser_debug_level > 2)                \
+                if (unlikely(iser_debug_level > 2))      \
                         printk(KERN_DEBUG PFX "%s: " fmt,\
                                 __func__ , ## arg);      \
         } while (0)
 
 #define iser_warn(fmt, arg...)                          \
         do {                                            \
-                if (iser_debug_level > 0)               \
+                if (unlikely(iser_debug_level > 0))     \
                         pr_warn(PFX "%s: " fmt,         \
                                 __func__ , ## arg);     \
         } while (0)
 
 #define iser_info(fmt, arg...)                          \
         do {                                            \
-                if (iser_debug_level > 1)               \
+                if (unlikely(iser_debug_level > 1))     \
                         pr_info(PFX "%s: " fmt,         \
                                 __func__ , ## arg);     \
         } while (0)
 
-#define iser_err(fmt, arg...)                           \
-        do {                                            \
-                printk(KERN_ERR PFX "%s: " fmt,         \
-                       __func__ , ## arg);              \
-        } while (0)
+#define iser_err(fmt, arg...) \
+        pr_err(PFX "%s: " fmt, __func__ , ## arg)
 
 #define SHIFT_4K        12
 #define SIZE_4K (1ULL << SHIFT_4K)