Patch Detail
get:
Show a patch.
patch:
Update a patch.
put:
Update a patch.
GET /api/patches/91511/?format=api
{ "id": 91511, "url": "https://patches.dpdk.org/api/patches/91511/?format=api", "web_url": "https://patches.dpdk.org/project/dpdk/patch/1618451359-20693-28-git-send-email-timothy.mcdaniel@intel.com/", "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": "<1618451359-20693-28-git-send-email-timothy.mcdaniel@intel.com>", "list_archive_url": "https://inbox.dpdk.org/dev/1618451359-20693-28-git-send-email-timothy.mcdaniel@intel.com", "date": "2021-04-15T01:49:19", "name": "[v4,27/27] event/dlb: move rte config defines to runtime devargs", "commit_ref": null, "pull_url": null, "state": "superseded", "archived": true, "hash": "396f2834d8ba2260e52cffd7863e1e94f1ab7f66", "submitter": { "id": 826, "url": "https://patches.dpdk.org/api/people/826/?format=api", "name": "Timothy McDaniel", "email": "timothy.mcdaniel@intel.com" }, "delegate": { "id": 310, "url": "https://patches.dpdk.org/api/users/310/?format=api", "username": "jerin", "first_name": "Jerin", "last_name": "Jacob", "email": "jerinj@marvell.com" }, "mbox": "https://patches.dpdk.org/project/dpdk/patch/1618451359-20693-28-git-send-email-timothy.mcdaniel@intel.com/mbox/", "series": [ { "id": 16383, "url": "https://patches.dpdk.org/api/series/16383/?format=api", "web_url": "https://patches.dpdk.org/project/dpdk/list/?series=16383", "date": "2021-04-15T01:48:52", "name": "Add DLB v2.5", "version": 4, "mbox": "https://patches.dpdk.org/series/16383/mbox/" } ], "comments": "https://patches.dpdk.org/api/patches/91511/comments/", "check": "fail", "checks": "https://patches.dpdk.org/api/patches/91511/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 7F536A0562;\n\tThu, 15 Apr 2021 03:53:53 +0200 (CEST)", "from [217.70.189.124] (localhost [127.0.0.1])\n\tby mails.dpdk.org (Postfix) with ESMTP id 6BEDF161ED0;\n\tThu, 15 Apr 2021 03:51:12 +0200 (CEST)", "from mga09.intel.com (mga09.intel.com [134.134.136.24])\n by mails.dpdk.org (Postfix) with ESMTP id ED96D161E4E\n for <dev@dpdk.org>; Thu, 15 Apr 2021 03:50:49 +0200 (CEST)", "from orsmga003.jf.intel.com ([10.7.209.27])\n by orsmga102.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384;\n 14 Apr 2021 18:50:48 -0700", "from txasoft-yocto.an.intel.com ([10.123.72.192])\n by orsmga003.jf.intel.com with ESMTP; 14 Apr 2021 18:50:47 -0700" ], "IronPort-SDR": [ "\n YoBwpH2byBduYfzenSi0KntJpzqNjokKVsVBcS/wcc7Ixya/q4WJK2gD3a3HfyyinSqat7dn67\n Q2oIDCI2rBXg==", "\n Oo8KkBPs0BKUtZE0MaJ58xIxhCeEIparQXOC5xoJiqN5ha53wvUVyb5oKMrLL2MVH+az+vJcGn\n xrUlL7Ad56rw==" ], "X-IronPort-AV": [ "E=McAfee;i=\"6200,9189,9954\"; a=\"194881866\"", "E=Sophos;i=\"5.82,223,1613462400\"; d=\"scan'208\";a=\"194881866\"", "E=Sophos;i=\"5.82,223,1613462400\"; d=\"scan'208\";a=\"382569927\"" ], "X-ExtLoop1": "1", "From": "Timothy McDaniel <timothy.mcdaniel@intel.com>", "To": "", "Cc": "dev@dpdk.org, erik.g.carrillo@intel.com, harry.van.haaren@intel.com,\n jerinj@marvell.com, thomas@monjalon.net", "Date": "Wed, 14 Apr 2021 20:49:19 -0500", "Message-Id": "<1618451359-20693-28-git-send-email-timothy.mcdaniel@intel.com>", "X-Mailer": "git-send-email 1.7.10", "In-Reply-To": "<1618451359-20693-1-git-send-email-timothy.mcdaniel@intel.com>", "References": "<20210316221857.2254-2-timothy.mcdaniel@intel.com>\n <1618451359-20693-1-git-send-email-timothy.mcdaniel@intel.com>", "Subject": "[dpdk-dev] [PATCH v4 27/27] event/dlb: move rte config defines to\n runtime devargs", "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": "The new devarg names and their default values\nare listed below. The defaults have not changed, and\nnone of these parameters are accessed in the fast path.\n\npoll_interval=1000\nsw_credit_quantai=32\ndefault_depth_thresh=256\n\nSigned-off-by: Timothy McDaniel <timothy.mcdaniel@intel.com>\n---\n config/rte_config.h | 3 -\n drivers/event/dlb/dlb2.c | 109 +++++++++++++++++++++++++++++++--\n drivers/event/dlb/dlb2_priv.h | 14 +++++\n drivers/event/dlb/pf/dlb2_pf.c | 5 +-\n 4 files changed, 121 insertions(+), 10 deletions(-)", "diff": "diff --git a/config/rte_config.h b/config/rte_config.h\nindex 1aa852cd7..836aca3c2 100644\n--- a/config/rte_config.h\n+++ b/config/rte_config.h\n@@ -140,9 +140,6 @@\n #define RTE_LIBRTE_QEDE_FW \"\"\n \n /* DLB defines */\n-#define RTE_LIBRTE_PMD_DLB_POLL_INTERVAL 1000\n #undef RTE_LIBRTE_PMD_DLB_QUELL_STATS\n-#define RTE_LIBRTE_PMD_DLB_SW_CREDIT_QUANTA 32\n-#define RTE_LIBRTE_PMD_DLB_DEFAULT_DEPTH_THRESH 256\n \n #endif /* _RTE_CONFIG_H_ */\ndiff --git a/drivers/event/dlb/dlb2.c b/drivers/event/dlb/dlb2.c\nindex e5def9357..818b1c367 100644\n--- a/drivers/event/dlb/dlb2.c\n+++ b/drivers/event/dlb/dlb2.c\n@@ -315,6 +315,66 @@ set_cos(const char *key __rte_unused,\n \treturn 0;\n }\n \n+static int\n+set_poll_interval(const char *key __rte_unused,\n+\tconst char *value,\n+\tvoid *opaque)\n+{\n+\tint *poll_interval = opaque;\n+\tint ret;\n+\n+\tif (value == NULL || opaque == NULL) {\n+\t\tDLB2_LOG_ERR(\"NULL pointer\\n\");\n+\t\treturn -EINVAL;\n+\t}\n+\n+\tret = dlb2_string_to_int(poll_interval, value);\n+\tif (ret < 0)\n+\t\treturn ret;\n+\n+\treturn 0;\n+}\n+\n+static int\n+set_sw_credit_quanta(const char *key __rte_unused,\n+\tconst char *value,\n+\tvoid *opaque)\n+{\n+\tint *sw_credit_quanta = opaque;\n+\tint ret;\n+\n+\tif (value == NULL || opaque == NULL) {\n+\t\tDLB2_LOG_ERR(\"NULL pointer\\n\");\n+\t\treturn -EINVAL;\n+\t}\n+\n+\tret = dlb2_string_to_int(sw_credit_quanta, value);\n+\tif (ret < 0)\n+\t\treturn ret;\n+\n+\treturn 0;\n+}\n+\n+static int\n+set_default_depth_thresh(const char *key __rte_unused,\n+\tconst char *value,\n+\tvoid *opaque)\n+{\n+\tint *default_depth_thresh = opaque;\n+\tint ret;\n+\n+\tif (value == NULL || opaque == NULL) {\n+\t\tDLB2_LOG_ERR(\"NULL pointer\\n\");\n+\t\treturn -EINVAL;\n+\t}\n+\n+\tret = dlb2_string_to_int(default_depth_thresh, value);\n+\tif (ret < 0)\n+\t\treturn ret;\n+\n+\treturn 0;\n+}\n+\n static int\n set_qid_depth_thresh(const char *key __rte_unused,\n \t\t const char *value,\n@@ -923,9 +983,9 @@ dlb2_hw_create_ldb_queue(struct dlb2_eventdev *dlb2,\n \t}\n \n \tif (ev_queue->depth_threshold == 0) {\n-\t\tcfg.depth_threshold = RTE_LIBRTE_PMD_DLB_DEFAULT_DEPTH_THRESH;\n+\t\tcfg.depth_threshold = dlb2->default_depth_thresh;\n \t\tev_queue->depth_threshold =\n-\t\t\tRTE_LIBRTE_PMD_DLB_DEFAULT_DEPTH_THRESH;\n+\t\t\tdlb2->default_depth_thresh;\n \t} else\n \t\tcfg.depth_threshold = ev_queue->depth_threshold;\n \n@@ -1617,7 +1677,7 @@ dlb2_eventdev_port_setup(struct rte_eventdev *dev,\n \t\t RTE_EVENT_PORT_CFG_DISABLE_IMPL_REL);\n \tev_port->outstanding_releases = 0;\n \tev_port->inflight_credits = 0;\n-\tev_port->credit_update_quanta = RTE_LIBRTE_PMD_DLB_SW_CREDIT_QUANTA;\n+\tev_port->credit_update_quanta = dlb2->sw_credit_quanta;\n \tev_port->dlb2 = dlb2; /* reverse link */\n \n \t/* Tear down pre-existing port->queue links */\n@@ -1712,9 +1772,9 @@ dlb2_hw_create_dir_queue(struct dlb2_eventdev *dlb2,\n \tcfg.port_id = qm_port_id;\n \n \tif (ev_queue->depth_threshold == 0) {\n-\t\tcfg.depth_threshold = RTE_LIBRTE_PMD_DLB_DEFAULT_DEPTH_THRESH;\n+\t\tcfg.depth_threshold = dlb2->default_depth_thresh;\n \t\tev_queue->depth_threshold =\n-\t\t\tRTE_LIBRTE_PMD_DLB_DEFAULT_DEPTH_THRESH;\n+\t\t\tdlb2->default_depth_thresh;\n \t} else\n \t\tcfg.depth_threshold = ev_queue->depth_threshold;\n \n@@ -3065,7 +3125,7 @@ dlb2_dequeue_wait(struct dlb2_eventdev *dlb2,\n \n \t\tDLB2_INC_STAT(ev_port->stats.traffic.rx_umonitor_umwait, 1);\n \t} else {\n-\t\tuint64_t poll_interval = RTE_LIBRTE_PMD_DLB_POLL_INTERVAL;\n+\t\tuint64_t poll_interval = dlb2->poll_interval;\n \t\tuint64_t curr_ticks = rte_get_timer_cycles();\n \t\tuint64_t init_ticks = curr_ticks;\n \n@@ -4020,6 +4080,9 @@ dlb2_primary_eventdev_probe(struct rte_eventdev *dev,\n \tdlb2->max_num_events_override = dlb2_args->max_num_events;\n \tdlb2->num_dir_credits_override = dlb2_args->num_dir_credits_override;\n \tdlb2->qm_instance.cos_id = dlb2_args->cos_id;\n+\tdlb2->poll_interval = dlb2_args->poll_interval;\n+\tdlb2->sw_credit_quanta = dlb2_args->sw_credit_quanta;\n+\tdlb2->default_depth_thresh = dlb2_args->default_depth_thresh;\n \n \terr = dlb2_iface_open(&dlb2->qm_instance, name);\n \tif (err < 0) {\n@@ -4120,6 +4183,9 @@ dlb2_parse_params(const char *params,\n \t\t\t\t\t DEV_ID_ARG,\n \t\t\t\t\t DLB2_QID_DEPTH_THRESH_ARG,\n \t\t\t\t\t DLB2_COS_ARG,\n+\t\t\t\t\t DLB2_POLL_INTERVAL_ARG,\n+\t\t\t\t\t DLB2_SW_CREDIT_QUANTA_ARG,\n+\t\t\t\t\t DLB2_DEPTH_THRESH_ARG,\n \t\t\t\t\t NULL };\n \n \tif (params != NULL && params[0] != '\\0') {\n@@ -4202,6 +4268,37 @@ dlb2_parse_params(const char *params,\n \t\t\t\treturn ret;\n \t\t\t}\n \n+\t\t\tret = rte_kvargs_process(kvlist, DLB2_POLL_INTERVAL_ARG,\n+\t\t\t\t\t\t set_poll_interval,\n+\t\t\t\t\t\t &dlb2_args->poll_interval);\n+\t\t\tif (ret != 0) {\n+\t\t\t\tDLB2_LOG_ERR(\"%s: Error parsing poll interval parameter\",\n+\t\t\t\t\t name);\n+\t\t\t\trte_kvargs_free(kvlist);\n+\t\t\t\treturn ret;\n+\t\t\t}\n+\n+\t\t\tret = rte_kvargs_process(kvlist,\n+\t\t\t\t\t\t DLB2_SW_CREDIT_QUANTA_ARG,\n+\t\t\t\t\t\t set_sw_credit_quanta,\n+\t\t\t\t\t\t &dlb2_args->sw_credit_quanta);\n+\t\t\tif (ret != 0) {\n+\t\t\t\tDLB2_LOG_ERR(\"%s: Error parsing sw xredit quanta parameter\",\n+\t\t\t\t\t name);\n+\t\t\t\trte_kvargs_free(kvlist);\n+\t\t\t\treturn ret;\n+\t\t\t}\n+\n+\t\t\tret = rte_kvargs_process(kvlist, DLB2_DEPTH_THRESH_ARG,\n+\t\t\t\t\tset_default_depth_thresh,\n+\t\t\t\t\t&dlb2_args->default_depth_thresh);\n+\t\t\tif (ret != 0) {\n+\t\t\t\tDLB2_LOG_ERR(\"%s: Error parsing set depth thresh parameter\",\n+\t\t\t\t\t name);\n+\t\t\t\trte_kvargs_free(kvlist);\n+\t\t\t\treturn ret;\n+\t\t\t}\n+\n \t\t\trte_kvargs_free(kvlist);\n \t\t}\n \t}\ndiff --git a/drivers/event/dlb/dlb2_priv.h b/drivers/event/dlb/dlb2_priv.h\nindex f11e08fca..3c540a264 100644\n--- a/drivers/event/dlb/dlb2_priv.h\n+++ b/drivers/event/dlb/dlb2_priv.h\n@@ -23,6 +23,11 @@\n /* common name for all dlb devs (dlb v2.0, dlb v2.5 ...) */\n #define EVDEV_DLB2_NAME_PMD dlb_event\n \n+/* Default values for command line devargs */\n+#define DLB2_POLL_INTERVAL_DEFAULT 1000\n+#define DLB2_SW_CREDIT_QUANTA_DEFAULT 32\n+#define DLB2_DEPTH_THRESH_DEFAULT 256\n+\n /* command line arg strings */\n #define NUMA_NODE_ARG \"numa_node\"\n #define DLB2_MAX_NUM_EVENTS \"max_num_events\"\n@@ -31,6 +36,9 @@\n #define DLB2_DEFER_SCHED_ARG \"defer_sched\"\n #define DLB2_QID_DEPTH_THRESH_ARG \"qid_depth_thresh\"\n #define DLB2_COS_ARG \"cos\"\n+#define DLB2_POLL_INTERVAL_ARG \"poll_interval\"\n+#define DLB2_SW_CREDIT_QUANTA_ARG \"sw_credit_quanta\"\n+#define DLB2_DEPTH_THRESH_ARG \"default_depth_thresh\"\n \n /* Begin HW related defines and structs */\n \n@@ -571,6 +579,9 @@ struct dlb2_eventdev {\n \tbool global_dequeue_wait; /* Not using per dequeue wait if true */\n \tbool defer_sched;\n \tenum dlb2_cq_poll_modes poll_mode;\n+\tint poll_interval;\n+\tint sw_credit_quanta;\n+\tint default_depth_thresh;\n \tuint8_t revision;\n \tuint8_t version;\n \tbool configured;\n@@ -604,6 +615,9 @@ struct dlb2_devargs {\n \tint defer_sched;\n \tstruct dlb2_qid_depth_thresholds qid_depth_thresholds;\n \tenum dlb2_cos cos_id;\n+\tint poll_interval;\n+\tint sw_credit_quanta;\n+\tint default_depth_thresh;\n };\n \n /* End Eventdev related defines and structs */\ndiff --git a/drivers/event/dlb/pf/dlb2_pf.c b/drivers/event/dlb/pf/dlb2_pf.c\nindex f57dc1584..e9da89d65 100644\n--- a/drivers/event/dlb/pf/dlb2_pf.c\n+++ b/drivers/event/dlb/pf/dlb2_pf.c\n@@ -615,7 +615,10 @@ dlb2_eventdev_pci_init(struct rte_eventdev *eventdev)\n \t\t.max_num_events = DLB2_MAX_NUM_LDB_CREDITS,\n \t\t.num_dir_credits_override = -1,\n \t\t.qid_depth_thresholds = { {0} },\n-\t\t.cos_id = DLB2_COS_DEFAULT\n+\t\t.cos_id = DLB2_COS_DEFAULT,\n+\t\t.poll_interval = DLB2_POLL_INTERVAL_DEFAULT,\n+\t\t.sw_credit_quanta = DLB2_SW_CREDIT_QUANTA_DEFAULT,\n+\t\t.default_depth_thresh = DLB2_DEPTH_THRESH_DEFAULT\n \t};\n \tstruct dlb2_eventdev *dlb2;\n \n", "prefixes": [ "v4", "27/27" ] }