netdev - 2.6.29 ALB bonding printk()s every arp (was Re: [PATCH 2/3] bonding: Fix ALB mode to balance traffic on VLANs)

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <49E518F8.3030109@siriusit.co.uk>
Date:	Wed, 15 Apr 2009 00:15:04 +0100
From:	Duncan Gibb <duncan.gibb@...iusit.co.uk>
To:	Jay Vosburgh <fubar@...ibm.com>
CC:	netdev@...r.kernel.org, Jeff Garzik <jgarzik@...ox.com>
Subject: 2.6.29 ALB bonding printk()s every arp (was Re: [PATCH 2/3] bonding:
 Fix ALB mode to balance traffic on VLANs)

On 05 Nov 2008 at 01:51, Jay Vosburgh wrote:

JV> The current ALB function that processes incoming ARPs does
JV> not handle traffic for VLANs configured above bonding.  This
JV> causes traffic on those VLANs to all be assigned the same slave.
JV> This patch corrects that misbehavior by locating the bonding
JV> interface nested below the VLAN interface.

> diff --git a/drivers/net/bonding/bond_alb.c b/drivers/net/bonding/bond_alb.c
> index 87437c7..e170fa2 100644
> --- a/drivers/net/bonding/bond_alb.c
> +++ b/drivers/net/bonding/bond_alb.c

> @@ -368,6 +372,9 @@ static int rlb_arp_recv(struct sk_buff *skb, struct net_device *bond_dev, struct
>  
>  	if (arp->op_code == htons(ARPOP_REPLY)) {
>  		/* update rx hash table for this ARP */
> +		printk("rar: update orig %s bond_dev %s\n", orig_dev->name,
> +		       bond_dev->name);
> +		bond = bond_dev->priv;
>  		rlb_update_entry_from_arp(bond, arp);
>  		dprintk("Server received an ARP Reply from client\n");
>  	}

Is the printk() in this patch necessary?

We recently put a 2.6.29 kernel on a router with multiple ALB bonds each
of several e1000 devices.  It now generates a log entry every time an
arp is-at packet arrives at a bonded interface.  I'm not sure that's a
feature...

Cheers


Duncan

-- 
Duncan Gibb - Technical Director
Sirius Corporation plc - control through freedom
http://www.siriusit.co.uk/ || t: +44 870 608 0063
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html