s390/dasd: fix infinite loop during format

Error recovery requests may not be cleaned up correctly so that other
needed erp requests can not be build because of insufficient memory.
This would lead to an infinite loop trying to build erp requests.

Signed-off-by: Stefan Haberland <stefan.haberland@de.ibm.com>
Signed-off-by: Martin Schwidefsky <schwidefsky@de.ibm.com>
This commit is contained in:
Stefan Haberland 2014-10-01 13:04:54 +02:00 committed by Martin Schwidefsky
parent cfb0b24143
commit 362ce84f43
1 changed files with 15 additions and 10 deletions

View File

@ -2261,8 +2261,8 @@ static inline int _wait_for_wakeup_queue(struct list_head *ccw_queue)
static int _dasd_sleep_on_queue(struct list_head *ccw_queue, int interruptible) static int _dasd_sleep_on_queue(struct list_head *ccw_queue, int interruptible)
{ {
struct dasd_device *device; struct dasd_device *device;
int rc;
struct dasd_ccw_req *cqr, *n; struct dasd_ccw_req *cqr, *n;
int rc;
retry: retry:
list_for_each_entry_safe(cqr, n, ccw_queue, blocklist) { list_for_each_entry_safe(cqr, n, ccw_queue, blocklist) {
@ -2310,21 +2310,26 @@ static int _dasd_sleep_on_queue(struct list_head *ccw_queue, int interruptible)
/* /*
* for alias devices simplify error recovery and * for alias devices simplify error recovery and
* return to upper layer * return to upper layer
* do not skip ERP requests
*/ */
if (cqr->startdev != cqr->basedev && if (cqr->startdev != cqr->basedev && !cqr->refers &&
(cqr->status == DASD_CQR_TERMINATED || (cqr->status == DASD_CQR_TERMINATED ||
cqr->status == DASD_CQR_NEED_ERP)) cqr->status == DASD_CQR_NEED_ERP))
return -EAGAIN; return -EAGAIN;
else {
/* normal recovery for basedev IO */ /* normal recovery for basedev IO */
if (__dasd_sleep_on_erp(cqr)) { if (__dasd_sleep_on_erp(cqr)) {
if (!cqr->status == DASD_CQR_TERMINATED && goto retry;
!cqr->status == DASD_CQR_NEED_ERP) /* remember that ERP was needed */
break;
rc = 1; rc = 1;
/* skip processing for active cqr */
if (cqr->status != DASD_CQR_TERMINATED &&
cqr->status != DASD_CQR_NEED_ERP)
break;
} }
} }
}
/* start ERP requests in upper loop */
if (rc) if (rc)
goto retry; goto retry;