Bug 116916

Summary: tg3 driver doesn't support bonding driver's ALB mode
Product: Red Hat Enterprise Linux 3 Reporter: Martin Wilck <martin.wilck>
Component: kernelAssignee: David Miller <davem>
Status: CLOSED ERRATA QA Contact: Brian Brock <bbrock>
Severity: medium Docs Contact:
Priority: medium    
Version: 3.0CC: petrides, riel, rperkins, tao
Target Milestone: ---   
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard: tg3
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2004-05-12 01:08:35 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Attachments:
Description Flags
fix for MAC setting bug none

Description Martin Wilck 2004-02-26 13:30:22 UTC
Description of problem:

It is impossible to use the adaptive load balancing (ALB) mode of the
Linux bonding driver with the tg3 base driver because the tg3 driver
doesn't support changing the MAC address of a running interface.

Version-Release number of selected component (if applicable):
2.4.21-9.EL and earlier

How reproducible:
always

Steps to Reproduce:
1.install system with Broadcom NIC and tg3
2.configure bonding driver with mode=6 using the Broadcom NIC
  
Actual results: the following error message:

bonding: Error: alb_set_mac_addr: dev->set_mac_address of dev eth0
failed! ALB mode requires that the base driver support setting the hw
address also when the network device's interface is open

Expected results:

ALB mode works.

Additional info:

Works with Broadcomn's bcm5700 driver, thus it is no Hardware problem

The following lines in tg3_set_mac_addr()  (tg3.c line 4523f.) seem to
be responsible:

        if (netif_running(dev))
                return -EBUSY;

Comment 2 David Miller 2004-03-08 19:08:54 UTC
Yes, that test is not needed at all, the fix is to simply remove
those two lines.


Comment 3 David Miller 2004-03-08 19:09:27 UTC
Created attachment 98376 [details]
fix for MAC setting bug

Comment 4 Martin Wilck 2004-03-09 08:26:34 UTC
Great, thanks.


Comment 7 Martin Wilck 2004-04-06 12:54:23 UTC
When will this part become part of an official RHEL kernel?


Comment 8 Ernie Petrides 2004-04-08 02:39:38 UTC
The tg3 driver has just been upgraded to v3.1 in tonight's
respin of RHEL3 U2 (kernel version 2.4.21-13.EL) to address
this problem.

Comment 9 John Flanagan 2004-05-12 01:08:36 UTC
An errata has been issued which should help the problem described in this bug report. 
This report is therefore being closed with a resolution of ERRATA. For more information
on the solution and/or where to find the updated files, please follow the link below. You may reopen 
this bug report if the solution does not work for you.

http://rhn.redhat.com/errata/RHSA-2004-188.html