Merge branch 'upstream-fixes' into upstream
authorJeff Garzik <jeff@garzik.org>
Sat, 20 May 2006 04:36:08 +0000 (00:36 -0400)
committerJeff Garzik <jeff@garzik.org>
Sat, 20 May 2006 04:36:08 +0000 (00:36 -0400)
Conflicts:

drivers/scsi/libata-core.c

1  2 
drivers/scsi/libata-core.c
drivers/scsi/sata_mv.c

  
  #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);
 +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;
@@@ -73,10 -77,6 +73,10 @@@ 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)");
@@@ -397,22 -397,11 +397,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++;
        }
  }
@@@ -875,6 -864,9 +875,9 @@@ static unsigned int ata_id_xfermask(con
  /**
   *    ata_port_queue_task - Queue port_task
   *    @ap: The ata_port to queue port_task for
+  *    @fn: workqueue function to be scheduled
+  *    @data: data value to pass to workqueue function
+  *    @delay: delay time for workqueue function
   *
   *    Schedule @fn(@data) for execution after @delay jiffies using
   *    port_task.  There is one port_task per port and it's the
@@@ -951,14 -943,15 +954,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
   *    None.  Should be called with kernel context, might sleep.
   */
  
 -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;
  }
  
@@@ -1146,10 -1076,11 +1149,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)
 +static 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)
 +static 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;
        }
@@@ -1450,24 -1352,16 +1453,24 @@@ 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);
  
 +      for (i = 0; i < ATA_MAX_DEVICES; i++)
 +              tries[i] = ATA_PROBE_MAX_TRIES;
 +
 + retry:
 +      down_xfermask = 0;
 +
        /* reset and determine device classes */
        for (i = 0; i < ATA_MAX_DEVICES; i++)
                classes[i] = ATA_DEV_UNKNOWN;
        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);
 +                      ata_port_printk(ap, KERN_ERR,
 +                                      "reset failed (errno=%d)\n", rc);
                        return rc;
                }
        } else {
                ap->ops->phy_reset(ap);
  
 -              if (!(ap->flags & ATA_FLAG_PORT_DISABLED))
 -                      for (i = 0; i < ATA_MAX_DEVICES; i++)
 +              for (i = 0; i < ATA_MAX_DEVICES; i++) {
 +                      if (!(ap->flags & ATA_FLAG_DISABLED))
                                classes[i] = ap->device[i].class;
 +                      ap->device[i].class = ATA_DEV_UNKNOWN;
 +              }
  
                ata_port_probe(ap);
        }
  
        /* 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);
        }
  }
  
@@@ -1618,18 -1497,17 +1621,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;
  }
@@@ -1707,122 -1585,7 +1710,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;
  }
  
  /*
@@@ -1973,196 -1736,151 +1976,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;
 +      for (i = 0; i < ATA_MAX_DEVICES; i++) {
 +              dev = &ap->device[i];
 +              if (!ata_dev_enabled(dev))
 +                      continue;
 +
 +              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 */
 -      ata_host_set_dma(ap);
 +      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++) {
 -              struct ata_device *dev = &ap->device[i];
 +              dev = &ap->device[i];
  
 -              if (!ata_dev_present(dev))
 +              if (!ata_dev_enabled(dev))
                        continue;
  
 -              if (ata_dev_set_mode(ap, dev))
 -                      goto err_out;
 +              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.
 +      /* 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;
  }
  
  /**
@@@ -2212,8 -1930,8 +2215,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;
        }
  
@@@ -2315,10 -2033,8 +2318,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)
  {
        unsigned long timeout = jiffies + (HZ * 5);
 -      u32 sstatus;
 +      u32 scontrol, sstatus;
 +      int rc;
  
 -      scr_write_flush(ap, SCR_CONTROL, 0x300);
 +      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;
  
        /* Wait for phy to become ready, if necessary. */
        do {
                msleep(200);
 -              sstatus = scr_read(ap, SCR_STATUS);
 +              if ((rc = sata_scr_read(ap, SCR_STATUS, &sstatus)))
 +                      return rc;
                if ((sstatus & 0xf) != 1)
                        return 0;
        } while (time_before(jiffies, timeout));
  
 -      return -1;
 +      return -EBUSY;
  }
  
  /**
   */
  void ata_std_probeinit(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);
 +      u32 scontrol;
 +
 +      /* resume link */
 +      sata_phy_resume(ap);
 +
 +      /* init sata_spd_limit to the current value */
 +      if (sata_scr_read(ap, SCR_CONTROL, &scontrol) == 0) {
 +              int spd = (scontrol >> 4) & 0xf;
 +              ap->sata_spd_limit &= (1 << spd) - 1;
        }
 +
 +      /* wait for device */
 +      if (ata_port_online(ap))
 +              ata_busy_sleep(ap, ATA_TMOUT_BOOT_QUICK, ATA_TMOUT_BOOT);
  }
  
  /**
   *    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
   *    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.
   *    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;
 +
 +              scontrol = (scontrol & 0x0f0) | 0x302;
  
 -      /*
 -       * Couldn't find anything in SATA I/II specs, but AHCI-1.1
 +              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 */
 +      /* bring phy back */
        sata_phy_resume(ap);
  
        /* 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;
        }
  
   */
  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)
