mirror of
https://github.com/hardkernel/linux.git
synced 2026-06-08 20:07:46 +09:00
tg3: Fix TSO CAP for 5704 devs w / ASF enabled
[ Upstream commitcf9ecf4b63] On the earliest TSO capable devices, TSO was accomplished through firmware. The TSO cannot coexist with ASF management firmware though. The tg3 driver determines whether or not ASF is enabled by calling tg3_get_eeprom_hw_cfg(), which checks a particular bit of NIC memory. Commitdabc5c670d, entitled "tg3: Move TSO_CAPABLE assignment", accidentally moved the code that determines TSO capabilities earlier than the call to tg3_get_eeprom_hw_cfg(). As a consequence, the driver was attempting to determine TSO capabilities before it had all the data it needed to make the decision. This patch fixes the problem by revisiting and reevaluating the decision after tg3_get_eeprom_hw_cfg() is called. Signed-off-by: Matt Carlson <mcarlson@broadcom.com> Signed-off-by: Michael Chan <mchan@broadcom.com> Signed-off-by: David S. Miller <davem@davemloft.net> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
This commit is contained in:
committed by
Greg Kroah-Hartman
parent
dbbfb5ca29
commit
2cf80ae813
@@ -13647,9 +13647,13 @@ static int __devinit tg3_get_invariants(struct tg3 *tp)
|
||||
if (tg3_flag(tp, HW_TSO_1) ||
|
||||
tg3_flag(tp, HW_TSO_2) ||
|
||||
tg3_flag(tp, HW_TSO_3) ||
|
||||
(tp->fw_needed && !tg3_flag(tp, ENABLE_ASF)))
|
||||
tp->fw_needed) {
|
||||
/* For firmware TSO, assume ASF is disabled.
|
||||
* We'll disable TSO later if we discover ASF
|
||||
* is enabled in tg3_get_eeprom_hw_cfg().
|
||||
*/
|
||||
tg3_flag_set(tp, TSO_CAPABLE);
|
||||
else {
|
||||
} else {
|
||||
tg3_flag_clear(tp, TSO_CAPABLE);
|
||||
tg3_flag_clear(tp, TSO_BUG);
|
||||
tp->fw_needed = NULL;
|
||||
@@ -13887,6 +13891,12 @@ static int __devinit tg3_get_invariants(struct tg3 *tp)
|
||||
*/
|
||||
tg3_get_eeprom_hw_cfg(tp);
|
||||
|
||||
if (tp->fw_needed && tg3_flag(tp, ENABLE_ASF)) {
|
||||
tg3_flag_clear(tp, TSO_CAPABLE);
|
||||
tg3_flag_clear(tp, TSO_BUG);
|
||||
tp->fw_needed = NULL;
|
||||
}
|
||||
|
||||
if (tg3_flag(tp, ENABLE_APE)) {
|
||||
/* Allow reads and writes to the
|
||||
* APE register and memory space.
|
||||
|
||||
Reference in New Issue
Block a user