Re: [PATCH 0/2] : definion, code, and use of new SCSI ML host status DID_COND_REQUEUE

2007-02-02 Thread Edward Goggin
On Fri, 2007-02-02 at 17:18 -0600, James Bottomley wrote: > On Fri, 2007-02-02 at 18:11 -0500, Edward Goggin wrote: > > That solution doesn't work for the RDAC/MPP driver as the BUSY status > > handler retries indefinitely. We need a solution which works for both a > &g

Re: [PATCH 0/2] : definion, code, and use of new SCSI ML host status DID_COND_REQUEUE

2007-02-02 Thread Edward Goggin
On Fri, 2007-02-02 at 16:54 -0600, James Bottomley wrote: > On Fri, 2007-02-02 at 17:04 -0500, Edward Goggin wrote: > > Patch Set Summary: > > > > 1 Define new SCSI ML host status DID_COND_REQUEUE and > > add its handling code to scsi_decide_disposition. >

[PATCH 2/2] fusion: return DID_COND_REQUEUE if SCSI status is MPI_SCSI_STATUS_BUSY

2007-02-02 Thread Edward Goggin
From: Ed Goggin <[EMAIL PROTECTED]> Return DID_COND_REQUEUE instead of DID_BUS_BUSY for IOC status is SUCCESS and scsi_status is MPI_SCSI_STATUS_BUSY. Command will be retried via ADD_TO_MLQUEUE IFF not REQ_FAILFAST. Signed-off-by: Ed Goggin <[EMAIL PROTECTED]> diff --git a/drivers/message/fusio

[PATCH 1/2] scsi: add DID_COND_REQUEUE SCSI ML host status

2007-02-02 Thread Edward Goggin
From: Ed Goggin <[EMAIL PROTECTED]> Add new SCSI ML host status DID_COND_REQUEUE for ADD_TO_MLQUEUE IFF not REQ_FAILFAST. Signed-off-by: Ed Goggin <[EMAIL PROTECTED]> diff --git a/drivers/scsi/scsi_error.c b/drivers/scsi/scsi_error.c index 2dce06a..d8e884b 100644 --- a/drivers/scsi/scsi_error.c

[PATCH 0/2] : definion, code, and use of new SCSI ML host status DID_COND_REQUEUE

2007-02-02 Thread Edward Goggin
Patch Set Summary: 1 Define new SCSI ML host status DID_COND_REQUEUE and add its handling code to scsi_decide_disposition. Scsi_decide_disposition returns ADD_TO_MLQUEUE IFF not REQ_FAILFAST. 2 Return DID_COND_REQUEUE instead of DID_BUS_BUSY host status

why no cmd_flags parameter for scsi_execute_async?

2007-01-25 Thread Edward Goggin
Looks like a simple oversight, but does anyone know why there a cmd_flags parameter for scsi_execute but not for scsi_execute_async? Seems like this is lost functionality when scsi_execute_async replaced scsi_do_req. Previously, the caller of scsi_do_req could set the flags field of the sr_reques

Re: [PATCH 2/5] fusion: vmware bug fix prevent inifinite retries

2007-01-10 Thread Edward Goggin
On Wed, 2007-01-10 at 08:10 -0800, James Bottomley wrote: > On Tue, 2007-01-09 at 16:32 -0500, Edward Goggin wrote: > > The attached (untested) patch shows a VMware and scsi transport agnostic > > approach which introduces a new host status (DID_QUALIFIED_REQUEUE) to > > be us

Re: [PATCH 2/5] fusion: vmware bug fix prevent inifinite retries

2007-01-09 Thread Edward Goggin
On Tue, 2007-01-09 at 12:55 -0800, Petr Vandrovec wrote: > Adam Zimman wrote: > > Adding VMware engineering... > > > > -Original Message- > > From: Manon Goo [mailto:[EMAIL PROTECTED] > > Sent: Tuesday, January 09, 2007 9:49 AM > > To: Michael Reed; Moore, Eric; David Berghoff > > Cc: Ja