diff options
author | Harvey Harrison <harvey.harrison@gmail.com> | 2008-10-31 03:56:00 -0400 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2008-10-31 03:56:00 -0400 |
commit | 63779436ab4ad0867bcea53bf853b0004d7b895d (patch) | |
tree | bd4e49449eb852a54f62cc8bb1c807e26746dd43 /drivers/net/bonding/bond_main.c | |
parent | 21454aaad30651ba0dcc16fe5271bc12ee21f132 (diff) |
drivers: replace NIPQUAD()
Using NIPQUAD() with NIPQUAD_FMT, %d.%d.%d.%d or %u.%u.%u.%u
can be replaced with %pI4
Signed-off-by: Harvey Harrison <harvey.harrison@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/bonding/bond_main.c')
-rw-r--r-- | drivers/net/bonding/bond_main.c | 22 |
1 files changed, 10 insertions, 12 deletions
diff --git a/drivers/net/bonding/bond_main.c b/drivers/net/bonding/bond_main.c index fd9028d7c06e..56c823c175fe 100644 --- a/drivers/net/bonding/bond_main.c +++ b/drivers/net/bonding/bond_main.c | |||
@@ -2585,8 +2585,8 @@ static void bond_arp_send_all(struct bonding *bond, struct slave *slave) | |||
2585 | if (rv) { | 2585 | if (rv) { |
2586 | if (net_ratelimit()) { | 2586 | if (net_ratelimit()) { |
2587 | printk(KERN_WARNING DRV_NAME | 2587 | printk(KERN_WARNING DRV_NAME |
2588 | ": %s: no route to arp_ip_target %u.%u.%u.%u\n", | 2588 | ": %s: no route to arp_ip_target %pI4\n", |
2589 | bond->dev->name, NIPQUAD(fl.fl4_dst)); | 2589 | bond->dev->name, &fl.fl4_dst); |
2590 | } | 2590 | } |
2591 | continue; | 2591 | continue; |
2592 | } | 2592 | } |
@@ -2622,8 +2622,8 @@ static void bond_arp_send_all(struct bonding *bond, struct slave *slave) | |||
2622 | 2622 | ||
2623 | if (net_ratelimit()) { | 2623 | if (net_ratelimit()) { |
2624 | printk(KERN_WARNING DRV_NAME | 2624 | printk(KERN_WARNING DRV_NAME |
2625 | ": %s: no path to arp_ip_target %u.%u.%u.%u via rt.dev %s\n", | 2625 | ": %s: no path to arp_ip_target %pI4 via rt.dev %s\n", |
2626 | bond->dev->name, NIPQUAD(fl.fl4_dst), | 2626 | bond->dev->name, &fl.fl4_dst, |
2627 | rt->u.dst.dev ? rt->u.dst.dev->name : "NULL"); | 2627 | rt->u.dst.dev ? rt->u.dst.dev->name : "NULL"); |
2628 | } | 2628 | } |
2629 | ip_rt_put(rt); | 2629 | ip_rt_put(rt); |
@@ -2672,10 +2672,8 @@ static void bond_validate_arp(struct bonding *bond, struct slave *slave, __be32 | |||
2672 | 2672 | ||
2673 | targets = bond->params.arp_targets; | 2673 | targets = bond->params.arp_targets; |
2674 | for (i = 0; (i < BOND_MAX_ARP_TARGETS) && targets[i]; i++) { | 2674 | for (i = 0; (i < BOND_MAX_ARP_TARGETS) && targets[i]; i++) { |
2675 | dprintk("bva: sip %u.%u.%u.%u tip %u.%u.%u.%u t[%d] " | 2675 | dprintk("bva: sip %pI4 tip %pI4 t[%d] %pI4 bhti(tip) %d\n", |
2676 | "%u.%u.%u.%u bhti(tip) %d\n", | 2676 | &sip, &tip, i, &targets[i], bond_has_this_ip(bond, tip)); |
2677 | NIPQUAD(sip), NIPQUAD(tip), i, NIPQUAD(targets[i]), | ||
2678 | bond_has_this_ip(bond, tip)); | ||
2679 | if (sip == targets[i]) { | 2677 | if (sip == targets[i]) { |
2680 | if (bond_has_this_ip(bond, tip)) | 2678 | if (bond_has_this_ip(bond, tip)) |
2681 | slave->last_arp_rx = jiffies; | 2679 | slave->last_arp_rx = jiffies; |
@@ -2727,10 +2725,10 @@ static int bond_arp_rcv(struct sk_buff *skb, struct net_device *dev, struct pack | |||
2727 | arp_ptr += 4 + dev->addr_len; | 2725 | arp_ptr += 4 + dev->addr_len; |
2728 | memcpy(&tip, arp_ptr, 4); | 2726 | memcpy(&tip, arp_ptr, 4); |
2729 | 2727 | ||
2730 | dprintk("bond_arp_rcv: %s %s/%d av %d sv %d sip %u.%u.%u.%u" | 2728 | dprintk("bond_arp_rcv: %s %s/%d av %d sv %d sip %pI4 tip %pI4\n", |
2731 | " tip %u.%u.%u.%u\n", bond->dev->name, slave->dev->name, | 2729 | bond->dev->name, slave->dev->name, slave->state, |
2732 | slave->state, bond->params.arp_validate, | 2730 | bond->params.arp_validate, slave_do_arp_validate(bond, slave), |
2733 | slave_do_arp_validate(bond, slave), NIPQUAD(sip), NIPQUAD(tip)); | 2731 | &sip, &tip); |
2734 | 2732 | ||
2735 | /* | 2733 | /* |
2736 | * Backup slaves won't see the ARP reply, but do come through | 2734 | * Backup slaves won't see the ARP reply, but do come through |