Differences

This shows you the differences between two versions of the page.

--- printing_debugging [2018/09/25 12:55] – [drivers/base/core.c] rpjday
+++ printing_debugging [2018/09/25 13:27] (current) – [__netdev_printk()] rpjday
@@ Line 144: / Line 144: @@
 ==== drivers/base/core.c ====
+=== dev_printk() ===
 <code>
-int dev_vprintk_emit(int level, const struct device *dev,
+void dev_printk(const char *level, const struct device *dev,
-		     const char *fmt, va_list args)
+		const char *fmt, ...)
 {
-	char hdr[128];
+	struct va_format vaf;
-	size_t hdrlen;
+	va_list args;
-	hdrlen = create_syslog_header(dev, hdr, sizeof(hdr));
+	va_start(args, fmt);
-	return vprintk_emit(0, level, hdrlen ? hdr : NULL, hdrlen, fmt, args);
+	vaf.fmt = fmt;
+	vaf.va = &args;
+	__dev_printk(level, dev, &vaf);
+	va_end(args);
 }
-EXPORT_SYMBOL(dev_vprintk_emit);
+EXPORT_SYMBOL(dev_printk);
+</code>
+=== __dev_printk() ===
+<code>
 static void __dev_printk(const char *level, const struct device *dev,
 			struct va_format *vaf)
@@ Line 167: / Line 178: @@
 		printk("%s(NULL device *): %pV", level, vaf);
 }
+</code>
-void dev_printk(const char *level, const struct device *dev,
+=== dev_printk_emit() ===
-		const char *fmt, ...)
+<code>
+int dev_printk_emit(int level, const struct device *dev, const char *fmt, ...)
 {
-	struct va_format vaf;
 	va_list args;
+	int r;
 	va_start(args, fmt);
-	vaf.fmt = fmt;
+	r = dev_vprintk_emit(level, dev, fmt, args);
-	vaf.va = &args;
-	__dev_printk(level, dev, &vaf);
 	va_end(args);
+	return r;
 }
-EXPORT_SYMBOL(dev_printk);
+EXPORT_SYMBOL(dev_printk_emit);
 </code>
+=== dev_vprintk_emit() ===
+<code>
+int dev_vprintk_emit(int level, const struct device *dev,
+		     const char *fmt, va_list args)
+{
+	char hdr[128];
+	size_t hdrlen;
+	hdrlen = create_syslog_header(dev, hdr, sizeof(hdr));
+	return vprintk_emit(0, level, hdrlen ? hdr : NULL, hdrlen, fmt, args);
+}
+</code>
+===== netdev debugging (net/core/dev.c) =====
+==== Defining macros ====
+<code>
+#define define_netdev_printk_level(func, level)                 \
+void func(const struct net_device *dev, const char *fmt, ...)   \
+{                                                               \
+        struct va_format vaf;                                   \
+        va_list args;                                           \
+                                                                \
+        va_start(args, fmt);                                    \
+                                                                \
+        vaf.fmt = fmt;                                          \
+        vaf.va = &args;                                         \
+                                                                \
+        __netdev_printk(level, dev, &vaf);                      \
+                                                                \
+        va_end(args);                                           \
+}                                                               \
+EXPORT_SYMBOL(func);
+define_netdev_printk_level(netdev_emerg, KERN_EMERG);
+define_netdev_printk_level(netdev_alert, KERN_ALERT);
+define_netdev_printk_level(netdev_crit, KERN_CRIT);
+define_netdev_printk_level(netdev_err, KERN_ERR);
+define_netdev_printk_level(netdev_warn, KERN_WARNING);
+define_netdev_printk_level(netdev_notice, KERN_NOTICE);
+define_netdev_printk_level(netdev_info, KERN_INFO);
+</code>
+==== Alternate way of calling __netdev_printk() ====
+<code>
+void netdev_printk(const char *level, const struct net_device *dev,
+                   const char *format, ...)
+{
+        struct va_format vaf;
+        va_list args;
+        va_start(args, format);
+        vaf.fmt = format;
+        vaf.va = &args;
+        __netdev_printk(level, dev, &vaf);
+        va_end(args);
+}
+EXPORT_SYMBOL(netdev_printk);
+</code>
+==== __netdev_printk() ====
+<code>
+static void __netdev_printk(const char *level, const struct net_device *dev,
+                            struct va_format *vaf)
+{
+        if (dev && dev->dev.parent) {
+                dev_printk_emit(level[1] - '0',
+                                dev->dev.parent,
+                                "%s %s %s%s: %pV",
+                                dev_driver_string(dev->dev.parent),
+                                dev_name(dev->dev.parent),
+                                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);
+        } else {
+                printk("%s(NULL net_device): %pV", level, vaf);
+        }
+}
+</code>
+===== Dynamic debugging =====
+... coming ...