With CONFIG_BUG=n the __WARN() macro is not defined. Avoid that building with CONFIG_BUG=n fails by changing pr_err(...); _WARN() into WARN(true, ...). Signed-off-by: Bart Van Assche <bvanassche@xxxxxxx> Cc: Nicholas Bellinger <nab@xxxxxxxxxxxxxxx> Cc: Roland Dreier <roland@xxxxxxxxxxxxxxx> Cc: Christoph Hellwig <hch@xxxxxxxxxxxxx> Cc: Randy Dunlap <rdunlap@xxxxxxxxxxxx> Cc: Stephen Rothwell <sfr@xxxxxxxxxxxxxxxx> --- drivers/infiniband/ulp/srpt/ib_srpt.c | 14 +++++--------- 1 files changed, 5 insertions(+), 9 deletions(-) diff --git a/drivers/infiniband/ulp/srpt/ib_srpt.c b/drivers/infiniband/ulp/srpt/ib_srpt.c index 49ae767..6376e2a 100644 --- a/drivers/infiniband/ulp/srpt/ib_srpt.c +++ b/drivers/infiniband/ulp/srpt/ib_srpt.c @@ -1462,9 +1462,7 @@ static void srpt_handle_rdma_comp(struct srpt_rdma_ch *ch, } else if (opcode == SRPT_RDMA_ABORT) { ioctx->rdma_aborted = true; } else { - __WARN(); - printk(KERN_ERR "%s[%d]: scmnd == NULL (opcode %d)", __func__, - __LINE__, opcode); + WARN(true, "unexpected opcode %d\n", opcode); } } @@ -2735,7 +2733,7 @@ static void srpt_cm_dreq_recv(struct ib_cm_id *cm_id) break; case CH_DISCONNECTING: case CH_DRAINING: - __WARN(); + WARN(true, "unexpected channel state %d\n", ch->state); break; } spin_unlock_irqrestore(&ch->spinlock, flags); @@ -2963,8 +2961,7 @@ static int srpt_write_pending(struct se_cmd *se_cmd) ch_state = ch->state; switch (ch_state) { case CH_CONNECTING: - /* This code should never be reached. */ - __WARN(); + WARN(true, "unexpected channel state %d\n", ch_state); ret = -EINVAL; goto out; case CH_LIVE: @@ -3028,9 +3025,8 @@ static int srpt_queue_response(struct se_cmd *cmd) ioctx->state = SRPT_STATE_MGMT_RSP_SENT; break; default: - printk(KERN_ERR "ch %p; cmd %d: unexpected command state %d\n", - ch, ioctx->ioctx.index, ioctx->state); - __WARN(); + WARN(true, "ch %p; cmd %d: unexpected command state %d\n", + ch, ioctx->ioctx.index, ioctx->state); break; } spin_unlock_irqrestore(&ioctx->spinlock, flags); -- 1.7.3.4 -- To unsubscribe from this list: send the line "unsubscribe linux-next" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html