bus/vdev: don't double space log messages
PATCH 1/1 bus/vdev: don't double space log messages
The VDEV_LOG() macro already adds a newline, don't duplicate.
Signed-off-by Stephen Hemminger <[email protected]>
--- drivers/bus/vdev/vdev.c | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-)
drivers/bus/vdev/vdev.c +4 -4
@@ -142,7 +142,7 @@ vdev_probe_all_drivers(struct rte_vdev_device *dev) name =rte_vdev_device_name (dev); -VDEV_LOG (DEBUG ,"Search driver %s to probe device %s\n" , name, +VDEV_LOG (DEBUG ,"Search driver %s to probe device %s" , name,rte_vdev_device_name (dev));if (vdev_parse (name, &driver))
@@ -255,7 +255,7 @@ rte_vdev_init(const char *name, const char *args) ret =vdev_probe_all_drivers (dev);if (ret) {if (ret >0 ) -VDEV_LOG (ERR ,"no driver found for %s\n" , name); +VDEV_LOG (ERR ,"no driver found for %s" , name);/* If fails, remove it from vdev list */ devargs = dev->device.devargs;TAILQ_REMOVE (&vdev_device_list, dev, next);
@@ -274,7 +274,7 @@ vdev_remove_driver(struct rte_vdev_device *dev)const struct rte_vdev_driver *driver;if (!dev->device.driver) { -VDEV_LOG (DEBUG ,"no driver attach to device %s\n" , name); +VDEV_LOG (DEBUG ,"no driver attach to device %s" , name);return 1 ; }
@@ -482,7 +482,7 @@ vdev_probe(void)continue ;if (vdev_probe_all_drivers (dev)) { -VDEV_LOG (ERR ,"failed to initialize %s device\n" , +VDEV_LOG (ERR ,"failed to initialize %s device" ,rte_vdev_device_name (dev)); ret =-1 ; }
Signature · 1 line
2.17.0
Quoted 3 lines · Stephen Hemminger
The
VDEV_LOG()macro already adds a newline, don't duplicate.Signed-off-by Stephen Hemminger <[email protected]>
Fixes d22fcb225c24 ("bus/vdev: change log type")
Reviewed-by Ferruh Yigit <[email protected]>
Thomas Monjalon
Applied17/05/2018 11:33, Ferruh Yigit:
Quoted 8 lines · Stephen Hemminger
Quoted 3 lines
The
VDEV_LOG()macro already adds a newline, don't duplicate.Signed-off-by Stephen Hemminger <[email protected]>Fixes d22fcb225c24 ("bus/vdev: change log type")Reviewed-by Ferruh Yigit <[email protected]>
Applied, thanks