bnxt_en: add HWRM request assignment API
authorEdwin Peer <edwin.peer@broadcom.com>
Sun, 29 Aug 2021 07:35:01 +0000 (03:35 -0400)
committerDavid S. Miller <davem@davemloft.net>
Mon, 30 Aug 2021 08:35:04 +0000 (09:35 +0100)
hwrm_req_replace() provides an assignment like operation to replace a
managed HWRM request object with data from a pre-built source. This is
useful for handling request data provided by higher layer HWRM clients.

Signed-off-by: Edwin Peer <edwin.peer@broadcom.com>
Signed-off-by: Michael Chan <michael.chan@broadcom.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/broadcom/bnxt/bnxt_hwrm.c
drivers/net/ethernet/broadcom/bnxt/bnxt_hwrm.h

index 621daf687a0058f7af7d382de68eaaf6828ce81f..39ef65025e1758386767bcd56065c1209f38c5b0 100644 (file)
@@ -147,6 +147,61 @@ void hwrm_req_timeout(struct bnxt *bp, void *req, unsigned int timeout)
                ctx->timeout = timeout;
 }
 
+/**
+ * hwrm_req_replace() - Replace request data.
+ * @bp: The driver context.
+ * @req: The request to modify. A call to hwrm_req_replace() is conceptually
+ *     an assignment of new_req to req. Subsequent calls to HWRM API functions,
+ *     such as hwrm_req_send(), should thus use req and not new_req (in fact,
+ *     calls to HWRM API functions will fail if non-managed request objects
+ *     are passed).
+ * @len: The length of new_req.
+ * @new_req: The pre-built request to copy or reference.
+ *
+ * Replaces the request data in req with that of new_req. This is useful in
+ * scenarios where a request object has already been constructed by a third
+ * party prior to creating a resource managed request using hwrm_req_init().
+ * Depending on the length, hwrm_req_replace() will either copy the new
+ * request data into the DMA memory allocated for req, or it will simply
+ * reference the new request and use it in lieu of req during subsequent
+ * calls to hwrm_req_send(). The resource management is associated with
+ * req and is independent of and does not apply to new_req. The caller must
+ * ensure that the lifetime of new_req is least as long as req.
+ *
+ * Return: zero on success, negative error code otherwise:
+ *     E2BIG: Request is too large.
+ *     EINVAL: Invalid request to modify.
+ */
+int hwrm_req_replace(struct bnxt *bp, void *req, void *new_req, u32 len)
+{
+       struct bnxt_hwrm_ctx *ctx = __hwrm_ctx(bp, req);
+       struct input *internal_req = req;
+       u16 req_type;
+
+       if (!ctx)
+               return -EINVAL;
+
+       if (len > BNXT_HWRM_CTX_OFFSET)
+               return -E2BIG;
+
+       if ((bp->fw_cap & BNXT_FW_CAP_SHORT_CMD) || len > BNXT_HWRM_MAX_REQ_LEN) {
+               memcpy(internal_req, new_req, len);
+       } else {
+               internal_req->req_type = ((struct input *)new_req)->req_type;
+               ctx->req = new_req;
+       }
+
+       ctx->req_len = len;
+       ctx->req->resp_addr = cpu_to_le64(ctx->dma_handle +
+                                         BNXT_HWRM_RESP_OFFSET);
+
+       /* update sentinel for potentially new request type */
+       req_type = le16_to_cpu(internal_req->req_type);
+       ctx->sentinel = hwrm_calc_sentinel(ctx, req_type);
+
+       return 0;
+}
+
 /**
  * hwrm_req_flags() - Set non internal flags of the ctx
  * @bp: The driver context.
index 199c646f5e71293b685d65e1590decf87c78dca5..c58d84cc692aacd8b120ec08bbc07090f3367f4d 100644 (file)
@@ -139,4 +139,5 @@ void hwrm_req_flags(struct bnxt *bp, void *req, enum bnxt_hwrm_ctx_flags flags);
 void hwrm_req_timeout(struct bnxt *bp, void *req, unsigned int timeout);
 int hwrm_req_send(struct bnxt *bp, void *req);
 int hwrm_req_send_silent(struct bnxt *bp, void *req);
+int hwrm_req_replace(struct bnxt *bp, void *req, void *new_req, u32 len);
 #endif