Commit afc1990e authored by Dan Carpenter's avatar Dan Carpenter Committed by Jason Gunthorpe

RDMA/core: Fix a WARN() message

The first parameter of WARN_ONCE() is a condition, then following
parameters are the message.  In this case, we left out the condition so it
will just print the ops->type string.

Fixes: 3856ec4b ("RDMA/core: Add RDMA_NLDEV_CMD_NEWLINK/DELLINK support")
Signed-off-by: default avatarDan Carpenter <dan.carpenter@oracle.com>
Reviewed-by: default avatarMajd Dibbiny <majd@mellanox.com>
Reviewed-by: default avatarSteve Wise <swise@opengridcomputing.com>
Signed-off-by: default avatarJason Gunthorpe <jgg@mellanox.com>
parent c50866e2
......@@ -1222,10 +1222,8 @@ static const struct rdma_link_ops *link_ops_get(const char *type)
void rdma_link_register(struct rdma_link_ops *ops)
{
down_write(&link_ops_rwsem);
if (link_ops_get(ops->type)) {
WARN_ONCE("Duplicate rdma_link_ops! %s\n", ops->type);
if (WARN_ON_ONCE(link_ops_get(ops->type)))
goto out;
}
list_add(&ops->list, &link_ops);
out:
up_write(&link_ops_rwsem);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment