From: Gaetan Rivet <gaetan.rivet@6wind.com>
To: dev@dpdk.org
Cc: Gaetan Rivet <gaetan.rivet@6wind.com>
Subject: [PATCH v2 2/3] dev: remove vdev function dependency
Date: Thu, 1 Jun 2017 12:12:01 +0200 [thread overview]
Message-ID: <42654e975eee3b3a9650f1b1cfea26be08fa25d4.1496310671.git.gaetan.rivet@6wind.com> (raw)
In-Reply-To: <cover.1496310671.git.gaetan.rivet@6wind.com>
In-Reply-To: <cover.1496310671.git.gaetan.rivet@6wind.com>
Signed-off-by: Gaetan Rivet <gaetan.rivet@6wind.com>
---
lib/librte_eal/common/eal_common_dev.c | 49 +++++++++++++++++++---------------
1 file changed, 27 insertions(+), 22 deletions(-)
diff --git a/lib/librte_eal/common/eal_common_dev.c b/lib/librte_eal/common/eal_common_dev.c
index 968c66e..c286628 100644
--- a/lib/librte_eal/common/eal_common_dev.c
+++ b/lib/librte_eal/common/eal_common_dev.c
@@ -61,6 +61,7 @@ static int cmp_detached_dev_name(const struct rte_device *dev,
int rte_eal_dev_attach(const char *name, const char *devargs)
{
struct rte_device *dev;
+ struct rte_bus *bus;
int ret;
if (name == NULL || devargs == NULL) {
@@ -69,31 +70,35 @@ int rte_eal_dev_attach(const char *name, const char *devargs)
}
dev = rte_bus_find_device(NULL, cmp_detached_dev_name, name);
- if (dev) {
- struct rte_bus *bus;
-
+ if (dev)
bus = rte_bus_find_by_device(dev);
- if (!bus) {
- RTE_LOG(ERR, EAL, "Cannot find bus for device (%s)\n",
- name);
- return -EINVAL;
- }
-
- if (!bus->plug) {
- RTE_LOG(ERR, EAL, "Bus function not supported\n");
- return -ENOTSUP;
- }
-
+ else
+ bus = rte_bus_from_dev(name);
+ if (!bus) {
+ RTE_LOG(ERR, EAL, "Cannot find bus for device (%s)\n",
+ name);
+ return -EINVAL;
+ }
+ if (!bus->plug) {
+ RTE_LOG(ERR, EAL, "Device hotplug not supported on this bus\n");
+ return -ENOTSUP;
+ }
+ if (!dev) {
+ struct rte_devargs da;
+ char da_str[snprintf(NULL, 0, "%s,%s", name, devargs) + 1];
+
+ /*
+ * If we haven't found a bus device the user meant to hotplug
+ * a device instead.
+ */
+ snprintf(da_str, sizeof(da_str), "%s,%s", name, devargs);
+ ret = rte_eal_devargs_parse(da_str, &da);
+ if (ret)
+ return ret;
+ ret = bus->plug(&da);
+ } else {
ret = bus->plug(dev->devargs);
- goto out;
}
-
- /*
- * If we haven't found a bus device the user meant to "hotplug" a
- * virtual device instead.
- */
- ret = rte_vdev_init(name, devargs);
-out:
if (ret)
RTE_LOG(ERR, EAL, "Driver cannot attach the device (%s)\n",
name);
--
2.1.4
next prev parent reply other threads:[~2017-06-01 10:12 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-05-24 15:18 [PATCH 0/3] eal: complete attach / detach support Gaetan Rivet
2017-05-24 15:18 ` [PATCH 1/3] pci: implement attach/detach bus operation Gaetan Rivet
2017-05-24 15:18 ` [PATCH 2/3] vdev: implement attach " Gaetan Rivet
2017-05-24 15:18 ` [PATCH 3/3] vdev: use standard bus registration function Gaetan Rivet
2017-06-01 10:11 ` [PATCH v2 0/3] eal: complete attach / detach support Gaetan Rivet
2017-06-01 10:12 ` [PATCH v2 1/3] pci: implement plug/unplug bus operation Gaetan Rivet
2017-06-01 10:12 ` Gaetan Rivet [this message]
2017-06-01 10:12 ` [PATCH v2 3/3] vdev: use standard bus registration function Gaetan Rivet
2017-06-07 23:58 ` [PATCH v3 0/3] eal: complete attach / detach support Gaetan Rivet
2017-06-07 23:58 ` [PATCH v3 1/3] pci: implement hotplug bus operation Gaetan Rivet
2017-06-07 23:58 ` [PATCH v3 2/3] dev: remove vdev function dependency Gaetan Rivet
2017-06-21 14:42 ` Thomas Monjalon
2017-06-07 23:58 ` [PATCH v3 3/3] vdev: use standard bus registration function Gaetan Rivet
2017-06-21 14:41 ` Thomas Monjalon
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=42654e975eee3b3a9650f1b1cfea26be08fa25d4.1496310671.git.gaetan.rivet@6wind.com \
--to=gaetan.rivet@6wind.com \
--cc=dev@dpdk.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.