@@@ -2681,7 -2360,7 +2684,7 @@@ int ata_std_probe_reset(struct ata_por
        ata_reset_fn_t hardreset;
  
        hardreset = NULL;
 -      if (ap->flags & ATA_FLAG_SATA && ap->ops->scr_read)
 +      if (sata_scr_valid(ap))
                hardreset = sata_std_hardreset;
  
        return ata_drive_probe_reset(ap, ata_std_probeinit,
                                     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 ata_do_reset(struct ata_port *ap, ata_reset_fn_t reset,
 +               unsigned int *classes)
  {
        int i, rc;
  
        for (i = 0; i < ATA_MAX_DEVICES; i++)
                classes[i] = ATA_DEV_UNKNOWN;
  
 -      rc = reset(ap, 0, classes);
 +      rc = reset(ap, classes);
        if (rc)
                return rc;
  
                        if (classes[i] == ATA_DEV_UNKNOWN)
                                classes[i] = ATA_DEV_NONE;
  
 -      if (postreset)
 -              postreset(ap, classes);
 -
 -      return classes[0] != ATA_DEV_UNKNOWN ? 0 : -ENODEV;
 +      return 0;
  }
  
  /**
   *    - 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)
@@@ -2753,63 -2438,31 +2756,63 @@@ int ata_drive_probe_reset(struct ata_po
  {
        int rc = -EINVAL;
  
 +      ata_eh_freeze_port(ap);
 +
        if (probeinit)
                probeinit(ap);
  
 -      if (softreset) {
 -              rc = do_probe_reset(ap, softreset, postreset, classes);
 -              if (rc == 0)
 -                      return 0;
 +      if (softreset && !sata_set_spd_needed(ap)) {
 +              rc = ata_do_reset(ap, softreset, classes);
 +              if (rc == 0 && classes[0] != ATA_DEV_UNKNOWN)
 +                      goto done;
 +              ata_port_printk(ap, KERN_INFO, "softreset failed, "
 +                              "will try hardreset in 5 secs\n");
 +              ssleep(5);
        }
  
        if (!hardreset)
 -              return rc;
 +              goto done;
  
 -      rc = do_probe_reset(ap, hardreset, postreset, classes);
 -      if (rc == 0 || rc != -ENODEV)
 -              return rc;
 +      while (1) {
 +              rc = ata_do_reset(ap, hardreset, classes);
 +              if (rc == 0) {
 +                      if (classes[0] != ATA_DEV_UNKNOWN)
 +                              goto done;
 +                      break;
 +              }
 +
 +              if (sata_down_spd_limit(ap))
 +                      goto done;
 +
 +              ata_port_printk(ap, KERN_INFO, "hardreset failed, "
 +                              "will retry in 5 secs\n");
 +              ssleep(5);
 +      }
 +
 +      if (softreset) {
 +              ata_port_printk(ap, KERN_INFO,
 +                              "hardreset succeeded without classification, "
 +                              "will retry softreset in 5 secs\n");
 +              ssleep(5);
 +
 +              rc = ata_do_reset(ap, softreset, classes);
 +      }
  
 -      if (softreset)
 -              rc = do_probe_reset(ap, softreset, postreset, classes);
 + done:
 +      if (rc == 0) {
 +              if (postreset)
 +                      postreset(ap, classes);
  
 +              ata_eh_thaw_port(ap);
 +
 +              if (classes[0] == ATA_DEV_UNKNOWN)
 +                      rc = -ENODEV;
 +      }
        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;
  }
  
@@@ -2986,6 -2646,7 +2989,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
-  *    @sectors: Number of sectors
+  *    @heads: Number of heads (taskfile parameter)
+  *    @sectors: Number of sectors (taskfile parameter)
   *
   *    LOCKING:
   *    Kernel thread context (may sleep)
   *    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;
@@@ -3258,15 -2912,6 +3261,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;
  }
  /**
@@@ -3494,6 -3139,134 +3497,6 @@@ skip_map
        return 0;
  }
  
 -/**
 - *    ata_poll_qc_complete - turn irq back on and finish qc
 - *    @qc: Command to complete
 - *    @err_mask: ATA status register content
 - *
 - *    LOCKING:
 - *    None.  (grabs host lock)
 - */
 -
 -void ata_poll_qc_complete(struct ata_queued_cmd *qc)
 -{
 -      struct ata_port *ap = qc->ap;
 -      unsigned long flags;
 -
 -      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);
 -}
 -
 -/**
 - *    ata_pio_poll - poll using PIO, depending on current state
 - *    @ap: the target ata_port
 - *
 - *    LOCKING:
 - *    None.  (executing in kernel thread context)
 - *
 - *    RETURNS:
 - *    timeout value to use
 - */
 -
 -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
@@@ -3662,23 -3435,7 +3665,23 @@@ 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;
 +
 +              local_irq_save(flags);
 +              buf = kmap_atomic(page, KM_IRQ0);
 +
 +              /* do the actual data transfer */
 +              ata_data_xfer(ap, buf + offset, ATA_SECT_SIZE, do_write);
 +
 +              kunmap_atomic(buf, KM_IRQ0);
 +              local_irq_restore(flags);
 +      } else {
 +              buf = page_address(page);
 +              ata_data_xfer(ap, 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;
 +
 +              WARN_ON(qc->dev->multi_count == 0);
 +
 +              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.
 + */
  
 -      /* do the actual data transfer */
 -      do_write = (qc->tf.flags & ATA_TFLAG_WRITE);
 -      ata_data_xfer(ap, buf, ATA_SECT_SIZE, do_write);
 +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);
  
 -      kunmap(page);
 +      ata_data_xfer(ap, 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;
 +      }
  }
  
  /**
@@@ -3789,8 -3492,8 +3792,8 @@@ 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);
        /* 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;
 +
 +              local_irq_save(flags);
 +              buf = kmap_atomic(page, KM_IRQ0);
 +
 +              /* do the actual data transfer */
 +              ata_data_xfer(ap, buf + offset, count, do_write);
 +
 +              kunmap_atomic(buf, KM_IRQ0);
 +              local_irq_restore(flags);
 +      } else {
 +              buf = page_address(page);
 +              ata_data_xfer(ap, 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;
  }
@@@ -3877,347 -3571,302 +3880,347 @@@ static void atapi_pio_bytes(struct ata_
        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;
 -
 -      /*
 -       * 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 (qc->tf.flags & ATA_TFLAG_POLLING)
 +              return 1;
  
 -      qc = ata_qc_from_tag(ap, ap->active_tag);
 -      WARN_ON(qc == NULL);
 +      if (ap->hsm_task_state == HSM_ST_FIRST) {
 +              if (qc->tf.protocol == ATA_PROT_PIO &&
 +                  (qc->tf.flags & ATA_TFLAG_WRITE))
 +                  return 1;
  
 -      /* check error */
 -      if (status & (ATA_ERR | ATA_DF)) {
 -              qc->err_mask |= AC_ERR_DEV;
 -              ap->hsm_task_state = HSM_ST_ERR;
 -              return;
 +              if (is_atapi_taskfile(&qc->tf) &&
 +                  !(qc->dev->flags & ATA_DFLAG_CDB_INTR))
 +                      return 1;
        }
  
 -      /* 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;
 -              }
 -
 -              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;
 -              }
 -
 -              ata_pio_sector(qc);
 -      }
 -}
 -
 -static void ata_pio_error(struct ata_port *ap)
 -{
 -      struct ata_queued_cmd *qc;
 -
 -      qc = ata_qc_from_tag(ap, ap->active_tag);
 -      WARN_ON(qc == NULL);
 -
 -      if (qc->tf.command != ATA_CMD_PACKET)
 -              printk(KERN_WARNING "ata%u: PIO error\n", ap->id);
 -
 -      /* 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;
 -
 -      ata_poll_qc_complete(qc);
 +      return 0;
  }
  
 -static void ata_pio_task(void *_data)
 +/**
 + *    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 = _data;
 -      unsigned long timeout;
 -      int qc_completed;
 -
 -fsm_start:
 -      timeout = 0;
 -      qc_completed = 0;
 -
 -      switch (ap->hsm_task_state) {
 -      case HSM_ST_IDLE:
 -              return;
 -
 -      case HSM_ST:
 -              ata_pio_block(ap);
 -              break;
 +      struct ata_port *ap = qc->ap;
 +      unsigned long flags;
  
 -      case HSM_ST_LAST:
 -              qc_completed = ata_pio_complete(ap);
 -              break;
 +      if (ap->ops->error_handler) {
 +              if (in_wq) {
 +                      spin_lock_irqsave(&ap->host_set->lock, flags);
  
 -      case HSM_ST_POLL:
 -      case HSM_ST_LAST_POLL:
 -              timeout = ata_pio_poll(ap);
 -              break;
 +                      /* 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);
 +                      }
  
 -      case HSM_ST_TMOUT:
 -      case HSM_ST_ERR:
 -              ata_pio_error(ap);
 -              return;
 +                      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);
        }
 -
 -      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.
 + *    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
   *
 - *    LOCKING:
 - *    Kernel thread context (may sleep)
 + *    RETURNS:
 + *    1 when poll next status needed, 0 otherwise.
   */
  
 -static void atapi_packet_task(void *_data)
 +static int ata_hsm_move(struct ata_port *ap, struct ata_queued_cmd *qc,
 +                       u8 status, int in_wq)
  {
 -      struct ata_port *ap = _data;
 -      struct ata_queued_cmd *qc;
 -      u8 status;
 +      unsigned long flags = 0;
 +      int poll_next;
  
 -      qc = ata_qc_from_tag(ap, ap->active_tag);
 -      WARN_ON(qc == NULL);
 -      WARN_ON(!(qc->flags & ATA_QCFLAG_ACTIVE));
 +      WARN_ON((qc->flags & ATA_QCFLAG_ACTIVE) == 0);
  
 -      /* 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;
 -      }
 +      /* 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(in_wq != ata_hsm_ok_in_wq(ap, qc));
  
 -      /* 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;
 -      }
 +fsm_start:
 +      DPRINTK("ata%u: protocol %d task_state %d (dev_stat 0x%X)\n",
 +              ap->id, qc->tf.protocol, ap->hsm_task_state, status);
  
 -      /* send SCSI cdb */
 -      DPRINTK("send cdb\n");
 -      WARN_ON(qc->dev->cdb_len < 12);
 +      switch (ap->hsm_task_state) {
 +      case HSM_ST_FIRST:
 +              /* Send first data block or PACKET CDB */
  
 -      if (qc->tf.protocol == ATA_PROT_ATAPI_DMA ||
 -          qc->tf.protocol == ATA_PROT_ATAPI_NODATA) {
 -              unsigned long flags;
 +              /* 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_BUSY | ATA_DRQ)) != ATA_DRQ)) {
 +                      /* Wrong status. Let EH handle this */
 +                      qc->err_mask |= AC_ERR_HSM;
 +                      ap->hsm_task_state = HSM_ST_ERR;
 +                      goto fsm_start;
 +              }
  
 -              /* 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.
 +              /* 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.
                 */
 -              spin_lock_irqsave(&ap->host_set->lock, flags);
 -              ap->flags &= ~ATA_FLAG_NOINTR;
 -              ata_data_xfer(ap, qc->cdb, qc->dev->cdb_len, 1);
 -              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);
 +              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_DEV;
 +                      ap->hsm_task_state = HSM_ST_ERR;
 +                      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);
 -      }
 +              /* 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);
  
 -      return;
 +              if (qc->tf.protocol == ATA_PROT_PIO) {
 +                      /* PIO data out protocol.
 +                       * send first data block.
 +                       */
  
 -err_out:
 -      ata_poll_qc_complete(qc);
 -}
 +                      /* 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.
 +               */
 +              break;
  
 -/**
 - *    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)
 - */
 +      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 */
 +                              ap->hsm_task_state = HSM_ST_LAST;
 +                              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;
 +                      /* 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_DEV;
 +                              ap->hsm_task_state = HSM_ST_ERR;
 +                              goto fsm_start;
 +                      }
  
 -      DPRINTK("ENTER\n");
 +                      atapi_pio_bytes(qc);
  
 -      ap->hsm_task_state = HSM_ST_IDLE;
 +                      if (unlikely(ap->hsm_task_state == HSM_ST_ERR))
 +                              /* bad ireason reported by device */
 +                              goto fsm_start;
  
 -      spin_lock_irqsave(&host_set->lock, flags);
 +              } else {
 +                      /* ATA PIO protocol */
 +                      if (unlikely((status & ATA_DRQ) == 0)) {
 +                              /* handle BSY=0, DRQ=0 as error */
 +                              qc->err_mask |= AC_ERR_HSM;
 +                              ap->hsm_task_state = HSM_ST_ERR;
 +                              goto fsm_start;
 +                      }
  
 -      switch (qc->tf.protocol) {
 +                      /* 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);
 +                              }
 +
 +                              /* 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;
 +                      }
  
 -      case ATA_PROT_DMA:
 -      case ATA_PROT_ATAPI_DMA:
 -              host_stat = ap->ops->bmdma_status(ap);
 +                      ata_pio_sectors(qc);
 +
 +                      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;
 +                      }
 +              }
  
 -              /* before we do anything else, clear DMA-Start bit */
 -              ap->ops->bmdma_stop(qc);
 +              ata_altstatus(ap); /* flush */
 +              poll_next = 1;
 +              break;
  
 -              /* fall through */
 +      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;
 +              }
  
 -      default:
 -              ata_altstatus(ap);
 -              drv_stat = ata_chk_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);
  
 -              /* ack bmdma irq events */
 -              ap->ops->irq_clear(ap);
 +              WARN_ON(qc->err_mask);
  
 -              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);
 +              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;
 -      }
  
 -      spin_unlock_irqrestore(&host_set->lock, flags);
 +      case HSM_ST_ERR:
 +              /* make sure qc->err_mask is available to
 +               * know what's wrong and recover
 +               */
 +              WARN_ON(qc->err_mask == 0);
  
 -      ata_eh_qc_complete(qc);
 +              ap->hsm_task_state = HSM_ST_IDLE;
  
 -      DPRINTK("EXIT\n");
 -}
 +              /* complete taskfile transaction */
 +              ata_hsm_qc_complete(qc, in_wq);
  
 -/**
 - *    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)
 - */
 +              poll_next = 0;
 +              break;
 +      default:
 +              poll_next = 0;
 +              BUG();
 +      }
  
 -void ata_eng_timeout(struct ata_port *ap)
 +      return poll_next;
 +}
 +
 +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;
  }
  
  /**
@@@ -4234,14 -3883,9 +4237,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);
@@@ -4296,153 -3941,36 +4299,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;
@@@ -4477,22 -4005,8 +4480,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) {
@@@ -4542,105 -4056,43 +4545,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;
 +      u8 status, host_stat = 0;
  
 -              /* before we do anything else, clear DMA-Start bit */
 -              ap->ops->bmdma_stop(qc);
 +      VPRINTK("ata%u: protocol %d task_state %d\n",
 +              ap->id, qc->tf.protocol, ap->hsm_task_state);
  
 -              /* fall through */
 -
 -      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
@@@ -4776,11 -4214,11 +4779,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;
                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;
  }
@@@ -5135,7 -4434,7 +5139,7 @@@ static void ata_host_init(struct ata_po
        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->sata_spd_limit = UINT_MAX;
        ap->active_tag = ATA_TAG_POISON;
        ap->last_ctl = 0xFF;
  
        INIT_WORK(&ap->port_task, NULL, NULL);
        INIT_LIST_HEAD(&ap->eh_done_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;
@@@ -5216,7 -4509,7 +5220,7 @@@ static struct ata_port * ata_host_add(c
  
        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);
  
@@@ -5287,18 -4580,18 +5291,18 @@@ int ata_device_add(const struct ata_pro
                                (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++;
        }
  
  
                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,
@@@ -5428,12 -4722,15 +5432,12 @@@ void ata_host_set_remove(struct ata_hos
  
  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;
@@@ -5592,52 -4889,6 +5596,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
@@@ -5651,9 -4902,9 +5655,9 @@@ EXPORT_SYMBOL_GPL(ata_device_add)
  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_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);
@@@ -5674,13 -4925,7 +5678,13 @@@ 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_reset);
  EXPORT_SYMBOL_GPL(__sata_phy_reset);
  EXPORT_SYMBOL_GPL(ata_bus_reset);
@@@ -5695,24 -4940,18 +5699,24 @@@ 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_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);
@@@ -5734,13 -4973,3 +5738,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);
diff --combined drivers/scsi/sata_mv.c
@@@ -37,7 -37,7 +37,7 @@@
  #include <asm/io.h>
  
  #define DRV_NAME      "sata_mv"
