Merge branch 'master' into upstream
authorJeff Garzik <jeff@garzik.org>
Fri, 23 Jun 2006 02:11:56 +0000 (22:11 -0400)
committerJeff Garzik <jeff@garzik.org>
Fri, 23 Jun 2006 02:11:56 +0000 (22:11 -0400)
Conflicts:

drivers/scsi/libata-core.c
drivers/scsi/libata-scsi.c
include/linux/pci_ids.h

1  2 
drivers/scsi/Makefile
drivers/scsi/libata-core.c
drivers/scsi/libata-scsi.c
drivers/scsi/sata_vsc.c
drivers/scsi/scsi.c
drivers/scsi/scsi_error.c
drivers/scsi/scsi_lib.c
include/linux/pci_ids.h
include/scsi/scsi_cmnd.h

diff --combined drivers/scsi/Makefile
@@@ -33,7 -33,7 +33,7 @@@ obj-$(CONFIG_SCSI_FC_ATTRS)   += scsi_tr
  obj-$(CONFIG_SCSI_ISCSI_ATTRS)        += scsi_transport_iscsi.o
  obj-$(CONFIG_SCSI_SAS_ATTRS)  += scsi_transport_sas.o
  
- obj-$(CONFIG_ISCSI_TCP)       += iscsi_tcp.o
+ obj-$(CONFIG_ISCSI_TCP)       += libiscsi.o   iscsi_tcp.o
  obj-$(CONFIG_SCSI_AMIGA7XX)   += amiga7xx.o   53c7xx.o
  obj-$(CONFIG_A3000_SCSI)      += a3000.o      wd33c93.o
  obj-$(CONFIG_A2091_SCSI)      += a2091.o      wd33c93.o
@@@ -136,6 -136,7 +136,7 @@@ obj-$(CONFIG_SCSI_SATA_NV) += libata.o 
  obj-$(CONFIG_SCSI_SATA_ULI)   += libata.o sata_uli.o
  obj-$(CONFIG_SCSI_SATA_MV)    += libata.o sata_mv.o
  obj-$(CONFIG_SCSI_PDC_ADMA)   += libata.o pdc_adma.o
+ obj-$(CONFIG_SCSI_HPTIOP)     += hptiop.o
  
  obj-$(CONFIG_ARM)             += arm/
  
@@@ -164,7 -165,7 +165,7 @@@ ncr53c8xx-flags-$(CONFIG_SCSI_ZALON) 
  CFLAGS_ncr53c8xx.o    := $(ncr53c8xx-flags-y) $(ncr53c8xx-flags-m)
  zalon7xx-objs := zalon.o ncr53c8xx.o
  NCR_Q720_mod-objs     := NCR_Q720.o ncr53c8xx.o
 -libata-objs   := libata-core.o libata-scsi.o libata-bmdma.o
 +libata-objs   := libata-core.o libata-scsi.o libata-bmdma.o libata-eh.o
  oktagon_esp_mod-objs  := oktagon_esp.o oktagon_io.o
  
  # Files generated that shall be removed upon make clean
  
  #include "libata.h"
  
 -static unsigned int ata_dev_init_params(struct ata_port *ap,
 -                                      struct ata_device *dev,
 -                                      u16 heads,
 -                                      u16 sectors);
 -static void ata_set_mode(struct ata_port *ap);
 -static unsigned int ata_dev_set_xfermode(struct ata_port *ap,
 -                                       struct ata_device *dev);
 -static void ata_dev_xfermask(struct ata_port *ap, struct ata_device *dev);
 +/* debounce timing parameters in msecs { interval, duration, timeout } */
 +const unsigned long sata_deb_timing_boot[]            = {   5,  100, 2000 };
 +const unsigned long sata_deb_timing_eh[]              = {  25,  500, 2000 };
 +const unsigned long sata_deb_timing_before_fsrst[]    = { 100, 2000, 5000 };
 +
 +static unsigned int ata_dev_init_params(struct ata_device *dev,
 +                                      u16 heads, u16 sectors);
 +static unsigned int ata_dev_set_xfermode(struct ata_device *dev);
 +static void ata_dev_xfermask(struct ata_device *dev);
  
  static unsigned int ata_unique_id = 1;
  static struct workqueue_struct *ata_wq;
  
 +struct workqueue_struct *ata_aux_wq;
 +
  int atapi_enabled = 1;
  module_param(atapi_enabled, int, 0444);
  MODULE_PARM_DESC(atapi_enabled, "Enable discovery of ATAPI devices (0=off, 1=on)");
  
 +int atapi_dmadir = 0;
 +module_param(atapi_dmadir, int, 0444);
 +MODULE_PARM_DESC(atapi_dmadir, "Enable ATAPI DMADIR bridge support (0=off, 1=on)");
 +
  int libata_fua = 0;
  module_param_named(fua, libata_fua, int, 0444);
  MODULE_PARM_DESC(fua, "FUA support (0=off, 1=on)");
