From: Santiago Leon <santil@xxxxxxxxxx>
The "ibmvscsi: treat busy and error conditions separately" patch submitted by Dave Boutcher back in June incorrectly reenables the CRQ. The broken logic causes the adapter to get disabled if the CRQ connection happens to close temporarily. This patch "fixes that obviously wrong logic check" (Dave's words). Please apply.
Signed-off-by: Santiago Leon <santil@xxxxxxxxxx>
Signed-off-by: David Boutcher <sleddog@xxxxxxxxxx>
---
ibmvscsi.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
--- a/drivers/scsi/ibmvscsi/ibmvscsi.c 2006-09-27 16:38:48.000000000 -0500
+++ b/drivers/scsi/ibmvscsi/ibmvscsi.c 2006-10-13 13:34:22.000000000 -0500
@@ -1213,7 +1213,7 @@ void ibmvscsi_handle_crq(struct viosrp_c
"ibmvscsi: Re-enabling adapter!\n");
purge_requests(hostdata, DID_REQUEUE);
if ((ibmvscsi_reenable_crq_queue(&hostdata->queue,
- hostdata) == 0) ||
+ hostdata)) ||
(ibmvscsi_send_crq(hostdata,
0xC001000000000000LL, 0))) {
atomic_set(&hostdata->request_limit,
-
To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