- #define DRV_VERSION   "0.6"
+ #define DRV_VERSION   "0.7"
  
  enum {
        /* BAR's are enumerated in terms of pci_resource_start() terms */
  
        MV_PCI_REG_BASE         = 0,
        MV_IRQ_COAL_REG_BASE    = 0x18000,      /* 6xxx part only */
+       MV_IRQ_COAL_CAUSE               = (MV_IRQ_COAL_REG_BASE + 0x08),
+       MV_IRQ_COAL_CAUSE_LO            = (MV_IRQ_COAL_REG_BASE + 0x88),
+       MV_IRQ_COAL_CAUSE_HI            = (MV_IRQ_COAL_REG_BASE + 0x8c),
+       MV_IRQ_COAL_THRESHOLD           = (MV_IRQ_COAL_REG_BASE + 0xcc),
+       MV_IRQ_COAL_TIME_THRESHOLD      = (MV_IRQ_COAL_REG_BASE + 0xd0),
        MV_SATAHC0_REG_BASE     = 0x20000,
        MV_FLASH_CTL            = 0x1046c,
        MV_GPIO_PORT_CTL        = 0x104f0,
@@@ -87,7 -93,7 +93,7 @@@
        MV_FLAG_IRQ_COALESCE    = (1 << 29),  /* IRQ coalescing capability */
        MV_COMMON_FLAGS         = (ATA_FLAG_SATA | ATA_FLAG_NO_LEGACY |
                                   ATA_FLAG_SATA_RESET | ATA_FLAG_MMIO |
 -                                 ATA_FLAG_NO_ATAPI),
 +                                 ATA_FLAG_PIO_POLLING),
        MV_6XXX_FLAGS           = MV_FLAG_IRQ_COALESCE,
  
        CRQB_FLAG_READ          = (1 << 0),
@@@ -302,9 -308,6 +308,6 @@@ struct mv_port_priv 
        dma_addr_t              crpb_dma;
        struct mv_sg            *sg_tbl;
        dma_addr_t              sg_tbl_dma;
-       unsigned                req_producer;           /* cp of req_in_ptr */
-       unsigned                rsp_consumer;           /* cp of rsp_out_ptr */
        u32                     pp_flags;
  };
  
@@@ -680,7 -683,7 +683,7 @@@ static void mv_stop_dma(struct ata_por
        }
  
        if (EDMA_EN & reg) {
 -              printk(KERN_ERR "ata%u: Unable to stop eDMA\n", ap->id);
 +              ata_port_printk(ap, KERN_ERR, "Unable to stop eDMA\n");
                /* FIXME: Consider doing a reset here to recover */
        }
  }
@@@ -937,8 -940,6 +940,6 @@@ static int mv_port_start(struct ata_por
        writelfl(pp->crpb_dma & EDMA_RSP_Q_BASE_LO_MASK,
                 port_mmio + EDMA_RSP_Q_OUT_PTR_OFS);
  
-       pp->req_producer = pp->rsp_consumer = 0;
        /* Don't turn on EDMA here...do it before DMA commands only.  Else
         * we'll be unable to send non-data, PIO, etc due to restricted access
         * to shadow regs.
@@@ -1022,16 -1023,16 +1023,16 @@@ static void mv_fill_sg(struct ata_queue
        }
  }
  
- static inline unsigned mv_inc_q_index(unsigned *index)
+ static inline unsigned mv_inc_q_index(unsigned index)
  {
-       *index = (*index + 1) & MV_MAX_Q_DEPTH_MASK;
-       return *index;
+       return (index + 1) & MV_MAX_Q_DEPTH_MASK;
  }
  
  static inline void mv_crqb_pack_cmd(u16 *cmdw, u8 data, u8 addr, unsigned last)
  {
-       *cmdw = data | (addr << CRQB_CMD_ADDR_SHIFT) | CRQB_CMD_CS |
+       u16 tmp = data | (addr << CRQB_CMD_ADDR_SHIFT) | CRQB_CMD_CS |
                (last ? CRQB_CMD_LAST : 0);
+       *cmdw = cpu_to_le16(tmp);
  }
  
  /**
@@@ -1053,15 -1054,11 +1054,11 @@@ static void mv_qc_prep(struct ata_queue
        u16 *cw;
        struct ata_taskfile *tf;
        u16 flags = 0;
+       unsigned in_index;
  
        if (ATA_PROT_DMA != qc->tf.protocol)
                return;
  
-       /* the req producer index should be the same as we remember it */
-       WARN_ON(((readl(mv_ap_base(qc->ap) + EDMA_REQ_Q_IN_PTR_OFS) >>
-                 EDMA_REQ_Q_PTR_SHIFT) & MV_MAX_Q_DEPTH_MASK) !=
-               pp->req_producer);
        /* Fill in command request block
         */
        if (!(qc->tf.flags & ATA_TFLAG_WRITE))
        WARN_ON(MV_MAX_Q_DEPTH <= qc->tag);
        flags |= qc->tag << CRQB_TAG_SHIFT;
  
-       pp->crqb[pp->req_producer].sg_addr =
+       /* get current queue index from hardware */
+       in_index = (readl(mv_ap_base(ap) + EDMA_REQ_Q_IN_PTR_OFS)
+                       >> EDMA_REQ_Q_PTR_SHIFT) & MV_MAX_Q_DEPTH_MASK;
+       pp->crqb[in_index].sg_addr =
                cpu_to_le32(pp->sg_tbl_dma & 0xffffffff);
-       pp->crqb[pp->req_producer].sg_addr_hi =
+       pp->crqb[in_index].sg_addr_hi =
                cpu_to_le32((pp->sg_tbl_dma >> 16) >> 16);
-       pp->crqb[pp->req_producer].ctrl_flags = cpu_to_le16(flags);
+       pp->crqb[in_index].ctrl_flags = cpu_to_le16(flags);
  
-       cw = &pp->crqb[pp->req_producer].ata_cmd[0];
+       cw = &pp->crqb[in_index].ata_cmd[0];
        tf = &qc->tf;
  
        /* Sadly, the CRQB cannot accomodate all registers--there are
@@@ -1144,16 -1145,12 +1145,12 @@@ static void mv_qc_prep_iie(struct ata_q
        struct mv_port_priv *pp = ap->private_data;
        struct mv_crqb_iie *crqb;
        struct ata_taskfile *tf;
+       unsigned in_index;
        u32 flags = 0;
  
        if (ATA_PROT_DMA != qc->tf.protocol)
                return;
  
-       /* the req producer index should be the same as we remember it */
-       WARN_ON(((readl(mv_ap_base(qc->ap) + EDMA_REQ_Q_IN_PTR_OFS) >>
-                 EDMA_REQ_Q_PTR_SHIFT) & MV_MAX_Q_DEPTH_MASK) !=
-               pp->req_producer);
        /* Fill in Gen IIE command request block
         */
        if (!(qc->tf.flags & ATA_TFLAG_WRITE))
        WARN_ON(MV_MAX_Q_DEPTH <= qc->tag);
        flags |= qc->tag << CRQB_TAG_SHIFT;
  
-       crqb = (struct mv_crqb_iie *) &pp->crqb[pp->req_producer];
+       /* get current queue index from hardware */
+       in_index = (readl(mv_ap_base(ap) + EDMA_REQ_Q_IN_PTR_OFS)
+                       >> EDMA_REQ_Q_PTR_SHIFT) & MV_MAX_Q_DEPTH_MASK;
+       crqb = (struct mv_crqb_iie *) &pp->crqb[in_index];
        crqb->addr = cpu_to_le32(pp->sg_tbl_dma & 0xffffffff);
        crqb->addr_hi = cpu_to_le32((pp->sg_tbl_dma >> 16) >> 16);
        crqb->flags = cpu_to_le32(flags);
@@@ -1210,6 -1211,7 +1211,7 @@@ static unsigned int mv_qc_issue(struct 
  {
        void __iomem *port_mmio = mv_ap_base(qc->ap);
        struct mv_port_priv *pp = qc->ap->private_data;
+       unsigned in_index;
        u32 in_ptr;
  
        if (ATA_PROT_DMA != qc->tf.protocol) {
                return ata_qc_issue_prot(qc);
        }
  
-       in_ptr = readl(port_mmio + EDMA_REQ_Q_IN_PTR_OFS);
+       in_ptr   = readl(port_mmio + EDMA_REQ_Q_IN_PTR_OFS);
+       in_index = (in_ptr >> EDMA_REQ_Q_PTR_SHIFT) & MV_MAX_Q_DEPTH_MASK;
  
-       /* the req producer index should be the same as we remember it */
-       WARN_ON(((in_ptr >> EDMA_REQ_Q_PTR_SHIFT) & MV_MAX_Q_DEPTH_MASK) !=
-               pp->req_producer);
        /* until we do queuing, the queue should be empty at this point */
-       WARN_ON(((in_ptr >> EDMA_REQ_Q_PTR_SHIFT) & MV_MAX_Q_DEPTH_MASK) !=
-               ((readl(port_mmio + EDMA_REQ_Q_OUT_PTR_OFS) >>
-                 EDMA_REQ_Q_PTR_SHIFT) & MV_MAX_Q_DEPTH_MASK));
+       WARN_ON(in_index != ((readl(port_mmio + EDMA_REQ_Q_OUT_PTR_OFS)
+               >> EDMA_REQ_Q_PTR_SHIFT) & MV_MAX_Q_DEPTH_MASK));
  
-       mv_inc_q_index(&pp->req_producer);      /* now incr producer index */
+       in_index = mv_inc_q_index(in_index);    /* now incr producer index */
  
        mv_start_dma(port_mmio, pp);
  
        /* and write the request in pointer to kick the EDMA to life */
        in_ptr &= EDMA_REQ_Q_BASE_LO_MASK;
-       in_ptr |= pp->req_producer << EDMA_REQ_Q_PTR_SHIFT;
+       in_ptr |= in_index << EDMA_REQ_Q_PTR_SHIFT;
        writelfl(in_ptr, port_mmio + EDMA_REQ_Q_IN_PTR_OFS);
  
        return 0;
@@@ -1260,28 -1259,26 +1259,26 @@@ static u8 mv_get_crpb_status(struct ata
  {
        void __iomem *port_mmio = mv_ap_base(ap);
        struct mv_port_priv *pp = ap->private_data;
+       unsigned out_index;
        u32 out_ptr;
        u8 ata_status;
  
-       out_ptr = readl(port_mmio + EDMA_RSP_Q_OUT_PTR_OFS);
+       out_ptr   = readl(port_mmio + EDMA_RSP_Q_OUT_PTR_OFS);
+       out_index = (out_ptr >> EDMA_RSP_Q_PTR_SHIFT) & MV_MAX_Q_DEPTH_MASK;
  
-       /* the response consumer index should be the same as we remember it */
-       WARN_ON(((out_ptr >> EDMA_RSP_Q_PTR_SHIFT) & MV_MAX_Q_DEPTH_MASK) !=
-               pp->rsp_consumer);
-       ata_status = pp->crpb[pp->rsp_consumer].flags >> CRPB_FLAG_STATUS_SHIFT;
+       ata_status = le16_to_cpu(pp->crpb[out_index].flags)
+                                       >> CRPB_FLAG_STATUS_SHIFT;
  
        /* increment our consumer index... */
-       pp->rsp_consumer = mv_inc_q_index(&pp->rsp_consumer);
+       out_index = mv_inc_q_index(out_index);
  
        /* and, until we do NCQ, there should only be 1 CRPB waiting */
-       WARN_ON(((readl(port_mmio + EDMA_RSP_Q_IN_PTR_OFS) >>
-                 EDMA_RSP_Q_PTR_SHIFT) & MV_MAX_Q_DEPTH_MASK) !=
-               pp->rsp_consumer);
+       WARN_ON(out_index != ((readl(port_mmio + EDMA_RSP_Q_IN_PTR_OFS)
+               >> EDMA_RSP_Q_PTR_SHIFT) & MV_MAX_Q_DEPTH_MASK));
  
        /* write out our inc'd consumer index so EDMA knows we're caught up */
        out_ptr &= EDMA_RSP_Q_BASE_LO_MASK;
-       out_ptr |= pp->rsp_consumer << EDMA_RSP_Q_PTR_SHIFT;
+       out_ptr |= out_index << EDMA_RSP_Q_PTR_SHIFT;
        writelfl(out_ptr, port_mmio + EDMA_RSP_Q_OUT_PTR_OFS);
  
        /* Return ATA status register for completed CRPB */
  /**
   *      mv_err_intr - Handle error interrupts on the port
   *      @ap: ATA channel to manipulate
+  *      @reset_allowed: bool: 0 == don't trigger from reset here
   *
   *      In most cases, just clear the interrupt and move on.  However,
   *      some cases require an eDMA reset, which is done right before
   *      LOCKING:
   *      Inherited from caller.
   */
- static void mv_err_intr(struct ata_port *ap)
+ static void mv_err_intr(struct ata_port *ap, int reset_allowed)
  {
        void __iomem *port_mmio = mv_ap_base(ap);
        u32 edma_err_cause, serr = 0;
        edma_err_cause = readl(port_mmio + EDMA_ERR_IRQ_CAUSE_OFS);
  
        if (EDMA_ERR_SERR & edma_err_cause) {
 -              serr = scr_read(ap, SCR_ERROR);
 -              scr_write_flush(ap, SCR_ERROR, serr);
 +              sata_scr_read(ap, SCR_ERROR, &serr);
 +              sata_scr_write_flush(ap, SCR_ERROR, serr);
        }
        if (EDMA_ERR_SELF_DIS & edma_err_cause) {
                struct mv_port_priv *pp = ap->private_data;
        writelfl(0, port_mmio + EDMA_ERR_IRQ_CAUSE_OFS);
  
        /* check for fatal here and recover if needed */
-       if (EDMA_ERR_FATAL & edma_err_cause) {
+       if (reset_allowed && (EDMA_ERR_FATAL & edma_err_cause))
                mv_stop_and_reset(ap);
-       }
  }
  
  /**
@@@ -1374,12 -1371,12 +1371,12 @@@ static void mv_host_intr(struct ata_hos
                struct ata_port *ap = host_set->ports[port];
                struct mv_port_priv *pp = ap->private_data;
  
-               hard_port = port & MV_PORT_MASK;        /* range 0-3 */
+               hard_port = mv_hardport_from_port(port); /* range 0..3 */
                handled = 0;    /* ensure ata_status is set if handled++ */
  
                /* Note that DEV_IRQ might happen spuriously during EDMA,
-                * and should be ignored in such cases.  We could mask it,
-                * but it's pretty rare and may not be worth the overhead.
+                * and should be ignored in such cases.
+                * The cause of this is still under investigation.
                 */ 
                if (pp->pp_flags & MV_PP_FLAG_EDMA_EN) {
                        /* EDMA: check for response queue interrupt */
                                ata_status = readb((void __iomem *)
                                           ap->ioaddr.status_addr);
                                handled = 1;
+                               /* ignore spurious intr if drive still BUSY */
+                               if (ata_status & ATA_BUSY) {
+                                       ata_status = 0;
+                                       handled = 0;
+                               }
                        }
                }
  
 -              if (ap->flags & (ATA_FLAG_PORT_DISABLED | ATA_FLAG_NOINTR))
 +              if (ap && (ap->flags & ATA_FLAG_DISABLED))
                        continue;
  
                err_mask = ac_err_mask(ata_status);
                        shift++;        /* skip bit 8 in the HC Main IRQ reg */
                }
                if ((PORT0_ERR << shift) & relevant) {
-                       mv_err_intr(ap);
+                       mv_err_intr(ap, 1);
                        err_mask |= AC_ERR_OTHER;
                        handled = 1;
                }
                                VPRINTK("port %u IRQ found for qc, "
                                        "ata_status 0x%x\n", port,ata_status);
                                /* mark qc status appropriately */
 -                              if (!(qc->tf.ctl & ATA_NIEN)) {
 +                              if (!(qc->tf.flags & ATA_TFLAG_POLLING)) {
                                        qc->err_mask |= err_mask;
                                        ata_qc_complete(qc);
                                }
@@@ -1448,6 -1450,7 +1450,7 @@@ static irqreturn_t mv_interrupt(int irq
        struct ata_host_set *host_set = dev_instance;
        unsigned int hc, handled = 0, n_hcs;
        void __iomem *mmio = host_set->mmio_base;
+       struct mv_host_priv *hpriv;
        u32 irq_stat;
  
        irq_stat = readl(mmio + HC_MAIN_IRQ_CAUSE_OFS);
                        handled++;
                }
        }
+       hpriv = host_set->private_data;
+       if (IS_60XX(hpriv)) {
+               /* deal with the interrupt coalescing bits */
+               if (irq_stat & (TRAN_LO_DONE | TRAN_HI_DONE | PORTS_0_7_COAL_DONE)) {
+                       writelfl(0, mmio + MV_IRQ_COAL_CAUSE_LO);
+                       writelfl(0, mmio + MV_IRQ_COAL_CAUSE_HI);
+                       writelfl(0, mmio + MV_IRQ_COAL_CAUSE);
+               }
+       }
        if (PCI_ERR & irq_stat) {
                printk(KERN_ERR DRV_NAME ": PCI ERROR; PCI IRQ cause=0x%08x\n",
                       readl(mmio + PCI_IRQ_CAUSE_OFS));
@@@ -1867,7 -1881,8 +1881,8 @@@ static void mv_channel_reset(struct mv_
  
        if (IS_60XX(hpriv)) {
                u32 ifctl = readl(port_mmio + SATA_INTERFACE_CTL);
-               ifctl |= (1 << 12) | (1 << 7);
+               ifctl |= (1 << 7);              /* enable gen2i speed */
+               ifctl = (ifctl & 0xfff) | 0x9b1000; /* from chip spec */
                writelfl(ifctl, port_mmio + SATA_INTERFACE_CTL);
        }
  
@@@ -1934,16 -1949,15 +1949,16 @@@ static void __mv_phy_reset(struct ata_p
  
        /* Issue COMRESET via SControl */
  comreset_retry:
 -      scr_write_flush(ap, SCR_CONTROL, 0x301);
 +      sata_scr_write_flush(ap, SCR_CONTROL, 0x301);
        __msleep(1, can_sleep);
  
 -      scr_write_flush(ap, SCR_CONTROL, 0x300);
 +      sata_scr_write_flush(ap, SCR_CONTROL, 0x300);
        __msleep(20, can_sleep);
  
        timeout = jiffies + msecs_to_jiffies(200);
        do {
 -              sstatus = scr_read(ap, SCR_STATUS) & 0x3;
 +              sata_scr_read(ap, SCR_STATUS, &sstatus);
 +              sstatus &= 0x3;
                if ((sstatus == 3) || (sstatus == 0))
                        break;
  
                "SCtrl 0x%08x\n", mv_scr_read(ap, SCR_STATUS),
                mv_scr_read(ap, SCR_ERROR), mv_scr_read(ap, SCR_CONTROL));
  
 -      if (sata_dev_present(ap)) {
 +      if (ata_port_online(ap)) {
                ata_port_probe(ap);
        } else {
 -              printk(KERN_INFO "ata%u: no device found (phy stat %08x)\n",
 -                     ap->id, scr_read(ap, SCR_STATUS));
 +              sata_scr_read(ap, SCR_STATUS, &sstatus);
 +              ata_port_printk(ap, KERN_INFO,
 +                              "no device found (phy stat %08x)\n", sstatus);
                ata_port_disable(ap);
                return;
        }
        tf.nsect = readb((void __iomem *) ap->ioaddr.nsect_addr);
  
        dev->class = ata_dev_classify(&tf);
 -      if (!ata_dev_present(dev)) {
 +      if (!ata_dev_enabled(dev)) {
                VPRINTK("Port disabled post-sig: No device present.\n");
                ata_port_disable(ap);
        }
@@@ -2023,7 -2036,7 +2038,7 @@@ static void mv_eng_timeout(struct ata_p
  {
        struct ata_queued_cmd *qc;
  
 -      printk(KERN_ERR "ata%u: Entering mv_eng_timeout\n",ap->id);
 +      ata_port_printk(ap, KERN_ERR, "Entering mv_eng_timeout\n");
        DPRINTK("All regs @ start of eng_timeout\n");
        mv_dump_all_regs(ap->host_set->mmio_base, ap->port_no,
                         to_pci_dev(ap->host_set->dev));
               ap->host_set->mmio_base, ap, qc, qc->scsicmd,
               &qc->scsicmd->cmnd);
  
-       mv_err_intr(ap);
+       mv_err_intr(ap, 0);
        mv_stop_and_reset(ap);
  
-       qc->err_mask |= AC_ERR_TIMEOUT;
-       ata_eh_qc_complete(qc);
+       WARN_ON(!(qc->flags & ATA_QCFLAG_ACTIVE));
+       if (qc->flags & ATA_QCFLAG_ACTIVE) {
+               qc->err_mask |= AC_ERR_TIMEOUT;
+               ata_eh_qc_complete(qc);
+       }
  }
  
  /**
@@@ -2231,7 -2247,8 +2249,8 @@@ static int mv_init_host(struct pci_dev 
                        void __iomem *port_mmio = mv_port_base(mmio, port);
  
                        u32 ifctl = readl(port_mmio + SATA_INTERFACE_CTL);
-                       ifctl |= (1 << 12);
+                       ifctl |= (1 << 7);              /* enable gen2i speed */
+                       ifctl = (ifctl & 0xfff) | 0x9b1000; /* from chip spec */
                        writelfl(ifctl, port_mmio + SATA_INTERFACE_CTL);
                }
  
@@@ -2332,6 -2349,7 +2351,7 @@@ static int mv_init_one(struct pci_dev *
        if (rc) {
                return rc;
        }
+       pci_set_master(pdev);
  
        rc = pci_request_regions(pdev, DRV_NAME);
        if (rc) {