[PATCH] net/core/dev: print rtnl kind as driver name for virtual devices

Subsystems: networking [general], the rest

STALE2544d

2 messages, 2 authors, 2019-09-22 · open the first message on its own page

[PATCH] net/core/dev: print rtnl kind as driver name for virtual devices

From: Konstantin Khlebnikov <hidden>
Date: 2019-09-20 12:16:00

Device kind gives more information than only arbitrary device name.

Signed-off-by: Konstantin Khlebnikov <redacted>
---
 net/core/dev.c |   15 ++++++---------
 1 file changed, 6 insertions(+), 9 deletions(-)
diff --git a/net/core/dev.c b/net/core/dev.c
index 71b18e80389f..c84561634afd 100644
--- a/net/core/dev.c
+++ b/net/core/dev.c
@@ -9651,17 +9651,14 @@ static int __net_init netdev_init(struct net *net)
  */
 const char *netdev_drivername(const struct net_device *dev)
 {
-	const struct device_driver *driver;
 	const struct device *parent;
 	const char *empty = "";
 
 	parent = dev->dev.parent;
-	if (!parent)
-		return empty;
-
-	driver = parent->driver;
-	if (driver && driver->name)
-		return driver->name;
+	if (parent)
+		return dev_driver_string(parent);
+	if (dev->rtnl_link_ops)
+		return dev->rtnl_link_ops->kind;
 	return empty;
 }
 
@@ -9677,8 +9674,8 @@ static void __netdev_printk(const char *level, const struct net_device *dev,
 				netdev_name(dev), netdev_reg_state(dev),
 				vaf);
 	} else if (dev) {
-		printk("%s%s%s: %pV",
-		       level, netdev_name(dev), netdev_reg_state(dev), vaf);
+		printk("%s%s %s%s: %pV", level, netdev_drivername(dev),
+		       netdev_name(dev), netdev_reg_state(dev), vaf);
 	} else {
 		printk("%s(NULL net_device): %pV", level, vaf);
 	}

Re: [PATCH] net/core/dev: print rtnl kind as driver name for virtual devices

From: Jakub Kicinski <hidden>
Date: 2019-09-22 22:21:23

On Fri, 20 Sep 2019 15:15:55 +0300, Konstantin Khlebnikov wrote:
Device kind gives more information than only arbitrary device name.

Signed-off-by: Konstantin Khlebnikov <redacted>
Hi Konstantin!

These look like nice improvements, unfortunately the net-next tree is
now closed [1], and will reopen some time after the merge window is
over. Hopefully you can gather feedback now, but I'm dropping the patch
from patchwork, and you'll have to resubmit once Dave opens net-next
again.

[1] https://www.kernel.org/doc/html/latest/networking/netdev-FAQ.html

Same goes for:
ipv6/addrconf: use netdev_info()/netdev_warn()/netdev_dbg() for logging
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help