qeth: avoid recovery during device online setting
authorUrsula Braun <ursula.braun@de.ibm.com>
Mon, 11 Jan 2010 02:50:51 +0000 (02:50 +0000)
committerDavid S. Miller <davem@davemloft.net>
Thu, 14 Jan 2010 04:34:56 +0000 (20:34 -0800)
If a qeth device is set online, several initialisation steps are
performed. If a failure in one of these steps occurs, the qeth
device is reset into DOWN state. If due to the failure a qeth recovery
is scheduled and started in another thread, this might cause all kinds
of conflicts, even a kernel panic. The patch forbids scheduling of a
qeth recovery while online processing is performed till the card is in
state SOFTSETUP.

Signed-off-by: Ursula Braun <ursula.braun@de.ibm.com>
Signed-off-by: Frank Blaschka <frank.blaschka@de.ibm.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/s390/net/qeth_l2_main.c
drivers/s390/net/qeth_l3_main.c

index 74ba388a159f1eca5932873c3a4b5f8fca1c7c96..c3258b0dd649fa0c181e2ee49855901ae2a802f8 100644 (file)
@@ -927,7 +927,6 @@ static int __qeth_l2_set_online(struct ccwgroup_device *gdev, int recovery_mode)
        QETH_DBF_TEXT(SETUP, 2, "setonlin");
        QETH_DBF_HEX(SETUP, 2, &card, sizeof(void *));
 
-       qeth_set_allowed_threads(card, QETH_RECOVER_THREAD, 1);
        recover_flag = card->state;
        rc = qeth_core_hardsetup_card(card);
        if (rc) {
index 337d03fb045d76478a2a646564e760e308c2adb8..5475834ab916ba6dd08868bf3506877c17b55bee 100644 (file)
@@ -3338,8 +3338,6 @@ static int __qeth_l3_set_online(struct ccwgroup_device *gdev, int recovery_mode)
        QETH_DBF_TEXT(SETUP, 2, "setonlin");
        QETH_DBF_HEX(SETUP, 2, &card, sizeof(void *));
 
-       qeth_set_allowed_threads(card, QETH_RECOVER_THREAD, 1);
-
        recover_flag = card->state;
        rc = qeth_core_hardsetup_card(card);
        if (rc) {