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

patch confirmed by customer to solve the problem

linux-2.6-bonding-802.3ad-link-partner.patch (text/x-patch), 1.27 KB, created by Vince Worthington on 2008-03-17 21:05:03 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Vince Worthington

Created: 2008-03-17 21:05:03 UTC

Size: 1.27 KB

patch

obsolete

>---
>
> linux-2.6.9/drivers/net/bonding/bond_3ad.c |    9 ++++-----
> 1 file changed, 4 insertions(+), 5 deletions(-)
>
>diff -up linux-2.6.9/drivers/net/bonding/bond_3ad.c.partner linux-2.6.9/drivers/net/bonding/bond_3ad.c
>--- linux-2.6.9/drivers/net/bonding/bond_3ad.c.partner	2008-03-17 16:34:33.000000000 -0400
>+++ linux-2.6.9/drivers/net/bonding/bond_3ad.c	2008-03-17 16:38:29.000000000 -0400
>@@ -2330,19 +2330,18 @@ 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

Attachments on bug 437865: 298300 | 298306