From: Christoph Hellwig <hch@caldera.de>
To: Linus Torvalds <torvalds@transmeta.com>
Cc: linux-scsi@vger.kernel.org
Subject: [PATCH] remove old SCSI-EH methods from Scsi_Host_Template
Date: Mon, 15 Apr 2002 19:49:29 +0100 [thread overview]
Message-ID: <20020415194929.A8937@infradead.org> (raw)
Hi Linus,
when I removed the old SCSI error handling in 2.5.<early> I accidentally
left the Scsi_Host_Template methods for it in. The bloat cause by this
isn't much so this shouldn't be a real issue, but it allowed the drivers
not updated to new error handling to still compile and cause bad problems
in case of the need for error handling. The patch below nukes it to
break compilation for the huge number of drivers not yet updated.
Christoph
--- linux/drivers/scsi/hosts.h~ Tue Jan 15 10:59:08 2002
+++ linux/drivers/scsi/hosts.h Mon Apr 15 20:35:46 2002
@@ -166,40 +166,6 @@
int (*eh_host_reset_handler)(Scsi_Cmnd *);
/*
- * Since the mid level driver handles time outs, etc, we want to
- * be able to abort the current command. Abort returns 0 if the
- * abortion was successful. The field SCpnt->abort reason
- * can be filled in with the appropriate reason why we wanted
- * the abort in the first place, and this will be used
- * in the mid-level code instead of the host_byte().
- * If non-zero, the code passed to it
- * will be used as the return code, otherwise
- * DID_ABORT should be returned.
- *
- * Note that the scsi driver should "clean up" after itself,
- * resetting the bus, etc. if necessary.
- *
- * NOTE - this interface is depreciated, and will go away. Use
- * the eh_ routines instead.
- */
- int (* abort)(Scsi_Cmnd *);
-
- /*
- * The reset function will reset the SCSI bus. Any executing
- * commands should fail with a DID_RESET in the host byte.
- * The Scsi_Cmnd is passed so that the reset routine can figure
- * out which host adapter should be reset, and also which command
- * within the command block was responsible for the reset in
- * the first place. Some hosts do not implement a reset function,
- * and these hosts must call scsi_request_sense(SCpnt) to keep
- * the command alive.
- *
- * NOTE - this interface is depreciated, and will go away. Use
- * the eh_ routines instead.
- */
- int (* reset)(Scsi_Cmnd *, unsigned int);
-
- /*
* This function is used to select synchronous communications,
* which will result in a higher data throughput. Not implemented
* yet.
reply other threads:[~2002-04-15 18:49 UTC|newest]
Thread overview: [no followups] expand[flat|nested] mbox.gz Atom feed
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20020415194929.A8937@infradead.org \
--to=hch@caldera.de \
--cc=linux-scsi@vger.kernel.org \
--cc=torvalds@transmeta.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox