[dpdk-stable] patch 'bus/vdev: fix debug message on probing' has been queued to LTS release 17.11.7

Yongseok Koh yskoh at mellanox.com
Tue Jul 23 02:59:46 CEST 2019


Hi,

FYI, your patch has been queued to LTS release 17.11.7

Note it hasn't been pushed to http://dpdk.org/browse/dpdk-stable yet.
It will be pushed if I get no objection by 07/27/19. So please
shout if anyone has objection.

Also note that after the patch there's a diff of the upstream commit vs the
patch applied to the branch. This will indicate if there was any rebasing
needed to apply to the stable branch. If there were code changes for rebasing
(ie: not only metadata diffs), please double check that the rebase was
correctly done.

Thanks.

Yongseok

---
>From 1e8696459925a4655d695916cc6ab56020e87aaf Mon Sep 17 00:00:00 2001
From: Thomas Monjalon <thomas at monjalon.net>
Date: Thu, 21 Feb 2019 20:01:24 +0100
Subject: [PATCH] bus/vdev: fix debug message on probing

[ backported from upstream commit 4169ed6ed1c31b75f070cdccf3ee82561d4b79ce ]

The log was printing the device name two times,
first one being supposed to be the driver name.
As we don't know yet the driver name, the log is simplified.

Fixes: 9bf4901d1a11 ("bus/vdev: remove probe with driver name option")

Signed-off-by: Thomas Monjalon <thomas at monjalon.net>
Reviewed-by: Rami Rosen <ramirose at gmail.com>
Reviewed-by: Andrew Rybchenko <arybchenko at solarflare.com>
---
 drivers/bus/vdev/vdev.c | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/drivers/bus/vdev/vdev.c b/drivers/bus/vdev/vdev.c
index a0ffb534e8..18496e1fe5 100644
--- a/drivers/bus/vdev/vdev.c
+++ b/drivers/bus/vdev/vdev.c
@@ -105,9 +105,7 @@ vdev_probe_all_drivers(struct rte_vdev_device *dev)
 	int ret;
 
 	name = rte_vdev_device_name(dev);
-
-	VDEV_LOG(DEBUG, "Search driver %s to probe device %s\n", name,
-		rte_vdev_device_name(dev));
+	VDEV_LOG(DEBUG, "Search driver to probe device %s\n", name);
 
 	if (vdev_parse(name, &driver))
 		return -1;
-- 
2.21.0

---
  Diff of the applied patch vs upstream commit (please double-check if non-empty:
---
--- -	2019-07-22 17:55:07.543363633 -0700
+++ 0019-bus-vdev-fix-debug-message-on-probing.patch	2019-07-22 17:55:05.859470000 -0700
@@ -1,14 +1,15 @@
-From 4169ed6ed1c31b75f070cdccf3ee82561d4b79ce Mon Sep 17 00:00:00 2001
+From 1e8696459925a4655d695916cc6ab56020e87aaf Mon Sep 17 00:00:00 2001
 From: Thomas Monjalon <thomas at monjalon.net>
 Date: Thu, 21 Feb 2019 20:01:24 +0100
 Subject: [PATCH] bus/vdev: fix debug message on probing
 
+[ backported from upstream commit 4169ed6ed1c31b75f070cdccf3ee82561d4b79ce ]
+
 The log was printing the device name two times,
 first one being supposed to be the driver name.
 As we don't know yet the driver name, the log is simplified.
 
 Fixes: 9bf4901d1a11 ("bus/vdev: remove probe with driver name option")
-Cc: stable at dpdk.org
 
 Signed-off-by: Thomas Monjalon <thomas at monjalon.net>
 Reviewed-by: Rami Rosen <ramirose at gmail.com>
@@ -18,17 +19,17 @@
  1 file changed, 1 insertion(+), 3 deletions(-)
 
 diff --git a/drivers/bus/vdev/vdev.c b/drivers/bus/vdev/vdev.c
-index 2c03ca4185..7225411791 100644
+index a0ffb534e8..18496e1fe5 100644
 --- a/drivers/bus/vdev/vdev.c
 +++ b/drivers/bus/vdev/vdev.c
-@@ -144,9 +144,7 @@ vdev_probe_all_drivers(struct rte_vdev_device *dev)
+@@ -105,9 +105,7 @@ vdev_probe_all_drivers(struct rte_vdev_device *dev)
  	int ret;
  
  	name = rte_vdev_device_name(dev);
 -
--	VDEV_LOG(DEBUG, "Search driver %s to probe device %s", name,
+-	VDEV_LOG(DEBUG, "Search driver %s to probe device %s\n", name,
 -		rte_vdev_device_name(dev));
-+	VDEV_LOG(DEBUG, "Search driver to probe device %s", name);
++	VDEV_LOG(DEBUG, "Search driver to probe device %s\n", name);
  
  	if (vdev_parse(name, &driver))
  		return -1;


More information about the stable mailing list