The WARN_ON_ONCE() macro takes a condition, passing it this string literal sort of works as intended but it's better to use WARN_ONCE() instead. Signed-off-by: Dan Carpenter <dan.carpenter@xxxxxxxxxx> diff --git a/net/sunrpc/xprtrdma/svc_rdma_transport.c b/net/sunrpc/xprtrdma/svc_rdma_transport.c index abfbd02..4b397ad 100644 --- a/net/sunrpc/xprtrdma/svc_rdma_transport.c +++ b/net/sunrpc/xprtrdma/svc_rdma_transport.c @@ -221,7 +221,7 @@ out_empty: spin_lock_bh(&xprt->sc_ctxt_lock); xprt->sc_ctxt_used--; spin_unlock_bh(&xprt->sc_ctxt_lock); - WARN_ON_ONCE("svcrdma: empty RDMA ctxt list?\n"); + WARN_ONCE(1, "svcrdma: empty RDMA ctxt list?\n"); return NULL; } @@ -328,7 +328,7 @@ out_empty: if (map) goto out; - WARN_ON_ONCE("svcrdma: empty request map list?\n"); + WARN_ONCE(1, "svcrdma: empty request map list?\n"); return NULL; } -- To unsubscribe from this list: send the line "unsubscribe linux-nfs" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html