diff options
author | Tejun Heo <htejun@gmail.com> | 2006-02-10 01:10:48 -0500 |
---|---|---|
committer | Jeff Garzik <jgarzik@pobox.com> | 2006-02-10 06:50:47 -0500 |
commit | f63790201521ccadb63673ff121dac46ab300cf0 (patch) | |
tree | 86781b4efc31f9005c17de63df2606c35ae3b0c0 /drivers/scsi/sata_mv.c | |
parent | 35daeb8f9b41fd13180e8a6f8bec9fc5268938f9 (diff) |
[PATCH] libata: kill NULL qc handling from ->eng_timeout callbacks
->eng_timeout cannot be invoked with NULL qc anymore. Add an
assertion in ata_scsi_error() and kill NULL qc handling from all
->eng_timeout callbacks.
Signed-off-by: Tejun Heo <htejun@gmail.com>
Signed-off-by: Jeff Garzik <jgarzik@pobox.com>
Diffstat (limited to 'drivers/scsi/sata_mv.c')
-rw-r--r-- | drivers/scsi/sata_mv.c | 9 |
1 files changed, 2 insertions, 7 deletions
diff --git a/drivers/scsi/sata_mv.c b/drivers/scsi/sata_mv.c index 1db05f23f12f..6c80527ddd21 100644 --- a/drivers/scsi/sata_mv.c +++ b/drivers/scsi/sata_mv.c | |||
@@ -2027,13 +2027,8 @@ static void mv_eng_timeout(struct ata_port *ap) | |||
2027 | mv_err_intr(ap); | 2027 | mv_err_intr(ap); |
2028 | mv_stop_and_reset(ap); | 2028 | mv_stop_and_reset(ap); |
2029 | 2029 | ||
2030 | if (!qc) { | 2030 | qc->err_mask |= AC_ERR_TIMEOUT; |
2031 | printk(KERN_ERR "ata%u: BUG: timeout without command\n", | 2031 | ata_eh_qc_complete(qc); |
2032 | ap->id); | ||
2033 | } else { | ||
2034 | qc->err_mask |= AC_ERR_TIMEOUT; | ||
2035 | ata_eh_qc_complete(qc); | ||
2036 | } | ||
2037 | } | 2032 | } |
2038 | 2033 | ||
2039 | /** | 2034 | /** |