Attachment 298300 Details for Bug 437865 – patch confirmed by customer to solve problem

[patch] patch confirmed by customer to solve problem

linux-2.6-bonding-802.3ad-link-partner.patch (text/plain), 1.20 KB, created by Vince Worthington on 2008-03-17 20:01:07 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Vince Worthington

Created: 2008-03-17 20:01:07 UTC

Size: 1.20 KB

patch

obsolete

>diff -pruN kernel-2.6.9.orig/linux-2.6.9/drivers/net/bonding/bond_3ad.c kernel-2.6.9/linux-2.6.9/drivers/net/bonding/bond_3ad.c
>--- kernel-2.6.9.orig/linux-2.6.9/drivers/net/bonding/bond_3ad.c	2008-03-12 11:06:44.000000000 +0100
>+++ kernel-2.6.9/linux-2.6.9/drivers/net/bonding/bond_3ad.c	2008-03-12 11:10:15.000000000 +0100
>@@ -2330,19 +2330,19 @@ void bond_3ad_handle_link_change(struct 
> }
> 
> /*
>- * set link state for bonding master: if we have an active partnered
>+ * set link state for bonding master: if we have an active 
>  * aggregator, we're up, if not, we're down.  Presumes that we cannot
>  * have an active aggregator if there are no slaves with link up.
>  *
>+ * This behavior complies with IEEE 802.3 section 43.3.9.
>+ * 
>  * Called by bond_set_carrier(). Return zero if carrier state does not
>  * change, nonzero if it does.
>  */
> int bond_3ad_set_carrier(struct bonding *bond)
> {
>-	struct aggregator *agg;
> 
>-	agg = __get_active_agg(&(SLAVE_AD_INFO(bond->first_slave).aggregator));
>-	if (agg && MAC_ADDRESS_COMPARE(&agg->partner_system, &null_mac_addr)) {
>+  if (__get_active_agg(&(SLAVE_AD_INFO(bond->first_slave).aggregator))) {
> 		if (!netif_carrier_ok(bond->dev)) {
> 			netif_carrier_on(bond->dev);
> 			return 1;

Actions: View | Diff

Attachments on bug 437865: 298300 | 298306