RDMA/bnxt_re: Make use of create AH sleepable flag
When in a sleepable (non-atomic) context, wait for firmware completion instead of polling for it. Signed-off-by: Gal Pressman <galpress@amazon.com> Acked-by: Selvin Xavier <selvin.xavier@broadcom.com> Signed-off-by: Jason Gunthorpe <jgg@mellanox.com>
This commit is contained in:
parent
2553ba217e
commit
90e3edd8cc
@ -723,7 +723,8 @@ struct ib_ah *bnxt_re_create_ah(struct ib_pd *ib_pd,
|
|||||||
}
|
}
|
||||||
|
|
||||||
memcpy(ah->qplib_ah.dmac, ah_attr->roce.dmac, ETH_ALEN);
|
memcpy(ah->qplib_ah.dmac, ah_attr->roce.dmac, ETH_ALEN);
|
||||||
rc = bnxt_qplib_create_ah(&rdev->qplib_res, &ah->qplib_ah);
|
rc = bnxt_qplib_create_ah(&rdev->qplib_res, &ah->qplib_ah,
|
||||||
|
!(flags & RDMA_CREATE_AH_SLEEPABLE));
|
||||||
if (rc) {
|
if (rc) {
|
||||||
dev_err(rdev_to_dev(rdev), "Failed to allocate HW AH");
|
dev_err(rdev_to_dev(rdev), "Failed to allocate HW AH");
|
||||||
goto fail;
|
goto fail;
|
||||||
@ -959,7 +960,7 @@ static struct bnxt_re_ah *bnxt_re_create_shadow_qp_ah
|
|||||||
/* Have DMAC same as SMAC */
|
/* Have DMAC same as SMAC */
|
||||||
ether_addr_copy(ah->qplib_ah.dmac, rdev->netdev->dev_addr);
|
ether_addr_copy(ah->qplib_ah.dmac, rdev->netdev->dev_addr);
|
||||||
|
|
||||||
rc = bnxt_qplib_create_ah(&rdev->qplib_res, &ah->qplib_ah);
|
rc = bnxt_qplib_create_ah(&rdev->qplib_res, &ah->qplib_ah, false);
|
||||||
if (rc) {
|
if (rc) {
|
||||||
dev_err(rdev_to_dev(rdev),
|
dev_err(rdev_to_dev(rdev),
|
||||||
"Failed to allocate HW AH for Shadow QP");
|
"Failed to allocate HW AH for Shadow QP");
|
||||||
|
@ -488,7 +488,8 @@ int bnxt_qplib_add_pkey(struct bnxt_qplib_res *res,
|
|||||||
}
|
}
|
||||||
|
|
||||||
/* AH */
|
/* AH */
|
||||||
int bnxt_qplib_create_ah(struct bnxt_qplib_res *res, struct bnxt_qplib_ah *ah)
|
int bnxt_qplib_create_ah(struct bnxt_qplib_res *res, struct bnxt_qplib_ah *ah,
|
||||||
|
bool block)
|
||||||
{
|
{
|
||||||
struct bnxt_qplib_rcfw *rcfw = res->rcfw;
|
struct bnxt_qplib_rcfw *rcfw = res->rcfw;
|
||||||
struct cmdq_create_ah req;
|
struct cmdq_create_ah req;
|
||||||
@ -522,7 +523,7 @@ int bnxt_qplib_create_ah(struct bnxt_qplib_res *res, struct bnxt_qplib_ah *ah)
|
|||||||
req.dest_mac[2] = cpu_to_le16(temp16[2]);
|
req.dest_mac[2] = cpu_to_le16(temp16[2]);
|
||||||
|
|
||||||
rc = bnxt_qplib_rcfw_send_message(rcfw, (void *)&req, (void *)&resp,
|
rc = bnxt_qplib_rcfw_send_message(rcfw, (void *)&req, (void *)&resp,
|
||||||
NULL, 1);
|
NULL, block);
|
||||||
if (rc)
|
if (rc)
|
||||||
return rc;
|
return rc;
|
||||||
|
|
||||||
|
@ -241,7 +241,8 @@ int bnxt_qplib_get_dev_attr(struct bnxt_qplib_rcfw *rcfw,
|
|||||||
int bnxt_qplib_set_func_resources(struct bnxt_qplib_res *res,
|
int bnxt_qplib_set_func_resources(struct bnxt_qplib_res *res,
|
||||||
struct bnxt_qplib_rcfw *rcfw,
|
struct bnxt_qplib_rcfw *rcfw,
|
||||||
struct bnxt_qplib_ctx *ctx);
|
struct bnxt_qplib_ctx *ctx);
|
||||||
int bnxt_qplib_create_ah(struct bnxt_qplib_res *res, struct bnxt_qplib_ah *ah);
|
int bnxt_qplib_create_ah(struct bnxt_qplib_res *res, struct bnxt_qplib_ah *ah,
|
||||||
|
bool block);
|
||||||
int bnxt_qplib_destroy_ah(struct bnxt_qplib_res *res, struct bnxt_qplib_ah *ah);
|
int bnxt_qplib_destroy_ah(struct bnxt_qplib_res *res, struct bnxt_qplib_ah *ah);
|
||||||
int bnxt_qplib_alloc_mrw(struct bnxt_qplib_res *res,
|
int bnxt_qplib_alloc_mrw(struct bnxt_qplib_res *res,
|
||||||
struct bnxt_qplib_mrw *mrw);
|
struct bnxt_qplib_mrw *mrw);
|
||||||
|
Loading…
x
Reference in New Issue
Block a user