Patch Detail
get:
Show a patch.
patch:
Update a patch.
put:
Update a patch.
GET /api/patches/45463/?format=api
{ "id": 45463, "url": "http://patches.dpdk.org/api/patches/45463/?format=api", "web_url": "http://patches.dpdk.org/project/dpdk/patch/20180926214759.1856-5-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": "<20180926214759.1856-5-thomas@monjalon.net>", "list_archive_url": "https://inbox.dpdk.org/dev/20180926214759.1856-5-thomas@monjalon.net", "date": "2018-09-26T21:47:59", "name": "[v2,4/4] eal: simplify parameters of hotplug functions", "commit_ref": null, "pull_url": null, "state": "superseded", "archived": true, "hash": "411a6218aa81a1f6b2bbd9587407f2e12f2fe8aa", "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/20180926214759.1856-5-thomas@monjalon.net/mbox/", "series": [ { "id": 1528, "url": "http://patches.dpdk.org/api/series/1528/?format=api", "web_url": "http://patches.dpdk.org/project/dpdk/list/?series=1528", "date": "2018-09-26T21:47:55", "name": "eal: simplify devargs and hotplug functions", "version": 2, "mbox": "http://patches.dpdk.org/series/1528/mbox/" } ], "comments": "http://patches.dpdk.org/api/patches/45463/comments/", "check": "warning", "checks": "http://patches.dpdk.org/api/patches/45463/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 056CB1B495;\n\tWed, 26 Sep 2018 23:48:27 +0200 (CEST)", "from out1-smtp.messagingengine.com (out1-smtp.messagingengine.com\n\t[66.111.4.25]) by dpdk.org (Postfix) with ESMTP id 5E0731B476\n\tfor <dev@dpdk.org>; Wed, 26 Sep 2018 23:48:16 +0200 (CEST)", "from compute1.internal (compute1.nyi.internal [10.202.2.41])\n\tby mailout.nyi.internal (Postfix) with ESMTP id B3BC522012;\n\tWed, 26 Sep 2018 17:48:15 -0400 (EDT)", "from mailfrontend2 ([10.202.2.163])\n\tby compute1.internal (MEProxy); Wed, 26 Sep 2018 17:48:15 -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 93B6A102DE;\n\tWed, 26 Sep 2018 17:48:14 -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=98beZJkfU9s9fKJaJqKU66PYXrBmnCd+EQEthx\n\tpdOvY=; b=cbOTYn8oLcCGj5TMaGdYhJPl//OPGup5EQLk//Ag+5ZGYTNj3EkWar\n\tu5Ce8v1s0vSzYSggduuu+JyeT5fT0NhYqCRbZSVKyrrmZgu0ADkE6kmCAUcJ8RXJ\n\tIw0cNG3DmCtVuXMpfgIQJDUNysk6sGqfYD5dfVf80mjedgPZoBHMA=", "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=98beZJkfU9s9fKJaJ\n\tqKU66PYXrBmnCd+EQEthxpdOvY=; b=mADttBm0gh7g+pZXO0t26pjweetn8xZ/2\n\tM0YSFKvSWKZY32PAy6Wn59JGGLZkJOL/Q18obuDAH4cT2Infj90YhK1qJ1cUtBnb\n\tj+SfAXOYcsMfPJiLlhxKcvoLvLe8l636ZP2uGkOTv7/hW2px3VjhATbrYBMnjP0V\n\tbaQDjbGxkFzt0mWlScdbE514aHlj6H2AWpIKMVhWERZfMsoxvQxSIgcxxWddLZLo\n\tIfDS6JTBpbvRhe8NIPXV4rGjPowPUBEduTmmBMXHz92TgreLX7vs2J7hDTnKinlu\n\tJ7xC3F/736vYLHEajbsdi1Ix19afsNKAs8uarSHHFdOOVM3Cdw4jw==" ], "X-ME-Proxy": "<xmx:n_6rW9GPgrlY9lXA9YI7vrlehwpZApcAINLr7uhamcTWmAysoZMYug>\n\t<xmx:n_6rW0LtISJVm8YUQWvU24IEX1qOFTy16DGqcWrkp_OwSVya4kBJhg>\n\t<xmx:n_6rWw572JYuMQ6bpOPpHfxqn7SojW2t2qsR_uOCLrjUV3JZN87zcg>\n\t<xmx:n_6rW0TDxBWf-UXwXQMjaoOPNxQ9Oji1xAm5Vl7t-eiIKE5wKSAd5w>\n\t<xmx:n_6rW6CWStpyUTHXKwDaMi5f7dC3D0sIoMtLA0tpf799oXWvh2jthg>\n\t<xmx:n_6rWxqjc7ytnJEBRH_SPa_3Ak4YNl3ztvnsHa75cT_qYXrn0lJjuA>", "X-ME-Sender": "<xms:n_6rW13VZjzVTxrNjzMhl8B_OolNeLiMUyoNv4t__a317A5nqHv5rg>", "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": "Wed, 26 Sep 2018 23:47:59 +0200", "Message-Id": "<20180926214759.1856-5-thomas@monjalon.net>", "X-Mailer": "git-send-email 2.19.0", "In-Reply-To": "<20180926214759.1856-1-thomas@monjalon.net>", "References": "<20180907222727.20521-1-thomas@monjalon.net>\n\t<20180926214759.1856-1-thomas@monjalon.net>", "MIME-Version": "1.0", "Content-Transfer-Encoding": "8bit", "Subject": "[dpdk-dev] [PATCH v2 4/4] eal: simplify parameters of hotplug\n\tfunctions", "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": "All information about a device to probe can be grouped\nin a common string, which is what we usually call devargs.\nAn application should not have to parse this string before\ncalling the EAL probe function.\nAnd the syntax could evolve to be more complex and support\nmatching multiple devices in one string.\nThat's why the bus name and device name should be removed from\nrte_eal_hotplug_add().\nInstead of changing this function, a simpler one is added\nand used in the old one, which may be deprecated later.\n\nWhen removing a device, we already know its rte_device handle\nwhich can be directly passed as parameter of rte_eal_hotplug_remove().\nIf the rte_device is not known, it can be retrieved with the devargs,\nby iterating in the device list (future RTE_DEV_FOREACH()).\nSimilarly to the probing case, a new function is added\nand used in the old one, which may be deprecated later.\nThe new function is used in failsafe, because the replacement is easy.\n\nSigned-off-by: Thomas Monjalon <thomas@monjalon.net>\n---\n drivers/net/failsafe/failsafe_eal.c | 3 +-\n drivers/net/failsafe/failsafe_ether.c | 3 +-\n lib/librte_eal/common/eal_common_dev.c | 76 ++++++++++++++++---------\n lib/librte_eal/common/include/rte_dev.h | 30 +++++++++-\n lib/librte_eal/rte_eal_version.map | 2 +\n 5 files changed, 80 insertions(+), 34 deletions(-)", "diff": "diff --git a/drivers/net/failsafe/failsafe_eal.c b/drivers/net/failsafe/failsafe_eal.c\nindex ce1633f13..8a888b1ff 100644\n--- a/drivers/net/failsafe/failsafe_eal.c\n+++ b/drivers/net/failsafe/failsafe_eal.c\n@@ -144,8 +144,7 @@ fs_bus_uninit(struct rte_eth_dev *dev)\n \tint ret = 0;\n \n \tFOREACH_SUBDEV_STATE(sdev, i, dev, DEV_PROBED) {\n-\t\tsdev_ret = rte_eal_hotplug_remove(sdev->bus->name,\n-\t\t\t\t\t\t\tsdev->dev->name);\n+\t\tsdev_ret = rte_dev_remove(sdev->dev);\n \t\tif (sdev_ret) {\n \t\t\tERROR(\"Failed to remove requested device %s (err: %d)\",\n \t\t\t sdev->dev->name, sdev_ret);\ndiff --git a/drivers/net/failsafe/failsafe_ether.c b/drivers/net/failsafe/failsafe_ether.c\nindex 5b5cb3b49..cd7b97e1f 100644\n--- a/drivers/net/failsafe/failsafe_ether.c\n+++ b/drivers/net/failsafe/failsafe_ether.c\n@@ -265,8 +265,7 @@ fs_dev_remove(struct sub_device *sdev)\n \t\tsdev->state = DEV_PROBED;\n \t\t/* fallthrough */\n \tcase DEV_PROBED:\n-\t\tret = rte_eal_hotplug_remove(sdev->bus->name,\n-\t\t\t\t\t sdev->dev->name);\n+\t\tret = rte_dev_remove(sdev->dev);\n \t\tif (ret) {\n \t\t\tERROR(\"Bus detach failed for sub_device %u\",\n \t\t\t SUB_ID(sdev));\ndiff --git a/lib/librte_eal/common/eal_common_dev.c b/lib/librte_eal/common/eal_common_dev.c\nindex 43fdab395..85eb1569f 100644\n--- a/lib/librte_eal/common/eal_common_dev.c\n+++ b/lib/librte_eal/common/eal_common_dev.c\n@@ -129,46 +129,57 @@ int rte_eal_dev_detach(struct rte_device *dev)\n \n int\n rte_eal_hotplug_add(const char *busname, const char *devname,\n- const char *devargs)\n+ const char *drvargs)\n+{\n+\tchar *devargs = NULL;\n+\tint size, length = -1;\n+\n+\tdo { /* 2 iterations: first is to know string length */\n+\t\tsize = length + 1;\n+\t\tlength = snprintf(devargs, size, \"%s:%s,%s\", busname, devname, drvargs);\n+\t\tif (length >= size)\n+\t\t\tdevargs = malloc(length + 1);\n+\t\tif (devargs == NULL)\n+\t\t\treturn -ENOMEM;\n+\t} while (size == 0);\n+\n+\treturn rte_dev_probe(devargs);\n+}\n+\n+int __rte_experimental\n+rte_dev_probe(const char *devargs)\n {\n-\tstruct rte_bus *bus;\n \tstruct rte_device *dev;\n \tstruct rte_devargs *da;\n \tint ret;\n \n-\tbus = rte_bus_find_by_name(busname);\n-\tif (bus == NULL) {\n-\t\tRTE_LOG(ERR, EAL, \"Cannot find bus (%s)\\n\", busname);\n-\t\treturn -ENOENT;\n-\t}\n-\n-\tif (bus->plug == NULL) {\n-\t\tRTE_LOG(ERR, EAL, \"Function plug not supported by bus (%s)\\n\",\n-\t\t\tbus->name);\n-\t\treturn -ENOTSUP;\n-\t}\n-\n \tda = calloc(1, sizeof(*da));\n \tif (da == NULL)\n \t\treturn -ENOMEM;\n \n-\tret = rte_devargs_parsef(da, \"%s:%s,%s\",\n-\t\t\t\t busname, devname, devargs);\n+\tret = rte_devargs_parse(da, devargs);\n \tif (ret)\n \t\tgoto err_devarg;\n \n+\tif (da->bus->plug == NULL) {\n+\t\tRTE_LOG(ERR, EAL, \"Function plug not supported by bus (%s)\\n\",\n+\t\t\tda->bus->name);\n+\t\tret = -ENOTSUP;\n+\t\tgoto err_devarg;\n+\t}\n+\n \tret = rte_devargs_insert(da);\n \tif (ret)\n \t\tgoto err_devarg;\n \n-\tret = bus->scan();\n+\tret = da->bus->scan();\n \tif (ret)\n \t\tgoto err_devarg;\n \n-\tdev = bus->find_device(NULL, cmp_dev_name, devname);\n+\tdev = da->bus->find_device(NULL, cmp_dev_name, da->name);\n \tif (dev == NULL) {\n \t\tRTE_LOG(ERR, EAL, \"Cannot find device (%s)\\n\",\n-\t\t\tdevname);\n+\t\t\tda->name);\n \t\tret = -ENODEV;\n \t\tgoto err_devarg;\n \t}\n@@ -178,7 +189,7 @@ rte_eal_hotplug_add(const char *busname, const char *devname,\n \t\treturn -EEXIST;\n \t}\n \n-\tret = bus->plug(dev);\n+\tret = da->bus->plug(dev);\n \tif (ret) {\n \t\tRTE_LOG(ERR, EAL, \"Driver cannot attach the device (%s)\\n\",\n \t\t\tdev->name);\n@@ -197,9 +208,8 @@ rte_eal_hotplug_add(const char *busname, const char *devname,\n int\n rte_eal_hotplug_remove(const char *busname, const char *devname)\n {\n-\tstruct rte_bus *bus;\n \tstruct rte_device *dev;\n-\tint ret;\n+\tstruct rte_bus *bus;\n \n \tbus = rte_bus_find_by_name(busname);\n \tif (bus == NULL) {\n@@ -207,23 +217,33 @@ rte_eal_hotplug_remove(const char *busname, const char *devname)\n \t\treturn -ENOENT;\n \t}\n \n-\tif (bus->unplug == NULL) {\n-\t\tRTE_LOG(ERR, EAL, \"Function unplug not supported by bus (%s)\\n\",\n-\t\t\tbus->name);\n-\t\treturn -ENOTSUP;\n-\t}\n-\n \tdev = bus->find_device(NULL, cmp_dev_name, devname);\n \tif (dev == NULL) {\n \t\tRTE_LOG(ERR, EAL, \"Cannot find plugged device (%s)\\n\", devname);\n \t\treturn -EINVAL;\n \t}\n \n+\treturn rte_dev_remove(dev);\n+}\n+\n+int __rte_experimental\n+rte_dev_remove(struct rte_device *dev)\n+{\n+\tstruct rte_bus *bus;\n+\tint ret;\n+\n \tif (dev->driver == NULL) {\n \t\tRTE_LOG(ERR, EAL, \"Device is already unplugged\\n\");\n \t\treturn -ENOENT;\n \t}\n \n+\tbus = dev->devargs->bus;\n+\tif (bus->unplug == NULL) {\n+\t\tRTE_LOG(ERR, EAL, \"Function unplug not supported by bus (%s)\\n\",\n+\t\t\tbus->name);\n+\t\treturn -ENOTSUP;\n+\t}\n+\n \tret = bus->unplug(dev);\n \tif (ret)\n \t\tRTE_LOG(ERR, EAL, \"Driver cannot detach the device (%s)\\n\",\ndiff --git a/lib/librte_eal/common/include/rte_dev.h b/lib/librte_eal/common/include/rte_dev.h\nindex 4f5282459..7a30362c0 100644\n--- a/lib/librte_eal/common/include/rte_dev.h\n+++ b/lib/librte_eal/common/include/rte_dev.h\n@@ -196,13 +196,26 @@ int rte_eal_dev_detach(struct rte_device *dev);\n * @param devname\n * The device name. Based on this device name, eal will identify a driver\n * capable of handling it and pass it to the driver probing function.\n- * @param devargs\n+ * @param drvargs\n * Device arguments to be passed to the driver.\n * @return\n * 0 on success, negative on error.\n */\n int rte_eal_hotplug_add(const char *busname, const char *devname,\n- const char *devargs);\n+ const char *drvargs);\n+\n+/**\n+ * @warning\n+ * @b EXPERIMENTAL: this API may change without prior notice\n+ *\n+ * Add matching devices.\n+ *\n+ * @param devargs\n+ * Device arguments including bus, class and driver properties.\n+ * @return\n+ * 0 on success, negative on error.\n+ */\n+int __rte_experimental rte_dev_probe(const char *devargs);\n \n /**\n * Hotplug remove a given device from a specific bus.\n@@ -216,6 +229,19 @@ int rte_eal_hotplug_add(const char *busname, const char *devname,\n */\n int rte_eal_hotplug_remove(const char *busname, const char *devname);\n \n+/**\n+ * @warning\n+ * @b EXPERIMENTAL: this API may change without prior notice\n+ *\n+ * Remove one device.\n+ *\n+ * @param dev\n+ * Data structure of the device to remove.\n+ * @return\n+ * 0 on success, negative on error.\n+ */\n+int __rte_experimental rte_dev_remove(struct rte_device *dev);\n+\n /**\n * Device comparison function.\n *\ndiff --git a/lib/librte_eal/rte_eal_version.map b/lib/librte_eal/rte_eal_version.map\nindex 6bff37f4b..2ea7a870a 100644\n--- a/lib/librte_eal/rte_eal_version.map\n+++ b/lib/librte_eal/rte_eal_version.map\n@@ -281,6 +281,8 @@ EXPERIMENTAL {\n \trte_dev_event_monitor_stop;\n \trte_dev_iterator_init;\n \trte_dev_iterator_next;\n+\trte_dev_probe;\n+\trte_dev_remove;\n \trte_devargs_add;\n \trte_devargs_dump;\n \trte_devargs_insert;\n", "prefixes": [ "v2", "4/4" ] }