Patch Detail
get:
Show a patch.
patch:
Update a patch.
put:
Update a patch.
GET /api/patches/86783/?format=api
{ "id": 86783, "url": "https://patches.dpdk.org/api/patches/86783/?format=api", "web_url": "https://patches.dpdk.org/project/dpdk/patch/1610968623-31345-3-git-send-email-xuemingl@nvidia.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": "<1610968623-31345-3-git-send-email-xuemingl@nvidia.com>", "list_archive_url": "https://inbox.dpdk.org/dev/1610968623-31345-3-git-send-email-xuemingl@nvidia.com", "date": "2021-01-18T11:16:56", "name": "[v4,2/9] ethdev: support representor port list", "commit_ref": null, "pull_url": null, "state": "superseded", "archived": true, "hash": "69d8af8699b5f07176435ee4ac31f329a4285e3d", "submitter": { "id": 1904, "url": "https://patches.dpdk.org/api/people/1904/?format=api", "name": "Xueming Li", "email": "xuemingl@nvidia.com" }, "delegate": { "id": 319, "url": "https://patches.dpdk.org/api/users/319/?format=api", "username": "fyigit", "first_name": "Ferruh", "last_name": "Yigit", "email": "ferruh.yigit@amd.com" }, "mbox": "https://patches.dpdk.org/project/dpdk/patch/1610968623-31345-3-git-send-email-xuemingl@nvidia.com/mbox/", "series": [ { "id": 14809, "url": "https://patches.dpdk.org/api/series/14809/?format=api", "web_url": "https://patches.dpdk.org/project/dpdk/list/?series=14809", "date": "2021-01-18T11:16:55", "name": "ethdev: support SubFunction representor", "version": 4, "mbox": "https://patches.dpdk.org/series/14809/mbox/" } ], "comments": "https://patches.dpdk.org/api/patches/86783/comments/", "check": "success", "checks": "https://patches.dpdk.org/api/patches/86783/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 C9E04A0A03;\n\tMon, 18 Jan 2021 12:17:23 +0100 (CET)", "from [217.70.189.124] (localhost [127.0.0.1])\n\tby mails.dpdk.org (Postfix) with ESMTP id 7A73F140D23;\n\tMon, 18 Jan 2021 12:17:18 +0100 (CET)", "from mellanox.co.il (mail-il-dmz.mellanox.com [193.47.165.129])\n by mails.dpdk.org (Postfix) with ESMTP id 177D4140D1B\n for <dev@dpdk.org>; Mon, 18 Jan 2021 12:17:15 +0100 (CET)", "from Internal Mail-Server by MTLPINE1 (envelope-from\n xuemingl@nvidia.com) with SMTP; 18 Jan 2021 13:17:12 +0200", "from nvidia.com (pegasus05.mtr.labs.mlnx [10.210.16.100])\n by labmailer.mlnx (8.13.8/8.13.8) with ESMTP id 10IBHBTp021675;\n Mon, 18 Jan 2021 13:17:12 +0200" ], "From": "Xueming Li <xuemingl@nvidia.com>", "To": "Thomas Monjalon <thomas@monjalon.net>,\n Ferruh Yigit <ferruh.yigit@intel.com>,\n Andrew Rybchenko <andrew.rybchenko@oktetlabs.ru>,\n Olivier Matz <olivier.matz@6wind.com>", "Cc": "dev@dpdk.org, Viacheslav Ovsiienko <viacheslavo@nvidia.com>,\n xuemingl@nvidia.com, Asaf Penso <asafp@nvidia.com>", "Date": "Mon, 18 Jan 2021 11:16:56 +0000", "Message-Id": "<1610968623-31345-3-git-send-email-xuemingl@nvidia.com>", "X-Mailer": "git-send-email 1.8.3.1", "In-Reply-To": [ "<1610968623-31345-1-git-send-email-xuemingl@nvidia.com>", "<1608303356-13089-2-git-send-email-xuemingl@nvidia.com>" ], "References": [ "<1610968623-31345-1-git-send-email-xuemingl@nvidia.com>", "<1608303356-13089-2-git-send-email-xuemingl@nvidia.com>" ], "Subject": "[dpdk-dev] [PATCH v4 2/9] ethdev: support representor port list", "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": "To support extended representor syntax, this patch extends the\nrepresentor list parsing to support for representor port range in\ndevargs, examples:\n representor=[1,2,3] - single list\n representor=[1,3-5,7,9-11] - list with singles and ranges\n\nSigned-off-by: Xueming Li <xuemingl@nvidia.com>\nAcked-by: Viacheslav Ovsiienko <viacheslavo@nvidia.com>\n---\n lib/librte_ethdev/ethdev_private.c | 105 ++++++++++++++---------------\n lib/librte_ethdev/ethdev_private.h | 3 -\n lib/librte_ethdev/rte_class_eth.c | 4 +-\n lib/librte_ethdev/rte_ethdev.c | 5 +-\n 4 files changed, 54 insertions(+), 63 deletions(-)", "diff": "diff --git a/lib/librte_ethdev/ethdev_private.c b/lib/librte_ethdev/ethdev_private.c\nindex c1a411dba4..12bcc7e98d 100644\n--- a/lib/librte_ethdev/ethdev_private.c\n+++ b/lib/librte_ethdev/ethdev_private.c\n@@ -38,77 +38,71 @@ eth_find_device(const struct rte_eth_dev *start, rte_eth_cmp_t cmp,\n \treturn NULL;\n }\n \n-int\n-rte_eth_devargs_parse_list(char *str, rte_eth_devargs_callback_t callback,\n-\tvoid *data)\n+static int\n+rte_eth_devargs_enlist(uint16_t *list, uint16_t *len_list,\n+\t\t const uint16_t max_list, uint16_t val)\n {\n-\tchar *str_start;\n-\tint state;\n-\tint result;\n-\n-\tif (*str != '[')\n-\t\t/* Single element, not a list */\n-\t\treturn callback(str, data);\n-\n-\t/* Sanity check, then strip the brackets */\n-\tstr_start = &str[strlen(str) - 1];\n-\tif (*str_start != ']') {\n-\t\tRTE_LOG(ERR, EAL, \"(%s): List does not end with ']'\\n\", str);\n-\t\treturn -EINVAL;\n-\t}\n-\tstr++;\n-\t*str_start = '\\0';\n+\tuint16_t i;\n \n-\t/* Process list elements */\n-\tstate = 0;\n-\twhile (1) {\n-\t\tif (state == 0) {\n-\t\t\tif (*str == '\\0')\n-\t\t\t\tbreak;\n-\t\t\tif (*str != ',') {\n-\t\t\t\tstr_start = str;\n-\t\t\t\tstate = 1;\n-\t\t\t}\n-\t\t} else if (state == 1) {\n-\t\t\tif (*str == ',' || *str == '\\0') {\n-\t\t\t\tif (str > str_start) {\n-\t\t\t\t\t/* Non-empty string fragment */\n-\t\t\t\t\t*str = '\\0';\n-\t\t\t\t\tresult = callback(str_start, data);\n-\t\t\t\t\tif (result < 0)\n-\t\t\t\t\t\treturn result;\n-\t\t\t\t}\n-\t\t\t\tstate = 0;\n-\t\t\t}\n-\t\t}\n-\t\tstr++;\n+\tif (*len_list >= max_list)\n+\t\treturn -1;\n+\tfor (i = 0; i < *len_list; i++) {\n+\t\tif (list[i] == val)\n+\t\t\treturn 0;\n \t}\n+\tlist[(*len_list)++] = val;\n \treturn 0;\n }\n \n-static int\n+static char *\n rte_eth_devargs_process_range(char *str, uint16_t *list, uint16_t *len_list,\n \tconst uint16_t max_list)\n {\n \tuint16_t lo, hi, val;\n \tint result;\n+\tchar *pos = str;\n \n \tresult = sscanf(str, \"%hu-%hu\", &lo, &hi);\n \tif (result == 1) {\n-\t\tif (*len_list >= max_list)\n-\t\t\treturn -ENOMEM;\n-\t\tlist[(*len_list)++] = lo;\n+\t\tif (rte_eth_devargs_enlist(list, len_list, max_list, lo) != 0)\n+\t\t\treturn NULL;\n \t} else if (result == 2) {\n-\t\tif (lo >= hi || lo > RTE_MAX_ETHPORTS || hi > RTE_MAX_ETHPORTS)\n-\t\t\treturn -EINVAL;\n+\t\tif (lo >= hi)\n+\t\t\treturn NULL;\n \t\tfor (val = lo; val <= hi; val++) {\n-\t\t\tif (*len_list >= max_list)\n-\t\t\t\treturn -ENOMEM;\n-\t\t\tlist[(*len_list)++] = val;\n+\t\t\tif (rte_eth_devargs_enlist(list, len_list, max_list,\n+\t\t\t\t\t\t val) != 0)\n+\t\t\t\treturn NULL;\n \t\t}\n \t} else\n-\t\treturn -EINVAL;\n-\treturn 0;\n+\t\treturn NULL;\n+\twhile (*pos != 0 && ((*pos >= '0' && *pos <= '9') || *pos == '-'))\n+\t\tpos++;\n+\treturn pos;\n+}\n+\n+static char *\n+rte_eth_devargs_process_list(char *str, uint16_t *list, uint16_t *len_list,\n+\tconst uint16_t max_list)\n+{\n+\tchar *pos = str;\n+\n+\tif (*pos == '[')\n+\t\tpos++;\n+\twhile (1) {\n+\t\tpos = rte_eth_devargs_process_range(pos, list, len_list,\n+\t\t\t\t\t\t max_list);\n+\t\tif (pos == NULL)\n+\t\t\treturn NULL;\n+\t\tif (*pos != ',') /* end of list */\n+\t\t\tbreak;\n+\t\tpos++;\n+\t}\n+\tif (*str == '[' && *pos != ']')\n+\t\treturn NULL;\n+\tif (*pos == ']')\n+\t\tpos++;\n+\treturn pos;\n }\n \n /*\n@@ -121,6 +115,9 @@ rte_eth_devargs_parse_representor_ports(char *str, void *data)\n \tstruct rte_eth_devargs *eth_da = data;\n \n \teth_da->type = RTE_ETH_REPRESENTOR_VF;\n-\treturn rte_eth_devargs_process_range(str, eth_da->representor_ports,\n+\tstr = rte_eth_devargs_process_list(str, eth_da->representor_ports,\n \t\tð_da->nb_representor_ports, RTE_MAX_ETHPORTS);\n+\tif (str == NULL)\n+\t\tRTE_LOG(ERR, EAL, \"wrong representor format: %s\\n\", str);\n+\treturn str == NULL ? -1 : 0;\n }\ndiff --git a/lib/librte_ethdev/ethdev_private.h b/lib/librte_ethdev/ethdev_private.h\nindex 905a45c337..220ddd4408 100644\n--- a/lib/librte_ethdev/ethdev_private.h\n+++ b/lib/librte_ethdev/ethdev_private.h\n@@ -26,9 +26,6 @@ eth_find_device(const struct rte_eth_dev *_start, rte_eth_cmp_t cmp,\n \t\tconst void *data);\n \n /* Parse devargs value for representor parameter. */\n-typedef int (*rte_eth_devargs_callback_t)(char *str, void *data);\n-int rte_eth_devargs_parse_list(char *str, rte_eth_devargs_callback_t callback,\n-\tvoid *data);\n int rte_eth_devargs_parse_representor_ports(char *str, void *data);\n \n #ifdef __cplusplus\ndiff --git a/lib/librte_ethdev/rte_class_eth.c b/lib/librte_ethdev/rte_class_eth.c\nindex 6338355e25..efe6149df5 100644\n--- a/lib/librte_ethdev/rte_class_eth.c\n+++ b/lib/librte_ethdev/rte_class_eth.c\n@@ -77,9 +77,7 @@ eth_representor_cmp(const char *key __rte_unused,\n \tif (values == NULL)\n \t\treturn -1;\n \tmemset(&representors, 0, sizeof(representors));\n-\tret = rte_eth_devargs_parse_list(values,\n-\t\t\trte_eth_devargs_parse_representor_ports,\n-\t\t\t&representors);\n+\tret = rte_eth_devargs_parse_representor_ports(values, &representors);\n \tfree(values);\n \tif (ret != 0)\n \t\treturn -1; /* invalid devargs value */\ndiff --git a/lib/librte_ethdev/rte_ethdev.c b/lib/librte_ethdev/rte_ethdev.c\nindex 17ddacc78d..2ac51ac149 100644\n--- a/lib/librte_ethdev/rte_ethdev.c\n+++ b/lib/librte_ethdev/rte_ethdev.c\n@@ -5542,9 +5542,8 @@ rte_eth_devargs_parse(const char *dargs, struct rte_eth_devargs *eth_da)\n \tfor (i = 0; i < args.count; i++) {\n \t\tpair = &args.pairs[i];\n \t\tif (strcmp(\"representor\", pair->key) == 0) {\n-\t\t\tresult = rte_eth_devargs_parse_list(pair->value,\n-\t\t\t\trte_eth_devargs_parse_representor_ports,\n-\t\t\t\teth_da);\n+\t\t\tresult = rte_eth_devargs_parse_representor_ports(\n+\t\t\t\t\tpair->value, eth_da);\n \t\t\tif (result < 0)\n \t\t\t\tgoto parse_cleanup;\n \t\t}\n", "prefixes": [ "v4", "2/9" ] }