@@@ -404,22 -397,11 +404,22 @@@ static const char *ata_mode_string(unsi
        return "<n/a>";
  }
  
 -static void ata_dev_disable(struct ata_port *ap, struct ata_device *dev)
 +static const char *sata_spd_string(unsigned int spd)
 +{
 +      static const char * const spd_str[] = {
 +              "1.5 Gbps",
 +              "3.0 Gbps",
 +      };
 +
 +      if (spd == 0 || (spd - 1) >= ARRAY_SIZE(spd_str))
 +              return "<unknown>";
 +      return spd_str[spd - 1];
 +}
 +
 +void ata_dev_disable(struct ata_device *dev)
  {
 -      if (ata_dev_present(dev)) {
 -              printk(KERN_WARNING "ata%u: dev %u disabled\n",
 -                     ap->id, dev->devno);
 +      if (ata_dev_enabled(dev)) {
 +              ata_dev_printk(dev, KERN_WARNING, "disabled\n");
                dev->class++;
        }
  }
@@@ -961,14 -943,15 +961,14 @@@ void ata_qc_complete_internal(struct at
  {
        struct completion *waiting = qc->private_data;
  
 -      qc->ap->ops->tf_read(qc->ap, &qc->tf);
        complete(waiting);
  }
  
  /**
   *    ata_exec_internal - execute libata internal command
 - *    @ap: Port to which the command is sent
   *    @dev: Device to which the command is sent
   *    @tf: Taskfile registers for the command and the result
 + *    @cdb: CDB for packet command
   *    @dma_dir: Data tranfer direction of the command
   *    @buf: Data buffer of the command
   *    @buflen: Length of data buffer
   *
   *    LOCKING:
   *    None.  Should be called with kernel context, might sleep.
 + *
 + *    RETURNS:
 + *    Zero on success, AC_ERR_* mask on failure
   */
 -
 -static unsigned
 -ata_exec_internal(struct ata_port *ap, struct ata_device *dev,
 -                struct ata_taskfile *tf,
 -                int dma_dir, void *buf, unsigned int buflen)
 +unsigned ata_exec_internal(struct ata_device *dev,
 +                         struct ata_taskfile *tf, const u8 *cdb,
 +                         int dma_dir, void *buf, unsigned int buflen)
  {
 +      struct ata_port *ap = dev->ap;
        u8 command = tf->command;
        struct ata_queued_cmd *qc;
 +      unsigned int tag, preempted_tag;
 +      u32 preempted_sactive, preempted_qc_active;
        DECLARE_COMPLETION(wait);
        unsigned long flags;
        unsigned int err_mask;
 +      int rc;
  
        spin_lock_irqsave(&ap->host_set->lock, flags);
  
 -      qc = ata_qc_new_init(ap, dev);
 -      BUG_ON(qc == NULL);
 +      /* no internal command while frozen */
 +      if (ap->flags & ATA_FLAG_FROZEN) {
 +              spin_unlock_irqrestore(&ap->host_set->lock, flags);
 +              return AC_ERR_SYSTEM;
 +      }
 +
 +      /* initialize internal qc */
 +
 +      /* XXX: Tag 0 is used for drivers with legacy EH as some
 +       * drivers choke if any other tag is given.  This breaks
 +       * ata_tag_internal() test for those drivers.  Don't use new
 +       * EH stuff without converting to it.
 +       */
 +      if (ap->ops->error_handler)
 +              tag = ATA_TAG_INTERNAL;
 +      else
 +              tag = 0;
 +
 +      if (test_and_set_bit(tag, &ap->qc_allocated))
 +              BUG();
 +      qc = __ata_qc_from_tag(ap, tag);
 +
 +      qc->tag = tag;
 +      qc->scsicmd = NULL;
 +      qc->ap = ap;
 +      qc->dev = dev;
 +      ata_qc_reinit(qc);
 +
 +      preempted_tag = ap->active_tag;
 +      preempted_sactive = ap->sactive;
 +      preempted_qc_active = ap->qc_active;
 +      ap->active_tag = ATA_TAG_POISON;
 +      ap->sactive = 0;
 +      ap->qc_active = 0;
  
 +      /* prepare & issue qc */
        qc->tf = *tf;
 +      if (cdb)
 +              memcpy(qc->cdb, cdb, ATAPI_CDB_LEN);
 +      qc->flags |= ATA_QCFLAG_RESULT_TF;
        qc->dma_dir = dma_dir;
        if (dma_dir != DMA_NONE) {
                ata_sg_init_one(qc, buf, buflen);
  
        spin_unlock_irqrestore(&ap->host_set->lock, flags);
  
 -      if (!wait_for_completion_timeout(&wait, ATA_TMOUT_INTERNAL)) {
 -              ata_port_flush_task(ap);
 +      rc = wait_for_completion_timeout(&wait, ATA_TMOUT_INTERNAL);
 +
 +      ata_port_flush_task(ap);
  
 +      if (!rc) {
                spin_lock_irqsave(&ap->host_set->lock, flags);
  
                /* We're racing with irq here.  If we lose, the
                 * following test prevents us from completing the qc
 -               * again.  If completion irq occurs after here but
 -               * before the caller cleans up, it will result in a
 -               * spurious interrupt.  We can live with that.
 +               * twice.  If we win, the port is frozen and will be
 +               * cleaned up by ->post_internal_cmd().
                 */
                if (qc->flags & ATA_QCFLAG_ACTIVE) {
 -                      qc->err_mask = AC_ERR_TIMEOUT;
 -                      ata_qc_complete(qc);
 -                      printk(KERN_WARNING "ata%u: qc timeout (cmd 0x%x)\n",
 -                             ap->id, command);
 +                      qc->err_mask |= AC_ERR_TIMEOUT;
 +
 +                      if (ap->ops->error_handler)
 +                              ata_port_freeze(ap);
 +                      else
 +                              ata_qc_complete(qc);
 +
 +                      ata_dev_printk(dev, KERN_WARNING,
 +                                     "qc timeout (cmd 0x%x)\n", command);
                }
  
                spin_unlock_irqrestore(&ap->host_set->lock, flags);
        }
  
 -      *tf = qc->tf;
 +      /* do post_internal_cmd */
 +      if (ap->ops->post_internal_cmd)
 +              ap->ops->post_internal_cmd(qc);
 +
 +      if (qc->flags & ATA_QCFLAG_FAILED && !qc->err_mask) {
 +              ata_dev_printk(dev, KERN_WARNING, "zero err_mask for failed "
 +                             "internal command, assuming AC_ERR_OTHER\n");
 +              qc->err_mask |= AC_ERR_OTHER;
 +      }
 +
 +      /* finish up */
 +      spin_lock_irqsave(&ap->host_set->lock, flags);
 +
 +      *tf = qc->result_tf;
        err_mask = qc->err_mask;
  
        ata_qc_free(qc);
 +      ap->active_tag = preempted_tag;
 +      ap->sactive = preempted_sactive;
 +      ap->qc_active = preempted_qc_active;
  
        /* XXX - Some LLDDs (sata_mv) disable port on command failure.
         * Until those drivers are fixed, we detect the condition
         *
         * Kill the following code as soon as those drivers are fixed.
         */
 -      if (ap->flags & ATA_FLAG_PORT_DISABLED) {
 +      if (ap->flags & ATA_FLAG_DISABLED) {
                err_mask |= AC_ERR_SYSTEM;
                ata_port_probe(ap);
        }
  
 +      spin_unlock_irqrestore(&ap->host_set->lock, flags);
 +
        return err_mask;
  }
  
@@@ -1158,10 -1076,11 +1158,10 @@@ unsigned int ata_pio_need_iordy(const s
  
  /**
   *    ata_dev_read_id - Read ID data from the specified device
 - *    @ap: port on which target device resides
   *    @dev: target device
   *    @p_class: pointer to class of the target device (may be changed)
   *    @post_reset: is this read ID post-reset?
 - *    @p_id: read IDENTIFY page (newly allocated)
 + *    @id: buffer to read IDENTIFY data into
   *
   *    Read ID data from the specified device.  ATA_CMD_ID_ATA is
   *    performed on ATA devices and ATA_CMD_ID_ATAPI on ATAPI
   *    RETURNS:
   *    0 on success, -errno otherwise.
   */
 -static int ata_dev_read_id(struct ata_port *ap, struct ata_device *dev,
 -                         unsigned int *p_class, int post_reset, u16 **p_id)
 +int ata_dev_read_id(struct ata_device *dev, unsigned int *p_class,
 +                  int post_reset, u16 *id)
  {
 +      struct ata_port *ap = dev->ap;
        unsigned int class = *p_class;
        struct ata_taskfile tf;
        unsigned int err_mask = 0;
 -      u16 *id;
        const char *reason;
        int rc;
  
  
        ata_dev_select(ap, dev->devno, 1, 1); /* select device 0/1 */
  
 -      id = kmalloc(sizeof(id[0]) * ATA_ID_WORDS, GFP_KERNEL);
 -      if (id == NULL) {
 -              rc = -ENOMEM;
 -              reason = "out of memory";
 -              goto err_out;
 -      }
 -
   retry:
 -      ata_tf_init(ap, &tf, dev->devno);
 +      ata_tf_init(dev, &tf);
  
        switch (class) {
        case ATA_DEV_ATA:
  
        tf.protocol = ATA_PROT_PIO;
  
 -      err_mask = ata_exec_internal(ap, dev, &tf, DMA_FROM_DEVICE,
 +      err_mask = ata_exec_internal(dev, &tf, NULL, DMA_FROM_DEVICE,
                                     id, sizeof(id[0]) * ATA_ID_WORDS);
        if (err_mask) {
                rc = -EIO;
                 * Some drives were very specific about that exact sequence.
                 */
                if (ata_id_major_version(id) < 4 || !ata_id_has_lba(id)) {
 -                      err_mask = ata_dev_init_params(ap, dev, id[3], id[6]);
 +                      err_mask = ata_dev_init_params(dev, id[3], id[6]);
                        if (err_mask) {
                                rc = -EIO;
                                reason = "INIT_DEV_PARAMS failed";
        }
  
        *p_class = class;
 -      *p_id = id;
 +
        return 0;
  
   err_out:
 -      printk(KERN_WARNING "ata%u: dev %u failed to IDENTIFY (%s)\n",
 -             ap->id, dev->devno, reason);
 -      kfree(id);
 +      ata_dev_printk(dev, KERN_WARNING, "failed to IDENTIFY "
 +                     "(%s, err_mask=0x%x)\n", reason, err_mask);
        return rc;
  }
  
 -static inline u8 ata_dev_knobble(const struct ata_port *ap,
 -                               struct ata_device *dev)
 +static inline u8 ata_dev_knobble(struct ata_device *dev)
 +{
 +      return ((dev->ap->cbl == ATA_CBL_SATA) && (!ata_id_is_sata(dev->id)));
 +}
 +
 +static void ata_dev_config_ncq(struct ata_device *dev,
 +                             char *desc, size_t desc_sz)
  {
 -      return ((ap->cbl == ATA_CBL_SATA) && (!ata_id_is_sata(dev->id)));
 +      struct ata_port *ap = dev->ap;
 +      int hdepth = 0, ddepth = ata_id_queue_depth(dev->id);
 +
 +      if (!ata_id_has_ncq(dev->id)) {
 +              desc[0] = '\0';
 +              return;
 +      }
 +
 +      if (ap->flags & ATA_FLAG_NCQ) {
 +              hdepth = min(ap->host->can_queue, ATA_MAX_QUEUE - 1);
 +              dev->flags |= ATA_DFLAG_NCQ;
 +      }
 +
 +      if (hdepth >= ddepth)
 +              snprintf(desc, desc_sz, "NCQ (depth %d)", ddepth);
 +      else
 +              snprintf(desc, desc_sz, "NCQ (depth %d/%d)", hdepth, ddepth);
  }
  
  /**
   *    ata_dev_configure - Configure the specified ATA/ATAPI device
 - *    @ap: Port on which target device resides
   *    @dev: Target device to configure
   *    @print_info: Enable device info printout
   *
   *    RETURNS:
   *    0 on success, -errno otherwise
   */
 -static int ata_dev_configure(struct ata_port *ap, struct ata_device *dev,
 -                           int print_info)
 +int ata_dev_configure(struct ata_device *dev, int print_info)
  {
 +      struct ata_port *ap = dev->ap;
        const u16 *id = dev->id;
        unsigned int xfer_mask;
        int i, rc;
  
 -      if (!ata_dev_present(dev)) {
 +      if (!ata_dev_enabled(dev)) {
                DPRINTK("ENTER/EXIT (host %u, dev %u) -- nodev\n",
                        ap->id, dev->devno);
                return 0;
  
        /* print device capabilities */
        if (print_info)
 -              printk(KERN_DEBUG "ata%u: dev %u cfg 49:%04x 82:%04x 83:%04x "
 -                     "84:%04x 85:%04x 86:%04x 87:%04x 88:%04x\n",
 -                     ap->id, dev->devno, id[49], id[82], id[83],
 -                     id[84], id[85], id[86], id[87], id[88]);
 +              ata_dev_printk(dev, KERN_DEBUG, "cfg 49:%04x 82:%04x 83:%04x "
 +                             "84:%04x 85:%04x 86:%04x 87:%04x 88:%04x\n",
 +                             id[49], id[82], id[83], id[84],
 +                             id[85], id[86], id[87], id[88]);
  
        /* initialize to-be-configured parameters */
 -      dev->flags = 0;
 +      dev->flags &= ~ATA_DFLAG_CFG_MASK;
        dev->max_sectors = 0;
        dev->cdb_len = 0;
        dev->n_sectors = 0;
  
                if (ata_id_has_lba(id)) {
                        const char *lba_desc;
 +                      char ncq_desc[20];
  
                        lba_desc = "LBA";
                        dev->flags |= ATA_DFLAG_LBA;
                                lba_desc = "LBA48";
                        }
  
 +                      /* config NCQ */
 +                      ata_dev_config_ncq(dev, ncq_desc, sizeof(ncq_desc));
 +
                        /* print device info to dmesg */
                        if (print_info)
 -                              printk(KERN_INFO "ata%u: dev %u ATA-%d, "
 -                                     "max %s, %Lu sectors: %s\n",
 -                                     ap->id, dev->devno,
 -                                     ata_id_major_version(id),
 -                                     ata_mode_string(xfer_mask),
 -                                     (unsigned long long)dev->n_sectors,
 -                                     lba_desc);
 +                              ata_dev_printk(dev, KERN_INFO, "ATA-%d, "
 +                                      "max %s, %Lu sectors: %s %s\n",
 +                                      ata_id_major_version(id),
 +                                      ata_mode_string(xfer_mask),
 +                                      (unsigned long long)dev->n_sectors,
 +                                      lba_desc, ncq_desc);
                } else {
                        /* CHS */
  
  
                        /* print device info to dmesg */
                        if (print_info)
 -                              printk(KERN_INFO "ata%u: dev %u ATA-%d, "
 -                                     "max %s, %Lu sectors: CHS %u/%u/%u\n",
 -                                     ap->id, dev->devno,
 -                                     ata_id_major_version(id),
 -                                     ata_mode_string(xfer_mask),
 -                                     (unsigned long long)dev->n_sectors,
 -                                     dev->cylinders, dev->heads, dev->sectors);
 +                              ata_dev_printk(dev, KERN_INFO, "ATA-%d, "
 +                                      "max %s, %Lu sectors: CHS %u/%u/%u\n",
 +                                      ata_id_major_version(id),
 +                                      ata_mode_string(xfer_mask),
 +                                      (unsigned long long)dev->n_sectors,
 +                                      dev->cylinders, dev->heads, dev->sectors);
 +              }
 +
 +              if (dev->id[59] & 0x100) {
 +                      dev->multi_count = dev->id[59] & 0xff;
 +                      DPRINTK("ata%u: dev %u multi count %u\n",
 +                              ap->id, dev->devno, dev->multi_count);
                }
  
                dev->cdb_len = 16;
  
        /* ATAPI-specific feature tests */
        else if (dev->class == ATA_DEV_ATAPI) {
 +              char *cdb_intr_string = "";
 +
                rc = atapi_cdb_len(id);
                if ((rc < 12) || (rc > ATAPI_CDB_LEN)) {
 -                      printk(KERN_WARNING "ata%u: unsupported CDB len\n", ap->id);
 +                      ata_dev_printk(dev, KERN_WARNING,
 +                                     "unsupported CDB len\n");
                        rc = -EINVAL;
                        goto err_out_nosup;
                }
                dev->cdb_len = (unsigned int) rc;
  
 +              if (ata_id_cdb_intr(dev->id)) {
 +                      dev->flags |= ATA_DFLAG_CDB_INTR;
 +                      cdb_intr_string = ", CDB intr";
 +              }
 +
                /* print device info to dmesg */
                if (print_info)
 -                      printk(KERN_INFO "ata%u: dev %u ATAPI, max %s\n",
 -                             ap->id, dev->devno, ata_mode_string(xfer_mask));
 +                      ata_dev_printk(dev, KERN_INFO, "ATAPI, max %s%s\n",
 +                                     ata_mode_string(xfer_mask),
 +                                     cdb_intr_string);
        }
  
        ap->host->max_cmd_len = 0;
                                              ap->device[i].cdb_len);
  
        /* limit bridge transfers to udma5, 200 sectors */
 -      if (ata_dev_knobble(ap, dev)) {
 +      if (ata_dev_knobble(dev)) {
                if (print_info)
 -                      printk(KERN_INFO "ata%u(%u): applying bridge limits\n",
 -                             ap->id, dev->devno);
 +                      ata_dev_printk(dev, KERN_INFO,
 +                                     "applying bridge limits\n");
                dev->udma_mask &= ATA_UDMA5;
                dev->max_sectors = ATA_MAX_SECTORS;
        }
@@@ -1462,104 -1352,79 +1462,104 @@@ err_out_nosup
   *    PCI/etc. bus probe sem.
   *
   *    RETURNS:
 - *    Zero on success, non-zero on error.
 + *    Zero on success, negative errno otherwise.
   */
  
  static int ata_bus_probe(struct ata_port *ap)
  {
        unsigned int classes[ATA_MAX_DEVICES];
 -      unsigned int i, rc, found = 0;
 +      int tries[ATA_MAX_DEVICES];
 +      int i, rc, down_xfermask;
 +      struct ata_device *dev;
  
        ata_port_probe(ap);
  
 -      /* reset and determine device classes */
        for (i = 0; i < ATA_MAX_DEVICES; i++)
 -              classes[i] = ATA_DEV_UNKNOWN;
 +              tries[i] = ATA_PROBE_MAX_TRIES;
  
 -      if (ap->ops->probe_reset) {
 -              rc = ap->ops->probe_reset(ap, classes);
 -              if (rc) {
 -                      printk("ata%u: reset failed (errno=%d)\n", ap->id, rc);
 -                      return rc;
 -              }
 -      } else {
 -              ap->ops->phy_reset(ap);
 + retry:
 +      down_xfermask = 0;
  
 -              if (!(ap->flags & ATA_FLAG_PORT_DISABLED))
 -                      for (i = 0; i < ATA_MAX_DEVICES; i++)
 -                              classes[i] = ap->device[i].class;
 +      /* reset and determine device classes */
 +      ap->ops->phy_reset(ap);
  
 -              ata_port_probe(ap);
 +      for (i = 0; i < ATA_MAX_DEVICES; i++) {
 +              dev = &ap->device[i];
 +
 +              if (!(ap->flags & ATA_FLAG_DISABLED) &&
 +                  dev->class != ATA_DEV_UNKNOWN)
 +                      classes[dev->devno] = dev->class;
 +              else
 +                      classes[dev->devno] = ATA_DEV_NONE;
 +
 +              dev->class = ATA_DEV_UNKNOWN;
        }
  
 +      ata_port_probe(ap);
 +
 +      /* after the reset the device state is PIO 0 and the controller
 +         state is undefined. Record the mode */
 +
        for (i = 0; i < ATA_MAX_DEVICES; i++)
 -              if (classes[i] == ATA_DEV_UNKNOWN)
 -                      classes[i] = ATA_DEV_NONE;
 +              ap->device[i].pio_mode = XFER_PIO_0;
  
        /* read IDENTIFY page and configure devices */
        for (i = 0; i < ATA_MAX_DEVICES; i++) {
 -              struct ata_device *dev = &ap->device[i];
 +              dev = &ap->device[i];
  
 -              dev->class = classes[i];
 +              if (tries[i])
 +                      dev->class = classes[i];
  
 -              if (!ata_dev_present(dev))
 +              if (!ata_dev_enabled(dev))
                        continue;
  
 -              WARN_ON(dev->id != NULL);
 -              if (ata_dev_read_id(ap, dev, &dev->class, 1, &dev->id)) {
 -                      dev->class = ATA_DEV_NONE;
 -                      continue;
 -              }
 +              rc = ata_dev_read_id(dev, &dev->class, 1, dev->id);
 +              if (rc)
 +                      goto fail;
  
 -              if (ata_dev_configure(ap, dev, 1)) {
 -                      ata_dev_disable(ap, dev);
 -                      continue;
 -              }
 +              rc = ata_dev_configure(dev, 1);
 +              if (rc)
 +                      goto fail;
 +      }
  
 -              found = 1;
 +      /* configure transfer mode */
 +      rc = ata_set_mode(ap, &dev);
 +      if (rc) {
 +              down_xfermask = 1;
 +              goto fail;
        }
  
 -      if (!found)
 -              goto err_out_disable;
 +      for (i = 0; i < ATA_MAX_DEVICES; i++)
 +              if (ata_dev_enabled(&ap->device[i]))
 +                      return 0;
  
 -      if (ap->ops->set_mode)
 -              ap->ops->set_mode(ap);
 -      else
 -              ata_set_mode(ap);
 +      /* no device present, disable port */
 +      ata_port_disable(ap);
 +      ap->ops->port_disable(ap);
 +      return -ENODEV;
  
 -      if (ap->flags & ATA_FLAG_PORT_DISABLED)
 -              goto err_out_disable;
 + fail:
 +      switch (rc) {
 +      case -EINVAL:
 +      case -ENODEV:
 +              tries[dev->devno] = 0;
 +              break;
 +      case -EIO:
 +              sata_down_spd_limit(ap);
 +              /* fall through */
 +      default:
 +              tries[dev->devno]--;
 +              if (down_xfermask &&
 +                  ata_down_xfermask_limit(dev, tries[dev->devno] == 1))
 +                      tries[dev->devno] = 0;
 +      }
  
 -      return 0;
 +      if (!tries[dev->devno]) {
 +              ata_down_xfermask_limit(dev, 1);
 +              ata_dev_disable(dev);
 +      }
  
 -err_out_disable:
 -      ap->ops->port_disable(ap);
 -      return -1;
 +      goto retry;
  }
  
  /**
  
  void ata_port_probe(struct ata_port *ap)
  {
 -      ap->flags &= ~ATA_FLAG_PORT_DISABLED;
 +      ap->flags &= ~ATA_FLAG_DISABLED;
  }
  
  /**
   */
  static void sata_print_link_status(struct ata_port *ap)
  {
 -      u32 sstatus, tmp;
 -      const char *speed;
 +      u32 sstatus, scontrol, tmp;
  
 -      if (!ap->ops->scr_read)
 +      if (sata_scr_read(ap, SCR_STATUS, &sstatus))
                return;
 +      sata_scr_read(ap, SCR_CONTROL, &scontrol);
  
 -      sstatus = scr_read(ap, SCR_STATUS);
 -
 -      if (sata_dev_present(ap)) {
 +      if (ata_port_online(ap)) {
                tmp = (sstatus >> 4) & 0xf;
 -              if (tmp & (1 << 0))
 -                      speed = "1.5";
 -              else if (tmp & (1 << 1))
 -                      speed = "3.0";
 -              else
 -                      speed = "<unknown>";
 -              printk(KERN_INFO "ata%u: SATA link up %s Gbps (SStatus %X)\n",
 -                     ap->id, speed, sstatus);
 +              ata_port_printk(ap, KERN_INFO,
 +                              "SATA link up %s (SStatus %X SControl %X)\n",
 +                              sata_spd_string(tmp), sstatus, scontrol);
        } else {
 -              printk(KERN_INFO "ata%u: SATA link down (SStatus %X)\n",
 -                     ap->id, sstatus);
 +              ata_port_printk(ap, KERN_INFO,
 +                              "SATA link down (SStatus %X SControl %X)\n",
 +                              sstatus, scontrol);
        }
  }
  
@@@ -1626,18 -1497,17 +1626,18 @@@ void __sata_phy_reset(struct ata_port *
  
        if (ap->flags & ATA_FLAG_SATA_RESET) {
                /* issue phy wake/reset */
 -              scr_write_flush(ap, SCR_CONTROL, 0x301);
 +              sata_scr_write_flush(ap, SCR_CONTROL, 0x301);
                /* Couldn't find anything in SATA I/II specs, but
                 * AHCI-1.1 10.4.2 says at least 1 ms. */
                mdelay(1);
        }
 -      scr_write_flush(ap, SCR_CONTROL, 0x300); /* phy wake/clear reset */
 +      /* phy wake/clear reset */
 +      sata_scr_write_flush(ap, SCR_CONTROL, 0x300);
  
        /* wait for phy to become ready, if necessary */
        do {
                msleep(200);
 -              sstatus = scr_read(ap, SCR_STATUS);
 +              sata_scr_read(ap, SCR_STATUS, &sstatus);
                if ((sstatus & 0xf) != 1)
                        break;
        } while (time_before(jiffies, timeout));
        sata_print_link_status(ap);
  
        /* TODO: phy layer with polling, timeouts, etc. */
 -      if (sata_dev_present(ap))
 +      if (!ata_port_offline(ap))
                ata_port_probe(ap);
        else
                ata_port_disable(ap);
  
 -      if (ap->flags & ATA_FLAG_PORT_DISABLED)
 +      if (ap->flags & ATA_FLAG_DISABLED)
                return;
  
        if (ata_busy_sleep(ap, ATA_TMOUT_BOOT_QUICK, ATA_TMOUT_BOOT)) {
  void sata_phy_reset(struct ata_port *ap)
  {
        __sata_phy_reset(ap);
 -      if (ap->flags & ATA_FLAG_PORT_DISABLED)
 +      if (ap->flags & ATA_FLAG_DISABLED)
                return;
        ata_bus_reset(ap);
  }
  
  /**
   *    ata_dev_pair            -       return other device on cable
 - *    @ap: port
   *    @adev: device
   *
   *    Obtain the other device on the same cable, or if none is
   *    present NULL is returned
   */
  
 -struct ata_device *ata_dev_pair(struct ata_port *ap, struct ata_device *adev)
 +struct ata_device *ata_dev_pair(struct ata_device *adev)
  {
 +      struct ata_port *ap = adev->ap;
        struct ata_device *pair = &ap->device[1 - adev->devno];
 -      if (!ata_dev_present(pair))
 +      if (!ata_dev_enabled(pair))
                return NULL;
        return pair;
  }
@@@ -1715,122 -1585,7 +1715,122 @@@ void ata_port_disable(struct ata_port *
  {
        ap->device[0].class = ATA_DEV_NONE;
        ap->device[1].class = ATA_DEV_NONE;
 -      ap->flags |= ATA_FLAG_PORT_DISABLED;
 +      ap->flags |= ATA_FLAG_DISABLED;
 +}
 +
 +/**
 + *    sata_down_spd_limit - adjust SATA spd limit downward
 + *    @ap: Port to adjust SATA spd limit for
 + *
 + *    Adjust SATA spd limit of @ap downward.  Note that this
 + *    function only adjusts the limit.  The change must be applied
 + *    using sata_set_spd().
 + *
 + *    LOCKING:
 + *    Inherited from caller.
 + *
 + *    RETURNS:
 + *    0 on success, negative errno on failure
 + */
 +int sata_down_spd_limit(struct ata_port *ap)
 +{
 +      u32 sstatus, spd, mask;
 +      int rc, highbit;
 +
 +      rc = sata_scr_read(ap, SCR_STATUS, &sstatus);
 +      if (rc)
 +              return rc;
 +
 +      mask = ap->sata_spd_limit;
 +      if (mask <= 1)
 +              return -EINVAL;
 +      highbit = fls(mask) - 1;
 +      mask &= ~(1 << highbit);
 +
 +      spd = (sstatus >> 4) & 0xf;
 +      if (spd <= 1)
 +              return -EINVAL;
 +      spd--;
 +      mask &= (1 << spd) - 1;
 +      if (!mask)
 +              return -EINVAL;
 +
 +      ap->sata_spd_limit = mask;
 +
 +      ata_port_printk(ap, KERN_WARNING, "limiting SATA link speed to %s\n",
 +                      sata_spd_string(fls(mask)));
 +
 +      return 0;
 +}
 +
 +static int __sata_set_spd_needed(struct ata_port *ap, u32 *scontrol)
 +{
 +      u32 spd, limit;
 +
 +      if (ap->sata_spd_limit == UINT_MAX)
 +              limit = 0;
 +      else
 +              limit = fls(ap->sata_spd_limit);
 +
 +      spd = (*scontrol >> 4) & 0xf;
 +      *scontrol = (*scontrol & ~0xf0) | ((limit & 0xf) << 4);
 +
 +      return spd != limit;
 +}
 +
 +/**
 + *    sata_set_spd_needed - is SATA spd configuration needed
 + *    @ap: Port in question
 + *
 + *    Test whether the spd limit in SControl matches
 + *    @ap->sata_spd_limit.  This function is used to determine
 + *    whether hardreset is necessary to apply SATA spd
 + *    configuration.
 + *
 + *    LOCKING:
 + *    Inherited from caller.
 + *
 + *    RETURNS:
 + *    1 if SATA spd configuration is needed, 0 otherwise.
 + */
 +int sata_set_spd_needed(struct ata_port *ap)
 +{
 +      u32 scontrol;
 +
 +      if (sata_scr_read(ap, SCR_CONTROL, &scontrol))
 +              return 0;
 +
 +      return __sata_set_spd_needed(ap, &scontrol);
 +}
 +
 +/**
 + *    sata_set_spd - set SATA spd according to spd limit
 + *    @ap: Port to set SATA spd for
 + *
 + *    Set SATA spd of @ap according to sata_spd_limit.
 + *
 + *    LOCKING:
 + *    Inherited from caller.
 + *
 + *    RETURNS:
 + *    0 if spd doesn't need to be changed, 1 if spd has been
 + *    changed.  Negative errno if SCR registers are inaccessible.
 + */
 +int sata_set_spd(struct ata_port *ap)
 +{
 +      u32 scontrol;
 +      int rc;
 +
 +      if ((rc = sata_scr_read(ap, SCR_CONTROL, &scontrol)))
 +              return rc;
 +
 +      if (!__sata_set_spd_needed(ap, &scontrol))
 +              return 0;
 +
 +      if ((rc = sata_scr_write(ap, SCR_CONTROL, scontrol)))
 +              return rc;
 +
 +      return 1;
  }
  
  /*
@@@ -1981,196 -1736,151 +1981,196 @@@ int ata_timing_compute(struct ata_devic
        return 0;
  }
  
 -static int ata_dev_set_mode(struct ata_port *ap, struct ata_device *dev)
 +/**
 + *    ata_down_xfermask_limit - adjust dev xfer masks downward
 + *    @dev: Device to adjust xfer masks
 + *    @force_pio0: Force PIO0
 + *
 + *    Adjust xfer masks of @dev downward.  Note that this function
 + *    does not apply the change.  Invoking ata_set_mode() afterwards
 + *    will apply the limit.
 + *
 + *    LOCKING:
 + *    Inherited from caller.
 + *
 + *    RETURNS:
 + *    0 on success, negative errno on failure
 + */
 +int ata_down_xfermask_limit(struct ata_device *dev, int force_pio0)
 +{
 +      unsigned long xfer_mask;
 +      int highbit;
 +
 +      xfer_mask = ata_pack_xfermask(dev->pio_mask, dev->mwdma_mask,
 +                                    dev->udma_mask);
 +
 +      if (!xfer_mask)
 +              goto fail;
 +      /* don't gear down to MWDMA from UDMA, go directly to PIO */
 +      if (xfer_mask & ATA_MASK_UDMA)
 +              xfer_mask &= ~ATA_MASK_MWDMA;
 +
 +      highbit = fls(xfer_mask) - 1;
 +      xfer_mask &= ~(1 << highbit);
 +      if (force_pio0)
 +              xfer_mask &= 1 << ATA_SHIFT_PIO;
 +      if (!xfer_mask)
 +              goto fail;
 +
 +      ata_unpack_xfermask(xfer_mask, &dev->pio_mask, &dev->mwdma_mask,
 +                          &dev->udma_mask);
 +
 +      ata_dev_printk(dev, KERN_WARNING, "limiting speed to %s\n",
 +                     ata_mode_string(xfer_mask));
 +
 +      return 0;
 +
 + fail:
 +      return -EINVAL;
 +}
 +
 +static int ata_dev_set_mode(struct ata_device *dev)
  {
        unsigned int err_mask;
        int rc;
  
 +      dev->flags &= ~ATA_DFLAG_PIO;
        if (dev->xfer_shift == ATA_SHIFT_PIO)
                dev->flags |= ATA_DFLAG_PIO;
  
 -      err_mask = ata_dev_set_xfermode(ap, dev);
 +      err_mask = ata_dev_set_xfermode(dev);
        if (err_mask) {
 -              printk(KERN_ERR
 -                     "ata%u: failed to set xfermode (err_mask=0x%x)\n",
 -                     ap->id, err_mask);
 +              ata_dev_printk(dev, KERN_ERR, "failed to set xfermode "
 +                             "(err_mask=0x%x)\n", err_mask);
                return -EIO;
        }
  
 -      rc = ata_dev_revalidate(ap, dev, 0);
 -      if (rc) {
 -              printk(KERN_ERR
 -                     "ata%u: failed to revalidate after set xfermode\n",
 -                     ap->id);
 +      rc = ata_dev_revalidate(dev, 0);
 +      if (rc)
                return rc;
 -      }
  
        DPRINTK("xfer_shift=%u, xfer_mode=0x%x\n",
                dev->xfer_shift, (int)dev->xfer_mode);
  
 -      printk(KERN_INFO "ata%u: dev %u configured for %s\n",
 -             ap->id, dev->devno,
 -             ata_mode_string(ata_xfer_mode2mask(dev->xfer_mode)));
 -      return 0;
 -}
 -
 -static int ata_host_set_pio(struct ata_port *ap)
 -{
 -      int i;
 -
 -      for (i = 0; i < ATA_MAX_DEVICES; i++) {
 -              struct ata_device *dev = &ap->device[i];
 -
 -              if (!ata_dev_present(dev))
 -                      continue;
 -
 -              if (!dev->pio_mode) {
 -                      printk(KERN_WARNING "ata%u: no PIO support for device %d.\n", ap->id, i);
 -                      return -1;
 -              }
 -
 -              dev->xfer_mode = dev->pio_mode;
 -              dev->xfer_shift = ATA_SHIFT_PIO;
 -              if (ap->ops->set_piomode)
 -                      ap->ops->set_piomode(ap, dev);
 -      }
 -
 +      ata_dev_printk(dev, KERN_INFO, "configured for %s\n",
 +                     ata_mode_string(ata_xfer_mode2mask(dev->xfer_mode)));
        return 0;
  }
  
 -static void ata_host_set_dma(struct ata_port *ap)
 -{
 -      int i;
 -
 -      for (i = 0; i < ATA_MAX_DEVICES; i++) {
 -              struct ata_device *dev = &ap->device[i];
 -
 -              if (!ata_dev_present(dev) || !dev->dma_mode)
 -                      continue;
 -
 -              dev->xfer_mode = dev->dma_mode;
 -              dev->xfer_shift = ata_xfer_mode2shift(dev->dma_mode);
 -              if (ap->ops->set_dmamode)
 -                      ap->ops->set_dmamode(ap, dev);
 -      }
 -}
 -
  /**
   *    ata_set_mode - Program timings and issue SET FEATURES - XFER
   *    @ap: port on which timings will be programmed
 + *    @r_failed_dev: out paramter for failed device
   *
 - *    Set ATA device disk transfer mode (PIO3, UDMA6, etc.).
 + *    Set ATA device disk transfer mode (PIO3, UDMA6, etc.).  If
 + *    ata_set_mode() fails, pointer to the failing device is
 + *    returned in @r_failed_dev.
   *
   *    LOCKING:
   *    PCI/etc. bus probe sem.
 + *
 + *    RETURNS:
 + *    0 on success, negative errno otherwise
   */
 -static void ata_set_mode(struct ata_port *ap)
 +int ata_set_mode(struct ata_port *ap, struct ata_device **r_failed_dev)
  {
 -      int i, rc, used_dma = 0;
 +      struct ata_device *dev;
 +      int i, rc = 0, used_dma = 0, found = 0;
 +
 +      /* has private set_mode? */
 +      if (ap->ops->set_mode) {
 +              /* FIXME: make ->set_mode handle no device case and
 +               * return error code and failing device on failure.
 +               */
 +              for (i = 0; i < ATA_MAX_DEVICES; i++) {
 +                      if (ata_dev_enabled(&ap->device[i])) {
 +                              ap->ops->set_mode(ap);
 +                              break;
 +                      }
 +              }
 +              return 0;
 +      }
  
        /* step 1: calculate xfer_mask */
        for (i = 0; i < ATA_MAX_DEVICES; i++) {
 -              struct ata_device *dev = &ap->device[i];
                unsigned int pio_mask, dma_mask;
  
 -              if (!ata_dev_present(dev))
 -                      continue;
 +              dev = &ap->device[i];
  
 -              ata_dev_xfermask(ap, dev);
 +              if (!ata_dev_enabled(dev))
 +                      continue;
  
 -              /* TODO: let LLDD filter dev->*_mask here */
 +              ata_dev_xfermask(dev);
  
                pio_mask = ata_pack_xfermask(dev->pio_mask, 0, 0);
                dma_mask = ata_pack_xfermask(0, dev->mwdma_mask, dev->udma_mask);
                dev->pio_mode = ata_xfer_mask2mode(pio_mask);
                dev->dma_mode = ata_xfer_mask2mode(dma_mask);
  
 +              found = 1;
                if (dev->dma_mode)
                        used_dma = 1;
        }
 +      if (!found)
 +              goto out;
  
        /* step 2: always set host PIO timings */
 -      rc = ata_host_set_pio(ap);
 -      if (rc)
 -              goto err_out;
 -
 -      /* step 3: set host DMA timings */
 -      ata_host_set_dma(ap);
 -
 -      /* step 4: update devices' xfer mode */
        for (i = 0; i < ATA_MAX_DEVICES; i++) {
 -              struct ata_device *dev = &ap->device[i];
 -
 -              if (!ata_dev_present(dev))
 +              dev = &ap->device[i];
 +              if (!ata_dev_enabled(dev))
                        continue;
  
 -              if (ata_dev_set_mode(ap, dev))
 -                      goto err_out;
 -      }
 -
 -      /*
 -       *      Record simplex status. If we selected DMA then the other
 -       *      host channels are not permitted to do so.
 -       */
 +              if (!dev->pio_mode) {
 +                      ata_dev_printk(dev, KERN_WARNING, "no PIO support\n");
 +                      rc = -EINVAL;
 +                      goto out;
 +              }
 +
 +              dev->xfer_mode = dev->pio_mode;
 +              dev->xfer_shift = ATA_SHIFT_PIO;
 +              if (ap->ops->set_piomode)
 +                      ap->ops->set_piomode(ap, dev);
 +      }
 +
 +      /* step 3: set host DMA timings */
 +      for (i = 0; i < ATA_MAX_DEVICES; i++) {
 +              dev = &ap->device[i];
 +
 +              if (!ata_dev_enabled(dev) || !dev->dma_mode)
 +                      continue;
  
 +              dev->xfer_mode = dev->dma_mode;
 +              dev->xfer_shift = ata_xfer_mode2shift(dev->dma_mode);
 +              if (ap->ops->set_dmamode)
 +                      ap->ops->set_dmamode(ap, dev);
 +      }
 +
 +      /* step 4: update devices' xfer mode */
 +      for (i = 0; i < ATA_MAX_DEVICES; i++) {
 +              dev = &ap->device[i];
 +
 +              if (!ata_dev_enabled(dev))
 +                      continue;
 +
 +              rc = ata_dev_set_mode(dev);
 +              if (rc)
 +                      goto out;
 +      }
 +
 +      /* Record simplex status. If we selected DMA then the other
 +       * host channels are not permitted to do so.
 +       */
        if (used_dma && (ap->host_set->flags & ATA_HOST_SIMPLEX))
                ap->host_set->simplex_claimed = 1;
  
 -      /*
 -       *      Chip specific finalisation
 -       */
 +      /* step5: chip specific finalisation */
        if (ap->ops->post_set_mode)
                ap->ops->post_set_mode(ap);
  
 -      return;
 -
 -err_out:
 -      ata_port_disable(ap);
 + out:
 +      if (rc)
 +              *r_failed_dev = dev;
 +      return rc;
  }
  
  /**
@@@ -2220,8 -1930,8 +2220,8 @@@ unsigned int ata_busy_sleep (struct ata
        }
  
        if (status & ATA_BUSY)
 -              printk(KERN_WARNING "ata%u is slow to respond, "
 -                     "please be patient\n", ap->id);
 +              ata_port_printk(ap, KERN_WARNING,
 +                              "port is slow to respond, please be patient\n");
  
        timeout = timer_start + tmout;
        while ((status & ATA_BUSY) && (time_before(jiffies, timeout))) {
        }
  
        if (status & ATA_BUSY) {
 -              printk(KERN_ERR "ata%u failed to respond (%lu secs)\n",
 -                     ap->id, tmout / HZ);
 +              ata_port_printk(ap, KERN_ERR, "port failed to respond "
 +                              "(%lu secs)\n", tmout / HZ);
                return 1;
        }
  
@@@ -2323,10 -2033,8 +2323,10 @@@ static unsigned int ata_bus_softreset(s
         * the bus shows 0xFF because the odd clown forgets the D7
         * pulldown resistor.
         */
 -      if (ata_check_status(ap) == 0xFF)
 +      if (ata_check_status(ap) == 0xFF) {
 +              ata_port_printk(ap, KERN_ERR, "SRST failed (status 0xFF)\n");
                return AC_ERR_OTHER;
 +      }
  
        ata_bus_post_reset(ap, devmask);
  
   *    Obtains host_set lock.
   *
   *    SIDE EFFECTS:
 - *    Sets ATA_FLAG_PORT_DISABLED if bus reset fails.
 + *    Sets ATA_FLAG_DISABLED if bus reset fails.
   */
  
  void ata_bus_reset(struct ata_port *ap)
        return;
  
  err_out:
 -      printk(KERN_ERR "ata%u: disabling port\n", ap->id);
 +      ata_port_printk(ap, KERN_ERR, "disabling port\n");
        ap->ops->port_disable(ap);
  
        DPRINTK("EXIT\n");
  }
  
 -static int sata_phy_resume(struct ata_port *ap)
 +/**
 + *    sata_phy_debounce - debounce SATA phy status
 + *    @ap: ATA port to debounce SATA phy status for
 + *    @params: timing parameters { interval, duratinon, timeout } in msec
 + *
 + *    Make sure SStatus of @ap reaches stable state, determined by
 + *    holding the same value where DET is not 1 for @duration polled
 + *    every @interval, before @timeout.  Timeout constraints the
 + *    beginning of the stable state.  Because, after hot unplugging,
 + *    DET gets stuck at 1 on some controllers, this functions waits
 + *    until timeout then returns 0 if DET is stable at 1.
 + *
 + *    LOCKING:
 + *    Kernel thread context (may sleep)
 + *
 + *    RETURNS:
 + *    0 on success, -errno on failure.
 + */
 +int sata_phy_debounce(struct ata_port *ap, const unsigned long *params)
  {
 -      unsigned long timeout = jiffies + (HZ * 5);
 -      u32 sstatus;
 +      unsigned long interval_msec = params[0];
 +      unsigned long duration = params[1] * HZ / 1000;
 +      unsigned long timeout = jiffies + params[2] * HZ / 1000;
 +      unsigned long last_jiffies;
 +      u32 last, cur;
 +      int rc;
  
 -      scr_write_flush(ap, SCR_CONTROL, 0x300);
 +      if ((rc = sata_scr_read(ap, SCR_STATUS, &cur)))
 +              return rc;
 +      cur &= 0xf;
  
 -      /* Wait for phy to become ready, if necessary. */
 -      do {
 -              msleep(200);
 -              sstatus = scr_read(ap, SCR_STATUS);
 -              if ((sstatus & 0xf) != 1)
 -                      return 0;
 -      } while (time_before(jiffies, timeout));
 +      last = cur;
 +      last_jiffies = jiffies;
  
 -      return -1;
 +      while (1) {
 +              msleep(interval_msec);
 +              if ((rc = sata_scr_read(ap, SCR_STATUS, &cur)))
 +                      return rc;
 +              cur &= 0xf;
 +
 +              /* DET stable? */
 +              if (cur == last) {
 +                      if (cur == 1 && time_before(jiffies, timeout))
 +                              continue;
 +                      if (time_after(jiffies, last_jiffies + duration))
 +                              return 0;
 +                      continue;
 +              }
 +
 +              /* unstable, start over */
 +              last = cur;
 +              last_jiffies = jiffies;
 +
 +              /* check timeout */
 +              if (time_after(jiffies, timeout))
 +                      return -EBUSY;
 +      }
  }
  
  /**
 - *    ata_std_probeinit - initialize probing
 - *    @ap: port to be probed
 + *    sata_phy_resume - resume SATA phy
 + *    @ap: ATA port to resume SATA phy for
 + *    @params: timing parameters { interval, duratinon, timeout } in msec
   *
 - *    @ap is about to be probed.  Initialize it.  This function is
 - *    to be used as standard callback for ata_drive_probe_reset().
 + *    Resume SATA phy of @ap and debounce it.
   *
 - *    NOTE!!! Do not use this function as probeinit if a low level
 - *    driver implements only hardreset.  Just pass NULL as probeinit
 - *    in that case.  Using this function is probably okay but doing
 - *    so makes reset sequence different from the original
 - *    ->phy_reset implementation and Jeff nervous.  :-P
 + *    LOCKING:
 + *    Kernel thread context (may sleep)
 + *
 + *    RETURNS:
 + *    0 on success, -errno on failure.
 + */
 +int sata_phy_resume(struct ata_port *ap, const unsigned long *params)
 +{
 +      u32 scontrol;
 +      int rc;
 +
 +      if ((rc = sata_scr_read(ap, SCR_CONTROL, &scontrol)))
 +              return rc;
 +
 +      scontrol = (scontrol & 0x0f0) | 0x300;
 +
 +      if ((rc = sata_scr_write(ap, SCR_CONTROL, scontrol)))
 +              return rc;
 +
 +      /* Some PHYs react badly if SStatus is pounded immediately
 +       * after resuming.  Delay 200ms before debouncing.
 +       */
 +      msleep(200);
 +
 +      return sata_phy_debounce(ap, params);
 +}
 +
 +static void ata_wait_spinup(struct ata_port *ap)
 +{
 +      struct ata_eh_context *ehc = &ap->eh_context;
 +      unsigned long end, secs;
 +      int rc;
 +
 +      /* first, debounce phy if SATA */
 +      if (ap->cbl == ATA_CBL_SATA) {
 +              rc = sata_phy_debounce(ap, sata_deb_timing_eh);
 +
 +              /* if debounced successfully and offline, no need to wait */
 +              if ((rc == 0 || rc == -EOPNOTSUPP) && ata_port_offline(ap))
 +                      return;
 +      }
 +
 +      /* okay, let's give the drive time to spin up */
 +      end = ehc->i.hotplug_timestamp + ATA_SPINUP_WAIT * HZ / 1000;
 +      secs = ((end - jiffies) + HZ - 1) / HZ;
 +
 +      if (time_after(jiffies, end))
 +              return;
 +
 +      if (secs > 5)
 +              ata_port_printk(ap, KERN_INFO, "waiting for device to spin up "
 +                              "(%lu secs)\n", secs);
 +
 +      schedule_timeout_uninterruptible(end - jiffies);
 +}
 +
 +/**
 + *    ata_std_prereset - prepare for reset
 + *    @ap: ATA port to be reset
 + *
 + *    @ap is about to be reset.  Initialize it.
 + *
 + *    LOCKING:
 + *    Kernel thread context (may sleep)
 + *
 + *    RETURNS:
 + *    0 on success, -errno otherwise.
   */
 -void ata_std_probeinit(struct ata_port *ap)
 +int ata_std_prereset(struct ata_port *ap)
  {
 -      if ((ap->flags & ATA_FLAG_SATA) && ap->ops->scr_read) {
 -              sata_phy_resume(ap);
 -              if (sata_dev_present(ap))
 -                      ata_busy_sleep(ap, ATA_TMOUT_BOOT_QUICK, ATA_TMOUT_BOOT);
 +      struct ata_eh_context *ehc = &ap->eh_context;
 +      const unsigned long *timing;
 +      int rc;
 +
 +      /* hotplug? */
 +      if (ehc->i.flags & ATA_EHI_HOTPLUGGED) {
 +              if (ap->flags & ATA_FLAG_HRST_TO_RESUME)
 +                      ehc->i.action |= ATA_EH_HARDRESET;
 +              if (ap->flags & ATA_FLAG_SKIP_D2H_BSY)
 +                      ata_wait_spinup(ap);
 +      }
 +
 +      /* if we're about to do hardreset, nothing more to do */
 +      if (ehc->i.action & ATA_EH_HARDRESET)
 +              return 0;
 +
 +      /* if SATA, resume phy */
 +      if (ap->cbl == ATA_CBL_SATA) {
 +              if (ap->flags & ATA_FLAG_LOADING)
 +                      timing = sata_deb_timing_boot;
 +              else
 +                      timing = sata_deb_timing_eh;
 +
 +              rc = sata_phy_resume(ap, timing);
 +              if (rc && rc != -EOPNOTSUPP) {
 +                      /* phy resume failed */
 +                      ata_port_printk(ap, KERN_WARNING, "failed to resume "
 +                                      "link for reset (errno=%d)\n", rc);
 +                      return rc;
 +              }
        }
 +
 +      /* Wait for !BSY if the controller can wait for the first D2H
 +       * Reg FIS and we don't know that no device is attached.
 +       */
 +      if (!(ap->flags & ATA_FLAG_SKIP_D2H_BSY) && !ata_port_offline(ap))
 +              ata_busy_sleep(ap, ATA_TMOUT_BOOT_QUICK, ATA_TMOUT_BOOT);
 +
 +      return 0;
  }
  
  /**
   *    ata_std_softreset - reset host port via ATA SRST
   *    @ap: port to reset
 - *    @verbose: fail verbosely
   *    @classes: resulting classes of attached devices
   *
 - *    Reset host port using ATA SRST.  This function is to be used
 - *    as standard callback for ata_drive_*_reset() functions.
 + *    Reset host port using ATA SRST.
   *
   *    LOCKING:
   *    Kernel thread context (may sleep)
   *    RETURNS:
   *    0 on success, -errno otherwise.
   */
 -int ata_std_softreset(struct ata_port *ap, int verbose, unsigned int *classes)
 +int ata_std_softreset(struct ata_port *ap, unsigned int *classes)
  {
        unsigned int slave_possible = ap->flags & ATA_FLAG_SLAVE_POSS;
        unsigned int devmask = 0, err_mask;
  
        DPRINTK("ENTER\n");
  
 -      if (ap->ops->scr_read && !sata_dev_present(ap)) {
 +      if (ata_port_offline(ap)) {
                classes[0] = ATA_DEV_NONE;
                goto out;
        }
        DPRINTK("about to softreset, devmask=%x\n", devmask);
        err_mask = ata_bus_softreset(ap, devmask);
        if (err_mask) {
 -              if (verbose)
 -                      printk(KERN_ERR "ata%u: SRST failed (err_mask=0x%x)\n",
 -                             ap->id, err_mask);
 -              else
 -                      DPRINTK("EXIT, softreset failed (err_mask=0x%x)\n",
 +              ata_port_printk(ap, KERN_ERR, "SRST failed (err_mask=0x%x)\n",
                                err_mask);
                return -EIO;
        }
  /**
   *    sata_std_hardreset - reset host port via SATA phy reset
   *    @ap: port to reset
 - *    @verbose: fail verbosely
   *    @class: resulting class of attached device
   *
   *    SATA phy-reset host port using DET bits of SControl register.
 - *    This function is to be used as standard callback for
 - *    ata_drive_*_reset().
   *
   *    LOCKING:
   *    Kernel thread context (may sleep)
   *    RETURNS:
   *    0 on success, -errno otherwise.
   */
 -int sata_std_hardreset(struct ata_port *ap, int verbose, unsigned int *class)
 +int sata_std_hardreset(struct ata_port *ap, unsigned int *class)
  {
 +      u32 scontrol;
 +      int rc;
 +
        DPRINTK("ENTER\n");
  
 -      /* Issue phy wake/reset */
 -      scr_write_flush(ap, SCR_CONTROL, 0x301);
 +      if (sata_set_spd_needed(ap)) {
 +              /* SATA spec says nothing about how to reconfigure
 +               * spd.  To be on the safe side, turn off phy during
 +               * reconfiguration.  This works for at least ICH7 AHCI
 +               * and Sil3124.
 +               */
 +              if ((rc = sata_scr_read(ap, SCR_CONTROL, &scontrol)))
 +                      return rc;
  
 -      /*
 -       * Couldn't find anything in SATA I/II specs, but AHCI-1.1
 +              scontrol = (scontrol & 0x0f0) | 0x302;
 +
 +              if ((rc = sata_scr_write(ap, SCR_CONTROL, scontrol)))
 +                      return rc;
 +
 +              sata_set_spd(ap);
 +      }
 +
 +      /* issue phy wake/reset */
 +      if ((rc = sata_scr_read(ap, SCR_CONTROL, &scontrol)))
 +              return rc;
 +
 +      scontrol = (scontrol & 0x0f0) | 0x301;
 +
 +      if ((rc = sata_scr_write_flush(ap, SCR_CONTROL, scontrol)))
 +              return rc;
 +
 +      /* Couldn't find anything in SATA I/II specs, but AHCI-1.1
         * 10.4.2 says at least 1 ms.
         */
        msleep(1);
  
 -      /* Bring phy back */
 -      sata_phy_resume(ap);
 +      /* bring phy back */
 +      sata_phy_resume(ap, sata_deb_timing_eh);
  
        /* TODO: phy layer with polling, timeouts, etc. */
 -      if (!sata_dev_present(ap)) {
 +      if (ata_port_offline(ap)) {
                *class = ATA_DEV_NONE;
                DPRINTK("EXIT, link offline\n");
                return 0;
        }
  
        if (ata_busy_sleep(ap, ATA_TMOUT_BOOT_QUICK, ATA_TMOUT_BOOT)) {
 -              if (verbose)
 -                      printk(KERN_ERR "ata%u: COMRESET failed "
 -                             "(device not ready)\n", ap->id);
 -              else
 -                      DPRINTK("EXIT, device not ready\n");
 +              ata_port_printk(ap, KERN_ERR,
 +                              "COMRESET failed (device not ready)\n");
                return -EIO;
        }
  
   *    the device might have been reset more than once using
   *    different reset methods before postreset is invoked.
   *
 - *    This function is to be used as standard callback for
 - *    ata_drive_*_reset().
 - *
   *    LOCKING:
   *    Kernel thread context (may sleep)
   */
  void ata_std_postreset(struct ata_port *ap, unsigned int *classes)
  {
 -      DPRINTK("ENTER\n");
 +      u32 serror;
  
 -      /* set cable type if it isn't already set */
 -      if (ap->cbl == ATA_CBL_NONE && ap->flags & ATA_FLAG_SATA)
 -              ap->cbl = ATA_CBL_SATA;
 +      DPRINTK("ENTER\n");
  
        /* print link status */
 -      if (ap->cbl == ATA_CBL_SATA)
 -              sata_print_link_status(ap);
 +      sata_print_link_status(ap);
 +
 +      /* clear SError */
 +      if (sata_scr_read(ap, SCR_ERROR, &serror) == 0)
 +              sata_scr_write(ap, SCR_ERROR, serror);
  
        /* re-enable interrupts */
 -      if (ap->ioaddr.ctl_addr)        /* FIXME: hack. create a hook instead */
 -              ata_irq_on(ap);
 +      if (!ap->ops->error_handler) {
 +              /* FIXME: hack. create a hook instead */
 +              if (ap->ioaddr.ctl_addr)
 +                      ata_irq_on(ap);
 +      }
  
        /* is double-select really necessary? */
        if (classes[0] != ATA_DEV_NONE)
        DPRINTK("EXIT\n");
  }
  
 -/**
 - *    ata_std_probe_reset - standard probe reset method
 - *    @ap: prot to perform probe-reset
 - *    @classes: resulting classes of attached devices
 - *
 - *    The stock off-the-shelf ->probe_reset method.
 - *
 - *    LOCKING:
 - *    Kernel thread context (may sleep)
 - *
 - *    RETURNS:
 - *    0 on success, -errno otherwise.
 - */
 -int ata_std_probe_reset(struct ata_port *ap, unsigned int *classes)
 -{
 -      ata_reset_fn_t hardreset;
 -
 -      hardreset = NULL;
 -      if (ap->flags & ATA_FLAG_SATA && ap->ops->scr_read)
 -              hardreset = sata_std_hardreset;
 -
 -      return ata_drive_probe_reset(ap, ata_std_probeinit,
 -                                   ata_std_softreset, hardreset,
 -                                   ata_std_postreset, classes);
 -}
 -
 -static int do_probe_reset(struct ata_port *ap, ata_reset_fn_t reset,
 -                        ata_postreset_fn_t postreset,
 -                        unsigned int *classes)
 -{
 -      int i, rc;
 -
 -      for (i = 0; i < ATA_MAX_DEVICES; i++)
 -              classes[i] = ATA_DEV_UNKNOWN;
 -
 -      rc = reset(ap, 0, classes);
 -      if (rc)
 -              return rc;
 -
 -      /* If any class isn't ATA_DEV_UNKNOWN, consider classification
 -       * is complete and convert all ATA_DEV_UNKNOWN to
 -       * ATA_DEV_NONE.
 -       */
 -      for (i = 0; i < ATA_MAX_DEVICES; i++)
 -              if (classes[i] != ATA_DEV_UNKNOWN)
 -                      break;
 -
 -      if (i < ATA_MAX_DEVICES)
 -              for (i = 0; i < ATA_MAX_DEVICES; i++)
 -                      if (classes[i] == ATA_DEV_UNKNOWN)
 -                              classes[i] = ATA_DEV_NONE;
 -
 -      if (postreset)
 -              postreset(ap, classes);
 -
 -      return classes[0] != ATA_DEV_UNKNOWN ? 0 : -ENODEV;
 -}
 -
 -/**
 - *    ata_drive_probe_reset - Perform probe reset with given methods
 - *    @ap: port to reset
 - *    @probeinit: probeinit method (can be NULL)
 - *    @softreset: softreset method (can be NULL)
 - *    @hardreset: hardreset method (can be NULL)
 - *    @postreset: postreset method (can be NULL)
 - *    @classes: resulting classes of attached devices
 - *
 - *    Reset the specified port and classify attached devices using
 - *    given methods.  This function prefers softreset but tries all
 - *    possible reset sequences to reset and classify devices.  This
 - *    function is intended to be used for constructing ->probe_reset
 - *    callback by low level drivers.
 - *
 - *    Reset methods should follow the following rules.
 - *
 - *    - Return 0 on sucess, -errno on failure.
 - *    - If classification is supported, fill classes[] with
 - *      recognized class codes.
 - *    - If classification is not supported, leave classes[] alone.
 - *    - If verbose is non-zero, print error message on failure;
 - *      otherwise, shut up.
 - *
 - *    LOCKING:
 - *    Kernel thread context (may sleep)
 - *
 - *    RETURNS:
 - *    0 on success, -EINVAL if no reset method is avaliable, -ENODEV
 - *    if classification fails, and any error code from reset
 - *    methods.
 - */
 -int ata_drive_probe_reset(struct ata_port *ap, ata_probeinit_fn_t probeinit,
 -                        ata_reset_fn_t softreset, ata_reset_fn_t hardreset,
 -                        ata_postreset_fn_t postreset, unsigned int *classes)
 -{
 -      int rc = -EINVAL;
 -
 -      if (probeinit)
 -              probeinit(ap);
 -
 -      if (softreset) {
 -              rc = do_probe_reset(ap, softreset, postreset, classes);
 -              if (rc == 0)
 -                      return 0;
 -      }
 -
 -      if (!hardreset)
 -              return rc;
 -
 -      rc = do_probe_reset(ap, hardreset, postreset, classes);
 -      if (rc == 0 || rc != -ENODEV)
 -              return rc;
 -
 -      if (softreset)
 -              rc = do_probe_reset(ap, softreset, postreset, classes);
 -
 -      return rc;
 -}
 -
  /**
   *    ata_dev_same_device - Determine whether new ID matches configured device
 - *    @ap: port on which the device to compare against resides
   *    @dev: device to compare against
   *    @new_class: class of the new device
   *    @new_id: IDENTIFY page of the new device
   *    RETURNS:
   *    1 if @dev matches @new_class and @new_id, 0 otherwise.
   */
 -static int ata_dev_same_device(struct ata_port *ap, struct ata_device *dev,
 -                             unsigned int new_class, const u16 *new_id)
 +static int ata_dev_same_device(struct ata_device *dev, unsigned int new_class,
 +                             const u16 *new_id)
  {
        const u16 *old_id = dev->id;
        unsigned char model[2][41], serial[2][21];
        u64 new_n_sectors;
  
        if (dev->class != new_class) {
 -              printk(KERN_INFO
 -                     "ata%u: dev %u class mismatch %d != %d\n",
 -                     ap->id, dev->devno, dev->class, new_class);
 +              ata_dev_printk(dev, KERN_INFO, "class mismatch %d != %d\n",
 +                             dev->class, new_class);
                return 0;
        }
  
        new_n_sectors = ata_id_n_sectors(new_id);
  
        if (strcmp(model[0], model[1])) {
 -              printk(KERN_INFO
 -                     "ata%u: dev %u model number mismatch '%s' != '%s'\n",
 -                     ap->id, dev->devno, model[0], model[1]);
 +              ata_dev_printk(dev, KERN_INFO, "model number mismatch "
 +                             "'%s' != '%s'\n", model[0], model[1]);
                return 0;
        }
  
        if (strcmp(serial[0], serial[1])) {
 -              printk(KERN_INFO
 -                     "ata%u: dev %u serial number mismatch '%s' != '%s'\n",
 -                     ap->id, dev->devno, serial[0], serial[1]);
 +              ata_dev_printk(dev, KERN_INFO, "serial number mismatch "
 +                             "'%s' != '%s'\n", serial[0], serial[1]);
                return 0;
        }
  
        if (dev->class == ATA_DEV_ATA && dev->n_sectors != new_n_sectors) {
 -              printk(KERN_INFO
 -                     "ata%u: dev %u n_sectors mismatch %llu != %llu\n",
 -                     ap->id, dev->devno, (unsigned long long)dev->n_sectors,
 -                     (unsigned long long)new_n_sectors);
 +              ata_dev_printk(dev, KERN_INFO, "n_sectors mismatch "
 +                             "%llu != %llu\n",
 +                             (unsigned long long)dev->n_sectors,
 +                             (unsigned long long)new_n_sectors);
                return 0;
        }
  
  
  /**
   *    ata_dev_revalidate - Revalidate ATA device
 - *    @ap: port on which the device to revalidate resides
   *    @dev: device to revalidate
   *    @post_reset: is this revalidation after reset?
   *
   *    RETURNS:
   *    0 on success, negative errno otherwise
   */
 -int ata_dev_revalidate(struct ata_port *ap, struct ata_device *dev,
 -                     int post_reset)
 +int ata_dev_revalidate(struct ata_device *dev, int post_reset)
  {
 -      unsigned int class;
 -      u16 *id;
 +      unsigned int class = dev->class;
 +      u16 *id = (void *)dev->ap->sector_buf;
        int rc;
  
 -      if (!ata_dev_present(dev))
 -              return -ENODEV;
 -
 -      class = dev->class;
 -      id = NULL;
 +      if (!ata_dev_enabled(dev)) {
 +              rc = -ENODEV;
 +              goto fail;
 +      }
  
 -      /* allocate & read ID data */
 -      rc = ata_dev_read_id(ap, dev, &class, post_reset, &id);
 +      /* read ID data */
 +      rc = ata_dev_read_id(dev, &class, post_reset, id);
        if (rc)
                goto fail;
  
        /* is the device still there? */
 -      if (!ata_dev_same_device(ap, dev, class, id)) {
 +      if (!ata_dev_same_device(dev, class, id)) {
                rc = -ENODEV;
                goto fail;
        }
  
 -      kfree(dev->id);
 -      dev->id = id;
 +      memcpy(dev->id, id, sizeof(id[0]) * ATA_ID_WORDS);
  
        /* configure device according to the new ID */
 -      return ata_dev_configure(ap, dev, 0);
 +      rc = ata_dev_configure(dev, 0);
 +      if (rc == 0)
 +              return 0;
  
   fail:
 -      printk(KERN_ERR "ata%u: dev %u revalidation failed (errno=%d)\n",
 -             ap->id, dev->devno, rc);
 -      kfree(id);
 +      ata_dev_printk(dev, KERN_ERR, "revalidation failed (errno=%d)\n", rc);
        return rc;
  }
  
@@@ -2963,6 -2646,7 +2963,6 @@@ static int ata_dma_blacklisted(const st
  
  /**
   *    ata_dev_xfermask - Compute supported xfermask of the given device
 - *    @ap: Port on which the device to compute xfermask for resides
   *    @dev: Device to compute xfermask for
   *
   *    Compute supported xfermask of @dev and store it in
   *    LOCKING:
   *    None.
   */
 -static void ata_dev_xfermask(struct ata_port *ap, struct ata_device *dev)
 +static void ata_dev_xfermask(struct ata_device *dev)
  {
 +      struct ata_port *ap = dev->ap;
        struct ata_host_set *hs = ap->host_set;
        unsigned long xfer_mask;
        int i;
  
 -      xfer_mask = ata_pack_xfermask(ap->pio_mask, ap->mwdma_mask,
 -                                    ap->udma_mask);
 +      xfer_mask = ata_pack_xfermask(ap->pio_mask,
 +                                    ap->mwdma_mask, ap->udma_mask);
 +
 +      /* Apply cable rule here.  Don't apply it early because when
 +       * we handle hot plug the cable type can itself change.
 +       */
 +      if (ap->cbl == ATA_CBL_PATA40)
 +              xfer_mask &= ~(0xF8 << ATA_SHIFT_UDMA);
  
        /* FIXME: Use port-wide xfermask for now */
        for (i = 0; i < ATA_MAX_DEVICES; i++) {
                struct ata_device *d = &ap->device[i];
 -              if (!ata_dev_present(d))
 +
 +              if (ata_dev_absent(d))
 +                      continue;
 +
 +              if (ata_dev_disabled(d)) {
 +                      /* to avoid violating device selection timing */
 +                      xfer_mask &= ata_pack_xfermask(d->pio_mask,
 +                                                     UINT_MAX, UINT_MAX);
                        continue;
 -              xfer_mask &= ata_pack_xfermask(d->pio_mask, d->mwdma_mask,
 -                                             d->udma_mask);
 +              }
 +
 +              xfer_mask &= ata_pack_xfermask(d->pio_mask,
 +                                             d->mwdma_mask, d->udma_mask);
                xfer_mask &= ata_id_xfermask(d->id);
                if (ata_dma_blacklisted(d))
                        xfer_mask &= ~(ATA_MASK_MWDMA | ATA_MASK_UDMA);
 -              /* Apply cable rule here. Don't apply it early because when
 -                 we handle hot plug the cable type can itself change */
 -              if (ap->cbl == ATA_CBL_PATA40)
 -                      xfer_mask &= ~(0xF8 << ATA_SHIFT_UDMA);
        }
  
        if (ata_dma_blacklisted(dev))
 -              printk(KERN_WARNING "ata%u: dev %u is on DMA blacklist, "
 -                     "disabling DMA\n", ap->id, dev->devno);
 +              ata_dev_printk(dev, KERN_WARNING,
 +                             "device is on DMA blacklist, disabling DMA\n");
  
        if (hs->flags & ATA_HOST_SIMPLEX) {
                if (hs->simplex_claimed)
                        xfer_mask &= ~(ATA_MASK_MWDMA | ATA_MASK_UDMA);
        }
 +
        if (ap->ops->mode_filter)
                xfer_mask = ap->ops->mode_filter(ap, dev, xfer_mask);
  
 -      ata_unpack_xfermask(xfer_mask, &dev->pio_mask, &dev->mwdma_mask,
 -                          &dev->udma_mask);
 +      ata_unpack_xfermask(xfer_mask, &dev->pio_mask,
 +                          &dev->mwdma_mask, &dev->udma_mask);
  }
  
  /**
   *    ata_dev_set_xfermode - Issue SET FEATURES - XFER MODE command
 - *    @ap: Port associated with device @dev
   *    @dev: Device to which command will be sent
   *
   *    Issue SET FEATURES - XFER MODE command to device @dev
   *    0 on success, AC_ERR_* mask otherwise.
   */
  
 -static unsigned int ata_dev_set_xfermode(struct ata_port *ap,
 -                                       struct ata_device *dev)
 +static unsigned int ata_dev_set_xfermode(struct ata_device *dev)
  {
        struct ata_taskfile tf;
        unsigned int err_mask;
        /* set up set-features taskfile */
        DPRINTK("set features - xfer mode\n");
  
 -      ata_tf_init(ap, &tf, dev->devno);
 +      ata_tf_init(dev, &tf);
        tf.command = ATA_CMD_SET_FEATURES;
        tf.feature = SETFEATURES_XFER;
        tf.flags |= ATA_TFLAG_ISADDR | ATA_TFLAG_DEVICE;
        tf.protocol = ATA_PROT_NODATA;
        tf.nsect = dev->xfer_mode;
  
 -      err_mask = ata_exec_internal(ap, dev, &tf, DMA_NONE, NULL, 0);
 +      err_mask = ata_exec_internal(dev, &tf, NULL, DMA_NONE, NULL, 0);
  
        DPRINTK("EXIT, err_mask=%x\n", err_mask);
        return err_mask;
  
  /**
   *    ata_dev_init_params - Issue INIT DEV PARAMS command
 - *    @ap: Port associated with device @dev
   *    @dev: Device to which command will be sent
   *    @heads: Number of heads (taskfile parameter)
   *    @sectors: Number of sectors (taskfile parameter)
   *    RETURNS:
   *    0 on success, AC_ERR_* mask otherwise.
   */
 -
 -static unsigned int ata_dev_init_params(struct ata_port *ap,
 -                                      struct ata_device *dev,
 -                                      u16 heads,
 -                                      u16 sectors)
 +static unsigned int ata_dev_init_params(struct ata_device *dev,
 +                                      u16 heads, u16 sectors)
  {
        struct ata_taskfile tf;
        unsigned int err_mask;
        /* set up init dev params taskfile */
        DPRINTK("init dev params \n");
  
 -      ata_tf_init(ap, &tf, dev->devno);
 +      ata_tf_init(dev, &tf);
        tf.command = ATA_CMD_INIT_DEV_PARAMS;
        tf.flags |= ATA_TFLAG_ISADDR | ATA_TFLAG_DEVICE;
        tf.protocol = ATA_PROT_NODATA;
        tf.nsect = sectors;
        tf.device |= (heads - 1) & 0x0f; /* max head = num. of heads - 1 */
  
 -      err_mask = ata_exec_internal(ap, dev, &tf, DMA_NONE, NULL, 0);
 +      err_mask = ata_exec_internal(dev, &tf, NULL, DMA_NONE, NULL, 0);
  
        DPRINTK("EXIT, err_mask=%x\n", err_mask);
        return err_mask;
@@@ -3235,15 -2912,6 +3235,15 @@@ int ata_check_atapi_dma(struct ata_queu
        if (ap->ops->check_atapi_dma)
                rc = ap->ops->check_atapi_dma(qc);
  
 +      /* We don't support polling DMA.
 +       * Use PIO if the LLDD handles only interrupts in
 +       * the HSM_ST_LAST state and the ATAPI device
 +       * generates CDB interrupts.
 +       */
 +      if ((ap->flags & ATA_FLAG_PIO_POLLING) &&
 +          (qc->dev->flags & ATA_DFLAG_CDB_INTR))
 +              rc = 1;
 +
        return rc;
  }
  /**
@@@ -3289,7 -2957,6 +3289,7 @@@ void ata_sg_init_one(struct ata_queued_
        qc->n_elem = 1;
        qc->orig_n_elem = 1;
        qc->buf_virt = buf;
 +      qc->nbytes = buflen;
  
        sg = qc->__sg;
        sg_init_one(sg, buf, buflen);
@@@ -3473,44 -3140,171 +3473,44 @@@ skip_map
  }
  
  /**
 - *    ata_poll_qc_complete - turn irq back on and finish qc
 - *    @qc: Command to complete
 - *    @err_mask: ATA status register content
 + *    swap_buf_le16 - swap halves of 16-bit words in place
 + *    @buf:  Buffer to swap
 + *    @buf_words:  Number of 16-bit words in buffer.
 + *
 + *    Swap halves of 16-bit words if needed to convert from
 + *    little-endian byte order to native cpu byte order, or
 + *    vice-versa.
   *
   *    LOCKING:
 - *    None.  (grabs host lock)
 + *    Inherited from caller.
   */
 -
 -void ata_poll_qc_complete(struct ata_queued_cmd *qc)
 +void swap_buf_le16(u16 *buf, unsigned int buf_words)
  {
 -      struct ata_port *ap = qc->ap;
 -      unsigned long flags;
 +#ifdef __BIG_ENDIAN
 +      unsigned int i;
  
 -      spin_lock_irqsave(&ap->host_set->lock, flags);
 -      ap->flags &= ~ATA_FLAG_NOINTR;
 -      ata_irq_on(ap);
 -      ata_qc_complete(qc);
 -      spin_unlock_irqrestore(&ap->host_set->lock, flags);
 +      for (i = 0; i < buf_words; i++)
 +              buf[i] = le16_to_cpu(buf[i]);
 +#endif /* __BIG_ENDIAN */
  }
  
  /**
 - *    ata_pio_poll - poll using PIO, depending on current state
 - *    @ap: the target ata_port
 + *    ata_mmio_data_xfer - Transfer data by MMIO
 + *    @adev: device for this I/O
 + *    @buf: data buffer
 + *    @buflen: buffer length
 + *    @write_data: read/write
   *
 - *    LOCKING:
 - *    None.  (executing in kernel thread context)
 + *    Transfer data from/to the device data register by MMIO.
   *
 - *    RETURNS:
 - *    timeout value to use
 + *    LOCKING:
 + *    Inherited from caller.
   */
  
 -static unsigned long ata_pio_poll(struct ata_port *ap)
 -{
 -      struct ata_queued_cmd *qc;
 -      u8 status;
 -      unsigned int poll_state = HSM_ST_UNKNOWN;
 -      unsigned int reg_state = HSM_ST_UNKNOWN;
 -
 -      qc = ata_qc_from_tag(ap, ap->active_tag);
 -      WARN_ON(qc == NULL);
 -
 -      switch (ap->hsm_task_state) {
 -      case HSM_ST:
 -      case HSM_ST_POLL:
 -              poll_state = HSM_ST_POLL;
 -              reg_state = HSM_ST;
 -              break;
 -      case HSM_ST_LAST:
 -      case HSM_ST_LAST_POLL:
 -              poll_state = HSM_ST_LAST_POLL;
 -              reg_state = HSM_ST_LAST;
 -              break;
 -      default:
 -              BUG();
 -              break;
 -      }
 -
 -      status = ata_chk_status(ap);
 -      if (status & ATA_BUSY) {
 -              if (time_after(jiffies, ap->pio_task_timeout)) {
 -                      qc->err_mask |= AC_ERR_TIMEOUT;
 -                      ap->hsm_task_state = HSM_ST_TMOUT;
 -                      return 0;
 -              }
 -              ap->hsm_task_state = poll_state;
 -              return ATA_SHORT_PAUSE;
 -      }
 -
 -      ap->hsm_task_state = reg_state;
 -      return 0;
 -}
 -
 -/**
 - *    ata_pio_complete - check if drive is busy or idle
 - *    @ap: the target ata_port
 - *
 - *    LOCKING:
 - *    None.  (executing in kernel thread context)
 - *
 - *    RETURNS:
 - *    Non-zero if qc completed, zero otherwise.
 - */
 -
 -static int ata_pio_complete (struct ata_port *ap)
 -{
 -      struct ata_queued_cmd *qc;
 -      u8 drv_stat;
 -
 -      /*
 -       * This is purely heuristic.  This is a fast path.  Sometimes when
 -       * we enter, BSY will be cleared in a chk-status or two.  If not,
 -       * the drive is probably seeking or something.  Snooze for a couple
 -       * msecs, then chk-status again.  If still busy, fall back to
 -       * HSM_ST_POLL state.
 -       */
 -      drv_stat = ata_busy_wait(ap, ATA_BUSY, 10);
 -      if (drv_stat & ATA_BUSY) {
 -              msleep(2);
 -              drv_stat = ata_busy_wait(ap, ATA_BUSY, 10);
 -              if (drv_stat & ATA_BUSY) {
 -                      ap->hsm_task_state = HSM_ST_LAST_POLL;
 -                      ap->pio_task_timeout = jiffies + ATA_TMOUT_PIO;
 -                      return 0;
 -              }
 -      }
 -
 -      qc = ata_qc_from_tag(ap, ap->active_tag);
 -      WARN_ON(qc == NULL);
 -
 -      drv_stat = ata_wait_idle(ap);
 -      if (!ata_ok(drv_stat)) {
 -              qc->err_mask |= __ac_err_mask(drv_stat);
 -              ap->hsm_task_state = HSM_ST_ERR;
 -              return 0;
 -      }
 -
 -      ap->hsm_task_state = HSM_ST_IDLE;
 -
 -      WARN_ON(qc->err_mask);
 -      ata_poll_qc_complete(qc);
 -
 -      /* another command may start at this point */
 -
 -      return 1;
 -}
 -
 -
 -/**
 - *    swap_buf_le16 - swap halves of 16-bit words in place
 - *    @buf:  Buffer to swap
 - *    @buf_words:  Number of 16-bit words in buffer.
 - *
 - *    Swap halves of 16-bit words if needed to convert from
 - *    little-endian byte order to native cpu byte order, or
 - *    vice-versa.
 - *
 - *    LOCKING:
 - *    Inherited from caller.
 - */
 -void swap_buf_le16(u16 *buf, unsigned int buf_words)
 -{
 -#ifdef __BIG_ENDIAN
 -      unsigned int i;
 -
 -      for (i = 0; i < buf_words; i++)
 -              buf[i] = le16_to_cpu(buf[i]);
 -#endif /* __BIG_ENDIAN */
 -}
 -
 -/**
 - *    ata_mmio_data_xfer - Transfer data by MMIO
 - *    @ap: port to read/write
 - *    @buf: data buffer
 - *    @buflen: buffer length
 - *    @write_data: read/write
 - *
 - *    Transfer data from/to the device data register by MMIO.
 - *
 - *    LOCKING:
 - *    Inherited from caller.
 - */
 -
 -static void ata_mmio_data_xfer(struct ata_port *ap, unsigned char *buf,
 -                             unsigned int buflen, int write_data)
 +void ata_mmio_data_xfer(struct ata_device *adev, unsigned char *buf, 
 +                      unsigned int buflen, int write_data)
  {
 +      struct ata_port *ap = adev->ap;
        unsigned int i;
        unsigned int words = buflen >> 1;
        u16 *buf16 = (u16 *) buf;
  
  /**
   *    ata_pio_data_xfer - Transfer data by PIO
 - *    @ap: port to read/write
 + *    @adev: device to target
   *    @buf: data buffer
   *    @buflen: buffer length
   *    @write_data: read/write
   *    Inherited from caller.
   */
  
 -static void ata_pio_data_xfer(struct ata_port *ap, unsigned char *buf,
 -                            unsigned int buflen, int write_data)
 +void ata_pio_data_xfer(struct ata_device *adev, unsigned char *buf, 
 +                     unsigned int buflen, int write_data)
  {
 +      struct ata_port *ap = adev->ap;
        unsigned int words = buflen >> 1;
  
        /* Transfer multiple of 2 bytes */
  }
  
  /**
 - *    ata_data_xfer - Transfer data from/to the data register.
 - *    @ap: port to read/write
 + *    ata_pio_data_xfer_noirq - Transfer data by PIO
 + *    @adev: device to target
   *    @buf: data buffer
   *    @buflen: buffer length
 - *    @do_write: read/write
 + *    @write_data: read/write
   *
 - *    Transfer data from/to the device data register.
 + *    Transfer data from/to the device data register by PIO. Do the 
 + *    transfer with interrupts disabled.
   *
   *    LOCKING:
   *    Inherited from caller.
   */
  
 -static void ata_data_xfer(struct ata_port *ap, unsigned char *buf,
 -                        unsigned int buflen, int do_write)
 +void ata_pio_data_xfer_noirq(struct ata_device *adev, unsigned char *buf,
 +                                  unsigned int buflen, int write_data)
  {
 -      /* Make the crap hardware pay the costs not the good stuff */
 -      if (unlikely(ap->flags & ATA_FLAG_IRQ_MASK)) {
 -              unsigned long flags;
 -              local_irq_save(flags);
 -              if (ap->flags & ATA_FLAG_MMIO)
 -                      ata_mmio_data_xfer(ap, buf, buflen, do_write);
 -              else
 -                      ata_pio_data_xfer(ap, buf, buflen, do_write);
 -              local_irq_restore(flags);
 -      } else {
 -              if (ap->flags & ATA_FLAG_MMIO)
 -                      ata_mmio_data_xfer(ap, buf, buflen, do_write);
 -              else
 -                      ata_pio_data_xfer(ap, buf, buflen, do_write);
 -      }
 +      unsigned long flags;
 +      local_irq_save(flags);
 +      ata_pio_data_xfer(adev, buf, buflen, write_data);
 +      local_irq_restore(flags);
  }
  
 +
  /**
   *    ata_pio_sector - Transfer ATA_SECT_SIZE (512 bytes) of data.
   *    @qc: Command on going
@@@ -3633,24 -3435,7 +3633,24 @@@ static void ata_pio_sector(struct ata_q
        page = nth_page(page, (offset >> PAGE_SHIFT));
        offset %= PAGE_SIZE;
  
 -      buf = kmap(page) + offset;
 +      DPRINTK("data %s\n", qc->tf.flags & ATA_TFLAG_WRITE ? "write" : "read");
 +
 +      if (PageHighMem(page)) {
 +              unsigned long flags;
 +
 +              /* FIXME: use a bounce buffer */
 +              local_irq_save(flags);
 +              buf = kmap_atomic(page, KM_IRQ0);
 +
 +              /* do the actual data transfer */
 +              ap->ops->data_xfer(qc->dev, buf + offset, ATA_SECT_SIZE, do_write);
 +
 +              kunmap_atomic(buf, KM_IRQ0);
 +              local_irq_restore(flags);
 +      } else {
 +              buf = page_address(page);
 +              ap->ops->data_xfer(qc->dev, buf + offset, ATA_SECT_SIZE, do_write);
 +      }
  
        qc->cursect++;
        qc->cursg_ofs++;
                qc->cursg++;
                qc->cursg_ofs = 0;
        }
 +}
  
 -      DPRINTK("data %s\n", qc->tf.flags & ATA_TFLAG_WRITE ? "write" : "read");
 +/**
 + *    ata_pio_sectors - Transfer one or many 512-byte sectors.
 + *    @qc: Command on going
 + *
 + *    Transfer one or many ATA_SECT_SIZE of data from/to the
 + *    ATA device for the DRQ request.
 + *
 + *    LOCKING:
 + *    Inherited from caller.
 + */
 +
 +static void ata_pio_sectors(struct ata_queued_cmd *qc)
 +{
 +      if (is_multi_taskfile(&qc->tf)) {
 +              /* READ/WRITE MULTIPLE */
 +              unsigned int nsect;
  
 -      /* do the actual data transfer */
 -      do_write = (qc->tf.flags & ATA_TFLAG_WRITE);
 -      ata_data_xfer(ap, buf, ATA_SECT_SIZE, do_write);
 +              WARN_ON(qc->dev->multi_count == 0);
  
 -      kunmap(page);
 +              nsect = min(qc->nsect - qc->cursect, qc->dev->multi_count);
 +              while (nsect--)
 +                      ata_pio_sector(qc);
 +      } else
 +              ata_pio_sector(qc);
 +}
 +
 +/**
 + *    atapi_send_cdb - Write CDB bytes to hardware
 + *    @ap: Port to which ATAPI device is attached.
 + *    @qc: Taskfile currently active
 + *
 + *    When device has indicated its readiness to accept
 + *    a CDB, this function is called.  Send the CDB.
 + *
 + *    LOCKING:
 + *    caller.
 + */
 +
 +static void atapi_send_cdb(struct ata_port *ap, struct ata_queued_cmd *qc)
 +{
 +      /* send SCSI cdb */
 +      DPRINTK("send cdb\n");
 +      WARN_ON(qc->dev->cdb_len < 12);
 +
 +      ap->ops->data_xfer(qc->dev, qc->cdb, qc->dev->cdb_len, 1);
 +      ata_altstatus(ap); /* flush */
 +
 +      switch (qc->tf.protocol) {
 +      case ATA_PROT_ATAPI:
 +              ap->hsm_task_state = HSM_ST;
 +              break;
 +      case ATA_PROT_ATAPI_NODATA:
 +              ap->hsm_task_state = HSM_ST_LAST;
 +              break;
 +      case ATA_PROT_ATAPI_DMA:
 +              ap->hsm_task_state = HSM_ST_LAST;
 +              /* initiate bmdma */
 +              ap->ops->bmdma_start(qc);
 +              break;
 +      }
  }
  
  /**
@@@ -3761,11 -3492,11 +3761,11 @@@ next_sg
                unsigned int i;
  
                if (words) /* warning if bytes > 1 */
 -                      printk(KERN_WARNING "ata%u: %u bytes trailing data\n",
 -                             ap->id, bytes);
 +                      ata_dev_printk(qc->dev, KERN_WARNING,
 +                                     "%u bytes trailing data\n", bytes);
  
                for (i = 0; i < words; i++)
 -                      ata_data_xfer(ap, (unsigned char*)pad_buf, 2, do_write);
 +                      ap->ops->data_xfer(qc->dev, (unsigned char*)pad_buf, 2, do_write);
  
                ap->hsm_task_state = HSM_ST_LAST;
                return;
        /* don't cross page boundaries */
        count = min(count, (unsigned int)PAGE_SIZE - offset);
  
 -      buf = kmap(page) + offset;
 +      DPRINTK("data %s\n", qc->tf.flags & ATA_TFLAG_WRITE ? "write" : "read");
 +
 +      if (PageHighMem(page)) {
 +              unsigned long flags;
 +
 +              /* FIXME: use bounce buffer */
 +              local_irq_save(flags);
 +              buf = kmap_atomic(page, KM_IRQ0);
 +
 +              /* do the actual data transfer */
 +              ap->ops->data_xfer(qc->dev,  buf + offset, count, do_write);
 +
 +              kunmap_atomic(buf, KM_IRQ0);
 +              local_irq_restore(flags);
 +      } else {
 +              buf = page_address(page);
 +              ap->ops->data_xfer(qc->dev,  buf + offset, count, do_write);
 +      }
  
        bytes -= count;
        qc->curbytes += count;
                qc->cursg_ofs = 0;
        }
  
 -      DPRINTK("data %s\n", qc->tf.flags & ATA_TFLAG_WRITE ? "write" : "read");
 -
 -      /* do the actual data transfer */
 -      ata_data_xfer(ap, buf, count, do_write);
 -
 -      kunmap(page);
 -
        if (bytes)
                goto next_sg;
  }
@@@ -3835,16 -3556,10 +3835,16 @@@ static void atapi_pio_bytes(struct ata_
        unsigned int ireason, bc_lo, bc_hi, bytes;
        int i_write, do_write = (qc->tf.flags & ATA_TFLAG_WRITE) ? 1 : 0;
  
 -      ap->ops->tf_read(ap, &qc->tf);
 -      ireason = qc->tf.nsect;
 -      bc_lo = qc->tf.lbam;
 -      bc_hi = qc->tf.lbah;
 +      /* Abuse qc->result_tf for temp storage of intermediate TF
 +       * here to save some kernel stack usage.
 +       * For normal completion, qc->result_tf is not relevant. For
 +       * error, qc->result_tf is later overwritten by ata_qc_complete().
 +       * So, the correctness of qc->result_tf is not affected.
 +       */
 +      ap->ops->tf_read(ap, &qc->result_tf);
 +      ireason = qc->result_tf.nsect;
 +      bc_lo = qc->result_tf.lbam;
 +      bc_hi = qc->result_tf.lbah;
        bytes = (bc_hi << 8) | bc_lo;
  
        /* shall be cleared to zero, indicating xfer of data */
        if (do_write != i_write)
                goto err_out;
  
 +      VPRINTK("ata%u: xfering %d bytes\n", ap->id, bytes);
 +
        __atapi_pio_bytes(qc, bytes);
  
        return;
  
  err_out:
 -      printk(KERN_INFO "ata%u: dev %u: ATAPI check failed\n",
 -            ap->id, dev->devno);
 +      ata_dev_printk(dev, KERN_INFO, "ATAPI check failed\n");
        qc->err_mask |= AC_ERR_HSM;
        ap->hsm_task_state = HSM_ST_ERR;
  }
  
  /**
 - *    ata_pio_block - start PIO on a block
 + *    ata_hsm_ok_in_wq - Check if the qc can be handled in the workqueue.
   *    @ap: the target ata_port
 + *    @qc: qc on going
   *
 - *    LOCKING:
 - *    None.  (executing in kernel thread context)
 + *    RETURNS:
 + *    1 if ok in workqueue, 0 otherwise.
   */
  
 -static void ata_pio_block(struct ata_port *ap)
 +static inline int ata_hsm_ok_in_wq(struct ata_port *ap, struct ata_queued_cmd *qc)
  {
 -      struct ata_queued_cmd *qc;
 -      u8 status;
 +      if (qc->tf.flags & ATA_TFLAG_POLLING)
 +              return 1;
  
 -      /*
 -       * This is purely heuristic.  This is a fast path.
 -       * Sometimes when we enter, BSY will be cleared in
 -       * a chk-status or two.  If not, the drive is probably seeking
 -       * or something.  Snooze for a couple msecs, then
 -       * chk-status again.  If still busy, fall back to
 -       * HSM_ST_POLL state.
 -       */
 -      status = ata_busy_wait(ap, ATA_BUSY, 5);
 -      if (status & ATA_BUSY) {
 -              msleep(2);
 -              status = ata_busy_wait(ap, ATA_BUSY, 10);
 -              if (status & ATA_BUSY) {
 -                      ap->hsm_task_state = HSM_ST_POLL;
 -                      ap->pio_task_timeout = jiffies + ATA_TMOUT_PIO;
 -                      return;
 -              }
 +      if (ap->hsm_task_state == HSM_ST_FIRST) {
 +              if (qc->tf.protocol == ATA_PROT_PIO &&
 +                  (qc->tf.flags & ATA_TFLAG_WRITE))
 +                  return 1;
 +
 +              if (is_atapi_taskfile(&qc->tf) &&
 +                  !(qc->dev->flags & ATA_DFLAG_CDB_INTR))
 +                      return 1;
        }
  
 -      qc = ata_qc_from_tag(ap, ap->active_tag);
 -      WARN_ON(qc == NULL);
 +      return 0;
 +}
  
 -      /* check error */
 -      if (status & (ATA_ERR | ATA_DF)) {
 -              qc->err_mask |= AC_ERR_DEV;
 -              ap->hsm_task_state = HSM_ST_ERR;
 -              return;
 -      }
 +/**
 + *    ata_hsm_qc_complete - finish a qc running on standard HSM
 + *    @qc: Command to complete
 + *    @in_wq: 1 if called from workqueue, 0 otherwise
 + *
 + *    Finish @qc which is running on standard HSM.
 + *
 + *    LOCKING:
 + *    If @in_wq is zero, spin_lock_irqsave(host_set lock).
 + *    Otherwise, none on entry and grabs host lock.
 + */
 +static void ata_hsm_qc_complete(struct ata_queued_cmd *qc, int in_wq)
 +{
 +      struct ata_port *ap = qc->ap;
 +      unsigned long flags;
  
 -      /* transfer data if any */
 -      if (is_atapi_taskfile(&qc->tf)) {
 -              /* DRQ=0 means no more data to transfer */
 -              if ((status & ATA_DRQ) == 0) {
 -                      ap->hsm_task_state = HSM_ST_LAST;
 -                      return;
 -              }
 +      if (ap->ops->error_handler) {
 +              if (in_wq) {
 +                      spin_lock_irqsave(&ap->host_set->lock, flags);
  
 -              atapi_pio_bytes(qc);
 -      } else {
 -              /* handle BSY=0, DRQ=0 as error */
 -              if ((status & ATA_DRQ) == 0) {
 -                      qc->err_mask |= AC_ERR_HSM;
 -                      ap->hsm_task_state = HSM_ST_ERR;
 -                      return;
 -              }
 +                      /* EH might have kicked in while host_set lock
 +                       * is released.
 +                       */
 +                      qc = ata_qc_from_tag(ap, qc->tag);
 +                      if (qc) {
 +                              if (likely(!(qc->err_mask & AC_ERR_HSM))) {
 +                                      ata_irq_on(ap);
 +                                      ata_qc_complete(qc);
 +                              } else
 +                                      ata_port_freeze(ap);
 +                      }
  
 -              ata_pio_sector(qc);
 +                      spin_unlock_irqrestore(&ap->host_set->lock, flags);
 +              } else {
 +                      if (likely(!(qc->err_mask & AC_ERR_HSM)))
 +                              ata_qc_complete(qc);
 +                      else
 +                              ata_port_freeze(ap);
 +              }
 +      } else {
 +              if (in_wq) {
 +                      spin_lock_irqsave(&ap->host_set->lock, flags);
 +                      ata_irq_on(ap);
 +                      ata_qc_complete(qc);
 +                      spin_unlock_irqrestore(&ap->host_set->lock, flags);
 +              } else
 +                      ata_qc_complete(qc);
        }
  
        ata_altstatus(ap); /* flush */
  }
  
 -static void ata_pio_error(struct ata_port *ap)
 +/**
 + *    ata_hsm_move - move the HSM to the next state.
 + *    @ap: the target ata_port
 + *    @qc: qc on going
 + *    @status: current device status
 + *    @in_wq: 1 if called from workqueue, 0 otherwise
 + *
 + *    RETURNS:
 + *    1 when poll next status needed, 0 otherwise.
 + */
 +int ata_hsm_move(struct ata_port *ap, struct ata_queued_cmd *qc,
 +               u8 status, int in_wq)
  {
 -      struct ata_queued_cmd *qc;
 -
 -      qc = ata_qc_from_tag(ap, ap->active_tag);
 -      WARN_ON(qc == NULL);
 +      unsigned long flags = 0;
 +      int poll_next;
  
 -      if (qc->tf.command != ATA_CMD_PACKET)
 -              printk(KERN_WARNING "ata%u: PIO error\n", ap->id);
 +      WARN_ON((qc->flags & ATA_QCFLAG_ACTIVE) == 0);
  
 -      /* make sure qc->err_mask is available to
 -       * know what's wrong and recover
 +      /* Make sure ata_qc_issue_prot() does not throw things
 +       * like DMA polling into the workqueue. Notice that
 +       * in_wq is not equivalent to (qc->tf.flags & ATA_TFLAG_POLLING).
         */
 -      WARN_ON(qc->err_mask == 0);
 -
 -      ap->hsm_task_state = HSM_ST_IDLE;
 -
 -      ata_poll_qc_complete(qc);
 -}
 -
 -static void ata_pio_task(void *_data)
 -{
 -      struct ata_port *ap = _data;
 -      unsigned long timeout;
 -      int qc_completed;
 +      WARN_ON(in_wq != ata_hsm_ok_in_wq(ap, qc));
  
  fsm_start:
 -      timeout = 0;
 -      qc_completed = 0;
 +      DPRINTK("ata%u: protocol %d task_state %d (dev_stat 0x%X)\n",
 +              ap->id, qc->tf.protocol, ap->hsm_task_state, status);
  
        switch (ap->hsm_task_state) {
 -      case HSM_ST_IDLE:
 -              return;
 -
 -      case HSM_ST:
 -              ata_pio_block(ap);
 -              break;
 -
 -      case HSM_ST_LAST:
 -              qc_completed = ata_pio_complete(ap);
 -              break;
 -
 -      case HSM_ST_POLL:
 -      case HSM_ST_LAST_POLL:
 -              timeout = ata_pio_poll(ap);
 -              break;
 -
 -      case HSM_ST_TMOUT:
 -      case HSM_ST_ERR:
 -              ata_pio_error(ap);
 -              return;
 -      }
 +      case HSM_ST_FIRST:
 +              /* Send first data block or PACKET CDB */
  
 -      if (timeout)
 -              ata_port_queue_task(ap, ata_pio_task, ap, timeout);
 -      else if (!qc_completed)
 -              goto fsm_start;
 -}
 -
 -/**
 - *    atapi_packet_task - Write CDB bytes to hardware
 - *    @_data: Port to which ATAPI device is attached.
 - *
 - *    When device has indicated its readiness to accept
 - *    a CDB, this function is called.  Send the CDB.
 - *    If DMA is to be performed, exit immediately.
 - *    Otherwise, we are in polling mode, so poll
 - *    status under operation succeeds or fails.
 - *
 - *    LOCKING:
 - *    Kernel thread context (may sleep)
 - */
 -
 -static void atapi_packet_task(void *_data)
 -{
 -      struct ata_port *ap = _data;
 -      struct ata_queued_cmd *qc;
 -      u8 status;
 -
 -      qc = ata_qc_from_tag(ap, ap->active_tag);
 -      WARN_ON(qc == NULL);
 -      WARN_ON(!(qc->flags & ATA_QCFLAG_ACTIVE));
 +              /* If polling, we will stay in the work queue after
 +               * sending the data. Otherwise, interrupt handler
 +               * takes over after sending the data.
 +               */
 +              poll_next = (qc->tf.flags & ATA_TFLAG_POLLING);
 +
 +              /* check device status */
 +              if (unlikely((status & ATA_DRQ) == 0)) {
 +                      /* handle BSY=0, DRQ=0 as error */
 +                      if (likely(status & (ATA_ERR | ATA_DF)))
 +                              /* device stops HSM for abort/error */
 +                              qc->err_mask |= AC_ERR_DEV;
 +                      else
 +                              /* HSM violation. Let EH handle this */
 +                              qc->err_mask |= AC_ERR_HSM;
  
 -      /* sleep-wait for BSY to clear */
 -      DPRINTK("busy wait\n");
 -      if (ata_busy_sleep(ap, ATA_TMOUT_CDB_QUICK, ATA_TMOUT_CDB)) {
 -              qc->err_mask |= AC_ERR_TIMEOUT;
 -              goto err_out;
 -      }
 +                      ap->hsm_task_state = HSM_ST_ERR;
 +                      goto fsm_start;
 +              }
  
 -      /* make sure DRQ is set */
 -      status = ata_chk_status(ap);
 -      if ((status & (ATA_BUSY | ATA_DRQ)) != ATA_DRQ) {
 -              qc->err_mask |= AC_ERR_HSM;
 -              goto err_out;
 -      }
 +              /* Device should not ask for data transfer (DRQ=1)
 +               * when it finds something wrong.
 +               * We ignore DRQ here and stop the HSM by
 +               * changing hsm_task_state to HSM_ST_ERR and
 +               * let the EH abort the command or reset the device.
 +               */
 +              if (unlikely(status & (ATA_ERR | ATA_DF))) {
 +                      printk(KERN_WARNING "ata%d: DRQ=1 with device error, dev_stat 0x%X\n",
 +                             ap->id, status);
 +                      qc->err_mask |= AC_ERR_HSM;
 +                      ap->hsm_task_state = HSM_ST_ERR;
 +                      goto fsm_start;
 +              }
  
 -      /* send SCSI cdb */
 -      DPRINTK("send cdb\n");
 -      WARN_ON(qc->dev->cdb_len < 12);
 +              /* Send the CDB (atapi) or the first data block (ata pio out).
 +               * During the state transition, interrupt handler shouldn't
 +               * be invoked before the data transfer is complete and
 +               * hsm_task_state is changed. Hence, the following locking.
 +               */
 +              if (in_wq)
 +                      spin_lock_irqsave(&ap->host_set->lock, flags);
  
 -      if (qc->tf.protocol == ATA_PROT_ATAPI_DMA ||
 -          qc->tf.protocol == ATA_PROT_ATAPI_NODATA) {
 -              unsigned long flags;
 +              if (qc->tf.protocol == ATA_PROT_PIO) {
 +                      /* PIO data out protocol.
 +                       * send first data block.
 +                       */
  
 -              /* Once we're done issuing command and kicking bmdma,
 -               * irq handler takes over.  To not lose irq, we need
 -               * to clear NOINTR flag before sending cdb, but
 -               * interrupt handler shouldn't be invoked before we're
 -               * finished.  Hence, the following locking.
 +                      /* ata_pio_sectors() might change the state
 +                       * to HSM_ST_LAST. so, the state is changed here
 +                       * before ata_pio_sectors().
 +                       */
 +                      ap->hsm_task_state = HSM_ST;
 +                      ata_pio_sectors(qc);
 +                      ata_altstatus(ap); /* flush */
 +              } else
 +                      /* send CDB */
 +                      atapi_send_cdb(ap, qc);
 +
 +              if (in_wq)
 +                      spin_unlock_irqrestore(&ap->host_set->lock, flags);
 +
 +              /* if polling, ata_pio_task() handles the rest.
 +               * otherwise, interrupt handler takes over from here.
                 */
 -              spin_lock_irqsave(&ap->host_set->lock, flags);
 -              ap->flags &= ~ATA_FLAG_NOINTR;
 -              ata_data_xfer(ap, qc->cdb, qc->dev->cdb_len, 1);
 -              ata_altstatus(ap); /* flush */
 +              break;
  
 -              if (qc->tf.protocol == ATA_PROT_ATAPI_DMA)
 -                      ap->ops->bmdma_start(qc);       /* initiate bmdma */
 -              spin_unlock_irqrestore(&ap->host_set->lock, flags);
 -      } else {
 -              ata_data_xfer(ap, qc->cdb, qc->dev->cdb_len, 1);
 -              ata_altstatus(ap); /* flush */
 +      case HSM_ST:
 +              /* complete command or read/write the data register */
 +              if (qc->tf.protocol == ATA_PROT_ATAPI) {
 +                      /* ATAPI PIO protocol */
 +                      if ((status & ATA_DRQ) == 0) {
 +                              /* No more data to transfer or device error.
 +                               * Device error will be tagged in HSM_ST_LAST.
 +                               */
 +                              ap->hsm_task_state = HSM_ST_LAST;
 +                              goto fsm_start;
 +                      }
  
 -              /* PIO commands are handled by polling */
 -              ap->hsm_task_state = HSM_ST;
 -              ata_port_queue_task(ap, ata_pio_task, ap, 0);
 -      }
 +                      /* Device should not ask for data transfer (DRQ=1)
 +                       * when it finds something wrong.
 +                       * We ignore DRQ here and stop the HSM by
 +                       * changing hsm_task_state to HSM_ST_ERR and
 +                       * let the EH abort the command or reset the device.
 +                       */
 +                      if (unlikely(status & (ATA_ERR | ATA_DF))) {
 +                              printk(KERN_WARNING "ata%d: DRQ=1 with device error, dev_stat 0x%X\n",
 +                                     ap->id, status);
 +                              qc->err_mask |= AC_ERR_HSM;
 +                              ap->hsm_task_state = HSM_ST_ERR;
 +                              goto fsm_start;
 +                      }
  
 -      return;
 +                      atapi_pio_bytes(qc);
  
 -err_out:
 -      ata_poll_qc_complete(qc);
 -}
 +                      if (unlikely(ap->hsm_task_state == HSM_ST_ERR))
 +                              /* bad ireason reported by device */
 +                              goto fsm_start;
  
 -/**
 - *    ata_qc_timeout - Handle timeout of queued command
 - *    @qc: Command that timed out
 - *
 - *    Some part of the kernel (currently, only the SCSI layer)
 - *    has noticed that the active command on port @ap has not
 - *    completed after a specified length of time.  Handle this
 - *    condition by disabling DMA (if necessary) and completing
 - *    transactions, with error if necessary.
 - *
 - *    This also handles the case of the "lost interrupt", where
 - *    for some reason (possibly hardware bug, possibly driver bug)
 - *    an interrupt was not delivered to the driver, even though the
 - *    transaction completed successfully.
 - *
 - *    LOCKING:
 - *    Inherited from SCSI layer (none, can sleep)
 - */
 +              } else {
 +                      /* ATA PIO protocol */
 +                      if (unlikely((status & ATA_DRQ) == 0)) {
 +                              /* handle BSY=0, DRQ=0 as error */
 +                              if (likely(status & (ATA_ERR | ATA_DF)))
 +                                      /* device stops HSM for abort/error */
 +                                      qc->err_mask |= AC_ERR_DEV;
 +                              else
 +                                      /* HSM violation. Let EH handle this */
 +                                      qc->err_mask |= AC_ERR_HSM;
 +
 +                              ap->hsm_task_state = HSM_ST_ERR;
 +                              goto fsm_start;
 +                      }
  
 -static void ata_qc_timeout(struct ata_queued_cmd *qc)
 -{
 -      struct ata_port *ap = qc->ap;
 -      struct ata_host_set *host_set = ap->host_set;
 -      u8 host_stat = 0, drv_stat;
 -      unsigned long flags;
 +                      /* For PIO reads, some devices may ask for
 +                       * data transfer (DRQ=1) alone with ERR=1.
 +                       * We respect DRQ here and transfer one
 +                       * block of junk data before changing the
 +                       * hsm_task_state to HSM_ST_ERR.
 +                       *
 +                       * For PIO writes, ERR=1 DRQ=1 doesn't make
 +                       * sense since the data block has been
 +                       * transferred to the device.
 +                       */
 +                      if (unlikely(status & (ATA_ERR | ATA_DF))) {
 +                              /* data might be corrputed */
 +                              qc->err_mask |= AC_ERR_DEV;
 +
 +                              if (!(qc->tf.flags & ATA_TFLAG_WRITE)) {
 +                                      ata_pio_sectors(qc);
 +                                      ata_altstatus(ap);
 +                                      status = ata_wait_idle(ap);
 +                              }
 +
 +                              if (status & (ATA_BUSY | ATA_DRQ))
 +                                      qc->err_mask |= AC_ERR_HSM;
 +
 +                              /* ata_pio_sectors() might change the
 +                               * state to HSM_ST_LAST. so, the state
 +                               * is changed after ata_pio_sectors().
 +                               */
 +                              ap->hsm_task_state = HSM_ST_ERR;
 +                              goto fsm_start;
 +                      }
  
 -      DPRINTK("ENTER\n");
 +                      ata_pio_sectors(qc);
  
 -      ap->hsm_task_state = HSM_ST_IDLE;
 +                      if (ap->hsm_task_state == HSM_ST_LAST &&
 +                          (!(qc->tf.flags & ATA_TFLAG_WRITE))) {
 +                              /* all data read */
 +                              ata_altstatus(ap);
 +                              status = ata_wait_idle(ap);
 +                              goto fsm_start;
 +                      }
 +              }
  
 -      spin_lock_irqsave(&host_set->lock, flags);
 +              ata_altstatus(ap); /* flush */
 +              poll_next = 1;
 +              break;
  
 -      switch (qc->tf.protocol) {
 +      case HSM_ST_LAST:
 +              if (unlikely(!ata_ok(status))) {
 +                      qc->err_mask |= __ac_err_mask(status);
 +                      ap->hsm_task_state = HSM_ST_ERR;
 +                      goto fsm_start;
 +              }
  
 -      case ATA_PROT_DMA:
 -      case ATA_PROT_ATAPI_DMA:
 -              host_stat = ap->ops->bmdma_status(ap);
 +              /* no more data to transfer */
 +              DPRINTK("ata%u: dev %u command complete, drv_stat 0x%x\n",
 +                      ap->id, qc->dev->devno, status);
  
 -              /* before we do anything else, clear DMA-Start bit */
 -              ap->ops->bmdma_stop(qc);
 +              WARN_ON(qc->err_mask);
  
 -              /* fall through */
 +              ap->hsm_task_state = HSM_ST_IDLE;
  
 -      default:
 -              ata_altstatus(ap);
 -              drv_stat = ata_chk_status(ap);
 +              /* complete taskfile transaction */
 +              ata_hsm_qc_complete(qc, in_wq);
  
 -              /* ack bmdma irq events */
 -              ap->ops->irq_clear(ap);
 +              poll_next = 0;
 +              break;
  
 -              printk(KERN_ERR "ata%u: command 0x%x timeout, stat 0x%x host_stat 0x%x\n",
 -                     ap->id, qc->tf.command, drv_stat, host_stat);
 +      case HSM_ST_ERR:
 +              /* make sure qc->err_mask is available to
 +               * know what's wrong and recover
 +               */
 +              WARN_ON(qc->err_mask == 0);
 +
 +              ap->hsm_task_state = HSM_ST_IDLE;
  
                /* complete taskfile transaction */
 -              qc->err_mask |= ac_err_mask(drv_stat);
 +              ata_hsm_qc_complete(qc, in_wq);
 +
 +              poll_next = 0;
                break;
 +      default:
 +              poll_next = 0;
 +              BUG();
        }
  
 -      spin_unlock_irqrestore(&host_set->lock, flags);
 -
 -      ata_eh_qc_complete(qc);
 -
 -      DPRINTK("EXIT\n");
 +      return poll_next;
  }
  
 -/**
 - *    ata_eng_timeout - Handle timeout of queued command
 - *    @ap: Port on which timed-out command is active
 - *
 - *    Some part of the kernel (currently, only the SCSI layer)
 - *    has noticed that the active command on port @ap has not
 - *    completed after a specified length of time.  Handle this
 - *    condition by disabling DMA (if necessary) and completing
 - *    transactions, with error if necessary.
 - *
 - *    This also handles the case of the "lost interrupt", where
 - *    for some reason (possibly hardware bug, possibly driver bug)
 - *    an interrupt was not delivered to the driver, even though the
 - *    transaction completed successfully.
 - *
 - *    LOCKING:
 - *    Inherited from SCSI layer (none, can sleep)
 - */
 -
 -void ata_eng_timeout(struct ata_port *ap)
 +static void ata_pio_task(void *_data)
  {
 -      DPRINTK("ENTER\n");
 +      struct ata_queued_cmd *qc = _data;
 +      struct ata_port *ap = qc->ap;
 +      u8 status;
 +      int poll_next;
  
 -      ata_qc_timeout(ata_qc_from_tag(ap, ap->active_tag));
 +fsm_start:
 +      WARN_ON(ap->hsm_task_state == HSM_ST_IDLE);
  
 -      DPRINTK("EXIT\n");
 +      /*
 +       * This is purely heuristic.  This is a fast path.
 +       * Sometimes when we enter, BSY will be cleared in
 +       * a chk-status or two.  If not, the drive is probably seeking
 +       * or something.  Snooze for a couple msecs, then
 +       * chk-status again.  If still busy, queue delayed work.
 +       */
 +      status = ata_busy_wait(ap, ATA_BUSY, 5);
 +      if (status & ATA_BUSY) {
 +              msleep(2);
 +              status = ata_busy_wait(ap, ATA_BUSY, 10);
 +              if (status & ATA_BUSY) {
 +                      ata_port_queue_task(ap, ata_pio_task, qc, ATA_SHORT_PAUSE);
 +                      return;
 +              }
 +      }
 +
 +      /* move the HSM */
 +      poll_next = ata_hsm_move(ap, qc, status, 1);
 +
 +      /* another command or interrupt handler
 +       * may be running at this point.
 +       */
 +      if (poll_next)
 +              goto fsm_start;
  }
  
  /**
@@@ -4231,14 -3888,9 +4231,14 @@@ static struct ata_queued_cmd *ata_qc_ne
        struct ata_queued_cmd *qc = NULL;
        unsigned int i;
  
 -      for (i = 0; i < ATA_MAX_QUEUE; i++)
 -              if (!test_and_set_bit(i, &ap->qactive)) {
 -                      qc = ata_qc_from_tag(ap, i);
 +      /* no command while frozen */
 +      if (unlikely(ap->flags & ATA_FLAG_FROZEN))
 +              return NULL;
 +
 +      /* the last tag is reserved for internal command. */
 +      for (i = 0; i < ATA_MAX_QUEUE - 1; i++)
 +              if (!test_and_set_bit(i, &ap->qc_allocated)) {
 +                      qc = __ata_qc_from_tag(ap, i);
                        break;
                }
  
  
  /**
   *    ata_qc_new_init - Request an available ATA command, and initialize it
 - *    @ap: Port associated with device @dev
   *    @dev: Device from whom we request an available command structure
   *
   *    LOCKING:
   *    None.
   */
  
 -struct ata_queued_cmd *ata_qc_new_init(struct ata_port *ap,
 -                                    struct ata_device *dev)
 +struct ata_queued_cmd *ata_qc_new_init(struct ata_device *dev)
  {
 +      struct ata_port *ap = dev->ap;
        struct ata_queued_cmd *qc;
  
        qc = ata_qc_new(ap);
@@@ -4293,153 -3946,36 +4293,153 @@@ void ata_qc_free(struct ata_queued_cmd 
        qc->flags = 0;
        tag = qc->tag;
        if (likely(ata_tag_valid(tag))) {
 -              if (tag == ap->active_tag)
 -                      ap->active_tag = ATA_TAG_POISON;
                qc->tag = ATA_TAG_POISON;
 -              clear_bit(tag, &ap->qactive);
 +              clear_bit(tag, &ap->qc_allocated);
        }
  }
  
  void __ata_qc_complete(struct ata_queued_cmd *qc)
  {
 +      struct ata_port *ap = qc->ap;
 +
        WARN_ON(qc == NULL);    /* ata_qc_from_tag _might_ return NULL */
        WARN_ON(!(qc->flags & ATA_QCFLAG_ACTIVE));
  
        if (likely(qc->flags & ATA_QCFLAG_DMAMAP))
                ata_sg_clean(qc);
  
 +      /* command should be marked inactive atomically with qc completion */
 +      if (qc->tf.protocol == ATA_PROT_NCQ)
 +              ap->sactive &= ~(1 << qc->tag);
 +      else
 +              ap->active_tag = ATA_TAG_POISON;
 +
        /* atapi: mark qc as inactive to prevent the interrupt handler
         * from completing the command twice later, before the error handler
         * is called. (when rc != 0 and atapi request sense is needed)
         */
        qc->flags &= ~ATA_QCFLAG_ACTIVE;
 +      ap->qc_active &= ~(1 << qc->tag);
  
        /* call completion callback */
        qc->complete_fn(qc);
  }
  
 +/**
 + *    ata_qc_complete - Complete an active ATA command
 + *    @qc: Command to complete
 + *    @err_mask: ATA Status register contents
 + *
 + *    Indicate to the mid and upper layers that an ATA
 + *    command has completed, with either an ok or not-ok status.
 + *
 + *    LOCKING:
 + *    spin_lock_irqsave(host_set lock)
 + */
 +void ata_qc_complete(struct ata_queued_cmd *qc)
 +{
 +      struct ata_port *ap = qc->ap;
 +
 +      /* XXX: New EH and old EH use different mechanisms to
 +       * synchronize EH with regular execution path.
 +       *
 +       * In new EH, a failed qc is marked with ATA_QCFLAG_FAILED.
 +       * Normal execution path is responsible for not accessing a
 +       * failed qc.  libata core enforces the rule by returning NULL
 +       * from ata_qc_from_tag() for failed qcs.
 +       *
 +       * Old EH depends on ata_qc_complete() nullifying completion
 +       * requests if ATA_QCFLAG_EH_SCHEDULED is set.  Old EH does
 +       * not synchronize with interrupt handler.  Only PIO task is
 +       * taken care of.
 +       */
 +      if (ap->ops->error_handler) {
 +              WARN_ON(ap->flags & ATA_FLAG_FROZEN);
 +
 +              if (unlikely(qc->err_mask))
 +                      qc->flags |= ATA_QCFLAG_FAILED;
 +
 +              if (unlikely(qc->flags & ATA_QCFLAG_FAILED)) {
 +                      if (!ata_tag_internal(qc->tag)) {
 +                              /* always fill result TF for failed qc */
 +                              ap->ops->tf_read(ap, &qc->result_tf);
 +                              ata_qc_schedule_eh(qc);
 +                              return;
 +                      }
 +              }
 +
 +              /* read result TF if requested */
 +              if (qc->flags & ATA_QCFLAG_RESULT_TF)
 +                      ap->ops->tf_read(ap, &qc->result_tf);
 +
 +              __ata_qc_complete(qc);
 +      } else {
 +              if (qc->flags & ATA_QCFLAG_EH_SCHEDULED)
 +                      return;
 +
 +              /* read result TF if failed or requested */
 +              if (qc->err_mask || qc->flags & ATA_QCFLAG_RESULT_TF)
 +                      ap->ops->tf_read(ap, &qc->result_tf);
 +
 +              __ata_qc_complete(qc);
 +      }
 +}
 +
 +/**
 + *    ata_qc_complete_multiple - Complete multiple qcs successfully
 + *    @ap: port in question
 + *    @qc_active: new qc_active mask
 + *    @finish_qc: LLDD callback invoked before completing a qc
 + *
 + *    Complete in-flight commands.  This functions is meant to be
 + *    called from low-level driver's interrupt routine to complete
 + *    requests normally.  ap->qc_active and @qc_active is compared
 + *    and commands are completed accordingly.
 + *
 + *    LOCKING:
 + *    spin_lock_irqsave(host_set lock)
 + *
 + *    RETURNS:
 + *    Number of completed commands on success, -errno otherwise.
 + */
 +int ata_qc_complete_multiple(struct ata_port *ap, u32 qc_active,
 +                           void (*finish_qc)(struct ata_queued_cmd *))
 +{
 +      int nr_done = 0;
 +      u32 done_mask;
 +      int i;
 +
 +      done_mask = ap->qc_active ^ qc_active;
 +
 +      if (unlikely(done_mask & qc_active)) {
 +              ata_port_printk(ap, KERN_ERR, "illegal qc_active transition "
 +                              "(%08x->%08x)\n", ap->qc_active, qc_active);
 +              return -EINVAL;
 +      }
 +
 +      for (i = 0; i < ATA_MAX_QUEUE; i++) {
 +              struct ata_queued_cmd *qc;
 +
 +              if (!(done_mask & (1 << i)))
 +                      continue;
 +
 +              if ((qc = ata_qc_from_tag(ap, i))) {
 +                      if (finish_qc)
 +                              finish_qc(qc);
 +                      ata_qc_complete(qc);
 +                      nr_done++;
 +              }
 +      }
 +
 +      return nr_done;
 +}
 +
  static inline int ata_should_dma_map(struct ata_queued_cmd *qc)
  {
        struct ata_port *ap = qc->ap;
  
        switch (qc->tf.protocol) {
 +      case ATA_PROT_NCQ:
        case ATA_PROT_DMA:
        case ATA_PROT_ATAPI_DMA:
                return 1;
@@@ -4474,22 -4010,8 +4474,22 @@@ void ata_qc_issue(struct ata_queued_cm
  {
        struct ata_port *ap = qc->ap;
  
 -      qc->ap->active_tag = qc->tag;
 +      /* Make sure only one non-NCQ command is outstanding.  The
 +       * check is skipped for old EH because it reuses active qc to
 +       * request ATAPI sense.
 +       */
 +      WARN_ON(ap->ops->error_handler && ata_tag_valid(ap->active_tag));
 +
 +      if (qc->tf.protocol == ATA_PROT_NCQ) {
 +              WARN_ON(ap->sactive & (1 << qc->tag));
 +              ap->sactive |= 1 << qc->tag;
 +      } else {
 +              WARN_ON(ap->sactive);
 +              ap->active_tag = qc->tag;
 +      }
 +
        qc->flags |= ATA_QCFLAG_ACTIVE;
 +      ap->qc_active |= 1 << qc->tag;
  
        if (ata_should_dma_map(qc)) {
                if (qc->flags & ATA_QCFLAG_SG) {
@@@ -4539,105 -4061,43 +4539,105 @@@ unsigned int ata_qc_issue_prot(struct a
  {
        struct ata_port *ap = qc->ap;
  
 +      /* Use polling pio if the LLD doesn't handle
 +       * interrupt driven pio and atapi CDB interrupt.
 +       */
 +      if (ap->flags & ATA_FLAG_PIO_POLLING) {
 +              switch (qc->tf.protocol) {
 +              case ATA_PROT_PIO:
 +              case ATA_PROT_ATAPI:
 +              case ATA_PROT_ATAPI_NODATA:
 +                      qc->tf.flags |= ATA_TFLAG_POLLING;
 +                      break;
 +              case ATA_PROT_ATAPI_DMA:
 +                      if (qc->dev->flags & ATA_DFLAG_CDB_INTR)
 +                              /* see ata_check_atapi_dma() */
 +                              BUG();
 +                      break;
 +              default:
 +                      break;
 +              }
 +      }
 +
 +      /* select the device */
        ata_dev_select(ap, qc->dev->devno, 1, 0);
  
 +      /* start the command */
        switch (qc->tf.protocol) {
        case ATA_PROT_NODATA:
 +              if (qc->tf.flags & ATA_TFLAG_POLLING)
 +                      ata_qc_set_polling(qc);
 +
                ata_tf_to_host(ap, &qc->tf);
 +              ap->hsm_task_state = HSM_ST_LAST;
 +
 +              if (qc->tf.flags & ATA_TFLAG_POLLING)
 +                      ata_port_queue_task(ap, ata_pio_task, qc, 0);
 +
                break;
  
        case ATA_PROT_DMA:
 +              WARN_ON(qc->tf.flags & ATA_TFLAG_POLLING);
 +
                ap->ops->tf_load(ap, &qc->tf);   /* load tf registers */
                ap->ops->bmdma_setup(qc);           /* set up bmdma */
                ap->ops->bmdma_start(qc);           /* initiate bmdma */
 +              ap->hsm_task_state = HSM_ST_LAST;
                break;
  
 -      case ATA_PROT_PIO: /* load tf registers, initiate polling pio */
 -              ata_qc_set_polling(qc);
 -              ata_tf_to_host(ap, &qc->tf);
 -              ap->hsm_task_state = HSM_ST;
 -              ata_port_queue_task(ap, ata_pio_task, ap, 0);
 -              break;
 +      case ATA_PROT_PIO:
 +              if (qc->tf.flags & ATA_TFLAG_POLLING)
 +                      ata_qc_set_polling(qc);
  
 -      case ATA_PROT_ATAPI:
 -              ata_qc_set_polling(qc);
                ata_tf_to_host(ap, &qc->tf);
 -              ata_port_queue_task(ap, atapi_packet_task, ap, 0);
 +
 +              if (qc->tf.flags & ATA_TFLAG_WRITE) {
 +                      /* PIO data out protocol */
 +                      ap->hsm_task_state = HSM_ST_FIRST;
 +                      ata_port_queue_task(ap, ata_pio_task, qc, 0);
 +
 +                      /* always send first data block using
 +                       * the ata_pio_task() codepath.
 +                       */
 +              } else {
 +                      /* PIO data in protocol */
 +                      ap->hsm_task_state = HSM_ST;
 +
 +                      if (qc->tf.flags & ATA_TFLAG_POLLING)
 +                              ata_port_queue_task(ap, ata_pio_task, qc, 0);
 +
 +                      /* if polling, ata_pio_task() handles the rest.
 +                       * otherwise, interrupt handler takes over from here.
 +                       */
 +              }
 +
                break;
  
 +      case ATA_PROT_ATAPI:
        case ATA_PROT_ATAPI_NODATA:
 -              ap->flags |= ATA_FLAG_NOINTR;
 +              if (qc->tf.flags & ATA_TFLAG_POLLING)
 +                      ata_qc_set_polling(qc);
 +
                ata_tf_to_host(ap, &qc->tf);
 -              ata_port_queue_task(ap, atapi_packet_task, ap, 0);
 +
 +              ap->hsm_task_state = HSM_ST_FIRST;
 +
 +              /* send cdb by polling if no cdb interrupt */
 +              if ((!(qc->dev->flags & ATA_DFLAG_CDB_INTR)) ||
 +                  (qc->tf.flags & ATA_TFLAG_POLLING))
 +                      ata_port_queue_task(ap, ata_pio_task, qc, 0);
                break;
  
        case ATA_PROT_ATAPI_DMA:
 -              ap->flags |= ATA_FLAG_NOINTR;
 +              WARN_ON(qc->tf.flags & ATA_TFLAG_POLLING);
 +
                ap->ops->tf_load(ap, &qc->tf);   /* load tf registers */
                ap->ops->bmdma_setup(qc);           /* set up bmdma */
 -              ata_port_queue_task(ap, atapi_packet_task, ap, 0);
 +              ap->hsm_task_state = HSM_ST_FIRST;
 +
 +              /* send cdb by polling if no cdb interrupt */
 +              if (!(qc->dev->flags & ATA_DFLAG_CDB_INTR))
 +                      ata_port_queue_task(ap, ata_pio_task, qc, 0);
                break;
  
        default:
  inline unsigned int ata_host_intr (struct ata_port *ap,
                                   struct ata_queued_cmd *qc)
  {
 -      u8 status, host_stat;
 -
 -      switch (qc->tf.protocol) {
 -
 -      case ATA_PROT_DMA:
 -      case ATA_PROT_ATAPI_DMA:
 -      case ATA_PROT_ATAPI:
 -              /* check status of DMA engine */
 -              host_stat = ap->ops->bmdma_status(ap);
 -              VPRINTK("ata%u: host_stat 0x%X\n", ap->id, host_stat);
 -
 -              /* if it's not our irq... */
 -              if (!(host_stat & ATA_DMA_INTR))
 -                      goto idle_irq;
 -
 -              /* before we do anything else, clear DMA-Start bit */
 -              ap->ops->bmdma_stop(qc);
 +      u8 status, host_stat = 0;
  
 -              /* fall through */
 +      VPRINTK("ata%u: protocol %d task_state %d\n",
 +              ap->id, qc->tf.protocol, ap->hsm_task_state);
  
 -      case ATA_PROT_ATAPI_NODATA:
 -      case ATA_PROT_NODATA:
 -              /* check altstatus */
 -              status = ata_altstatus(ap);
 -              if (status & ATA_BUSY)
 -                      goto idle_irq;
 +      /* Check whether we are expecting interrupt in this state */
 +      switch (ap->hsm_task_state) {
 +      case HSM_ST_FIRST:
 +              /* Some pre-ATAPI-4 devices assert INTRQ
 +               * at this state when ready to receive CDB.
 +               */
  
 -              /* check main status, clearing INTRQ */
 -              status = ata_chk_status(ap);
 -              if (unlikely(status & ATA_BUSY))
 +              /* Check the ATA_DFLAG_CDB_INTR flag is enough here.
 +               * The flag was turned on only for atapi devices.
 +               * No need to check is_atapi_taskfile(&qc->tf) again.
 +               */
 +              if (!(qc->dev->flags & ATA_DFLAG_CDB_INTR))
                        goto idle_irq;
 -              DPRINTK("ata%u: protocol %d (dev_stat 0x%X)\n",
 -                      ap->id, qc->tf.protocol, status);
 -
 -              /* ack bmdma irq events */
 -              ap->ops->irq_clear(ap);
 -
 -              /* complete taskfile transaction */
 -              qc->err_mask |= ac_err_mask(status);
 -              ata_qc_complete(qc);
                break;
 -
 +      case HSM_ST_LAST:
 +              if (qc->tf.protocol == ATA_PROT_DMA ||
 +                  qc->tf.protocol == ATA_PROT_ATAPI_DMA) {
 +                      /* check status of DMA engine */
 +                      host_stat = ap->ops->bmdma_status(ap);
 +                      VPRINTK("ata%u: host_stat 0x%X\n", ap->id, host_stat);
 +
 +                      /* if it's not our irq... */
 +                      if (!(host_stat & ATA_DMA_INTR))
 +                              goto idle_irq;
 +
 +                      /* before we do anything else, clear DMA-Start bit */
 +                      ap->ops->bmdma_stop(qc);
 +
 +                      if (unlikely(host_stat & ATA_DMA_ERR)) {
 +                              /* error when transfering data to/from memory */
 +                              qc->err_mask |= AC_ERR_HOST_BUS;
 +                              ap->hsm_task_state = HSM_ST_ERR;
 +                      }
 +              }
 +              break;
 +      case HSM_ST:
 +              break;
        default:
                goto idle_irq;
        }
  
 +      /* check altstatus */
 +      status = ata_altstatus(ap);
 +      if (status & ATA_BUSY)
 +              goto idle_irq;
 +
 +      /* check main status, clearing INTRQ */
 +      status = ata_chk_status(ap);
 +      if (unlikely(status & ATA_BUSY))
 +              goto idle_irq;
 +
 +      /* ack bmdma irq events */
 +      ap->ops->irq_clear(ap);
 +
 +      ata_hsm_move(ap, qc, status, 0);
        return 1;       /* irq handled */
  
  idle_irq:
  #ifdef ATA_IRQ_TRAP
        if ((ap->stats.idle_irq % 1000) == 0) {
                ata_irq_ack(ap, 0); /* debug trap */
 -              printk(KERN_WARNING "ata%d: irq trap\n", ap->id);
 +              ata_port_printk(ap, KERN_WARNING, "irq trap\n");
                return 1;
        }
  #endif
@@@ -4773,11 -4219,11 +4773,11 @@@ irqreturn_t ata_interrupt (int irq, voi
  
                ap = host_set->ports[i];
                if (ap &&
 -                  !(ap->flags & (ATA_FLAG_PORT_DISABLED | ATA_FLAG_NOINTR))) {
 +                  !(ap->flags & ATA_FLAG_DISABLED)) {
                        struct ata_queued_cmd *qc;
  
                        qc = ata_qc_from_tag(ap, ap->active_tag);
 -                      if (qc && (!(qc->tf.ctl & ATA_NIEN)) &&
 +                      if (qc && (!(qc->tf.flags & ATA_TFLAG_POLLING)) &&
                            (qc->flags & ATA_QCFLAG_ACTIVE))
                                handled |= ata_host_intr(ap, qc);
                }
        return IRQ_RETVAL(handled);
  }
  
 +/**
 + *    sata_scr_valid - test whether SCRs are accessible
 + *    @ap: ATA port to test SCR accessibility for
 + *
 + *    Test whether SCRs are accessible for @ap.
 + *
 + *    LOCKING:
 + *    None.
 + *
 + *    RETURNS:
 + *    1 if SCRs are accessible, 0 otherwise.
 + */
 +int sata_scr_valid(struct ata_port *ap)
 +{
 +      return ap->cbl == ATA_CBL_SATA && ap->ops->scr_read;
 +}
 +
 +/**
 + *    sata_scr_read - read SCR register of the specified port
 + *    @ap: ATA port to read SCR for
 + *    @reg: SCR to read
 + *    @val: Place to store read value
 + *
 + *    Read SCR register @reg of @ap into *@val.  This function is
 + *    guaranteed to succeed if the cable type of the port is SATA
 + *    and the port implements ->scr_read.
 + *
 + *    LOCKING:
 + *    None.
 + *
 + *    RETURNS:
 + *    0 on success, negative errno on failure.
 + */
 +int sata_scr_read(struct ata_port *ap, int reg, u32 *val)
 +{
 +      if (sata_scr_valid(ap)) {
 +              *val = ap->ops->scr_read(ap, reg);
 +              return 0;
 +      }
 +      return -EOPNOTSUPP;
 +}
 +
 +/**
 + *    sata_scr_write - write SCR register of the specified port
 + *    @ap: ATA port to write SCR for
 + *    @reg: SCR to write
 + *    @val: value to write
 + *
 + *    Write @val to SCR register @reg of @ap.  This function is
 + *    guaranteed to succeed if the cable type of the port is SATA
 + *    and the port implements ->scr_read.
 + *
 + *    LOCKING:
 + *    None.
 + *
 + *    RETURNS:
 + *    0 on success, negative errno on failure.
 + */
 +int sata_scr_write(struct ata_port *ap, int reg, u32 val)
 +{
 +      if (sata_scr_valid(ap)) {
 +              ap->ops->scr_write(ap, reg, val);
 +              return 0;
 +      }
 +      return -EOPNOTSUPP;
 +}
 +
 +/**
 + *    sata_scr_write_flush - write SCR register of the specified port and flush
 + *    @ap: ATA port to write SCR for
 + *    @reg: SCR to write
 + *    @val: value to write
 + *
 + *    This function is identical to sata_scr_write() except that this
 + *    function performs flush after writing to the register.
 + *
 + *    LOCKING:
 + *    None.
 + *
 + *    RETURNS:
 + *    0 on success, negative errno on failure.
 + */
 +int sata_scr_write_flush(struct ata_port *ap, int reg, u32 val)
 +{
 +      if (sata_scr_valid(ap)) {
 +              ap->ops->scr_write(ap, reg, val);
 +              ap->ops->scr_read(ap, reg);
 +              return 0;
 +      }
 +      return -EOPNOTSUPP;
 +}
 +
 +/**
 + *    ata_port_online - test whether the given port is online
 + *    @ap: ATA port to test
 + *
 + *    Test whether @ap is online.  Note that this function returns 0
 + *    if online status of @ap cannot be obtained, so
 + *    ata_port_online(ap) != !ata_port_offline(ap).
 + *
 + *    LOCKING:
 + *    None.
 + *
 + *    RETURNS:
 + *    1 if the port online status is available and online.
 + */
 +int ata_port_online(struct ata_port *ap)
 +{
 +      u32 sstatus;
 +
 +      if (!sata_scr_read(ap, SCR_STATUS, &sstatus) && (sstatus & 0xf) == 0x3)
 +              return 1;
 +      return 0;
 +}
 +
 +/**
 + *    ata_port_offline - test whether the given port is offline
 + *    @ap: ATA port to test
 + *
 + *    Test whether @ap is offline.  Note that this function returns
 + *    0 if offline status of @ap cannot be obtained, so
 + *    ata_port_online(ap) != !ata_port_offline(ap).
 + *
 + *    LOCKING:
 + *    None.
 + *
 + *    RETURNS:
 + *    1 if the port offline status is available and offline.
 + */
 +int ata_port_offline(struct ata_port *ap)
 +{
 +      u32 sstatus;
 +
 +      if (!sata_scr_read(ap, SCR_STATUS, &sstatus) && (sstatus & 0xf) != 0x3)
 +              return 1;
 +      return 0;
 +}
  
  /*
   * Execute a 'simple' command, that only consists of the opcode 'cmd' itself,
   * without filling any other registers
   */
 -static int ata_do_simple_cmd(struct ata_port *ap, struct ata_device *dev,
 -                           u8 cmd)
 +static int ata_do_simple_cmd(struct ata_device *dev, u8 cmd)
  {
        struct ata_taskfile tf;
        int err;
  
 -      ata_tf_init(ap, &tf, dev->devno);
 +      ata_tf_init(dev, &tf);
  
        tf.command = cmd;
        tf.flags |= ATA_TFLAG_DEVICE;
        tf.protocol = ATA_PROT_NODATA;
  
 -      err = ata_exec_internal(ap, dev, &tf, DMA_NONE, NULL, 0);
 +      err = ata_exec_internal(dev, &tf, NULL, DMA_NONE, NULL, 0);
        if (err)
 -              printk(KERN_ERR "%s: ata command failed: %d\n",
 -                              __FUNCTION__, err);
 +              ata_dev_printk(dev, KERN_ERR, "%s: ata command failed: %d\n",
 +                             __FUNCTION__, err);
  
        return err;
  }
  
 -static int ata_flush_cache(struct ata_port *ap, struct ata_device *dev)
 +static int ata_flush_cache(struct ata_device *dev)
  {
        u8 cmd;
  
        else
                cmd = ATA_CMD_FLUSH;
  
 -      return ata_do_simple_cmd(ap, dev, cmd);
 +      return ata_do_simple_cmd(dev, cmd);
  }
  
 -static int ata_standby_drive(struct ata_port *ap, struct ata_device *dev)
 +static int ata_standby_drive(struct ata_device *dev)
  {
 -      return ata_do_simple_cmd(ap, dev, ATA_CMD_STANDBYNOW1);
 +      return ata_do_simple_cmd(dev, ATA_CMD_STANDBYNOW1);
  }
  
 -static int ata_start_drive(struct ata_port *ap, struct ata_device *dev)
 +static int ata_start_drive(struct ata_device *dev)
  {
 -      return ata_do_simple_cmd(ap, dev, ATA_CMD_IDLEIMMEDIATE);
 +      return ata_do_simple_cmd(dev, ATA_CMD_IDLEIMMEDIATE);
  }
  
  /**
   *    ata_device_resume - wakeup a previously suspended devices
 - *    @ap: port the device is connected to
   *    @dev: the device to resume
   *
   *    Kick the drive back into action, by sending it an idle immediate
   *    and host.
   *
   */
 -int ata_device_resume(struct ata_port *ap, struct ata_device *dev)
 +int ata_device_resume(struct ata_device *dev)
  {
 +      struct ata_port *ap = dev->ap;
 +
        if (ap->flags & ATA_FLAG_SUSPENDED) {
 +              struct ata_device *failed_dev;
 +
+               ata_busy_sleep(ap, ATA_TMOUT_BOOT_QUICK, ATA_TMOUT_BOOT);
                ata_busy_wait(ap, ATA_BUSY | ATA_DRQ, 200000);
 +
                ap->flags &= ~ATA_FLAG_SUSPENDED;
 -              ata_set_mode(ap);
 +              while (ata_set_mode(ap, &failed_dev))
 +                      ata_dev_disable(failed_dev);
        }
 -      if (!ata_dev_present(dev))
 +      if (!ata_dev_enabled(dev))
                return 0;
        if (dev->class == ATA_DEV_ATA)
 -              ata_start_drive(ap, dev);
 +              ata_start_drive(dev);
  
        return 0;
  }
  
  /**
   *    ata_device_suspend - prepare a device for suspend
 - *    @ap: port the device is connected to
   *    @dev: the device to suspend
   *    @state: target power management state
   *
   *    Flush the cache on the drive, if appropriate, then issue a
   *    standbynow command.
   */
 -int ata_device_suspend(struct ata_port *ap, struct ata_device *dev, pm_message_t state)
 +int ata_device_suspend(struct ata_device *dev, pm_message_t state)
  {
 -      if (!ata_dev_present(dev))
 +      struct ata_port *ap = dev->ap;
 +
 +      if (!ata_dev_enabled(dev))
                return 0;
        if (dev->class == ATA_DEV_ATA)
 -              ata_flush_cache(ap, dev);
 +              ata_flush_cache(dev);
  
        if (state.event != PM_EVENT_FREEZE)
 -              ata_standby_drive(ap, dev);
 +              ata_standby_drive(dev);
        ap->flags |= ATA_FLAG_SUSPENDED;
        return 0;
  }
@@@ -5109,38 -4414,6 +5110,38 @@@ static void ata_host_remove(struct ata_
        ap->ops->port_stop(ap);
  }
  
 +/**
 + *    ata_dev_init - Initialize an ata_device structure
 + *    @dev: Device structure to initialize
 + *
 + *    Initialize @dev in preparation for probing.
 + *
 + *    LOCKING:
 + *    Inherited from caller.
 + */
 +void ata_dev_init(struct ata_device *dev)
 +{
 +      struct ata_port *ap = dev->ap;
 +      unsigned long flags;
 +
 +      /* SATA spd limit is bound to the first device */
 +      ap->sata_spd_limit = ap->hw_sata_spd_limit;
 +
 +      /* High bits of dev->flags are used to record warm plug
 +       * requests which occur asynchronously.  Synchronize using
 +       * host_set lock.
 +       */
 +      spin_lock_irqsave(&ap->host_set->lock, flags);
 +      dev->flags &= ~ATA_DFLAG_INIT_MASK;
 +      spin_unlock_irqrestore(&ap->host_set->lock, flags);
 +
 +      memset((void *)dev + ATA_DEVICE_CLEAR_OFFSET, 0,
 +             sizeof(*dev) - ATA_DEVICE_CLEAR_OFFSET);
 +      dev->pio_mask = UINT_MAX;
 +      dev->mwdma_mask = UINT_MAX;
 +      dev->udma_mask = UINT_MAX;
 +}
 +
  /**
   *    ata_host_init - Initialize an ata_port structure
   *    @ap: Structure to initialize
   *    LOCKING:
   *    Inherited from caller.
   */
 -
  static void ata_host_init(struct ata_port *ap, struct Scsi_Host *host,
                          struct ata_host_set *host_set,
                          const struct ata_probe_ent *ent, unsigned int port_no)
        host->unique_id = ata_unique_id++;
        host->max_cmd_len = 12;
  
 -      ap->flags = ATA_FLAG_PORT_DISABLED;
 +      ap->flags = ATA_FLAG_DISABLED;
        ap->id = host->unique_id;
        ap->host = host;
        ap->ctl = ATA_DEVCTL_OBS;
        ap->udma_mask = ent->udma_mask;
        ap->flags |= ent->host_flags;
        ap->ops = ent->port_ops;
 -      ap->cbl = ATA_CBL_NONE;
 +      ap->hw_sata_spd_limit = UINT_MAX;
        ap->active_tag = ATA_TAG_POISON;
        ap->last_ctl = 0xFF;
  
 +#if defined(ATA_VERBOSE_DEBUG)
 +      /* turn on all debugging levels */
 +      ap->msg_enable = 0x00FF;
 +#elif defined(ATA_DEBUG)
 +      ap->msg_enable = ATA_MSG_DRV | ATA_MSG_INFO | ATA_MSG_CTL | ATA_MSG_WARN | ATA_MSG_ERR;
 +#else 
 +      ap->msg_enable = ATA_MSG_DRV | ATA_MSG_ERR;
 +#endif
 +
        INIT_WORK(&ap->port_task, NULL, NULL);
 +      INIT_WORK(&ap->hotplug_task, ata_scsi_hotplug, ap);
 +      INIT_WORK(&ap->scsi_rescan_task, ata_scsi_dev_rescan, ap);
        INIT_LIST_HEAD(&ap->eh_done_q);
 +      init_waitqueue_head(&ap->eh_wait_q);
 +
 +      /* set cable type */
 +      ap->cbl = ATA_CBL_NONE;
 +      if (ap->flags & ATA_FLAG_SATA)
 +              ap->cbl = ATA_CBL_SATA;
  
        for (i = 0; i < ATA_MAX_DEVICES; i++) {
                struct ata_device *dev = &ap->device[i];
 +              dev->ap = ap;
                dev->devno = i;
 -              dev->pio_mask = UINT_MAX;
 -              dev->mwdma_mask = UINT_MAX;
 -              dev->udma_mask = UINT_MAX;
 +              ata_dev_init(dev);
        }
  
  #ifdef ATA_IRQ_TRAP
@@@ -5245,7 -4503,7 +5246,7 @@@ static struct ata_port * ata_host_add(c
  
        DPRINTK("ENTER\n");
  
 -      if (!ent->port_ops->probe_reset &&
 +      if (!ent->port_ops->error_handler &&
            !(ent->host_flags & (ATA_FLAG_SATA_RESET | ATA_FLAG_SRST))) {
                printk(KERN_ERR "ata%u: no reset mechanism available\n",
                       port_no);
  
        host->transportt = &ata_scsi_transport_template;
  
 -      ap = (struct ata_port *) &host->hostdata[0];
 +      ap = ata_shost_to_port(host);
  
        ata_host_init(ap, host, host_set, ent, port_no);
  
@@@ -5291,12 -4549,12 +5292,12 @@@ err_out
   *    RETURNS:
   *    Number of ports registered.  Zero on error (no ports registered).
   */
 -
  int ata_device_add(const struct ata_probe_ent *ent)
  {
        unsigned int count = 0, i;
        struct device *dev = ent->dev;
        struct ata_host_set *host_set;
 +      int rc;
  
        DPRINTK("ENTER\n");
        /* alloc a container for our list of ATA ports (buses) */
                                (ap->pio_mask << ATA_SHIFT_PIO);
  
                /* print per-port info to dmesg */
 -              printk(KERN_INFO "ata%u: %cATA max %s cmd 0x%lX ctl 0x%lX "
 -                               "bmdma 0x%lX irq %lu\n",
 -                      ap->id,
 -                      ap->flags & ATA_FLAG_SATA ? 'S' : 'P',
 -                      ata_mode_string(xfer_mode_mask),
 -                      ap->ioaddr.cmd_addr,
 -                      ap->ioaddr.ctl_addr,
 -                      ap->ioaddr.bmdma_addr,
 -                      ent->irq);
 +              ata_port_printk(ap, KERN_INFO, "%cATA max %s cmd 0x%lX "
 +                              "ctl 0x%lX bmdma 0x%lX irq %lu\n",
 +                              ap->flags & ATA_FLAG_SATA ? 'S' : 'P',
 +                              ata_mode_string(xfer_mode_mask),
 +                              ap->ioaddr.cmd_addr,
 +                              ap->ioaddr.ctl_addr,
 +                              ap->ioaddr.bmdma_addr,
 +                              ent->irq);
  
                ata_chk_status(ap);
                host_set->ops->irq_clear(ap);
 +              ata_eh_freeze_port(ap); /* freeze port before requesting IRQ */
                count++;
        }
  
                goto err_free_ret;
  
        /* obtain irq, that is shared between channels */
 -      if (request_irq(ent->irq, ent->port_ops->irq_handler, ent->irq_flags,
 -                      DRV_NAME, host_set))
 +      rc = request_irq(ent->irq, ent->port_ops->irq_handler, ent->irq_flags,
 +                       DRV_NAME, host_set);
 +      if (rc) {
 +              dev_printk(KERN_ERR, dev, "irq %lu request failed: %d\n",
 +                         ent->irq, rc);
                goto err_out;
 +      }
  
        /* perform each probe synchronously */
        DPRINTK("probe begin\n");
        for (i = 0; i < count; i++) {
                struct ata_port *ap;
 +              u32 scontrol;
                int rc;
  
                ap = host_set->ports[i];
  
 -              DPRINTK("ata%u: bus probe begin\n", ap->id);
 -              rc = ata_bus_probe(ap);
 -              DPRINTK("ata%u: bus probe end\n", ap->id);
 -
 -              if (rc) {
 -                      /* FIXME: do something useful here?
 -                       * Current libata behavior will
 -                       * tear down everything when
 -                       * the module is removed
 -                       * or the h/w is unplugged.
 -                       */
 +              /* init sata_spd_limit to the current value */
 +              if (sata_scr_read(ap, SCR_CONTROL, &scontrol) == 0) {
 +                      int spd = (scontrol >> 4) & 0xf;
 +                      ap->hw_sata_spd_limit &= (1 << spd) - 1;
                }
 +              ap->sata_spd_limit = ap->hw_sata_spd_limit;
  
                rc = scsi_add_host(ap->host, dev);
                if (rc) {
 -                      printk(KERN_ERR "ata%u: scsi_add_host failed\n",
 -                             ap->id);
 +                      ata_port_printk(ap, KERN_ERR, "scsi_add_host failed\n");
                        /* FIXME: do something useful here */
                        /* FIXME: handle unconditional calls to
                         * scsi_scan_host and ata_host_remove, below,
                         * at the very least
                         */
                }
 +
 +              if (ap->ops->error_handler) {
 +                      unsigned long flags;
 +
 +                      ata_port_probe(ap);
 +
 +                      /* kick EH for boot probing */
 +                      spin_lock_irqsave(&ap->host_set->lock, flags);
 +
 +                      ap->eh_info.probe_mask = (1 << ATA_MAX_DEVICES) - 1;
 +                      ap->eh_info.action |= ATA_EH_SOFTRESET;
 +
 +                      ap->flags |= ATA_FLAG_LOADING;
 +                      ata_port_schedule_eh(ap);
 +
 +                      spin_unlock_irqrestore(&ap->host_set->lock, flags);
 +
 +                      /* wait for EH to finish */
 +                      ata_port_wait_eh(ap);
 +              } else {
 +                      DPRINTK("ata%u: bus probe begin\n", ap->id);
 +                      rc = ata_bus_probe(ap);
 +                      DPRINTK("ata%u: bus probe end\n", ap->id);
 +
 +                      if (rc) {
 +                              /* FIXME: do something useful here?
 +                               * Current libata behavior will
 +                               * tear down everything when
 +                               * the module is removed
 +                               * or the h/w is unplugged.
 +                               */
 +                      }
 +              }
        }
  
        /* probes are done, now scan each port's disk(s) */
@@@ -5440,63 -4667,6 +5441,63 @@@ err_free_ret
        return 0;
  }
  
 +/**
 + *    ata_port_detach - Detach ATA port in prepration of device removal
 + *    @ap: ATA port to be detached
 + *
 + *    Detach all ATA devices and the associated SCSI devices of @ap;
 + *    then, remove the associated SCSI host.  @ap is guaranteed to
 + *    be quiescent on return from this function.
 + *
 + *    LOCKING:
 + *    Kernel thread context (may sleep).
 + */
 +void ata_port_detach(struct ata_port *ap)
 +{
 +      unsigned long flags;
 +      int i;
 +
 +      if (!ap->ops->error_handler)
 +              return;
 +
 +      /* tell EH we're leaving & flush EH */
 +      spin_lock_irqsave(&ap->host_set->lock, flags);
 +      ap->flags |= ATA_FLAG_UNLOADING;
 +      spin_unlock_irqrestore(&ap->host_set->lock, flags);
 +
 +      ata_port_wait_eh(ap);
 +
 +      /* EH is now guaranteed to see UNLOADING, so no new device
 +       * will be attached.  Disable all existing devices.
 +       */
 +      spin_lock_irqsave(&ap->host_set->lock, flags);
 +
 +      for (i = 0; i < ATA_MAX_DEVICES; i++)
 +              ata_dev_disable(&ap->device[i]);
 +
 +      spin_unlock_irqrestore(&ap->host_set->lock, flags);
 +
 +      /* Final freeze & EH.  All in-flight commands are aborted.  EH
 +       * will be skipped and retrials will be terminated with bad
 +       * target.
 +       */
 +      spin_lock_irqsave(&ap->host_set->lock, flags);
 +      ata_port_freeze(ap);    /* won't be thawed */
 +      spin_unlock_irqrestore(&ap->host_set->lock, flags);
 +
 +      ata_port_wait_eh(ap);
 +
 +      /* Flush hotplug task.  The sequence is similar to
 +       * ata_port_flush_task().
 +       */
 +      flush_workqueue(ata_aux_wq);
 +      cancel_delayed_work(&ap->hotplug_task);
 +      flush_workqueue(ata_aux_wq);
 +
 +      /* remove the associated SCSI host */
 +      scsi_remove_host(ap->host);
 +}
 +
  /**
   *    ata_host_set_remove - PCI layer callback for device removal
   *    @host_set: ATA host set that was removed
  
  void ata_host_set_remove(struct ata_host_set *host_set)
  {
 -      struct ata_port *ap;
        unsigned int i;
  
 -      for (i = 0; i < host_set->n_ports; i++) {
 -              ap = host_set->ports[i];
 -              scsi_remove_host(ap->host);
 -      }
 +      for (i = 0; i < host_set->n_ports; i++)
 +              ata_port_detach(host_set->ports[i]);
  
        free_irq(host_set->irq, host_set);
  
        for (i = 0; i < host_set->n_ports; i++) {
 -              ap = host_set->ports[i];
 +              struct ata_port *ap = host_set->ports[i];
  
                ata_scsi_release(ap->host);
  
  
  int ata_scsi_release(struct Scsi_Host *host)
  {
 -      struct ata_port *ap = (struct ata_port *) &host->hostdata[0];
 -      int i;
 +      struct ata_port *ap = ata_shost_to_port(host);
  
        DPRINTK("ENTER\n");
  
        ap->ops->port_disable(ap);
        ata_host_remove(ap, 0);
 -      for (i = 0; i < ATA_MAX_DEVICES; i++)
 -              kfree(ap->device[i].id);
  
        DPRINTK("EXIT\n");
        return 1;
@@@ -5621,12 -4797,8 +5622,12 @@@ void ata_pci_remove_one (struct pci_de
  {
        struct device *dev = pci_dev_to_dev(pdev);
        struct ata_host_set *host_set = dev_get_drvdata(dev);
 +      struct ata_host_set *host_set2 = host_set->next;
  
        ata_host_set_remove(host_set);
 +      if (host_set2)
 +              ata_host_set_remove(host_set2);
 +
        pci_release_regions(pdev);
        pci_disable_device(pdev);
        dev_set_drvdata(dev, NULL);
@@@ -5691,12 -4863,6 +5692,12 @@@ static int __init ata_init(void
        if (!ata_wq)
                return -ENOMEM;
  
 +      ata_aux_wq = create_singlethread_workqueue("ata_aux");
 +      if (!ata_aux_wq) {
 +              destroy_workqueue(ata_wq);
 +              return -ENOMEM;
 +      }
 +
        printk(KERN_DEBUG "libata version " DRV_VERSION " loaded.\n");
        return 0;
  }
  static void __exit ata_exit(void)
  {
        destroy_workqueue(ata_wq);
 +      destroy_workqueue(ata_aux_wq);
  }
  
  module_init(ata_init);
@@@ -5731,52 -4896,6 +5732,52 @@@ int ata_ratelimit(void
        return rc;
  }
  
 +/**
 + *    ata_wait_register - wait until register value changes
 + *    @reg: IO-mapped register
 + *    @mask: Mask to apply to read register value
 + *    @val: Wait condition
 + *    @interval_msec: polling interval in milliseconds
 + *    @timeout_msec: timeout in milliseconds
 + *
 + *    Waiting for some bits of register to change is a common
 + *    operation for ATA controllers.  This function reads 32bit LE
 + *    IO-mapped register @reg and tests for the following condition.
 + *
 + *    (*@reg & mask) != val
 + *
 + *    If the condition is met, it returns; otherwise, the process is
 + *    repeated after @interval_msec until timeout.
 + *
 + *    LOCKING:
 + *    Kernel thread context (may sleep)
 + *
 + *    RETURNS:
 + *    The final register value.
 + */
 +u32 ata_wait_register(void __iomem *reg, u32 mask, u32 val,
 +                    unsigned long interval_msec,
 +                    unsigned long timeout_msec)
 +{
 +      unsigned long timeout;
 +      u32 tmp;
 +
 +      tmp = ioread32(reg);
 +
 +      /* Calculate timeout _after_ the first read to make sure
 +       * preceding writes reach the controller before starting to
 +       * eat away the timeout.
 +       */
 +      timeout = jiffies + (timeout_msec * HZ) / 1000;
 +
 +      while ((tmp & mask) == val && time_before(jiffies, timeout)) {
 +              msleep(interval_msec);
 +              tmp = ioread32(reg);
 +      }
 +
 +      return tmp;
 +}
 +
  /*
   * libata is essentially a library of internal helper functions for
   * low-level ATA host controller drivers.  As such, the API/ABI is
   * Do not depend on ABI/API stability.
   */
  
 +EXPORT_SYMBOL_GPL(sata_deb_timing_boot);
 +EXPORT_SYMBOL_GPL(sata_deb_timing_eh);
 +EXPORT_SYMBOL_GPL(sata_deb_timing_before_fsrst);
  EXPORT_SYMBOL_GPL(ata_std_bios_param);
  EXPORT_SYMBOL_GPL(ata_std_ports);
  EXPORT_SYMBOL_GPL(ata_device_add);
 +EXPORT_SYMBOL_GPL(ata_port_detach);
  EXPORT_SYMBOL_GPL(ata_host_set_remove);
  EXPORT_SYMBOL_GPL(ata_sg_init);
  EXPORT_SYMBOL_GPL(ata_sg_init_one);
 -EXPORT_SYMBOL_GPL(__ata_qc_complete);
 +EXPORT_SYMBOL_GPL(ata_hsm_move);
 +EXPORT_SYMBOL_GPL(ata_qc_complete);
 +EXPORT_SYMBOL_GPL(ata_qc_complete_multiple);
  EXPORT_SYMBOL_GPL(ata_qc_issue_prot);
 -EXPORT_SYMBOL_GPL(ata_eng_timeout);
  EXPORT_SYMBOL_GPL(ata_tf_load);
  EXPORT_SYMBOL_GPL(ata_tf_read);
  EXPORT_SYMBOL_GPL(ata_noop_dev_select);
@@@ -5811,9 -4925,6 +5812,9 @@@ EXPORT_SYMBOL_GPL(ata_port_start)
  EXPORT_SYMBOL_GPL(ata_port_stop);
  EXPORT_SYMBOL_GPL(ata_host_stop);
  EXPORT_SYMBOL_GPL(ata_interrupt);
 +EXPORT_SYMBOL_GPL(ata_mmio_data_xfer);
 +EXPORT_SYMBOL_GPL(ata_pio_data_xfer);
 +EXPORT_SYMBOL_GPL(ata_pio_data_xfer_noirq);
  EXPORT_SYMBOL_GPL(ata_qc_prep);
  EXPORT_SYMBOL_GPL(ata_noop_qc_prep);
  EXPORT_SYMBOL_GPL(ata_bmdma_setup);
@@@ -5821,46 -4932,33 +5822,46 @@@ EXPORT_SYMBOL_GPL(ata_bmdma_start)
  EXPORT_SYMBOL_GPL(ata_bmdma_irq_clear);
  EXPORT_SYMBOL_GPL(ata_bmdma_status);
  EXPORT_SYMBOL_GPL(ata_bmdma_stop);
 +EXPORT_SYMBOL_GPL(ata_bmdma_freeze);
 +EXPORT_SYMBOL_GPL(ata_bmdma_thaw);
 +EXPORT_SYMBOL_GPL(ata_bmdma_drive_eh);
 +EXPORT_SYMBOL_GPL(ata_bmdma_error_handler);
 +EXPORT_SYMBOL_GPL(ata_bmdma_post_internal_cmd);
  EXPORT_SYMBOL_GPL(ata_port_probe);
 +EXPORT_SYMBOL_GPL(sata_set_spd);
 +EXPORT_SYMBOL_GPL(sata_phy_debounce);
 +EXPORT_SYMBOL_GPL(sata_phy_resume);
  EXPORT_SYMBOL_GPL(sata_phy_reset);
  EXPORT_SYMBOL_GPL(__sata_phy_reset);
  EXPORT_SYMBOL_GPL(ata_bus_reset);
 -EXPORT_SYMBOL_GPL(ata_std_probeinit);
 +EXPORT_SYMBOL_GPL(ata_std_prereset);
  EXPORT_SYMBOL_GPL(ata_std_softreset);
  EXPORT_SYMBOL_GPL(sata_std_hardreset);
  EXPORT_SYMBOL_GPL(ata_std_postreset);
 -EXPORT_SYMBOL_GPL(ata_std_probe_reset);
 -EXPORT_SYMBOL_GPL(ata_drive_probe_reset);
  EXPORT_SYMBOL_GPL(ata_dev_revalidate);
  EXPORT_SYMBOL_GPL(ata_dev_classify);
  EXPORT_SYMBOL_GPL(ata_dev_pair);
  EXPORT_SYMBOL_GPL(ata_port_disable);
  EXPORT_SYMBOL_GPL(ata_ratelimit);
 +EXPORT_SYMBOL_GPL(ata_wait_register);
  EXPORT_SYMBOL_GPL(ata_busy_sleep);
  EXPORT_SYMBOL_GPL(ata_port_queue_task);
  EXPORT_SYMBOL_GPL(ata_scsi_ioctl);
  EXPORT_SYMBOL_GPL(ata_scsi_queuecmd);
  EXPORT_SYMBOL_GPL(ata_scsi_slave_config);
 +EXPORT_SYMBOL_GPL(ata_scsi_slave_destroy);
 +EXPORT_SYMBOL_GPL(ata_scsi_change_queue_depth);
  EXPORT_SYMBOL_GPL(ata_scsi_release);
  EXPORT_SYMBOL_GPL(ata_host_intr);
 +EXPORT_SYMBOL_GPL(sata_scr_valid);
 +EXPORT_SYMBOL_GPL(sata_scr_read);
 +EXPORT_SYMBOL_GPL(sata_scr_write);
 +EXPORT_SYMBOL_GPL(sata_scr_write_flush);
 +EXPORT_SYMBOL_GPL(ata_port_online);
 +EXPORT_SYMBOL_GPL(ata_port_offline);
  EXPORT_SYMBOL_GPL(ata_id_string);
  EXPORT_SYMBOL_GPL(ata_id_c_string);
  EXPORT_SYMBOL_GPL(ata_scsi_simulate);
 -EXPORT_SYMBOL_GPL(ata_eh_qc_complete);
 -EXPORT_SYMBOL_GPL(ata_eh_qc_retry);
  
  EXPORT_SYMBOL_GPL(ata_pio_need_iordy);
  EXPORT_SYMBOL_GPL(ata_timing_compute);
@@@ -5882,13 -4980,3 +5883,13 @@@ EXPORT_SYMBOL_GPL(ata_device_suspend)
  EXPORT_SYMBOL_GPL(ata_device_resume);
  EXPORT_SYMBOL_GPL(ata_scsi_device_suspend);
  EXPORT_SYMBOL_GPL(ata_scsi_device_resume);
 +
 +EXPORT_SYMBOL_GPL(ata_eng_timeout);
 +EXPORT_SYMBOL_GPL(ata_port_schedule_eh);
 +EXPORT_SYMBOL_GPL(ata_port_abort);
 +EXPORT_SYMBOL_GPL(ata_port_freeze);
 +EXPORT_SYMBOL_GPL(ata_eh_freeze_port);
 +EXPORT_SYMBOL_GPL(ata_eh_thaw_port);
 +EXPORT_SYMBOL_GPL(ata_eh_qc_complete);
 +EXPORT_SYMBOL_GPL(ata_eh_qc_retry);
 +EXPORT_SYMBOL_GPL(ata_do_eh);
  #include <linux/spinlock.h>
  #include <scsi/scsi.h>
  #include <scsi/scsi_host.h>
+ #include <scsi/scsi_cmnd.h>
  #include <scsi/scsi_eh.h>
  #include <scsi/scsi_device.h>
- #include <scsi/scsi_request.h>
 +#include <scsi/scsi_tcq.h>
  #include <scsi/scsi_transport.h>
  #include <linux/libata.h>
  #include <linux/hdreg.h>
  #define SECTOR_SIZE   512
  
  typedef unsigned int (*ata_xlat_func_t)(struct ata_queued_cmd *qc, const u8 *scsicmd);
 -static struct ata_device *
 -ata_scsi_find_dev(struct ata_port *ap, const struct scsi_device *scsidev);
 -static void ata_scsi_error(struct Scsi_Host *host);
 -enum scsi_eh_timer_return ata_scsi_timed_out(struct scsi_cmnd *cmd);
 +
 +static struct ata_device * __ata_scsi_find_dev(struct ata_port *ap,
 +                                      const struct scsi_device *scsidev);
 +static struct ata_device * ata_scsi_find_dev(struct ata_port *ap,
 +                                          const struct scsi_device *scsidev);
 +static int ata_scsi_user_scan(struct Scsi_Host *shost, unsigned int channel,
 +                            unsigned int id, unsigned int lun);
 +
  
  #define RW_RECOVERY_MPAGE 0x1
  #define RW_RECOVERY_MPAGE_LEN 12
@@@ -107,7 -102,6 +107,7 @@@ static const u8 def_control_mpage[CONTR
  struct scsi_transport_template ata_scsi_transport_template = {
        .eh_strategy_handler    = ata_scsi_error,
        .eh_timed_out           = ata_scsi_timed_out,
 +      .user_scan              = ata_scsi_user_scan,
  };
  
  
@@@ -310,6 -304,7 +310,6 @@@ int ata_scsi_ioctl(struct scsi_device *
  
  /**
   *    ata_scsi_qc_new - acquire new ata_queued_cmd reference
 - *    @ap: ATA port to which the new command is attached
   *    @dev: ATA device to which the new command is attached
   *    @cmd: SCSI command that originated this ATA command
   *    @done: SCSI command completion function
   *    RETURNS:
   *    Command allocated, or %NULL if none available.
   */
 -struct ata_queued_cmd *ata_scsi_qc_new(struct ata_port *ap,
 -                                     struct ata_device *dev,
 +struct ata_queued_cmd *ata_scsi_qc_new(struct ata_device *dev,
                                       struct scsi_cmnd *cmd,
                                       void (*done)(struct scsi_cmnd *))
  {
        struct ata_queued_cmd *qc;
  
 -      qc = ata_qc_new_init(ap, dev);
 +      qc = ata_qc_new_init(dev);
        if (qc) {
                qc->scsicmd = cmd;
                qc->scsidone = done;
@@@ -401,18 -397,18 +401,18 @@@ void ata_dump_status(unsigned id, struc
  
  int ata_scsi_device_resume(struct scsi_device *sdev)
  {
 -      struct ata_port *ap = (struct ata_port *) &sdev->host->hostdata[0];
 -      struct ata_device *dev = &ap->device[sdev->id];
 +      struct ata_port *ap = ata_shost_to_port(sdev->host);
 +      struct ata_device *dev = __ata_scsi_find_dev(ap, sdev);
  
 -      return ata_device_resume(ap, dev);
 +      return ata_device_resume(dev);
  }
  
  int ata_scsi_device_suspend(struct scsi_device *sdev, pm_message_t state)
  {
 -      struct ata_port *ap = (struct ata_port *) &sdev->host->hostdata[0];
 -      struct ata_device *dev = &ap->device[sdev->id];
 +      struct ata_port *ap = ata_shost_to_port(sdev->host);
 +      struct ata_device *dev = __ata_scsi_find_dev(ap, sdev);
  
 -      return ata_device_suspend(ap, dev, state);
 +      return ata_device_suspend(dev, state);
  }
  
  /**
   *    @sk: the sense key we'll fill out
   *    @asc: the additional sense code we'll fill out
   *    @ascq: the additional sense code qualifier we'll fill out
 + *    @verbose: be verbose
   *
   *    Converts an ATA error into a SCSI error.  Fill out pointers to
   *    SK, ASC, and ASCQ bytes for later use in fixed or descriptor
   *    spin_lock_irqsave(host_set lock)
   */
  void ata_to_sense_error(unsigned id, u8 drv_stat, u8 drv_err, u8 *sk, u8 *asc,
 -                      u8 *ascq)
 +                      u8 *ascq, int verbose)
  {
        int i;
  
                        }
                }
                /* No immediate match */
 -              printk(KERN_WARNING "ata%u: no sense translation for "
 -                     "error 0x%02x\n", id, drv_err);
 +              if (verbose)
 +                      printk(KERN_WARNING "ata%u: no sense translation for "
 +                             "error 0x%02x\n", id, drv_err);
        }
  
        /* Fall back to interpreting status bits */
                }
        }
        /* No error?  Undecoded? */
 -      printk(KERN_WARNING "ata%u: no sense translation for status: 0x%02x\n",
 -             id, drv_stat);
 +      if (verbose)
 +              printk(KERN_WARNING "ata%u: no sense translation for "
 +                     "status: 0x%02x\n", id, drv_stat);
  
        /* We need a sensible error return here, which is tricky, and one
           that won't cause people to do things like return a disk wrongly */
        *ascq = 0x00;
  
   translate_done:
 -      printk(KERN_ERR "ata%u: translated ATA stat/err 0x%02x/%02x to "
 -             "SCSI SK/ASC/ASCQ 0x%x/%02x/%02x\n", id, drv_stat, drv_err,
 -             *sk, *asc, *ascq);
 +      if (verbose)
 +              printk(KERN_ERR "ata%u: translated ATA stat/err 0x%02x/%02x "
 +                     "to SCSI SK/ASC/ASCQ 0x%x/%02x/%02x\n",
 +                     id, drv_stat, drv_err, *sk, *asc, *ascq);
        return;
  }
  
  void ata_gen_ata_desc_sense(struct ata_queued_cmd *qc)
  {
        struct scsi_cmnd *cmd = qc->scsicmd;
 -      struct ata_taskfile *tf = &qc->tf;
 +      struct ata_taskfile *tf = &qc->result_tf;
        unsigned char *sb = cmd->sense_buffer;
        unsigned char *desc = sb + 8;
 +      int verbose = qc->ap->ops->error_handler == NULL;
  
        memset(sb, 0, SCSI_SENSE_BUFFERSIZE);
  
        cmd->result = (DRIVER_SENSE << 24) | SAM_STAT_CHECK_CONDITION;
  
 -      /*
 -       * Read the controller registers.
 -       */
 -      WARN_ON(qc->ap->ops->tf_read == NULL);
 -      qc->ap->ops->tf_read(qc->ap, tf);
 -
        /*
         * Use ata_to_sense_error() to map status register bits
         * onto sense key, asc & ascq.
         */
 -      if (tf->command & (ATA_BUSY | ATA_DF | ATA_ERR | ATA_DRQ)) {
 +      if (qc->err_mask ||
 +          tf->command & (ATA_BUSY | ATA_DF | ATA_ERR | ATA_DRQ)) {
                ata_to_sense_error(qc->ap->id, tf->command, tf->feature,
 -                                 &sb[1], &sb[2], &sb[3]);
 +                                 &sb[1], &sb[2], &sb[3], verbose);
                sb[1] &= 0x0f;
        }
  
  void ata_gen_fixed_sense(struct ata_queued_cmd *qc)
  {
        struct scsi_cmnd *cmd = qc->scsicmd;
 -      struct ata_taskfile *tf = &qc->tf;
 +      struct ata_taskfile *tf = &qc->result_tf;
        unsigned char *sb = cmd->sense_buffer;
 +      int verbose = qc->ap->ops->error_handler == NULL;
  
        memset(sb, 0, SCSI_SENSE_BUFFERSIZE);
  
        cmd->result = (DRIVER_SENSE << 24) | SAM_STAT_CHECK_CONDITION;
  
 -      /*
 -       * Read the controller registers.
 -       */
 -      WARN_ON(qc->ap->ops->tf_read == NULL);
 -      qc->ap->ops->tf_read(qc->ap, tf);
 -
        /*
         * Use ata_to_sense_error() to map status register bits
         * onto sense key, asc & ascq.
         */
 -      if (tf->command & (ATA_BUSY | ATA_DF | ATA_ERR | ATA_DRQ)) {
 +      if (qc->err_mask ||
 +          tf->command & (ATA_BUSY | ATA_DF | ATA_ERR | ATA_DRQ)) {
                ata_to_sense_error(qc->ap->id, tf->command, tf->feature,
 -                                 &sb[2], &sb[12], &sb[13]);
 +                                 &sb[2], &sb[12], &sb[13], verbose);
                sb[2] &= 0x0f;
        }
  
@@@ -677,7 -677,7 +677,7 @@@ static void ata_scsi_dev_config(struct 
         */
        max_sectors = ATA_MAX_SECTORS;
        if (dev->flags & ATA_DFLAG_LBA48)
 -              max_sectors = 2048;
 +              max_sectors = ATA_MAX_SECTORS_LBA48;
        if (dev->max_sectors)
                max_sectors = dev->max_sectors;
  
                request_queue_t *q = sdev->request_queue;
                blk_queue_max_hw_segments(q, q->max_hw_segments - 1);
        }
 +
 +      if (dev->flags & ATA_DFLAG_NCQ) {
 +              int depth;
 +
 +              depth = min(sdev->host->can_queue, ata_id_queue_depth(dev->id));
 +              depth = min(ATA_MAX_QUEUE - 1, depth);
 +              scsi_adjust_queue_depth(sdev, MSG_SIMPLE_TAG, depth);
 +      }
  }
  
  /**
  
  int ata_scsi_slave_config(struct scsi_device *sdev)
  {
 +      struct ata_port *ap = ata_shost_to_port(sdev->host);
 +      struct ata_device *dev = __ata_scsi_find_dev(ap, sdev);
 +
        ata_scsi_sdev_config(sdev);
  
        blk_queue_max_phys_segments(sdev->request_queue, LIBATA_MAX_PRD);
  
 -      if (sdev->id < ATA_MAX_DEVICES) {
 -              struct ata_port *ap;
 -              struct ata_device *dev;
 -
 -              ap = (struct ata_port *) &sdev->host->hostdata[0];
 -              dev = &ap->device[sdev->id];
 -
 +      if (dev)
                ata_scsi_dev_config(sdev, dev);
 -      }
  
        return 0;       /* scsi layer doesn't check return value, sigh */
  }
  
  /**
 - *    ata_scsi_timed_out - SCSI layer time out callback
 - *    @cmd: timed out SCSI command
 + *    ata_scsi_slave_destroy - SCSI device is about to be destroyed
 + *    @sdev: SCSI device to be destroyed
   *
 - *    Handles SCSI layer timeout.  We race with normal completion of
 - *    the qc for @cmd.  If the qc is already gone, we lose and let
 - *    the scsi command finish (EH_HANDLED).  Otherwise, the qc has
 - *    timed out and EH should be invoked.  Prevent ata_qc_complete()
 - *    from finishing it by setting EH_SCHEDULED and return
 - *    EH_NOT_HANDLED.
 + *    @sdev is about to be destroyed for hot/warm unplugging.  If
 + *    this unplugging was initiated by libata as indicated by NULL
 + *    dev->sdev, this function doesn't have to do anything.
 + *    Otherwise, SCSI layer initiated warm-unplug is in progress.
 + *    Clear dev->sdev, schedule the device for ATA detach and invoke
 + *    EH.
   *
   *    LOCKING:
 - *    Called from timer context
 - *
 - *    RETURNS:
 - *    EH_HANDLED or EH_NOT_HANDLED
 + *    Defined by SCSI layer.  We don't really care.
   */
 -enum scsi_eh_timer_return ata_scsi_timed_out(struct scsi_cmnd *cmd)
 +void ata_scsi_slave_destroy(struct scsi_device *sdev)
  {
 -      struct Scsi_Host *host = cmd->device->host;
 -      struct ata_port *ap = (struct ata_port *) &host->hostdata[0];
 +      struct ata_port *ap = ata_shost_to_port(sdev->host);
        unsigned long flags;
 -      struct ata_queued_cmd *qc;
 -      enum scsi_eh_timer_return ret = EH_HANDLED;
 +      struct ata_device *dev;
  
 -      DPRINTK("ENTER\n");
 +      if (!ap->ops->error_handler)
 +              return;
  
        spin_lock_irqsave(&ap->host_set->lock, flags);
 -      qc = ata_qc_from_tag(ap, ap->active_tag);
 -      if (qc) {
 -              WARN_ON(qc->scsicmd != cmd);
 -              qc->flags |= ATA_QCFLAG_EH_SCHEDULED;
 -              qc->err_mask |= AC_ERR_TIMEOUT;
 -              ret = EH_NOT_HANDLED;
 +      dev = __ata_scsi_find_dev(ap, sdev);
 +      if (dev && dev->sdev) {
 +              /* SCSI device already in CANCEL state, no need to offline it */
 +              dev->sdev = NULL;
 +              dev->flags |= ATA_DFLAG_DETACH;
 +              ata_port_schedule_eh(ap);
        }
        spin_unlock_irqrestore(&ap->host_set->lock, flags);
 -
 -      DPRINTK("EXIT, ret=%d\n", ret);
 -      return ret;
  }
  
  /**
 - *    ata_scsi_error - SCSI layer error handler callback
 - *    @host: SCSI host on which error occurred
 + *    ata_scsi_change_queue_depth - SCSI callback for queue depth config
 + *    @sdev: SCSI device to configure queue depth for
 + *    @queue_depth: new queue depth
   *
 - *    Handles SCSI-layer-thrown error events.
 + *    This is libata standard hostt->change_queue_depth callback.
 + *    SCSI will call into this callback when user tries to set queue
 + *    depth via sysfs.
   *
   *    LOCKING:
 - *    Inherited from SCSI layer (none, can sleep)
 + *    SCSI layer (we don't care)
 + *
 + *    RETURNS:
 + *    Newly configured queue depth.
   */
 -
 -static void ata_scsi_error(struct Scsi_Host *host)
 +int ata_scsi_change_queue_depth(struct scsi_device *sdev, int queue_depth)
  {
 -      struct ata_port *ap;
 -      unsigned long flags;
 -
 -      DPRINTK("ENTER\n");
 -
 -      ap = (struct ata_port *) &host->hostdata[0];
 -
 -      spin_lock_irqsave(&ap->host_set->lock, flags);
 -      WARN_ON(ap->flags & ATA_FLAG_IN_EH);
 -      ap->flags |= ATA_FLAG_IN_EH;
 -      WARN_ON(ata_qc_from_tag(ap, ap->active_tag) == NULL);
 -      spin_unlock_irqrestore(&ap->host_set->lock, flags);
 -
 -      ata_port_flush_task(ap);
 -
 -      ap->ops->eng_timeout(ap);
 -
 -      WARN_ON(host->host_failed || !list_empty(&host->eh_cmd_q));
 -
 -      scsi_eh_flush_done_q(&ap->eh_done_q);
 -
 -      spin_lock_irqsave(&ap->host_set->lock, flags);
 -      ap->flags &= ~ATA_FLAG_IN_EH;
 -      spin_unlock_irqrestore(&ap->host_set->lock, flags);
 -
 -      DPRINTK("EXIT\n");
 -}
 -
 -static void ata_eh_scsidone(struct scsi_cmnd *scmd)
 -{
 -      /* nada */
 -}
 +      struct ata_port *ap = ata_shost_to_port(sdev->host);
 +      struct ata_device *dev;
 +      int max_depth;
  
 -static void __ata_eh_qc_complete(struct ata_queued_cmd *qc)
 -{
 -      struct ata_port *ap = qc->ap;
 -      struct scsi_cmnd *scmd = qc->scsicmd;
 -      unsigned long flags;
 +      if (queue_depth < 1)
 +              return sdev->queue_depth;
  
 -      spin_lock_irqsave(&ap->host_set->lock, flags);
 -      qc->scsidone = ata_eh_scsidone;
 -      __ata_qc_complete(qc);
 -      WARN_ON(ata_tag_valid(qc->tag));
 -      spin_unlock_irqrestore(&ap->host_set->lock, flags);
 +      dev = ata_scsi_find_dev(ap, sdev);
 +      if (!dev || !ata_dev_enabled(dev))
 +              return sdev->queue_depth;
  
 -      scsi_eh_finish_cmd(scmd, &ap->eh_done_q);
 -}
 +      max_depth = min(sdev->host->can_queue, ata_id_queue_depth(dev->id));
 +      max_depth = min(ATA_MAX_QUEUE - 1, max_depth);
 +      if (queue_depth > max_depth)
 +              queue_depth = max_depth;
  
 -/**
 - *    ata_eh_qc_complete - Complete an active ATA command from EH
 - *    @qc: Command to complete
 - *
 - *    Indicate to the mid and upper layers that an ATA command has
 - *    completed.  To be used from EH.
 - */
 -void ata_eh_qc_complete(struct ata_queued_cmd *qc)
 -{
 -      struct scsi_cmnd *scmd = qc->scsicmd;
 -      scmd->retries = scmd->allowed;
 -      __ata_eh_qc_complete(qc);
 -}
 -
 -/**
 - *    ata_eh_qc_retry - Tell midlayer to retry an ATA command after EH
 - *    @qc: Command to retry
 - *
 - *    Indicate to the mid and upper layers that an ATA command
 - *    should be retried.  To be used from EH.
 - *
 - *    SCSI midlayer limits the number of retries to scmd->allowed.
 - *    This function might need to adjust scmd->retries for commands
 - *    which get retried due to unrelated NCQ failures.
 - */
 -void ata_eh_qc_retry(struct ata_queued_cmd *qc)
 -{
 -      __ata_eh_qc_complete(qc);
 +      scsi_adjust_queue_depth(sdev, MSG_SIMPLE_TAG, queue_depth);
 +      return queue_depth;
  }
  
  /**
@@@ -835,7 -891,7 +835,7 @@@ static unsigned int ata_scsi_start_stop
                tf->nsect = 1;  /* 1 sector, lba=0 */
  
                if (qc->dev->flags & ATA_DFLAG_LBA) {
 -                      qc->tf.flags |= ATA_TFLAG_LBA;
 +                      tf->flags |= ATA_TFLAG_LBA;
  
                        tf->lbah = 0x0;
                        tf->lbam = 0x0;
@@@ -1139,7 -1195,6 +1139,7 @@@ static unsigned int ata_scsi_rw_xlat(st
        u64 block;
        u32 n_block;
  
 +      qc->flags |= ATA_QCFLAG_IO;
        tf->flags |= ATA_TFLAG_ISADDR | ATA_TFLAG_DEVICE;
  
        if (scsicmd[0] == WRITE_10 || scsicmd[0] == WRITE_6 ||
                 */
                goto nothing_to_do;
  
 -      if (dev->flags & ATA_DFLAG_LBA) {
 +      if ((dev->flags & (ATA_DFLAG_PIO | ATA_DFLAG_NCQ)) == ATA_DFLAG_NCQ) {
 +              /* yay, NCQ */
 +              if (!lba_48_ok(block, n_block))
 +                      goto out_of_range;
 +
 +              tf->protocol = ATA_PROT_NCQ;
 +              tf->flags |= ATA_TFLAG_LBA | ATA_TFLAG_LBA48;
 +
 +              if (tf->flags & ATA_TFLAG_WRITE)
 +                      tf->command = ATA_CMD_FPDMA_WRITE;
 +              else
 +                      tf->command = ATA_CMD_FPDMA_READ;
 +
 +              qc->nsect = n_block;
 +
 +              tf->nsect = qc->tag << 3;
 +              tf->hob_feature = (n_block >> 8) & 0xff;
 +              tf->feature = n_block & 0xff;
 +
 +              tf->hob_lbah = (block >> 40) & 0xff;
 +              tf->hob_lbam = (block >> 32) & 0xff;
 +              tf->hob_lbal = (block >> 24) & 0xff;
 +              tf->lbah = (block >> 16) & 0xff;
 +              tf->lbam = (block >> 8) & 0xff;
 +              tf->lbal = block & 0xff;
 +
 +              tf->device = 1 << 6;
 +              if (tf->flags & ATA_TFLAG_FUA)
 +                      tf->device |= 1 << 7;
 +      } else if (dev->flags & ATA_DFLAG_LBA) {
                tf->flags |= ATA_TFLAG_LBA;
  
                if (lba_28_ok(block, n_block)) {
@@@ -1306,17 -1332,6 +1306,17 @@@ static void ata_scsi_qc_complete(struc
        u8 *cdb = cmd->cmnd;
        int need_sense = (qc->err_mask != 0);
  
 +      /* We snoop the SET_FEATURES - Write Cache ON/OFF command, and
 +       * schedule EH_REVALIDATE operation to update the IDENTIFY DEVICE
 +       * cache
 +       */
 +      if (!need_sense && (qc->tf.command == ATA_CMD_SET_FEATURES) &&
 +          ((qc->tf.feature == SETFEATURES_WC_ON) ||
 +           (qc->tf.feature == SETFEATURES_WC_OFF))) {
 +              qc->ap->eh_info.action |= ATA_EH_REVALIDATE;
 +              ata_port_schedule_eh(qc->ap);
 +      }
 +
        /* For ATA pass thru (SAT) commands, generate a sense block if
         * user mandated it or if there's an error.  Note that if we
         * generate because the user forced us to, a check condition
                }
        }
  
 -      if (need_sense) {
 -              /* The ata_gen_..._sense routines fill in tf */
 -              ata_dump_status(qc->ap->id, &qc->tf);
 -      }
 +      if (need_sense && !qc->ap->ops->error_handler)
 +              ata_dump_status(qc->ap->id, &qc->result_tf);
  
        qc->scsidone(cmd);
  
        ata_qc_free(qc);
  }
  
 +/**
 + *    ata_scmd_need_defer - Check whether we need to defer scmd
 + *    @dev: ATA device to which the command is addressed
 + *    @is_io: Is the command IO (and thus possibly NCQ)?
 + *
 + *    NCQ and non-NCQ commands cannot run together.  As upper layer
 + *    only knows the queue depth, we are responsible for maintaining
 + *    exclusion.  This function checks whether a new command can be
 + *    issued to @dev.
 + *
 + *    LOCKING:
 + *    spin_lock_irqsave(host_set lock)
 + *
 + *    RETURNS:
 + *    1 if deferring is needed, 0 otherwise.
 + */
 +static int ata_scmd_need_defer(struct ata_device *dev, int is_io)
 +{
 +      struct ata_port *ap = dev->ap;
 +
 +      if (!(dev->flags & ATA_DFLAG_NCQ))
 +              return 0;
 +
 +      if (is_io) {
 +              if (!ata_tag_valid(ap->active_tag))
 +                      return 0;
 +      } else {
 +              if (!ata_tag_valid(ap->active_tag) && !ap->sactive)
 +                      return 0;
 +      }
 +      return 1;
 +}
 +
  /**
   *    ata_scsi_translate - Translate then issue SCSI command to ATA device
 - *    @ap: ATA port to which the command is addressed
   *    @dev: ATA device to which the command is addressed
   *    @cmd: SCSI command to execute
   *    @done: SCSI command completion function
   *
   *    LOCKING:
   *    spin_lock_irqsave(host_set lock)
 + *
 + *    RETURNS:
 + *    0 on success, SCSI_ML_QUEUE_DEVICE_BUSY if the command
 + *    needs to be deferred.
   */
 -
 -static void ata_scsi_translate(struct ata_port *ap, struct ata_device *dev,
 -                            struct scsi_cmnd *cmd,
 +static int ata_scsi_translate(struct ata_device *dev, struct scsi_cmnd *cmd,
                              void (*done)(struct scsi_cmnd *),
                              ata_xlat_func_t xlat_func)
  {
        struct ata_queued_cmd *qc;
        u8 *scsicmd = cmd->cmnd;
 +      int is_io = xlat_func == ata_scsi_rw_xlat;
  
        VPRINTK("ENTER\n");
  
 -      qc = ata_scsi_qc_new(ap, dev, cmd, done);
 +      if (unlikely(ata_scmd_need_defer(dev, is_io)))
 +              goto defer;
 +
 +      qc = ata_scsi_qc_new(dev, cmd, done);
        if (!qc)
                goto err_mem;
  
        if (cmd->sc_data_direction == DMA_FROM_DEVICE ||
            cmd->sc_data_direction == DMA_TO_DEVICE) {
                if (unlikely(cmd->request_bufflen < 1)) {
 -                      printk(KERN_WARNING "ata%u(%u): WARNING: zero len r/w req\n",
 -                             ap->id, dev->devno);
 +                      ata_dev_printk(dev, KERN_WARNING,
 +                                     "WARNING: zero len r/w req\n");
                        goto err_did;
                }
  
        ata_qc_issue(qc);
  
        VPRINTK("EXIT\n");
 -      return;
 +      return 0;
  
  early_finish:
          ata_qc_free(qc);
        done(cmd);
        DPRINTK("EXIT - early finish (good or error)\n");
 -      return;
 +      return 0;
  
  err_did:
        ata_qc_free(qc);
@@@ -1467,11 -1446,7 +1467,11 @@@ err_mem
        cmd->result = (DID_ERROR << 16);
        done(cmd);
        DPRINTK("EXIT - internal\n");
 -      return;
 +      return 0;
 +
 +defer:
 +      DPRINTK("EXIT - defer\n");
 +      return SCSI_MLQUEUE_DEVICE_BUSY;
  }
  
  /**
@@@ -1969,7 -1944,7 +1969,7 @@@ unsigned int ata_scsiop_mode_sense(stru
                return 0;
  
        dpofua = 0;
 -      if (ata_dev_supports_fua(args->id) && dev->flags & ATA_DFLAG_LBA48 &&
 +      if (ata_dev_supports_fua(args->id) && (dev->flags & ATA_DFLAG_LBA48) &&
            (!(dev->flags & ATA_DFLAG_PIO) || dev->multi_count))
                dpofua = 1 << 4;
  
@@@ -2162,14 -2137,13 +2162,14 @@@ void ata_scsi_badcmd(struct scsi_cmnd *
  
  static void atapi_sense_complete(struct ata_queued_cmd *qc)
  {
 -      if (qc->err_mask && ((qc->err_mask & AC_ERR_DEV) == 0))
 +      if (qc->err_mask && ((qc->err_mask & AC_ERR_DEV) == 0)) {
                /* FIXME: not quite right; we don't want the
                 * translation of taskfile registers into
                 * a sense descriptors, since that's only
                 * correct for ATA, not ATAPI
                 */
                ata_gen_ata_desc_sense(qc);
 +      }
  
        qc->scsidone(qc->scsicmd);
        ata_qc_free(qc);
@@@ -2233,38 -2207,21 +2233,38 @@@ static void atapi_qc_complete(struct at
  
        VPRINTK("ENTER, err_mask 0x%X\n", err_mask);
  
 +      /* handle completion from new EH */
 +      if (unlikely(qc->ap->ops->error_handler &&
 +                   (err_mask || qc->flags & ATA_QCFLAG_SENSE_VALID))) {
 +
 +              if (!(qc->flags & ATA_QCFLAG_SENSE_VALID)) {
 +                      /* FIXME: not quite right; we don't want the
 +                       * translation of taskfile registers into a
 +                       * sense descriptors, since that's only
 +                       * correct for ATA, not ATAPI
 +                       */
 +                      ata_gen_ata_desc_sense(qc);
 +              }
 +
 +              qc->scsicmd->result = SAM_STAT_CHECK_CONDITION;
 +              qc->scsidone(cmd);
 +              ata_qc_free(qc);
 +              return;
 +      }
 +
 +      /* successful completion or old EH failure path */
        if (unlikely(err_mask & AC_ERR_DEV)) {
                cmd->result = SAM_STAT_CHECK_CONDITION;
                atapi_request_sense(qc);
                return;
 -      }
 -
 -      else if (unlikely(err_mask))
 +      } else if (unlikely(err_mask)) {
                /* FIXME: not quite right; we don't want the
                 * translation of taskfile registers into
                 * a sense descriptors, since that's only
                 * correct for ATA, not ATAPI
                 */
                ata_gen_ata_desc_sense(qc);
 -
 -      else {
 +      } else {
                u8 *scsicmd = cmd->cmnd;
  
                if ((scsicmd[0] == INQUIRY) && ((scsicmd[1] & 0x03) == 0)) {
@@@ -2346,33 -2303,18 +2346,33 @@@ static unsigned int atapi_xlat(struct a
                qc->tf.protocol = ATA_PROT_ATAPI_DMA;
                qc->tf.feature |= ATAPI_PKT_DMA;
  
 -#ifdef ATAPI_ENABLE_DMADIR
 -              /* some SATA bridges need us to indicate data xfer direction */
 -              if (cmd->sc_data_direction != DMA_TO_DEVICE)
 +              if (atapi_dmadir && (cmd->sc_data_direction != DMA_TO_DEVICE))
 +                      /* some SATA bridges need us to indicate data xfer direction */
                        qc->tf.feature |= ATAPI_DMADIR;
 -#endif
        }
  
-       qc->nbytes = cmd->bufflen;
+       qc->nbytes = cmd->request_bufflen;
  
        return 0;
  }
  
 +static struct ata_device * ata_find_dev(struct ata_port *ap, int id)
 +{
 +      if (likely(id < ATA_MAX_DEVICES))
 +              return &ap->device[id];
 +      return NULL;
 +}
 +
 +static struct ata_device * __ata_scsi_find_dev(struct ata_port *ap,
 +                                      const struct scsi_device *scsidev)
 +{
 +      /* skip commands not addressed to targets we simulate */
 +      if (unlikely(scsidev->channel || scsidev->lun))
 +              return NULL;
 +
 +      return ata_find_dev(ap, scsidev->id);
 +}
 +
  /**
   *    ata_scsi_find_dev - lookup ata_device from scsi_cmnd
   *    @ap: ATA port to which the device is attached
   *    RETURNS:
   *    Associated ATA device, or %NULL if not found.
   */
 -
  static struct ata_device *
  ata_scsi_find_dev(struct ata_port *ap, const struct scsi_device *scsidev)
  {
 -      struct ata_device *dev;
 -
 -      /* skip commands not addressed to targets we simulate */
 -      if (likely(scsidev->id < ATA_MAX_DEVICES))
 -              dev = &ap->device[scsidev->id];
 -      else
 -              return NULL;
 +      struct ata_device *dev = __ata_scsi_find_dev(ap, scsidev);
  
 -      if (unlikely((scsidev->channel != 0) ||
 -                   (scsidev->lun != 0)))
 -              return NULL;
 -
 -      if (unlikely(!ata_dev_present(dev)))
 +      if (unlikely(!dev || !ata_dev_enabled(dev)))
                return NULL;
  
        if (!atapi_enabled || (ap->flags & ATA_FLAG_NO_ATAPI)) {
                if (unlikely(dev->class == ATA_DEV_ATAPI)) {
 -                      printk(KERN_WARNING "ata%u(%u): WARNING: ATAPI is %s, device ignored.\n",
 -                             ap->id, dev->devno, atapi_enabled ? "not supported with this driver" : "disabled");
 +                      ata_dev_printk(dev, KERN_WARNING,
 +                              "WARNING: ATAPI is %s, device ignored.\n",
 +                              atapi_enabled ? "not supported with this driver" : "disabled");
                        return NULL;
                }
        }
@@@ -2462,15 -2414,10 +2462,15 @@@ ata_scsi_pass_thru(struct ata_queued_cm
  {
        struct ata_taskfile *tf = &(qc->tf);
        struct scsi_cmnd *cmd = qc->scsicmd;
 +      struct ata_device *dev = qc->dev;
  
        if ((tf->protocol = ata_scsi_map_proto(scsicmd[1])) == ATA_PROT_UNKNOWN)
                goto invalid_fld;
  
 +      /* We may not issue DMA commands if no DMA mode is set */
 +      if (tf->protocol == ATA_PROT_DMA && dev->dma_mode == 0)
 +              goto invalid_fld;
 +
        if (scsicmd[1] & 0xe0)
                /* PIO multi not supported yet */
                goto invalid_fld;
         * TODO: find out if we need to do more here to
         *       cover scatter/gather case.
         */
-       qc->nsect = cmd->bufflen / ATA_SECT_SIZE;
+       qc->nsect = cmd->request_bufflen / ATA_SECT_SIZE;
  
 +      /* request result TF */
 +      qc->flags |= ATA_QCFLAG_RESULT_TF;
 +
        return 0;
  
   invalid_fld:
@@@ -2634,24 -2578,19 +2634,24 @@@ static inline void ata_scsi_dump_cdb(st
  #endif
  }
  
 -static inline void __ata_scsi_queuecmd(struct scsi_cmnd *cmd, void (*done)(struct scsi_cmnd *),
 -                                     struct ata_port *ap, struct ata_device *dev)
 +static inline int __ata_scsi_queuecmd(struct scsi_cmnd *cmd,
 +                                    void (*done)(struct scsi_cmnd *),
 +                                    struct ata_device *dev)
  {
 +      int rc = 0;
 +
        if (dev->class == ATA_DEV_ATA) {
                ata_xlat_func_t xlat_func = ata_get_xlat_func(dev,
                                                              cmd->cmnd[0]);
  
                if (xlat_func)
 -                      ata_scsi_translate(ap, dev, cmd, done, xlat_func);
 +                      rc = ata_scsi_translate(dev, cmd, done, xlat_func);
                else
 -                      ata_scsi_simulate(ap, dev, cmd, done);
 +                      ata_scsi_simulate(dev, cmd, done);
        } else
 -              ata_scsi_translate(ap, dev, cmd, done, atapi_xlat);
 +              rc = ata_scsi_translate(dev, cmd, done, atapi_xlat);
 +
 +      return rc;
  }
  
  /**
   *    Releases scsi-layer-held lock, and obtains host_set lock.
   *
   *    RETURNS:
 - *    Zero.
 + *    Return value from __ata_scsi_queuecmd() if @cmd can be queued,
 + *    0 otherwise.
   */
 -
  int ata_scsi_queuecmd(struct scsi_cmnd *cmd, void (*done)(struct scsi_cmnd *))
  {
        struct ata_port *ap;
        struct ata_device *dev;
        struct scsi_device *scsidev = cmd->device;
        struct Scsi_Host *shost = scsidev->host;
 +      int rc = 0;
  
 -      ap = (struct ata_port *) &shost->hostdata[0];
 +      ap = ata_shost_to_port(shost);
  
        spin_unlock(shost->host_lock);
        spin_lock(&ap->host_set->lock);
  
        dev = ata_scsi_find_dev(ap, scsidev);
        if (likely(dev))
 -              __ata_scsi_queuecmd(cmd, done, ap, dev);
 +              rc = __ata_scsi_queuecmd(cmd, done, dev);
        else {
                cmd->result = (DID_BAD_TARGET << 16);
                done(cmd);
  
        spin_unlock(&ap->host_set->lock);
        spin_lock(shost->host_lock);
 -      return 0;
 +      return rc;
  }
  
  /**
   *    ata_scsi_simulate - simulate SCSI command on ATA device
 - *    @ap: port the device is connected to
   *    @dev: the target device
   *    @cmd: SCSI command being sent to device.
   *    @done: SCSI command completion function.
   *    spin_lock_irqsave(host_set lock)
   */
  
 -void ata_scsi_simulate(struct ata_port *ap, struct ata_device *dev,
 -                    struct scsi_cmnd *cmd,
 +void ata_scsi_simulate(struct ata_device *dev, struct scsi_cmnd *cmd,
                      void (*done)(struct scsi_cmnd *))
  {
        struct ata_scsi_args args;
        const u8 *scsicmd = cmd->cmnd;
  
 -      args.ap = ap;
        args.dev = dev;
        args.id = dev->id;
        args.cmd = cmd;
  
  void ata_scsi_scan_host(struct ata_port *ap)
  {
 -      struct ata_device *dev;
        unsigned int i;
  
 -      if (ap->flags & ATA_FLAG_PORT_DISABLED)
 +      if (ap->flags & ATA_FLAG_DISABLED)
                return;
  
        for (i = 0; i < ATA_MAX_DEVICES; i++) {
 -              dev = &ap->device[i];
 +              struct ata_device *dev = &ap->device[i];
 +              struct scsi_device *sdev;
 +
 +              if (!ata_dev_enabled(dev) || dev->sdev)
 +                      continue;
 +
 +              sdev = __scsi_add_device(ap->host, 0, i, 0, NULL);
 +              if (!IS_ERR(sdev)) {
 +                      dev->sdev = sdev;
 +                      scsi_device_put(sdev);
 +              }
 +      }
 +}
 +
 +/**
 + *    ata_scsi_offline_dev - offline attached SCSI device
 + *    @dev: ATA device to offline attached SCSI device for
 + *
 + *    This function is called from ata_eh_hotplug() and responsible
 + *    for taking the SCSI device attached to @dev offline.  This
 + *    function is called with host_set lock which protects dev->sdev
 + *    against clearing.
 + *
 + *    LOCKING:
 + *    spin_lock_irqsave(host_set lock)
 + *
 + *    RETURNS:
 + *    1 if attached SCSI device exists, 0 otherwise.
 + */
 +int ata_scsi_offline_dev(struct ata_device *dev)
 +{
 +      if (dev->sdev) {
 +              scsi_device_set_state(dev->sdev, SDEV_OFFLINE);
 +              return 1;
 +      }
 +      return 0;
 +}
 +
 +/**
 + *    ata_scsi_remove_dev - remove attached SCSI device
 + *    @dev: ATA device to remove attached SCSI device for
 + *
 + *    This function is called from ata_eh_scsi_hotplug() and
 + *    responsible for removing the SCSI device attached to @dev.
 + *
 + *    LOCKING:
 + *    Kernel thread context (may sleep).
 + */
 +static void ata_scsi_remove_dev(struct ata_device *dev)
 +{
 +      struct ata_port *ap = dev->ap;
 +      struct scsi_device *sdev;
 +      unsigned long flags;
 +
 +      /* Alas, we need to grab scan_mutex to ensure SCSI device
 +       * state doesn't change underneath us and thus
 +       * scsi_device_get() always succeeds.  The mutex locking can
 +       * be removed if there is __scsi_device_get() interface which
 +       * increments reference counts regardless of device state.
 +       */
 +      mutex_lock(&ap->host->scan_mutex);
 +      spin_lock_irqsave(&ap->host_set->lock, flags);
 +
 +      /* clearing dev->sdev is protected by host_set lock */
 +      sdev = dev->sdev;
 +      dev->sdev = NULL;
 +
 +      if (sdev) {
 +              /* If user initiated unplug races with us, sdev can go
 +               * away underneath us after the host_set lock and
 +               * scan_mutex are released.  Hold onto it.
 +               */
 +              if (scsi_device_get(sdev) == 0) {
 +                      /* The following ensures the attached sdev is
 +                       * offline on return from ata_scsi_offline_dev()
 +                       * regardless it wins or loses the race
 +                       * against this function.
 +                       */
 +                      scsi_device_set_state(sdev, SDEV_OFFLINE);
 +              } else {
 +                      WARN_ON(1);
 +                      sdev = NULL;
 +              }
 +      }
 +
 +      spin_unlock_irqrestore(&ap->host_set->lock, flags);
 +      mutex_unlock(&ap->host->scan_mutex);
 +
 +      if (sdev) {
 +              ata_dev_printk(dev, KERN_INFO, "detaching (SCSI %s)\n",
 +                             sdev->sdev_gendev.bus_id);
 +
 +              scsi_remove_device(sdev);
 +              scsi_device_put(sdev);
 +      }
 +}
  
 -              if (ata_dev_present(dev))
 -                      scsi_scan_target(&ap->host->shost_gendev, 0, i, 0, 0);
 +/**
 + *    ata_scsi_hotplug - SCSI part of hotplug
 + *    @data: Pointer to ATA port to perform SCSI hotplug on
 + *
 + *    Perform SCSI part of hotplug.  It's executed from a separate
 + *    workqueue after EH completes.  This is necessary because SCSI
 + *    hot plugging requires working EH and hot unplugging is
 + *    synchronized with hot plugging with a mutex.
 + *
 + *    LOCKING:
 + *    Kernel thread context (may sleep).
 + */
 +void ata_scsi_hotplug(void *data)
 +{
 +      struct ata_port *ap = data;
 +      int i;
 +
 +      if (ap->flags & ATA_FLAG_UNLOADING) {
 +              DPRINTK("ENTER/EXIT - unloading\n");
 +              return;
        }
 +
 +      DPRINTK("ENTER\n");
 +
 +      /* unplug detached devices */
 +      for (i = 0; i < ATA_MAX_DEVICES; i++) {
 +              struct ata_device *dev = &ap->device[i];
 +              unsigned long flags;
 +
 +              if (!(dev->flags & ATA_DFLAG_DETACHED))
 +                      continue;
 +
 +              spin_lock_irqsave(&ap->host_set->lock, flags);
 +              dev->flags &= ~ATA_DFLAG_DETACHED;
 +              spin_unlock_irqrestore(&ap->host_set->lock, flags);
 +
 +              ata_scsi_remove_dev(dev);
 +      }
 +
 +      /* scan for new ones */
 +      ata_scsi_scan_host(ap);
 +
 +      /* If we scanned while EH was in progress, scan would have
 +       * failed silently.  Requeue if there are enabled but
 +       * unattached devices.
 +       */
 +      for (i = 0; i < ATA_MAX_DEVICES; i++) {
 +              struct ata_device *dev = &ap->device[i];
 +              if (ata_dev_enabled(dev) && !dev->sdev) {
 +                      queue_delayed_work(ata_aux_wq, &ap->hotplug_task, HZ);
 +                      break;
 +              }
 +      }
 +
 +      DPRINTK("EXIT\n");
  }
  
 +/**
 + *    ata_scsi_user_scan - indication for user-initiated bus scan
 + *    @shost: SCSI host to scan
 + *    @channel: Channel to scan
 + *    @id: ID to scan
 + *    @lun: LUN to scan
 + *
 + *    This function is called when user explicitly requests bus
 + *    scan.  Set probe pending flag and invoke EH.
 + *
 + *    LOCKING:
 + *    SCSI layer (we don't care)
 + *
 + *    RETURNS:
 + *    Zero.
 + */
 +static int ata_scsi_user_scan(struct Scsi_Host *shost, unsigned int channel,
 +                            unsigned int id, unsigned int lun)
 +{
 +      struct ata_port *ap = ata_shost_to_port(shost);
 +      unsigned long flags;
 +      int rc = 0;
 +
 +      if (!ap->ops->error_handler)
 +              return -EOPNOTSUPP;
 +
 +      if ((channel != SCAN_WILD_CARD && channel != 0) ||
 +          (lun != SCAN_WILD_CARD && lun != 0))
 +              return -EINVAL;
 +
 +      spin_lock_irqsave(&ap->host_set->lock, flags);
 +
 +      if (id == SCAN_WILD_CARD) {
 +              ap->eh_info.probe_mask |= (1 << ATA_MAX_DEVICES) - 1;
 +              ap->eh_info.action |= ATA_EH_SOFTRESET;
 +      } else {
 +              struct ata_device *dev = ata_find_dev(ap, id);
 +
 +              if (dev) {
 +                      ap->eh_info.probe_mask |= 1 << dev->devno;
 +                      ap->eh_info.action |= ATA_EH_SOFTRESET;
 +              } else
 +                      rc = -EINVAL;
 +      }
 +
 +      if (rc == 0)
 +              ata_port_schedule_eh(ap);
 +
 +      spin_unlock_irqrestore(&ap->host_set->lock, flags);
 +
 +      return rc;
 +}
 +
 +/**
 + *    ata_scsi_dev_rescan - initiate scsi_rescan_device()
 + *    @data: Pointer to ATA port to perform scsi_rescan_device()
 + *
 + *    After ATA pass thru (SAT) commands are executed successfully,
 + *    libata need to propagate the changes to SCSI layer.  This
 + *    function must be executed from ata_aux_wq such that sdev
 + *    attach/detach don't race with rescan.
 + *
 + *    LOCKING:
 + *    Kernel thread context (may sleep).
 + */
 +void ata_scsi_dev_rescan(void *data)
 +{
 +      struct ata_port *ap = data;
 +      struct ata_device *dev;
 +      unsigned int i;
 +
 +      for (i = 0; i < ATA_MAX_DEVICES; i++) {
 +              dev = &ap->device[i];
 +
 +              if (ata_dev_enabled(dev) && dev->sdev)
 +                      scsi_rescan_device(&(dev->sdev->sdev_gendev));
 +      }
 +}
diff --combined drivers/scsi/sata_vsc.c
@@@ -221,21 -221,14 +221,21 @@@ static irqreturn_t vsc_sata_interrupt (
  
                        ap = host_set->ports[i];
  
 -                      if (ap && !(ap->flags &
 -                                  (ATA_FLAG_PORT_DISABLED|ATA_FLAG_NOINTR))) {
 +                      if (is_vsc_sata_int_err(i, int_status)) {
 +                              u32 err_status;
 +                              printk(KERN_DEBUG "%s: ignoring interrupt(s)\n", __FUNCTION__);
 +                              err_status = ap ? vsc_sata_scr_read(ap, SCR_ERROR) : 0;
 +                              vsc_sata_scr_write(ap, SCR_ERROR, err_status);
 +                              handled++;
 +                      }
 +
 +                      if (ap && !(ap->flags & ATA_FLAG_DISABLED)) {
                                struct ata_queued_cmd *qc;
  
                                qc = ata_qc_from_tag(ap, ap->active_tag);
 -                              if (qc && (!(qc->tf.ctl & ATA_NIEN))) {
 +                              if (qc && (!(qc->tf.flags & ATA_TFLAG_POLLING)))
                                        handled += ata_host_intr(ap, qc);
 -                              else if (is_vsc_sata_int_err(i, int_status)) {
 +                              else if (is_vsc_sata_int_err(i, int_status)) {
                                        /*
                                         * On some chips (i.e. Intel 31244), an error
                                         * interrupt will sneak in at initialization
@@@ -279,7 -272,6 +279,7 @@@ static struct scsi_host_template vsc_sa
        .proc_name              = DRV_NAME,
        .dma_boundary           = ATA_DMA_BOUNDARY,
        .slave_configure        = ata_scsi_slave_config,
 +      .slave_destroy          = ata_scsi_slave_destroy,
        .bios_param             = ata_std_bios_param,
  };
  
@@@ -291,17 -283,14 +291,17 @@@ static const struct ata_port_operation
        .exec_command           = ata_exec_command,
        .check_status           = ata_check_status,
        .dev_select             = ata_std_dev_select,
 -      .phy_reset              = sata_phy_reset,
        .bmdma_setup            = ata_bmdma_setup,
        .bmdma_start            = ata_bmdma_start,
        .bmdma_stop             = ata_bmdma_stop,
        .bmdma_status           = ata_bmdma_status,
        .qc_prep                = ata_qc_prep,
        .qc_issue               = ata_qc_issue_prot,
 -      .eng_timeout            = ata_eng_timeout,
 +      .data_xfer              = ata_pio_data_xfer,
 +      .freeze                 = ata_bmdma_freeze,
 +      .thaw                   = ata_bmdma_thaw,
 +      .error_handler          = ata_bmdma_error_handler,
 +      .post_internal_cmd      = ata_bmdma_post_internal_cmd,
        .irq_handler            = vsc_sata_interrupt,
        .irq_clear              = ata_bmdma_irq_clear,
        .scr_read               = vsc_sata_scr_read,
@@@ -396,7 -385,7 +396,7 @@@ static int __devinit vsc_sata_init_one 
  
        probe_ent->sht = &vsc_sata_sht;
        probe_ent->host_flags = ATA_FLAG_SATA | ATA_FLAG_NO_LEGACY |
 -                              ATA_FLAG_MMIO | ATA_FLAG_SATA_RESET;
 +                              ATA_FLAG_MMIO;
        probe_ent->port_ops = &vsc_sata_ops;
        probe_ent->n_ports = 4;
        probe_ent->irq = pdev->irq;
@@@ -444,13 -433,14 +444,14 @@@ err_out
  
  
  /*
-  * 0x1725/0x7174 is the Vitesse VSC-7174
-  * 0x8086/0x3200 is the Intel 31244, which is supposed to be identical
-  * compatibility is untested as of yet
+  * Intel 31244 is supposed to be identical.
+  * Compatibility is untested as of yet.
   */
  static const struct pci_device_id vsc_sata_pci_tbl[] = {
-       { 0x1725, 0x7174, PCI_ANY_ID, PCI_ANY_ID, 0x10600, 0xFFFFFF, 0 },
-       { 0x8086, 0x3200, PCI_ANY_ID, PCI_ANY_ID, 0x10600, 0xFFFFFF, 0 },
+       { PCI_VENDOR_ID_VITESSE, PCI_DEVICE_ID_VITESSE_VSC7174,
+         PCI_ANY_ID, PCI_ANY_ID, 0x10600, 0xFFFFFF, 0 },
+       { PCI_VENDOR_ID_INTEL, PCI_DEVICE_ID_INTEL_GD31244,
+         PCI_ANY_ID, PCI_ANY_ID, 0x10600, 0xFFFFFF, 0 },
        { }
  };
  
diff --combined drivers/scsi/scsi.c
@@@ -63,7 -63,6 +63,6 @@@
  #include <scsi/scsi_eh.h>
  #include <scsi/scsi_host.h>
  #include <scsi/scsi_tcq.h>
- #include <scsi/scsi_request.h>
  
  #include "scsi_priv.h"
  #include "scsi_logging.h"
@@@ -116,79 -115,6 +115,6 @@@ const char *const scsi_device_types[MAX
  };
  EXPORT_SYMBOL(scsi_device_types);
  
- /*
-  * Function:    scsi_allocate_request
-  *
-  * Purpose:     Allocate a request descriptor.
-  *
-  * Arguments:   device                - device for which we want a request
-  *            gfp_mask        - allocation flags passed to kmalloc
-  *
-  * Lock status: No locks assumed to be held.  This function is SMP-safe.
-  *
-  * Returns:     Pointer to request block.
-  */
- struct scsi_request *scsi_allocate_request(struct scsi_device *sdev,
-                                          gfp_t gfp_mask)
- {
-       const int offset = ALIGN(sizeof(struct scsi_request), 4);
-       const int size = offset + sizeof(struct request);
-       struct scsi_request *sreq;
-   
-       sreq = kzalloc(size, gfp_mask);
-       if (likely(sreq != NULL)) {
-               sreq->sr_request = (struct request *)(((char *)sreq) + offset);
-               sreq->sr_device = sdev;
-               sreq->sr_host = sdev->host;
-               sreq->sr_magic = SCSI_REQ_MAGIC;
-               sreq->sr_data_direction = DMA_BIDIRECTIONAL;
-       }
-       return sreq;
- }
- EXPORT_SYMBOL(scsi_allocate_request);
- void __scsi_release_request(struct scsi_request *sreq)
- {
-       struct request *req = sreq->sr_request;
-       /* unlikely because the tag was usually ended earlier by the
-        * mid-layer. However, for layering reasons ULD's don't end
-        * the tag of commands they generate. */
-       if (unlikely(blk_rq_tagged(req))) {
-               unsigned long flags;
-               struct request_queue *q = req->q;
-               spin_lock_irqsave(q->queue_lock, flags);
-               blk_queue_end_tag(q, req);
-               spin_unlock_irqrestore(q->queue_lock, flags);
-       }
-       if (likely(sreq->sr_command != NULL)) {
-               struct scsi_cmnd *cmd = sreq->sr_command;
-               sreq->sr_command = NULL;
-               scsi_next_command(cmd);
-       }
- }
- /*
-  * Function:    scsi_release_request
-  *
-  * Purpose:     Release a request descriptor.
-  *
-  * Arguments:   sreq    - request to release
-  *
-  * Lock status: No locks assumed to be held.  This function is SMP-safe.
-  */
- void scsi_release_request(struct scsi_request *sreq)
- {
-       __scsi_release_request(sreq);
-       kfree(sreq);
- }
- EXPORT_SYMBOL(scsi_release_request);
  struct scsi_host_cmd_pool {
        kmem_cache_t    *slab;
        unsigned int    users;
@@@ -646,97 -572,12 +572,30 @@@ int scsi_dispatch_cmd(struct scsi_cmnd 
        return rtn;
  }
  
- /*
-  * Function:    scsi_init_cmd_from_req
-  *
-  * Purpose:     Queue a SCSI command
-  * Purpose:     Initialize a struct scsi_cmnd from a struct scsi_request
-  *
-  * Arguments:   cmd       - command descriptor.
-  *              sreq      - Request from the queue.
-  *
-  * Lock status: None needed.
-  *
-  * Returns:     Nothing.
-  *
-  * Notes:       Mainly transfer data from the request structure to the
-  *              command structure.  The request structure is allocated
-  *              using the normal memory allocator, and requests can pile
-  *              up to more or less any depth.  The command structure represents
-  *              a consumable resource, as these are allocated into a pool
-  *              when the SCSI subsystem initializes.  The preallocation is
-  *              required so that in low-memory situations a disk I/O request
-  *              won't cause the memory manager to try and write out a page.
-  *              The request structure is generally used by ioctls and character
-  *              devices.
-  */
- void scsi_init_cmd_from_req(struct scsi_cmnd *cmd, struct scsi_request *sreq)
- {
-       sreq->sr_command = cmd;
-       cmd->cmd_len = sreq->sr_cmd_len;
-       cmd->use_sg = sreq->sr_use_sg;
-       cmd->request = sreq->sr_request;
-       memcpy(cmd->data_cmnd, sreq->sr_cmnd, sizeof(cmd->data_cmnd));
-       cmd->serial_number = 0;
-       cmd->bufflen = sreq->sr_bufflen;
-       cmd->buffer = sreq->sr_buffer;
-       cmd->retries = 0;
-       cmd->allowed = sreq->sr_allowed;
-       cmd->done = sreq->sr_done;
-       cmd->timeout_per_command = sreq->sr_timeout_per_command;
-       cmd->sc_data_direction = sreq->sr_data_direction;
-       cmd->sglist_len = sreq->sr_sglist_len;
-       cmd->underflow = sreq->sr_underflow;
-       cmd->sc_request = sreq;
-       memcpy(cmd->cmnd, sreq->sr_cmnd, sizeof(sreq->sr_cmnd));
-       /*
-        * Zero the sense buffer.  Some host adapters automatically request
-        * sense on error.  0 is not a valid sense code.
-        */
-       memset(cmd->sense_buffer, 0, sizeof(sreq->sr_sense_buffer));
-       cmd->request_buffer = sreq->sr_buffer;
-       cmd->request_bufflen = sreq->sr_bufflen;
-       cmd->old_use_sg = cmd->use_sg;
-       if (cmd->cmd_len == 0)
-               cmd->cmd_len = COMMAND_SIZE(cmd->cmnd[0]);
-       cmd->old_cmd_len = cmd->cmd_len;
-       cmd->sc_old_data_direction = cmd->sc_data_direction;
-       cmd->old_underflow = cmd->underflow;
-       /*
-        * Start the timer ticking.
-        */
-       cmd->result = 0;
-       SCSI_LOG_MLQUEUE(3, printk("Leaving scsi_init_cmd_from_req()\n"));
- }
  
  /*
   * Per-CPU I/O completion queue.
   */
  static DEFINE_PER_CPU(struct list_head, scsi_done_q);
  
 +/**
 + * scsi_req_abort_cmd -- Request command recovery for the specified command
 + * cmd: pointer to the SCSI command of interest
 + *
 + * This function requests that SCSI Core start recovery for the
 + * command by deleting the timer and adding the command to the eh
 + * queue.  It can be called by either LLDDs or SCSI Core.  LLDDs who
 + * implement their own error recovery MAY ignore the timeout event if
 + * they generated scsi_req_abort_cmd.
 + */
 +void scsi_req_abort_cmd(struct scsi_cmnd *cmd)
 +{
 +      if (!scsi_delete_timer(cmd))
 +              return;
 +      scsi_times_out(cmd);
 +}
 +EXPORT_SYMBOL(scsi_req_abort_cmd);
 +
  /**
   * scsi_done - Enqueue the finished SCSI command into the done queue.
   * @cmd: The SCSI Command for which a low-level device driver (LLDD) gives
@@@ -827,7 -668,6 +686,6 @@@ void scsi_finish_command(struct scsi_cm
  {
        struct scsi_device *sdev = cmd->device;
        struct Scsi_Host *shost = sdev->host;
-       struct scsi_request *sreq;
  
        scsi_device_unbusy(sdev);
  
         * We can get here with use_sg=0, causing a panic in the upper level
         */
        cmd->use_sg = cmd->old_use_sg;
-       /*
-        * If there is an associated request structure, copy the data over
-        * before we call the completion function.
-        */
-       sreq = cmd->sc_request;
-       if (sreq) {
-              sreq->sr_result = sreq->sr_command->result;
-              if (sreq->sr_result) {
-                      memcpy(sreq->sr_sense_buffer,
-                             sreq->sr_command->sense_buffer,
-                             sizeof(sreq->sr_sense_buffer));
-              }
-       }
        cmd->done(cmd);
  }
  EXPORT_SYMBOL(scsi_finish_command);
  #include <linux/delay.h>
  
  #include <scsi/scsi.h>
+ #include <scsi/scsi_cmnd.h>
  #include <scsi/scsi_dbg.h>
  #include <scsi/scsi_device.h>
  #include <scsi/scsi_eh.h>
  #include <scsi/scsi_transport.h>
  #include <scsi/scsi_host.h>
  #include <scsi/scsi_ioctl.h>
- #include <scsi/scsi_request.h>
  
  #include "scsi_priv.h"
  #include "scsi_logging.h"
@@@ -57,28 -57,6 +57,28 @@@ void scsi_eh_wakeup(struct Scsi_Host *s
        }
  }
  
 +/**
 + * scsi_schedule_eh - schedule EH for SCSI host
 + * @shost:    SCSI host to invoke error handling on.
 + *
 + * Schedule SCSI EH without scmd.
 + **/
 +void scsi_schedule_eh(struct Scsi_Host *shost)
 +{
 +      unsigned long flags;
 +
 +      spin_lock_irqsave(shost->host_lock, flags);
 +
 +      if (scsi_host_set_state(shost, SHOST_RECOVERY) == 0 ||
 +          scsi_host_set_state(shost, SHOST_CANCEL_RECOVERY) == 0) {
 +              shost->host_eh_scheduled++;
 +              scsi_eh_wakeup(shost);
 +      }
 +
 +      spin_unlock_irqrestore(shost->host_lock, flags);
 +}
 +EXPORT_SYMBOL_GPL(scsi_schedule_eh);
 +
  /**
   * scsi_eh_scmd_add - add scsi cmd to error handling.
   * @scmd:     scmd to run eh on.
@@@ -474,7 -452,6 +474,6 @@@ static int scsi_send_eh_cmnd(struct scs
                        (sdev->lun << 5 & 0xe0);
  
        shost->eh_action = &done;
-       scmd->request->rq_status = RQ_SCSI_BUSY;
  
        spin_lock_irqsave(shost->host_lock, flags);
        scsi_log_send(scmd);
  
        timeleft = wait_for_completion_timeout(&done, timeout);
  
-       scmd->request->rq_status = RQ_SCSI_DONE;
        shost->eh_action = NULL;
  
        scsi_log_completion(scmd, SUCCESS);
@@@ -1539,7 -1515,7 +1537,7 @@@ int scsi_error_handler(void *data
         */
        set_current_state(TASK_INTERRUPTIBLE);
        while (!kthread_should_stop()) {
 -              if (shost->host_failed == 0 ||
 +              if ((shost->host_failed == 0 && shost->host_eh_scheduled == 0) ||
                    shost->host_failed != shost->host_busy) {
                        SCSI_LOG_ERROR_RECOVERY(1,
                                printk("Error handler scsi_eh_%d sleeping\n",
@@@ -1679,7 -1655,6 +1677,6 @@@ scsi_reset_provider(struct scsi_device 
  
        scmd->request = &req;
        memset(&scmd->eh_timeout, 0, sizeof(scmd->eh_timeout));
-       scmd->request->rq_status        = RQ_SCSI_BUSY;
  
        memset(&scmd->cmnd, '\0', sizeof(scmd->cmnd));
      
        scmd->cmd_len                   = 0;
  
        scmd->sc_data_direction         = DMA_BIDIRECTIONAL;
-       scmd->sc_request                = NULL;
-       scmd->sc_magic                  = SCSI_CMND_MAGIC;
  
        init_timer(&scmd->eh_timeout);
  
@@@ -1791,14 -1764,6 +1786,6 @@@ int scsi_normalize_sense(const u8 *sens
  }
  EXPORT_SYMBOL(scsi_normalize_sense);
  
- int scsi_request_normalize_sense(struct scsi_request *sreq,
-                                struct scsi_sense_hdr *sshdr)
- {
-       return scsi_normalize_sense(sreq->sr_sense_buffer,
-                       sizeof(sreq->sr_sense_buffer), sshdr);
- }
- EXPORT_SYMBOL(scsi_request_normalize_sense);
  int scsi_command_normalize_sense(struct scsi_cmnd *cmd,
                                 struct scsi_sense_hdr *sshdr)
  {
diff --combined drivers/scsi/scsi_lib.c
  #include <linux/hardirq.h>
  
  #include <scsi/scsi.h>
+ #include <scsi/scsi_cmnd.h>
  #include <scsi/scsi_dbg.h>
  #include <scsi/scsi_device.h>
  #include <scsi/scsi_driver.h>
  #include <scsi/scsi_eh.h>
  #include <scsi/scsi_host.h>
- #include <scsi/scsi_request.h>
  
  #include "scsi_priv.h"
  #include "scsi_logging.h"
  
  
- #define SG_MEMPOOL_NR         (sizeof(scsi_sg_pools)/sizeof(struct scsi_host_sg_pool))
+ #define SG_MEMPOOL_NR         ARRAY_SIZE(scsi_sg_pools)
  #define SG_MEMPOOL_SIZE               32
  
  struct scsi_host_sg_pool {
@@@ -83,7 -83,7 +83,7 @@@ static void scsi_unprep_request(struct 
        struct scsi_cmnd *cmd = req->special;
  
        req->flags &= ~REQ_DONTPREP;
-       req->special = (req->flags & REQ_SPECIAL) ? cmd->sc_request : NULL;
+       req->special = NULL;
  
        scsi_put_command(cmd);
  }
@@@ -161,72 -161,6 +161,6 @@@ int scsi_queue_insert(struct scsi_cmnd 
        return 0;
  }
  
- /*
-  * Function:    scsi_do_req
-  *
-  * Purpose:     Queue a SCSI request
-  *
-  * Arguments:   sreq    - command descriptor.
-  *              cmnd      - actual SCSI command to be performed.
-  *              buffer    - data buffer.
-  *              bufflen   - size of data buffer.
-  *              done      - completion function to be run.
-  *              timeout   - how long to let it run before timeout.
-  *              retries   - number of retries we allow.
-  *
-  * Lock status: No locks held upon entry.
-  *
-  * Returns:     Nothing.
-  *
-  * Notes:     This function is only used for queueing requests for things
-  *            like ioctls and character device requests - this is because
-  *            we essentially just inject a request into the queue for the
-  *            device.
-  *
-  *            In order to support the scsi_device_quiesce function, we
-  *            now inject requests on the *head* of the device queue
-  *            rather than the tail.
-  */
- void scsi_do_req(struct scsi_request *sreq, const void *cmnd,
-                void *buffer, unsigned bufflen,
-                void (*done)(struct scsi_cmnd *),
-                int timeout, int retries)
- {
-       /*
-        * If the upper level driver is reusing these things, then
-        * we should release the low-level block now.  Another one will
-        * be allocated later when this request is getting queued.
-        */
-       __scsi_release_request(sreq);
-       /*
-        * Our own function scsi_done (which marks the host as not busy,
-        * disables the timeout counter, etc) will be called by us or by the
-        * scsi_hosts[host].queuecommand() function needs to also call
-        * the completion function for the high level driver.
-        */
-       memcpy(sreq->sr_cmnd, cmnd, sizeof(sreq->sr_cmnd));
-       sreq->sr_bufflen = bufflen;
-       sreq->sr_buffer = buffer;
-       sreq->sr_allowed = retries;
-       sreq->sr_done = done;
-       sreq->sr_timeout_per_command = timeout;
-       if (sreq->sr_cmd_len == 0)
-               sreq->sr_cmd_len = COMMAND_SIZE(sreq->sr_cmnd[0]);
-       /*
-        * head injection *required* here otherwise quiesce won't work
-        *
-        * Because users of this function are apt to reuse requests with no
-        * modification, we have to sanitise the request flags here
-        */
-       sreq->sr_request->flags &= ~REQ_DONTPREP;
-       blk_insert_request(sreq->sr_device->request_queue, sreq->sr_request,
-                          1, sreq);
- }
- EXPORT_SYMBOL(scsi_do_req);
  /**
   * scsi_execute - insert request and wait for the result
   * @sdev:     scsi device
@@@ -566,7 -500,7 +500,7 @@@ void scsi_device_unbusy(struct scsi_dev
        spin_lock_irqsave(shost->host_lock, flags);
        shost->host_busy--;
        if (unlikely(scsi_host_in_recovery(shost) &&
 -                   shost->host_failed))
 +                   (shost->host_failed || shost->host_eh_scheduled)))
                scsi_eh_wakeup(shost);
        spin_unlock(shost->host_lock);
        spin_lock(sdev->request_queue->queue_lock);
@@@ -1300,15 -1234,7 +1234,7 @@@ static int scsi_prep_fn(struct request_
         * at request->cmd, as this tells us the real story.
         */
        if (req->flags & REQ_SPECIAL && req->special) {
-               struct scsi_request *sreq = req->special;
-               if (sreq->sr_magic == SCSI_REQ_MAGIC) {
-                       cmd = scsi_get_command(sreq->sr_device, GFP_ATOMIC);
-                       if (unlikely(!cmd))
-                               goto defer;
-                       scsi_init_cmd_from_req(cmd, sreq);
-               } else
-                       cmd = req->special;
+               cmd = req->special;
        } else if (req->flags & (REQ_CMD | REQ_BLOCK_PC)) {
  
                if(unlikely(specials_only) && !(req->flags & REQ_SPECIAL)) {
@@@ -2363,3 -2289,61 +2289,61 @@@ scsi_target_unblock(struct device *dev
                device_for_each_child(dev, NULL, target_unblock);
  }
  EXPORT_SYMBOL_GPL(scsi_target_unblock);
+ /**
+  * scsi_kmap_atomic_sg - find and atomically map an sg-elemnt
+  * @sg:               scatter-gather list
+  * @sg_count: number of segments in sg
+  * @offset:   offset in bytes into sg, on return offset into the mapped area
+  * @len:      bytes to map, on return number of bytes mapped
+  *
+  * Returns virtual address of the start of the mapped page
+  */
+ void *scsi_kmap_atomic_sg(struct scatterlist *sg, int sg_count,
+                         size_t *offset, size_t *len)
+ {
+       int i;
+       size_t sg_len = 0, len_complete = 0;
+       struct page *page;
+       for (i = 0; i < sg_count; i++) {
+               len_complete = sg_len; /* Complete sg-entries */
+               sg_len += sg[i].length;
+               if (sg_len > *offset)
+                       break;
+       }
+       if (unlikely(i == sg_count)) {
+               printk(KERN_ERR "%s: Bytes in sg: %zu, requested offset %zu, "
+                       "elements %d\n",
+                      __FUNCTION__, sg_len, *offset, sg_count);
+               WARN_ON(1);
+               return NULL;
+       }
+       /* Offset starting from the beginning of first page in this sg-entry */
+       *offset = *offset - len_complete + sg[i].offset;
+       /* Assumption: contiguous pages can be accessed as "page + i" */
+       page = nth_page(sg[i].page, (*offset >> PAGE_SHIFT));
+       *offset &= ~PAGE_MASK;
+       /* Bytes in this sg-entry from *offset to the end of the page */
+       sg_len = PAGE_SIZE - *offset;
+       if (*len > sg_len)
+               *len = sg_len;
+       return kmap_atomic(page, KM_BIO_SRC_IRQ);
+ }
+ EXPORT_SYMBOL(scsi_kmap_atomic_sg);
+ /**
+  * scsi_kunmap_atomic_sg - atomically unmap a virtual address, previously
+  *                       mapped with scsi_kmap_atomic_sg
+  * @virt:     virtual address to be unmapped
+  */
+ void scsi_kunmap_atomic_sg(void *virt)
+ {
+       kunmap_atomic(virt, KM_BIO_SRC_IRQ);
+ }
+ EXPORT_SYMBOL(scsi_kunmap_atomic_sg);
diff --combined include/linux/pci_ids.h
  #define PCI_DEVICE_ID_ATI_RS480         0x5950
  /* ATI IXP Chipset */
  #define PCI_DEVICE_ID_ATI_IXP200_IDE  0x4349
+ #define PCI_DEVICE_ID_ATI_IXP200_SMBUS        0x4353
+ #define PCI_DEVICE_ID_ATI_IXP300_SMBUS        0x4363
  #define PCI_DEVICE_ID_ATI_IXP300_IDE  0x4369
  #define PCI_DEVICE_ID_ATI_IXP300_SATA   0x436e
+ #define PCI_DEVICE_ID_ATI_IXP400_SMBUS        0x4372
  #define PCI_DEVICE_ID_ATI_IXP400_IDE  0x4376
  #define PCI_DEVICE_ID_ATI_IXP400_SATA   0x4379
  #define PCI_DEVICE_ID_ATI_IXP400_SATA2        0x437a
  
  
  #define PCI_VENDOR_ID_QLOGIC          0x1077
+ #define PCI_DEVICE_ID_QLOGIC_ISP10160 0x1016
  #define PCI_DEVICE_ID_QLOGIC_ISP1020  0x1020
+ #define PCI_DEVICE_ID_QLOGIC_ISP1080  0x1080
+ #define PCI_DEVICE_ID_QLOGIC_ISP12160 0x1216
+ #define PCI_DEVICE_ID_QLOGIC_ISP1240  0x1240
+ #define PCI_DEVICE_ID_QLOGIC_ISP1280  0x1280
  #define PCI_DEVICE_ID_QLOGIC_ISP2100  0x2100
  #define PCI_DEVICE_ID_QLOGIC_ISP2200  0x2200
  #define PCI_DEVICE_ID_QLOGIC_ISP2300  0x2300
  #define PCI_DEVICE_ID_PLX_DJINN_ITOO  0x1151
  #define PCI_DEVICE_ID_PLX_R753                0x1152
  #define PCI_DEVICE_ID_PLX_OLITEC      0x1187
+ #define PCI_DEVICE_ID_PLX_PCI200SYN   0x3196
  #define PCI_DEVICE_ID_PLX_9050                0x9050
  #define PCI_DEVICE_ID_PLX_9080                0x9080
  #define PCI_DEVICE_ID_PLX_GTEK_SERIAL2        0xa001
  #define PCI_DEVICE_ID_NVIDIA_NVENET_8         0x0056
  #define PCI_DEVICE_ID_NVIDIA_NVENET_9         0x0057
  #define PCI_DEVICE_ID_NVIDIA_CK804_AUDIO      0x0059
+ #define PCI_DEVICE_ID_NVIDIA_CK804_PCIE               0x005d
  #define PCI_DEVICE_ID_NVIDIA_NFORCE2_SMBUS    0x0064
  #define PCI_DEVICE_ID_NVIDIA_NFORCE2_IDE      0x0065
  #define PCI_DEVICE_ID_NVIDIA_NVENET_2         0x0066
  #define PCI_DEVICE_ID_NVIDIA_QUADRO4_900XGL   0x0258
  #define PCI_DEVICE_ID_NVIDIA_QUADRO4_750XGL   0x0259
  #define PCI_DEVICE_ID_NVIDIA_QUADRO4_700XGL   0x025B
+ #define PCI_DEVICE_ID_NVIDIA_NFORCE_MCP51_SMBUS       0x0264
  #define PCI_DEVICE_ID_NVIDIA_NFORCE_MCP51_IDE 0x0265
  #define PCI_DEVICE_ID_NVIDIA_NFORCE_MCP51_SATA        0x0266
  #define PCI_DEVICE_ID_NVIDIA_NFORCE_MCP51_SATA2       0x0267
+ #define PCI_DEVICE_ID_NVIDIA_NFORCE_MCP55_SMBUS       0x0368
  #define PCI_DEVICE_ID_NVIDIA_NFORCE_MCP55_IDE 0x036E
  #define PCI_DEVICE_ID_NVIDIA_NFORCE_MCP55_SATA        0x037E
  #define PCI_DEVICE_ID_NVIDIA_NFORCE_MCP55_SATA2       0x037F
  #define PCI_DEVICE_ID_NVIDIA_QUADRO_FX_1100         0x034E
  #define PCI_DEVICE_ID_NVIDIA_NVENET_14              0x0372
  #define PCI_DEVICE_ID_NVIDIA_NVENET_15              0x0373
+ #define PCI_DEVICE_ID_NVIDIA_NVENET_16              0x03E5
+ #define PCI_DEVICE_ID_NVIDIA_NVENET_17              0x03E6
 +#define PCI_DEVICE_ID_NVIDIA_NFORCE_MCP61_SATA      0x03E7
 +#define PCI_DEVICE_ID_NVIDIA_NFORCE_MCP61_IDE       0x03EC
+ #define PCI_DEVICE_ID_NVIDIA_NVENET_18              0x03EE
+ #define PCI_DEVICE_ID_NVIDIA_NVENET_19              0x03EF
 +#define PCI_DEVICE_ID_NVIDIA_NFORCE_MCP61_SATA2     0x03F6
 +#define PCI_DEVICE_ID_NVIDIA_NFORCE_MCP61_SATA3     0x03F7
+ #define PCI_DEVICE_ID_NVIDIA_NVENET_20              0x0450
+ #define PCI_DEVICE_ID_NVIDIA_NVENET_21              0x0451
+ #define PCI_DEVICE_ID_NVIDIA_NVENET_22              0x0452
+ #define PCI_DEVICE_ID_NVIDIA_NVENET_23              0x0453
  
  #define PCI_VENDOR_ID_IMS             0x10e0
  #define PCI_DEVICE_ID_IMS_TT128               0x9128
  #define PCI_DEVICE_ID_VIA_PX8X0_0     0x0259
  #define PCI_DEVICE_ID_VIA_3269_0      0x0269
  #define PCI_DEVICE_ID_VIA_K8T800PRO_0 0x0282
 +#define PCI_DEVICE_ID_VIA_3296_0      0x0296
  #define PCI_DEVICE_ID_VIA_8363_0      0x0305
  #define PCI_DEVICE_ID_VIA_P4M800CE    0x0314
  #define PCI_DEVICE_ID_VIA_8371_0      0x0391
  #define PCI_DEVICE_ID_VIA_82C561      0x0561
  #define PCI_DEVICE_ID_VIA_82C586_1    0x0571
  #define PCI_DEVICE_ID_VIA_82C576      0x0576
 +#define PCI_DEVICE_ID_VIA_SATA_EIDE   0x0581
  #define PCI_DEVICE_ID_VIA_82C586_0    0x0586
  #define PCI_DEVICE_ID_VIA_82C596      0x0596
  #define PCI_DEVICE_ID_VIA_82C597_0    0x0597
  #define PCI_DEVICE_ID_VIA_8783_0      0x3208
  #define PCI_DEVICE_ID_VIA_8237                0x3227
  #define PCI_DEVICE_ID_VIA_8251                0x3287
 -#define PCI_DEVICE_ID_VIA_3296_0      0x0296
 +#define PCI_DEVICE_ID_VIA_8237A               0x3337
  #define PCI_DEVICE_ID_VIA_8231                0x8231
  #define PCI_DEVICE_ID_VIA_8231_4      0x8235
  #define PCI_DEVICE_ID_VIA_8365_1      0x8305
 +#define PCI_DEVICE_ID_VIA_CX700               0x8324
  #define PCI_DEVICE_ID_VIA_8371_1      0x8391
  #define PCI_DEVICE_ID_VIA_82C598_1    0x8598
  #define PCI_DEVICE_ID_VIA_838X_1      0xB188
  
  #define PCI_VENDOR_ID_SAMSUNG         0x144d
  
+ #define PCI_VENDOR_ID_MYRICOM         0x14c1
  
  #define PCI_VENDOR_ID_TITAN           0x14D2
  #define PCI_DEVICE_ID_TITAN_010L      0x8001
  #define PCI_DEVICE_ID_TIGON3_5751F    0x167e
  #define PCI_DEVICE_ID_TIGON3_5787M    0x1693
  #define PCI_DEVICE_ID_TIGON3_5782     0x1696
+ #define PCI_DEVICE_ID_TIGON3_5786     0x169a
  #define PCI_DEVICE_ID_TIGON3_5787     0x169b
  #define PCI_DEVICE_ID_TIGON3_5788     0x169c
  #define PCI_DEVICE_ID_TIGON3_5789     0x169d
  
  #define PCI_VENDOR_ID_MELLANOX                0x15b3
  #define PCI_DEVICE_ID_MELLANOX_TAVOR  0x5a44
+ #define PCI_DEVICE_ID_MELLANOX_TAVOR_BRIDGE   0x5a46
  #define PCI_DEVICE_ID_MELLANOX_ARBEL_COMPAT 0x6278
  #define PCI_DEVICE_ID_MELLANOX_ARBEL  0x6282
  #define PCI_DEVICE_ID_MELLANOX_SINAI_OLD 0x5e8c
  #define PCI_VENDOR_ID_NETCELL         0x169c
  #define PCI_DEVICE_ID_REVOLUTION      0x0044
  
+ #define PCI_VENDOR_ID_VITESSE         0x1725
+ #define PCI_DEVICE_ID_VITESSE_VSC7174 0x7174
  #define PCI_VENDOR_ID_LINKSYS         0x1737
  #define PCI_DEVICE_ID_LINKSYS_EG1064  0x1064
  
  #define PCI_DEVICE_ID_INTEL_80960_RP  0x1960
  #define PCI_DEVICE_ID_INTEL_82840_HB  0x1a21
  #define PCI_DEVICE_ID_INTEL_82845_HB  0x1a30
+ #define PCI_DEVICE_ID_INTEL_IOAT      0x1a38
  #define PCI_DEVICE_ID_INTEL_82801AA_0 0x2410
  #define PCI_DEVICE_ID_INTEL_82801AA_1 0x2411
  #define PCI_DEVICE_ID_INTEL_82801AA_3 0x2413
  #define PCI_DEVICE_ID_INTEL_ICH8_4    0x2815
  #define PCI_DEVICE_ID_INTEL_ICH8_5    0x283e
  #define PCI_DEVICE_ID_INTEL_ICH8_6    0x2850
+ #define PCI_DEVICE_ID_INTEL_GD31244   0x3200
  #define PCI_DEVICE_ID_INTEL_82855PM_HB        0x3340
  #define PCI_DEVICE_ID_INTEL_82830_HB  0x3575
  #define PCI_DEVICE_ID_INTEL_82830_CGC 0x3577
diff --combined include/scsi/scsi_cmnd.h
@@@ -9,7 -9,6 +9,6 @@@
  struct request;
  struct scatterlist;
  struct scsi_device;
- struct scsi_request;
  
  
  /* embedded in scsi_cmnd */
@@@ -29,13 -28,8 +28,8 @@@ struct scsi_pointer 
  };
  
  struct scsi_cmnd {
-       int     sc_magic;
        struct scsi_device *device;
-       struct scsi_request *sc_request;
        struct list_head list;  /* scsi_cmnd participates in queue lists */
        struct list_head eh_entry; /* entry for the host eh_cmd_q */
        int eh_eflags;          /* Used by error handlr */
        void (*done) (struct scsi_cmnd *);      /* Mid-level done function */
@@@ -151,6 -145,9 +145,10 @@@ extern struct scsi_cmnd *scsi_get_comma
  extern void scsi_put_command(struct scsi_cmnd *);
  extern void scsi_io_completion(struct scsi_cmnd *, unsigned int, unsigned int);
  extern void scsi_finish_command(struct scsi_cmnd *cmd);
 +extern void scsi_req_abort_cmd(struct scsi_cmnd *cmd);
  
+ extern void *scsi_kmap_atomic_sg(struct scatterlist *sg, int sg_count,
+                                size_t *offset, size_t *len);
+ extern void scsi_kunmap_atomic_sg(void *virt);
  #endif /* _SCSI_SCSI_CMND_H */