Patchwork [net-next] bonding: combine pr_debugs in bond_set_dev_addr into one

login
register
mail settings
Submitter Nikolay Aleksandrov
Date June 29, 2013, 11:16 a.m.
Message ID <1372504619-30123-1-git-send-email-nikolay@redhat.com>
Download mbox | patch
Permalink /patch/255731/
State Accepted
Delegated to: David Miller
Headers show

Comments

Nikolay Aleksandrov - June 29, 2013, 11:16 a.m.
Combine the multiple pr_debugs in bond_set_dev_addr into one pr_debug.

Signed-off-by: Nikolay Aleksandrov <nikolay@redhat.com>
---
 drivers/net/bonding/bond_main.c | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)
David Miller - June 29, 2013, 7:37 p.m.
From: Nikolay Aleksandrov <nikolay@redhat.com>
Date: Sat, 29 Jun 2013 13:16:59 +0200

> Combine the multiple pr_debugs in bond_set_dev_addr into one pr_debug.
> 
> Signed-off-by: Nikolay Aleksandrov <nikolay@redhat.com>

Applied, thanks.
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Patch

diff --git a/drivers/net/bonding/bond_main.c b/drivers/net/bonding/bond_main.c
index ec44580b..742c193 100644
--- a/drivers/net/bonding/bond_main.c
+++ b/drivers/net/bonding/bond_main.c
@@ -843,9 +843,8 @@  static void bond_hw_addr_swap(struct bonding *bond, struct slave *new_active,
 static void bond_set_dev_addr(struct net_device *bond_dev,
 			      struct net_device *slave_dev)
 {
-	pr_debug("bond_dev=%p\n", bond_dev);
-	pr_debug("slave_dev=%p\n", slave_dev);
-	pr_debug("slave_dev->addr_len=%d\n", slave_dev->addr_len);
+	pr_debug("bond_dev=%p slave_dev=%p slave_dev->addr_len=%d\n",
+		 bond_dev, slave_dev, slave_dev->addr_len);
 	memcpy(bond_dev->dev_addr, slave_dev->dev_addr, slave_dev->addr_len);
 	bond_dev->addr_assign_type = NET_ADDR_STOLEN;
 	call_netdevice_notifiers(NETDEV_CHANGEADDR, bond_dev);