All of lore.kernel.org
 help / color / mirror / Atom feed
* [patch 0/1] zfcp fixes for 2.6.28-rc6: missing patch
@ 2008-11-27 10:44 Christof Schmitt
  2008-11-27 10:44 ` [patch 1/1] zfcp: prevent double decrement on host_busy while being busy Christof Schmitt
  0 siblings, 1 reply; 2+ messages in thread
From: Christof Schmitt @ 2008-11-27 10:44 UTC (permalink / raw)
  To: James Bottomley; +Cc: linux-scsi, linux-s390, schwidefsky, heiko.carstens

James,

this fix should be part of yesterday's patch series for 2.6.28-rc6,
but it fell through some holes. It applies cleanly on top of the patch
series, please also include this fix.

thanks,

Christof

^ permalink raw reply	[flat|nested] 2+ messages in thread

* [patch 1/1] zfcp: prevent double decrement on host_busy while being busy
  2008-11-27 10:44 [patch 0/1] zfcp fixes for 2.6.28-rc6: missing patch Christof Schmitt
@ 2008-11-27 10:44 ` Christof Schmitt
  0 siblings, 0 replies; 2+ messages in thread
From: Christof Schmitt @ 2008-11-27 10:44 UTC (permalink / raw)
  To: James Bottomley
  Cc: linux-scsi, linux-s390, schwidefsky, heiko.carstens,
	Swen Schillig, Christof Schmitt

[-- Attachment #1: 711-zfcp-decrement.diff --]
[-- Type: text/plain, Size: 1092 bytes --]

From: Swen Schillig <swen@vnet.ibm.com>

The zfcp_scsi_queuecommand was not acting according to the standard
when the respective unit was not available. In this case an -EBUSY was
returned, which is not valid in itself, and in addition scsi_done
was called. This combination is not allowed and was leading to a
double finish of the request and therefor double decrement of the
host_busy counter.

Signed-off-by: Swen Schillig <swen@vnet.ibm.com>
Signed-off-by: Christof Schmitt <christof.schmitt@de.ibm.com>
---

 drivers/s390/scsi/zfcp_scsi.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

--- a/drivers/s390/scsi/zfcp_scsi.c	2008-11-26 17:21:58.000000000 +0100
+++ b/drivers/s390/scsi/zfcp_scsi.c	2008-11-27 10:58:30.000000000 +0100
@@ -88,7 +88,7 @@ static int zfcp_scsi_queuecommand(struct
 	ret = zfcp_fsf_send_fcp_command_task(adapter, unit, scpnt, 0,
 					     ZFCP_REQ_AUTO_CLEANUP);
 	if (unlikely(ret == -EBUSY))
-		zfcp_scsi_command_fail(scpnt, DID_NO_CONNECT);
+		return SCSI_MLQUEUE_DEVICE_BUSY;
 	else if (unlikely(ret < 0))
 		return SCSI_MLQUEUE_HOST_BUSY;
 

-- 

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2008-11-27 10:44 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2008-11-27 10:44 [patch 0/1] zfcp fixes for 2.6.28-rc6: missing patch Christof Schmitt
2008-11-27 10:44 ` [patch 1/1] zfcp: prevent double decrement on host_busy while being busy Christof Schmitt

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.