Patch Detail
get:
Show a patch.
patch:
Update a patch.
put:
Update a patch.
GET /api/patches/93513/?format=api
https://patches.dpdk.org/api/patches/93513/?format=api", "web_url": "https://patches.dpdk.org/project/dpdk/patch/20210527152510.1551026-15-andrew.rybchenko@oktetlabs.ru/", "project": { "id": 1, "url": "https://patches.dpdk.org/api/projects/1/?format=api", "name": "DPDK", "link_name": "dpdk", "list_id": "dev.dpdk.org", "list_email": "dev@dpdk.org", "web_url": "http://core.dpdk.org", "scm_url": "git://dpdk.org/dpdk", "webscm_url": "http://git.dpdk.org/dpdk", "list_archive_url": "https://inbox.dpdk.org/dev", "list_archive_url_format": "https://inbox.dpdk.org/dev/{}", "commit_url_format": "" }, "msgid": "<20210527152510.1551026-15-andrew.rybchenko@oktetlabs.ru>", "list_archive_url": "https://inbox.dpdk.org/dev/20210527152510.1551026-15-andrew.rybchenko@oktetlabs.ru", "date": "2021-05-27T15:25:04", "name": "[14/20] common/sfc_efx/base: add counter stream MCDI wrappers", "commit_ref": null, "pull_url": null, "state": "superseded", "archived": true, "hash": "11e1a5af6c188b5eff6856ea3c01dbaf177161be", "submitter": { "id": 2013, "url": "https://patches.dpdk.org/api/people/2013/?format=api", "name": "Andrew Rybchenko", "email": "Andrew.Rybchenko@oktetlabs.ru" }, "delegate": null, "mbox": "https://patches.dpdk.org/project/dpdk/patch/20210527152510.1551026-15-andrew.rybchenko@oktetlabs.ru/mbox/", "series": [ { "id": 17144, "url": "https://patches.dpdk.org/api/series/17144/?format=api", "web_url": "https://patches.dpdk.org/project/dpdk/list/?series=17144", "date": "2021-05-27T15:24:50", "name": "net/sfc: support flow API COUNT action", "version": 1, "mbox": "https://patches.dpdk.org/series/17144/mbox/" } ], "comments": "https://patches.dpdk.org/api/patches/93513/comments/", "check": "warning", "checks": "https://patches.dpdk.org/api/patches/93513/checks/", "tags": {}, "related": [], "headers": { "Return-Path": "<dev-bounces@dpdk.org>", "X-Original-To": "patchwork@inbox.dpdk.org", "Delivered-To": "patchwork@inbox.dpdk.org", "Received": [ "from mails.dpdk.org (mails.dpdk.org [217.70.189.124])\n\tby inbox.dpdk.org (Postfix) with ESMTP id 7B12DA0546;\n\tThu, 27 May 2021 17:27:26 +0200 (CEST)", "from [217.70.189.124] (localhost [127.0.0.1])\n\tby mails.dpdk.org (Postfix) with ESMTP id 204BD41153;\n\tThu, 27 May 2021 17:26:12 +0200 (CEST)", "from shelob.oktetlabs.ru (shelob.oktetlabs.ru [91.220.146.113])\n by mails.dpdk.org (Postfix) with ESMTP id B84D941103\n for <dev@dpdk.org>; Thu, 27 May 2021 17:26:09 +0200 (CEST)", "by shelob.oktetlabs.ru (Postfix, from userid 122)\n id 8EB877F603; Thu, 27 May 2021 18:26:09 +0300 (MSK)", "from aros.oktetlabs.ru (aros.oktetlabs.ru [192.168.38.17])\n by shelob.oktetlabs.ru (Postfix) with ESMTP id 24B707F607;\n Thu, 27 May 2021 18:25:43 +0300 (MSK)" ], "X-Spam-Checker-Version": "SpamAssassin 3.4.2 (2018-09-13) on shelob.oktetlabs.ru", "X-Spam-Level": "", "X-Spam-Status": "No, score=0.8 required=5.0 tests=ALL_TRUSTED,\n DKIM_ADSP_DISCARD,\n URIBL_BLOCKED autolearn=no autolearn_force=no version=3.4.2", "DKIM-Filter": "OpenDKIM Filter v2.11.0 shelob.oktetlabs.ru 24B707F607", "Authentication-Results": "shelob.oktetlabs.ru/24B707F607; dkim=none;\n dkim-atps=neutral", "From": "Andrew Rybchenko <andrew.rybchenko@oktetlabs.ru>", "To": "dev@dpdk.org", "Cc": "Igor Romanov <igor.romanov@oktetlabs.ru>,\n Andy Moreton <amoreton@xilinx.com>, Ivan Malov <ivan.malov@oktetlabs.ru>", "Date": "Thu, 27 May 2021 18:25:04 +0300", "Message-Id": "<20210527152510.1551026-15-andrew.rybchenko@oktetlabs.ru>", "X-Mailer": "git-send-email 2.30.2", "In-Reply-To": "<20210527152510.1551026-1-andrew.rybchenko@oktetlabs.ru>", "References": "<20210527152510.1551026-1-andrew.rybchenko@oktetlabs.ru>", "MIME-Version": "1.0", "Content-Transfer-Encoding": "8bit", "Subject": "[dpdk-dev] [PATCH 14/20] common/sfc_efx/base: add counter stream\n MCDI wrappers", "X-BeenThere": "dev@dpdk.org", "X-Mailman-Version": "2.1.29", "Precedence": "list", "List-Id": "DPDK patches and discussions <dev.dpdk.org>", "List-Unsubscribe": "<https://mails.dpdk.org/options/dev>,\n <mailto:dev-request@dpdk.org?subject=unsubscribe>", "List-Archive": "<http://mails.dpdk.org/archives/dev/>", "List-Post": "<mailto:dev@dpdk.org>", "List-Help": "<mailto:dev-request@dpdk.org?subject=help>", "List-Subscribe": "<https://mails.dpdk.org/listinfo/dev>,\n <mailto:dev-request@dpdk.org?subject=subscribe>", "Errors-To": "dev-bounces@dpdk.org", "Sender": "\"dev\" <dev-bounces@dpdk.org>" }, "content": "From: Igor Romanov <igor.romanov@oktetlabs.ru>\n\nThe MCDIs will be used to control counter Rx queue packet flow.\n\nSigned-off-by: Igor Romanov <igor.romanov@oktetlabs.ru>\nSigned-off-by: Andrew Rybchenko <andrew.rybchenko@oktetlabs.ru>\nReviewed-by: Andy Moreton <amoreton@xilinx.com>\nReviewed-by: Ivan Malov <ivan.malov@oktetlabs.ru>\n---\n drivers/common/sfc_efx/base/efx.h | 32 ++++++\n drivers/common/sfc_efx/base/efx_mae.c | 138 ++++++++++++++++++++++++++\n drivers/common/sfc_efx/version.map | 3 +\n 3 files changed, 173 insertions(+)", "diff": "diff --git a/drivers/common/sfc_efx/base/efx.h b/drivers/common/sfc_efx/base/efx.h\nindex d0f8bc10b3..cc173d13c6 100644\n--- a/drivers/common/sfc_efx/base/efx.h\n+++ b/drivers/common/sfc_efx/base/efx.h\n@@ -4455,6 +4455,38 @@ efx_mae_counters_free(\n \t__in_ecount(n_counters)\t\tconst efx_counter_t *countersp,\n \t__out_opt\t\t\tuint32_t *gen_countp);\n \n+/* When set, include counters with a value of zero */\n+#define\tEFX_MAE_COUNTERS_STREAM_IN_ZERO_SQUASH_DISABLE\t(1U << 0)\n+\n+/*\n+ * Set if credit-based flow control is used. In this case the driver\n+ * must call efx_mae_counters_stream_give_credits() to notify the\n+ * packetiser of descriptors written.\n+ */\n+#define\tEFX_MAE_COUNTERS_STREAM_OUT_USES_CREDITS\t(1U << 0)\n+\n+LIBEFX_API\n+extern\t__checkReturn\t\t\tefx_rc_t\n+efx_mae_counters_stream_start(\n+\t__in\t\t\t\tefx_nic_t *enp,\n+\t__in\t\t\t\tuint16_t rxq_id,\n+\t__in\t\t\t\tuint16_t packet_size,\n+\t__in\t\t\t\tuint32_t flags_in,\n+\t__out\t\t\t\tuint32_t *flags_out);\n+\n+LIBEFX_API\n+extern\t__checkReturn\t\t\tefx_rc_t\n+efx_mae_counters_stream_stop(\n+\t__in\t\t\t\tefx_nic_t *enp,\n+\t__in\t\t\t\tuint16_t rxq_id,\n+\t__out_opt\t\t\tuint32_t *gen_countp);\n+\n+LIBEFX_API\n+extern\t__checkReturn\t\t\tefx_rc_t\n+efx_mae_counters_stream_give_credits(\n+\t__in\t\t\t\tefx_nic_t *enp,\n+\t__in\t\t\t\tuint32_t n_credits);\n+\n LIBEFX_API\n extern\t__checkReturn\t\t\tefx_rc_t\n efx_mae_action_set_free(\ndiff --git a/drivers/common/sfc_efx/base/efx_mae.c b/drivers/common/sfc_efx/base/efx_mae.c\nindex 955f1d4353..0b3131161b 100644\n--- a/drivers/common/sfc_efx/base/efx_mae.c\n+++ b/drivers/common/sfc_efx/base/efx_mae.c\n@@ -2766,6 +2766,144 @@ efx_mae_counters_free(\n \tEFSYS_PROBE(fail2);\n fail1:\n \tEFSYS_PROBE1(fail1, efx_rc_t, rc);\n+\n+\treturn (rc);\n+}\n+\n+\t__checkReturn\t\t\tefx_rc_t\n+efx_mae_counters_stream_start(\n+\t__in\t\t\t\tefx_nic_t *enp,\n+\t__in\t\t\t\tuint16_t rxq_id,\n+\t__in\t\t\t\tuint16_t packet_size,\n+\t__in\t\t\t\tuint32_t flags_in,\n+\t__out\t\t\t\tuint32_t *flags_out)\n+{\n+\tefx_mcdi_req_t req;\n+\tEFX_MCDI_DECLARE_BUF(payload, MC_CMD_MAE_COUNTERS_STREAM_START_IN_LEN,\n+\t\t\t MC_CMD_MAE_COUNTERS_STREAM_START_OUT_LEN);\n+\tefx_rc_t rc;\n+\n+\tEFX_STATIC_ASSERT(EFX_MAE_COUNTERS_STREAM_IN_ZERO_SQUASH_DISABLE ==\n+\t 1U << MC_CMD_MAE_COUNTERS_STREAM_START_IN_ZERO_SQUASH_DISABLE_LBN);\n+\n+\tEFX_STATIC_ASSERT(EFX_MAE_COUNTERS_STREAM_OUT_USES_CREDITS ==\n+\t 1U << MC_CMD_MAE_COUNTERS_STREAM_START_OUT_USES_CREDITS_LBN);\n+\n+\treq.emr_cmd = MC_CMD_MAE_COUNTERS_STREAM_START;\n+\treq.emr_in_buf = payload;\n+\treq.emr_in_length = MC_CMD_MAE_COUNTERS_STREAM_START_IN_LEN;\n+\treq.emr_out_buf = payload;\n+\treq.emr_out_length = MC_CMD_MAE_COUNTERS_STREAM_START_OUT_LEN;\n+\n+\tMCDI_IN_SET_WORD(req, MAE_COUNTERS_STREAM_START_IN_QID, rxq_id);\n+\tMCDI_IN_SET_WORD(req, MAE_COUNTERS_STREAM_START_IN_PACKET_SIZE,\n+\t\t\t packet_size);\n+\tMCDI_IN_SET_DWORD(req, MAE_COUNTERS_STREAM_START_IN_FLAGS, flags_in);\n+\n+\tefx_mcdi_execute(enp, &req);\n+\n+\tif (req.emr_rc != 0) {\n+\t\trc = req.emr_rc;\n+\t\tgoto fail1;\n+\t}\n+\n+\tif (req.emr_out_length_used <\n+\t MC_CMD_MAE_COUNTERS_STREAM_START_OUT_LEN) {\n+\t\trc = EMSGSIZE;\n+\t\tgoto fail2;\n+\t}\n+\n+\t*flags_out = MCDI_OUT_DWORD(req, MAE_COUNTERS_STREAM_START_OUT_FLAGS);\n+\n+\treturn (0);\n+\n+fail2:\n+\tEFSYS_PROBE(fail2);\n+fail1:\n+\tEFSYS_PROBE1(fail1, efx_rc_t, rc);\n+\n+\treturn (rc);\n+}\n+\n+\t__checkReturn\t\t\tefx_rc_t\n+efx_mae_counters_stream_stop(\n+\t__in\t\t\t\tefx_nic_t *enp,\n+\t__in\t\t\t\tuint16_t rxq_id,\n+\t__out_opt\t\t\tuint32_t *gen_countp)\n+{\n+\tefx_mcdi_req_t req;\n+\tEFX_MCDI_DECLARE_BUF(payload, MC_CMD_MAE_COUNTERS_STREAM_STOP_IN_LEN,\n+\t\t\t MC_CMD_MAE_COUNTERS_STREAM_STOP_OUT_LEN);\n+\tefx_rc_t rc;\n+\n+\treq.emr_cmd = MC_CMD_MAE_COUNTERS_STREAM_STOP;\n+\treq.emr_in_buf = payload;\n+\treq.emr_in_length = MC_CMD_MAE_COUNTERS_STREAM_STOP_IN_LEN;\n+\treq.emr_out_buf = payload;\n+\treq.emr_out_length = MC_CMD_MAE_COUNTERS_STREAM_STOP_OUT_LEN;\n+\n+\tMCDI_IN_SET_WORD(req, MAE_COUNTERS_STREAM_STOP_IN_QID, rxq_id);\n+\n+\tefx_mcdi_execute(enp, &req);\n+\n+\tif (req.emr_rc != 0) {\n+\t\trc = req.emr_rc;\n+\t\tgoto fail1;\n+\t}\n+\n+\tif (req.emr_out_length_used <\n+\t MC_CMD_MAE_COUNTERS_STREAM_STOP_OUT_LEN) {\n+\t\trc = EMSGSIZE;\n+\t\tgoto fail2;\n+\t}\n+\n+\tif (gen_countp != NULL) {\n+\t\t*gen_countp = MCDI_OUT_DWORD(req,\n+\t\t\t MAE_COUNTERS_STREAM_STOP_OUT_GENERATION_COUNT);\n+\t}\n+\n+\treturn (0);\n+\n+fail2:\n+\tEFSYS_PROBE(fail2);\n+fail1:\n+\tEFSYS_PROBE1(fail1, efx_rc_t, rc);\n+\n+\treturn (rc);\n+}\n+\n+\t__checkReturn\t\t\tefx_rc_t\n+efx_mae_counters_stream_give_credits(\n+\t__in\t\t\t\tefx_nic_t *enp,\n+\t__in\t\t\t\tuint32_t n_credits)\n+{\n+\tefx_mcdi_req_t req;\n+\tEFX_MCDI_DECLARE_BUF(payload,\n+\t\t\t MC_CMD_MAE_COUNTERS_STREAM_GIVE_CREDITS_IN_LEN,\n+\t\t\t MC_CMD_MAE_COUNTERS_STREAM_GIVE_CREDITS_OUT_LEN);\n+\tefx_rc_t rc;\n+\n+\treq.emr_cmd = MC_CMD_MAE_COUNTERS_STREAM_GIVE_CREDITS;\n+\treq.emr_in_buf = payload;\n+\treq.emr_in_length = MC_CMD_MAE_COUNTERS_STREAM_GIVE_CREDITS_IN_LEN;\n+\treq.emr_out_buf = payload;\n+\treq.emr_out_length = MC_CMD_MAE_COUNTERS_STREAM_GIVE_CREDITS_OUT_LEN;\n+\n+\tMCDI_IN_SET_DWORD(req, MAE_COUNTERS_STREAM_GIVE_CREDITS_IN_NUM_CREDITS,\n+\t\t\t n_credits);\n+\n+\tefx_mcdi_execute(enp, &req);\n+\n+\tif (req.emr_rc != 0) {\n+\t\trc = req.emr_rc;\n+\t\tgoto fail1;\n+\t}\n+\n+\treturn (0);\n+\n+fail1:\n+\tEFSYS_PROBE1(fail1, efx_rc_t, rc);\n+\n \treturn (rc);\n }\n \ndiff --git a/drivers/common/sfc_efx/version.map b/drivers/common/sfc_efx/version.map\nindex 30b243a1e7..622f5d4cf5 100644\n--- a/drivers/common/sfc_efx/version.map\n+++ b/drivers/common/sfc_efx/version.map\n@@ -104,6 +104,9 @@ INTERNAL {\n \tefx_mae_action_set_specs_equal;\n \tefx_mae_counters_alloc;\n \tefx_mae_counters_free;\n+\tefx_mae_counters_stream_give_credits;\n+\tefx_mae_counters_stream_start;\n+\tefx_mae_counters_stream_stop;\n \tefx_mae_encap_header_alloc;\n \tefx_mae_encap_header_free;\n \tefx_mae_fini;\n", "prefixes": [ "14/20" ] }{ "id": 93513, "url": "