diff options
author | Ursula Braun <ursula.braun@de.ibm.com> | 2008-09-19 06:56:03 -0400 |
---|---|---|
committer | Jeff Garzik <jgarzik@redhat.com> | 2008-09-24 20:47:57 -0400 |
commit | 28a7e4c906bd86419eb8572b3b1343e619cd1470 (patch) | |
tree | 9d2c13335eeffdf3fe00b89576f6b1b55e7e04ab /drivers/s390/net/qeth_l3_main.c | |
parent | f1d58672de4e89478e0b02ea8fd1f60eb0626457 (diff) |
qeth: avoid qeth recovery problems
Do not touch IFF_UP flag during qeth recovery, but invoke dev_close()
in case of failing recovery.
Cancel outstanding control commands in case of Data Checks or
Channel Checks.
Do not invoke qeth_l2_del_all_mc() in case of a hard stop to speed up
removal of qeth devices.
Signed-off-by: Ursula Braun <ursula.braun@de.ibm.com>
Signed-off-by: Frank Blaschka <frank.blaschka@de.ibm.com>
Signed-off-by: Jeff Garzik <jgarzik@redhat.com>
Diffstat (limited to 'drivers/s390/net/qeth_l3_main.c')
-rw-r--r-- | drivers/s390/net/qeth_l3_main.c | 8 |
1 files changed, 5 insertions, 3 deletions
diff --git a/drivers/s390/net/qeth_l3_main.c b/drivers/s390/net/qeth_l3_main.c index dd72c3c20165..99547dea44de 100644 --- a/drivers/s390/net/qeth_l3_main.c +++ b/drivers/s390/net/qeth_l3_main.c | |||
@@ -2795,7 +2795,6 @@ static int qeth_l3_open(struct net_device *dev) | |||
2795 | return -ENODEV; | 2795 | return -ENODEV; |
2796 | card->data.state = CH_STATE_UP; | 2796 | card->data.state = CH_STATE_UP; |
2797 | card->state = CARD_STATE_UP; | 2797 | card->state = CARD_STATE_UP; |
2798 | card->dev->flags |= IFF_UP; | ||
2799 | netif_start_queue(dev); | 2798 | netif_start_queue(dev); |
2800 | 2799 | ||
2801 | if (!card->lan_online && netif_carrier_ok(dev)) | 2800 | if (!card->lan_online && netif_carrier_ok(dev)) |
@@ -2809,7 +2808,6 @@ static int qeth_l3_stop(struct net_device *dev) | |||
2809 | 2808 | ||
2810 | QETH_DBF_TEXT(TRACE, 4, "qethstop"); | 2809 | QETH_DBF_TEXT(TRACE, 4, "qethstop"); |
2811 | netif_tx_disable(dev); | 2810 | netif_tx_disable(dev); |
2812 | card->dev->flags &= ~IFF_UP; | ||
2813 | if (card->state == CARD_STATE_UP) | 2811 | if (card->state == CARD_STATE_UP) |
2814 | card->state = CARD_STATE_SOFTSETUP; | 2812 | card->state = CARD_STATE_SOFTSETUP; |
2815 | return 0; | 2813 | return 0; |
@@ -3218,9 +3216,13 @@ static int qeth_l3_recover(void *ptr) | |||
3218 | if (!rc) | 3216 | if (!rc) |
3219 | PRINT_INFO("Device %s successfully recovered!\n", | 3217 | PRINT_INFO("Device %s successfully recovered!\n", |
3220 | CARD_BUS_ID(card)); | 3218 | CARD_BUS_ID(card)); |
3221 | else | 3219 | else { |
3220 | rtnl_lock(); | ||
3221 | dev_close(card->dev); | ||
3222 | rtnl_unlock(); | ||
3222 | PRINT_INFO("Device %s could not be recovered!\n", | 3223 | PRINT_INFO("Device %s could not be recovered!\n", |
3223 | CARD_BUS_ID(card)); | 3224 | CARD_BUS_ID(card)); |
3225 | } | ||
3224 | return 0; | 3226 | return 0; |
3225 | } | 3227 | } |
3226 | 3228 | ||