patches for DPDK stable branches
 help / color / mirror / Atom feed
* [dpdk-stable] [PATCH] bus/vdev: fix vdev stops probing when device fails
@ 2018-02-05 17:15 Moti Haimovsky
  2018-02-05 23:27 ` [dpdk-stable] [dpdk-dev] " Thomas Monjalon
  0 siblings, 1 reply; 2+ messages in thread
From: Moti Haimovsky @ 2018-02-05 17:15 UTC (permalink / raw)
  To: jianfeng.tan; +Cc: dev, Moti Haimovsky, stable

This commit modifies vdev_probe to continue probing all the devices
regardless of device probing failures.

Fixes: 810e5fa3688b ("vdev: move virtual device probing into a bus")
Cc: stable@dpdk.org

Signed-off-by: Moti Haimovsky <motih@mellanox.com>
---
 drivers/bus/vdev/vdev.c | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/drivers/bus/vdev/vdev.c b/drivers/bus/vdev/vdev.c
index 0c8a6a8..e4bc724 100644
--- a/drivers/bus/vdev/vdev.c
+++ b/drivers/bus/vdev/vdev.c
@@ -364,6 +364,7 @@ struct vdev_custom_scan {
 vdev_probe(void)
 {
 	struct rte_vdev_device *dev;
+	int ret = 0;
 
 	/* call the init function for each virtual device */
 	TAILQ_FOREACH(dev, &vdev_device_list, next) {
@@ -374,11 +375,11 @@ struct vdev_custom_scan {
 		if (vdev_probe_all_drivers(dev)) {
 			VDEV_LOG(ERR, "failed to initialize %s device\n",
 				rte_vdev_device_name(dev));
-			return -1;
+			ret = -1;
 		}
 	}
 
-	return 0;
+	return ret;
 }
 
 static struct rte_device *
-- 
1.8.3.1

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: [dpdk-stable] [dpdk-dev] [PATCH] bus/vdev: fix vdev stops probing when device fails
  2018-02-05 17:15 [dpdk-stable] [PATCH] bus/vdev: fix vdev stops probing when device fails Moti Haimovsky
@ 2018-02-05 23:27 ` Thomas Monjalon
  0 siblings, 0 replies; 2+ messages in thread
From: Thomas Monjalon @ 2018-02-05 23:27 UTC (permalink / raw)
  To: Moti Haimovsky; +Cc: dev, jianfeng.tan, stable

05/02/2018 18:15, Moti Haimovsky:
> This commit modifies vdev_probe to continue probing all the devices
> regardless of device probing failures.
> 
> Fixes: 810e5fa3688b ("vdev: move virtual device probing into a bus")

It is not the origin commit.
Anyway it is not really a fix but a better behaviour.

> Cc: stable@dpdk.org

Exceptionnaly, Fixes tag is removed but Cc stable is kept.

> Signed-off-by: Moti Haimovsky <motih@mellanox.com>

Applied, thanks

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2018-02-05 23:27 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-02-05 17:15 [dpdk-stable] [PATCH] bus/vdev: fix vdev stops probing when device fails Moti Haimovsky
2018-02-05 23:27 ` [dpdk-stable] [dpdk-dev] " Thomas Monjalon

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).