Patch Detail
get:
Show a patch.
patch:
Update a patch.
put:
Update a patch.
GET /api/patches/45789/?format=api
{ "id": 45789, "url": "http://patches.dpdk.org/api/patches/45789/?format=api", "web_url": "http://patches.dpdk.org/project/dpdk/patch/20181001205228.3503-2-thomas@monjalon.net/", "project": { "id": 1, "url": "http://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": "<20181001205228.3503-2-thomas@monjalon.net>", "list_archive_url": "https://inbox.dpdk.org/dev/20181001205228.3503-2-thomas@monjalon.net", "date": "2018-10-01T20:52:25", "name": "[v4,1/4] devargs: remove deprecated functions", "commit_ref": null, "pull_url": null, "state": "superseded", "archived": true, "hash": "31910c5de039a0b86ad3cd2e1aa96b735193ad15", "submitter": { "id": 685, "url": "http://patches.dpdk.org/api/people/685/?format=api", "name": "Thomas Monjalon", "email": "thomas@monjalon.net" }, "delegate": { "id": 1, "url": "http://patches.dpdk.org/api/users/1/?format=api", "username": "tmonjalo", "first_name": "Thomas", "last_name": "Monjalon", "email": "thomas@monjalon.net" }, "mbox": "http://patches.dpdk.org/project/dpdk/patch/20181001205228.3503-2-thomas@monjalon.net/mbox/", "series": [ { "id": 1621, "url": "http://patches.dpdk.org/api/series/1621/?format=api", "web_url": "http://patches.dpdk.org/project/dpdk/list/?series=1621", "date": "2018-10-01T20:52:24", "name": "eal: simplify devargs and hotplug functions", "version": 4, "mbox": "http://patches.dpdk.org/series/1621/mbox/" } ], "comments": "http://patches.dpdk.org/api/patches/45789/comments/", "check": "success", "checks": "http://patches.dpdk.org/api/patches/45789/checks/", "tags": {}, "related": [], "headers": { "Return-Path": "<dev-bounces@dpdk.org>", "X-Original-To": "patchwork@dpdk.org", "Delivered-To": "patchwork@dpdk.org", "Received": [ "from [92.243.14.124] (localhost [127.0.0.1])\n\tby dpdk.org (Postfix) with ESMTP id 0E8A15F11;\n\tMon, 1 Oct 2018 22:52:37 +0200 (CEST)", "from out3-smtp.messagingengine.com (out3-smtp.messagingengine.com\n\t[66.111.4.27]) by dpdk.org (Postfix) with ESMTP id D4DAC5B26\n\tfor <dev@dpdk.org>; Mon, 1 Oct 2018 22:52:33 +0200 (CEST)", "from compute1.internal (compute1.nyi.internal [10.202.2.41])\n\tby mailout.nyi.internal (Postfix) with ESMTP id 729D32201F;\n\tMon, 1 Oct 2018 16:52:33 -0400 (EDT)", "from mailfrontend1 ([10.202.2.162])\n\tby compute1.internal (MEProxy); Mon, 01 Oct 2018 16:52:33 -0400", "from xps.monjalon.net (184.203.134.77.rev.sfr.net [77.134.203.184])\n\tby mail.messagingengine.com (Postfix) with ESMTPA id E5460E405E;\n\tMon, 1 Oct 2018 16:52:31 -0400 (EDT)" ], "DKIM-Signature": [ "v=1; a=rsa-sha256; c=relaxed/relaxed; d=monjalon.net; h=\n\tcc:content-transfer-encoding:date:from:in-reply-to:message-id\n\t:mime-version:references:subject:to:x-me-sender:x-me-sender\n\t:x-sasl-enc; s=mesmtp; bh=Ww7RTigvMXMBjAgM+XfSaHH3yS1bEIgHeE+Vgb\n\tzipHQ=; b=Kvrq121TsUwyQSGkgN6dNFr9deuXQpPoDppwTMvxAdJzBHQZXW3can\n\tSXTHj9LKMbjj0RvfTWo13iu/BC8I5BJX24hPOytWy9ZD8DywzUS/f4Il5m7pXXGu\n\tv+jS4m8otH8T161qBVSZQIKOj55OJISBzL1BnQQBVMza/p64UV7O4=", "v=1; a=rsa-sha256; c=relaxed/relaxed; d=\n\tmessagingengine.com; h=cc:content-transfer-encoding:date:from\n\t:in-reply-to:message-id:mime-version:references:subject:to\n\t:x-me-sender:x-me-sender:x-sasl-enc; s=fm3; bh=Ww7RTigvMXMBjAgM+\n\tXfSaHH3yS1bEIgHeE+VgbzipHQ=; b=rlLmf6GU7yJx4YrPBatlU7ivh8MLJ0U/2\n\tPs9kkV4ghWJ83epJGhQbTxk+wrsSoRVJ1/2cQ3/w6AFbP+gIGEOXITAv95acn4pu\n\todiKWhKwKg0MaJBAqlIYbdwZ0O/j9y8V1Em7O8vjZiOnoGmHTdpLjW0V6VxjMcto\n\tKDc+Pg3ZPASzopDvFEuAYRiawvPeesU7KcbWjEnEwlB21oA0kRYzH4U7PxT5LfTb\n\tzq1isiyPPBIUqra2B0VtshkwI5vbilQsfYYlmgfjGJABA8tc3bUHheEeq7h+vs52\n\tSaSvTLn5h707hQaRXI24tPsYc3grOfHrcplambxes4XgQ1tpZ5XFg==" ], "X-ME-Sender": "<xms:EImyWx8AeX_BdEq6_jbPssT8Ut3E6boYK2ok4KAp0iuW-Ox9zWf0Bg>", "X-ME-Proxy": "<xmx:EImyW6LXS1kUVzFi9A8RfsSIgyuloHMg533ZdwF26CK9yG1r5wCdAg>\n\t<xmx:EImyWwpvYggUGSkvTplTI8gJGzgiZWCOD7GjVLFqiLFe3al8fVUJmg>\n\t<xmx:EImyWxa5nVS8Ly7XDCK6h_-Mgq3BPWd2DBH6pO8Ai1771wUMWNuBuQ>\n\t<xmx:EImyW-h87vvSwJASTVOXsWZcGECz-ExHjznz7P_swe2O8WxaDad9ag>\n\t<xmx:EImyW2rv_tjrLhABC4yWQem2wOzCTZiBOKRXaJ0Qa60ZJqaedGgaHw>\n\t<xmx:EYmyWyLnGkBuoiAlFfRQr6Hf49c1YEY5yvTYdxUfMRTBuxsR5CAICA>", "From": "Thomas Monjalon <thomas@monjalon.net>", "To": "dev@dpdk.org", "Cc": "gaetan.rivet@6wind.com, ophirmu@mellanox.com, qi.z.zhang@intel.com,\n\tferruh.yigit@intel.com, ktraynor@redhat.com", "Date": "Mon, 1 Oct 2018 22:52:25 +0200", "Message-Id": "<20181001205228.3503-2-thomas@monjalon.net>", "X-Mailer": "git-send-email 2.19.0", "In-Reply-To": "<20181001205228.3503-1-thomas@monjalon.net>", "References": "<20180907222727.20521-1-thomas@monjalon.net>\n\t<20181001205228.3503-1-thomas@monjalon.net>", "MIME-Version": "1.0", "Content-Transfer-Encoding": "8bit", "Subject": "[dpdk-dev] [PATCH v4 1/4] devargs: remove deprecated functions", "X-BeenThere": "dev@dpdk.org", "X-Mailman-Version": "2.1.15", "Precedence": "list", "List-Id": "DPDK patches and discussions <dev.dpdk.org>", "List-Unsubscribe": "<https://mails.dpdk.org/options/dev>,\n\t<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\t<mailto:dev-request@dpdk.org?subject=subscribe>", "Errors-To": "dev-bounces@dpdk.org", "Sender": "\"dev\" <dev-bounces@dpdk.org>" }, "content": "rte_eal_parse_devargs_str() does not support parsing the bus name\nat the start of devargs. So it was renamed and deprecated.\n\nrte_eal_devargs_add(), rte_eal_devargs_type_count() and\nrte_eal_devargs_dump() were declared deprecated and had their\nimplementation body renamed.\n\nAll these functions were deprecated in release 18.05.\n\nSigned-off-by: Thomas Monjalon <thomas@monjalon.net>\nReviewed-by: Andrew Rybchenko <arybchenko@solarflare.com>\n---\nThe library version is not updated because it should be done\nin earlier patch https://patches.dpdk.org/patch/43903/\n---\n doc/guides/rel_notes/release_18_11.rst | 5 ++\n lib/librte_eal/common/eal_common_devargs.c | 30 ---------\n lib/librte_eal/common/include/rte_devargs.h | 71 ---------------------\n lib/librte_eal/rte_eal_version.map | 4 --\n 4 files changed, 5 insertions(+), 105 deletions(-)", "diff": "diff --git a/doc/guides/rel_notes/release_18_11.rst b/doc/guides/rel_notes/release_18_11.rst\nindex 9c00e33cc..ef902b028 100644\n--- a/doc/guides/rel_notes/release_18_11.rst\n+++ b/doc/guides/rel_notes/release_18_11.rst\n@@ -102,6 +102,11 @@ API Changes\n Also, make sure to start the actual text at the margin.\n =========================================================\n \n+* eal: The following devargs functions, which were deprecated in 18.05,\n+ are removed in 18.11:\n+ ``rte_eal_parse_devargs_str()``, ``rte_eal_devargs_add()``,\n+ ``rte_eal_devargs_type_count()``, and ``rte_eal_devargs_dump()``.\n+\n * mbuf: The ``__rte_mbuf_raw_free()`` and ``__rte_pktmbuf_prefree_seg()``\n functions were deprecated since 17.05 and are replaced by\n ``rte_mbuf_raw_free()`` and ``rte_pktmbuf_prefree_seg()``.\ndiff --git a/lib/librte_eal/common/eal_common_devargs.c b/lib/librte_eal/common/eal_common_devargs.c\nindex dac2402a4..f63d2c663 100644\n--- a/lib/librte_eal/common/eal_common_devargs.c\n+++ b/lib/librte_eal/common/eal_common_devargs.c\n@@ -31,36 +31,6 @@ TAILQ_HEAD(rte_devargs_list, rte_devargs);\n struct rte_devargs_list devargs_list =\n \tTAILQ_HEAD_INITIALIZER(devargs_list);\n \n-int\n-rte_eal_parse_devargs_str(const char *devargs_str,\n-\t\t\tchar **drvname, char **drvargs)\n-{\n-\tchar *sep;\n-\n-\tif ((devargs_str) == NULL || (drvname) == NULL || (drvargs == NULL))\n-\t\treturn -1;\n-\n-\t*drvname = strdup(devargs_str);\n-\tif (*drvname == NULL)\n-\t\treturn -1;\n-\n-\t/* set the first ',' to '\\0' to split name and arguments */\n-\tsep = strchr(*drvname, ',');\n-\tif (sep != NULL) {\n-\t\tsep[0] = '\\0';\n-\t\t*drvargs = strdup(sep + 1);\n-\t} else {\n-\t\t*drvargs = strdup(\"\");\n-\t}\n-\n-\tif (*drvargs == NULL) {\n-\t\tfree(*drvname);\n-\t\t*drvname = NULL;\n-\t\treturn -1;\n-\t}\n-\treturn 0;\n-}\n-\n static size_t\n devargs_layer_count(const char *s)\n {\ndiff --git a/lib/librte_eal/common/include/rte_devargs.h b/lib/librte_eal/common/include/rte_devargs.h\nindex 097a4ce7b..0eef6e9c4 100644\n--- a/lib/librte_eal/common/include/rte_devargs.h\n+++ b/lib/librte_eal/common/include/rte_devargs.h\n@@ -66,36 +66,6 @@ struct rte_devargs {\n \tconst char *data; /**< Device string storage. */\n };\n \n-/**\n- * @deprecated\n- * Parse a devargs string.\n- *\n- * For PCI devices, the format of arguments string is \"PCI_ADDR\" or\n- * \"PCI_ADDR,key=val,key2=val2,...\". Examples: \"08:00.1\", \"0000:5:00.0\",\n- * \"04:00.0,arg=val\".\n- *\n- * For virtual devices, the format of arguments string is \"DRIVER_NAME*\"\n- * or \"DRIVER_NAME*,key=val,key2=val2,...\". Examples: \"net_ring\",\n- * \"net_ring0\", \"net_pmdAnything,arg=0:arg2=1\".\n- *\n- * The function parses the arguments string to get driver name and driver\n- * arguments.\n- *\n- * @param devargs_str\n- * The arguments as given by the user.\n- * @param drvname\n- * The pointer to the string to store parsed driver name.\n- * @param drvargs\n- * The pointer to the string to store parsed driver arguments.\n- *\n- * @return\n- * - 0 on success\n- * - A negative value on error\n- */\n-__rte_deprecated\n-int rte_eal_parse_devargs_str(const char *devargs_str,\n-\t\t\t\tchar **drvname, char **drvargs);\n-\n /**\n * Parse a device string.\n *\n@@ -201,23 +171,6 @@ rte_devargs_insert(struct rte_devargs *da);\n __rte_experimental\n int rte_devargs_add(enum rte_devtype devtype, const char *devargs_str);\n \n-/**\n- * @deprecated\n- * Add a device to the user device list\n- * See rte_devargs_parse() for details.\n- *\n- * @param devtype\n- * The type of the device.\n- * @param devargs_str\n- * The arguments as given by the user.\n- *\n- * @return\n- * - 0 on success\n- * - A negative value on error\n- */\n-__rte_deprecated\n-int rte_eal_devargs_add(enum rte_devtype devtype, const char *devargs_str);\n-\n /**\n * Remove a device from the user device list.\n * Its resources are freed.\n@@ -251,20 +204,6 @@ __rte_experimental\n unsigned int\n rte_devargs_type_count(enum rte_devtype devtype);\n \n-/**\n- * @deprecated\n- * Count the number of user devices of a specified type\n- *\n- * @param devtype\n- * The type of the devices to counted.\n- *\n- * @return\n- * The number of devices.\n- */\n-__rte_deprecated\n-unsigned int\n-rte_eal_devargs_type_count(enum rte_devtype devtype);\n-\n /**\n * This function dumps the list of user device and their arguments.\n *\n@@ -274,16 +213,6 @@ rte_eal_devargs_type_count(enum rte_devtype devtype);\n __rte_experimental\n void rte_devargs_dump(FILE *f);\n \n-/**\n- * @deprecated\n- * This function dumps the list of user device and their arguments.\n- *\n- * @param f\n- * A pointer to a file for output\n- */\n-__rte_deprecated\n-void rte_eal_devargs_dump(FILE *f);\n-\n /**\n * Find next rte_devargs matching the provided bus name.\n *\ndiff --git a/lib/librte_eal/rte_eal_version.map b/lib/librte_eal/rte_eal_version.map\nindex 73282bbb0..3df7f9831 100644\n--- a/lib/librte_eal/rte_eal_version.map\n+++ b/lib/librte_eal/rte_eal_version.map\n@@ -19,9 +19,6 @@ DPDK_2.0 {\n \trte_dump_tailq;\n \trte_eal_alarm_cancel;\n \trte_eal_alarm_set;\n-\trte_eal_devargs_add;\n-\trte_eal_devargs_dump;\n-\trte_eal_devargs_type_count;\n \trte_eal_get_configuration;\n \trte_eal_get_lcore_state;\n \trte_eal_get_physmem_size;\n@@ -32,7 +29,6 @@ DPDK_2.0 {\n \trte_eal_lcore_role;\n \trte_eal_mp_remote_launch;\n \trte_eal_mp_wait_lcore;\n-\trte_eal_parse_devargs_str;\n \trte_eal_process_type;\n \trte_eal_remote_launch;\n \trte_eal_tailq_lookup;\n", "prefixes": [ "v4", "1/4" ] }