Re: host_self_blocked question/bug?

[Patrick Mansfield <patmans@us.ibm.com>]

On Tue, Nov 25, 2003 at 04:31:09PM -0600, Brian King wrote:
> James Bottomley wrote:
> > The original design was to allow short hiatuses when the HBA couldn't
> > accept I/O.  It doesn't work if there's I/O pending (unless the stop is
> > very short), because the SCSI timers are still ticking and error
> > recovery doesn't see this flag.
> > 
> > There has been talk of making this interface robust to pending commands
> > (halt the timers and freeze the error handler) for FC HBA's that take
> > ages to process loop events, but no work has been done on this---it's
> > quite a bit more work than simply not allowing the eh to emit TURs.

> I'd like a way to be able to stop the mid-layer from sending me any 
> commands. The scenarios I have today are:
> 
> 1. Fatal error on the adapter.
> 2. microcode download to the adapter.
> 3. Adapter cache recovery commands.
> 
> All of these cases require me to run BIST on the adapter and bring it 
> back up. To do this may take 20-30 seconds. I call scsi_block_requests, 
> fail all pending ops back with DID_ERROR, reset the adapter, then call 
> scsi_unblock_requests. My usage of it gets around the ticking timer 
> problem. I agree that the error recovery thread doesn't see this either 
> and that this is a potential problem. I had planned to work around that 
> by failing abort and device reset, forcing the host_reset to be called, 
> which would wait on the completion of the adapter reset, but it would be 
> nice if I didn't have to do that.

Given the above conditions: could we not start up the eh, and abort the eh
(and start it up again when unblocked) if already running and
we see host_self_blocked is set?

The following blocks the error handler from starting up, then we need code
to abort the error handler.

(There should be locking around all the setting and checking of
host_self_blocked.)

Untested, compiled only patch against main line bk:

===== drivers/scsi/scsi_error.c 1.65 vs edited =====
--- 1.65/drivers/scsi/scsi_error.c	Sun Sep 21 10:49:36 2003
+++ edited/drivers/scsi/scsi_error.c	Tue Nov 25 16:11:01 2003
@@ -47,7 +47,8 @@
 /* called with shost->host_lock held */
 void scsi_eh_wakeup(struct Scsi_Host *shost)
 {
-	if (shost->host_busy == shost->host_failed) {
+	if ((shost->host_busy == shost->host_failed) &&
+	    !shost->host_self_blocked) {
 		up(shost->eh_wait);
 		SCSI_LOG_ERROR_RECOVERY(5,
 				printk("Waking error handler thread\n"));
===== drivers/scsi/scsi_lib.c 1.113 vs edited =====
--- 1.113/drivers/scsi/scsi_lib.c	Sat Sep 20 06:53:02 2003
+++ edited/drivers/scsi/scsi_lib.c	Tue Nov 25 16:12:30 2003
@@ -1303,6 +1303,7 @@
 {
 	shost->host_self_blocked = 0;
 	scsi_run_host_queues(shost);
+	scsi_eh_wakeup(shost);
 }
 
 int __init scsi_init_queue(void)