netdev
[Top] [All Lists]

[PATCH 2.6.12-rc2 r11/17] bonding: add driver name to log messages

To: fubar@xxxxxxxxxx, bonding-devel@xxxxxxxxxxxxxxxxxxxxx
Subject: [PATCH 2.6.12-rc2 r11/17] bonding: add driver name to log messages
From: Radheka Godse <radheka.godse@xxxxxxxxx>
Date: Fri, 8 Apr 2005 17:01:12 -0700 (PDT)
Cc: netdev@xxxxxxxxxxx
Replyto: "Radheka Godse" <radheka.godse@xxxxxxxxx>
Sender: netdev-bounce@xxxxxxxxxxx
Trivial fix to include DRVNAME in logs printed to /var/log/messages

Signed-off-by: Radheka Godse <radheka.godse@xxxxxxxxx>
Signed-off-by: Mitch Williams <mitch.a.williams@xxxxxxxxx>


diff -urN -X dontdiff linux-2.6.12-rc2clean/drivers/net/bonding/bond_3ad.c 
linux-2.6.12-rc2/drivers/net/bonding/bond_3ad.c
--- linux-2.6.12-rc2clean/drivers/net/bonding/bond_3ad.c        2005-04-07 
11:24:41.000000000 -0700
+++ linux-2.6.12-rc2/drivers/net/bonding/bond_3ad.c     2005-04-07 
13:57:28.000000000 -0700
@@ -1915,7 +1915,8 @@
        struct aggregator *aggregator;

        if (bond == NULL) {
-               printk(KERN_ERR "The slave %s is not attached to its bond\n", 
slave->dev->name);
+               printk(KERN_ERR DRV_NAME ": %s: The slave %s is not attached to its 
bond\n",
+                      slave->dev->master->name, slave->dev->name);
                return -1;
        }

@@ -2085,7 +2086,8 @@
                                        // clear the aggregator
                                        ad_clear_agg(temp_aggregator);
                                        if (select_new_active_agg) {
-                                               printk(KERN_INFO "Removing an active 
aggregator\n");
+                                               printk(KERN_INFO DRV_NAME ": %s: 
Removing an active aggregator\n",
+                                                      
slave->dev->master->name);
                                                // select new active aggregator
                                                
ad_agg_selection_logic(__get_first_agg(port));
                                        }
@@ -2364,7 +2366,8 @@
        }

        if (bond_3ad_get_active_agg_info(bond, &ad_info)) {
-               printk(KERN_DEBUG "ERROR: bond_3ad_get_active_agg_info 
failed\n");
+               printk(KERN_DEBUG DRV_NAME ": %s: Error: "
+                      "bond_3ad_get_active_agg_info failed\n", dev->name);
                goto out;
        }

@@ -2373,7 +2376,9 @@

        if (slaves_in_agg == 0) {
                /*the aggregator is empty*/
-               printk(KERN_DEBUG "ERROR: active aggregator is empty\n");
+               printk(KERN_DEBUG DRV_NAME ": %s: Error: active "
+                      "aggregator is empty\n",
+                      dev->name);
                goto out;
        }

diff -urN -X dontdiff linux-2.6.12-rc2clean/drivers/net/bonding/bond_main.c 
linux-2.6.12-rc2/drivers/net/bonding/bond_main.c
--- linux-2.6.12-rc2clean/drivers/net/bonding/bond_main.c       2005-04-07 
11:24:41.000000000 -0700
+++ linux-2.6.12-rc2/drivers/net/bonding/bond_main.c    2005-04-07 
14:35:41.000000000 -0700
@@ -1846,10 +1846,10 @@
                       new_slave->dev->name);

                if (bond->params.mode == BOND_MODE_8023AD) {
-                       printk(KERN_WARNING
-                              "Operation of 802.3ad mode requires ETHTOOL "
+                       printk(KERN_WARNING DRV_NAME
+                              ": %s: Warning: Operation of 802.3ad mode requires 
ETHTOOL "
                               "support in base driver for proper aggregator "
-                              "selection.\n");
+                              "selection.\n", bond_dev->name);
                }
        }

@@ -2677,8 +2677,11 @@
                        break;
                default:
                        /* Should not happen */
-                       printk(KERN_ERR "bonding: Error: %s  Illegal value 
(link=%d)\n",
-                              slave->dev->name, slave->link);
+ printk(KERN_ERR DRV_NAME + ": %s: Error: %s Illegal value (link=%d)\n",
+                              bond_dev->name,
+ slave->dev->name, + slave->link);
                        goto out;
                } /* end of switch (slave->link) */


<Prev in Thread] Current Thread [Next in Thread>
  • [PATCH 2.6.12-rc2 r11/17] bonding: add driver name to log messages, Radheka Godse <=