[SCSI] libsas: fix timeout vs completion race
[linux-block.git] / drivers / scsi / libsas / sas_ata.c
1 /*
2  * Support for SATA devices on Serial Attached SCSI (SAS) controllers
3  *
4  * Copyright (C) 2006 IBM Corporation
5  *
6  * Written by: Darrick J. Wong <djwong@us.ibm.com>, IBM Corporation
7  *
8  * This program is free software; you can redistribute it and/or
9  * modify it under the terms of the GNU General Public License as
10  * published by the Free Software Foundation; either version 2 of the
11  * License, or (at your option) any later version.
12  *
13  * This program is distributed in the hope that it will be useful, but
14  * WITHOUT ANY WARRANTY; without even the implied warranty of
15  * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
16  * General Public License for more details.
17  *
18  * You should have received a copy of the GNU General Public License
19  * along with this program; if not, write to the Free Software
20  * Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307
21  * USA
22  */
23
24 #include <linux/scatterlist.h>
25 #include <linux/slab.h>
26
27 #include <scsi/sas_ata.h>
28 #include "sas_internal.h"
29 #include <scsi/scsi_host.h>
30 #include <scsi/scsi_device.h>
31 #include <scsi/scsi_tcq.h>
32 #include <scsi/scsi.h>
33 #include <scsi/scsi_transport.h>
34 #include <scsi/scsi_transport_sas.h>
35 #include "../scsi_sas_internal.h"
36 #include "../scsi_transport_api.h"
37 #include <scsi/scsi_eh.h>
38
39 static enum ata_completion_errors sas_to_ata_err(struct task_status_struct *ts)
40 {
41         /* Cheesy attempt to translate SAS errors into ATA.  Hah! */
42
43         /* transport error */
44         if (ts->resp == SAS_TASK_UNDELIVERED)
45                 return AC_ERR_ATA_BUS;
46
47         /* ts->resp == SAS_TASK_COMPLETE */
48         /* task delivered, what happened afterwards? */
49         switch (ts->stat) {
50                 case SAS_DEV_NO_RESPONSE:
51                         return AC_ERR_TIMEOUT;
52
53                 case SAS_INTERRUPTED:
54                 case SAS_PHY_DOWN:
55                 case SAS_NAK_R_ERR:
56                         return AC_ERR_ATA_BUS;
57
58
59                 case SAS_DATA_UNDERRUN:
60                         /*
61                          * Some programs that use the taskfile interface
62                          * (smartctl in particular) can cause underrun
63                          * problems.  Ignore these errors, perhaps at our
64                          * peril.
65                          */
66                         return 0;
67
68                 case SAS_DATA_OVERRUN:
69                 case SAS_QUEUE_FULL:
70                 case SAS_DEVICE_UNKNOWN:
71                 case SAS_SG_ERR:
72                         return AC_ERR_INVALID;
73
74                 case SAS_OPEN_TO:
75                 case SAS_OPEN_REJECT:
76                         SAS_DPRINTK("%s: Saw error %d.  What to do?\n",
77                                     __func__, ts->stat);
78                         return AC_ERR_OTHER;
79
80                 case SAM_STAT_CHECK_CONDITION:
81                 case SAS_ABORTED_TASK:
82                         return AC_ERR_DEV;
83
84                 case SAS_PROTO_RESPONSE:
85                         /* This means the ending_fis has the error
86                          * value; return 0 here to collect it */
87                         return 0;
88                 default:
89                         return 0;
90         }
91 }
92
93 static void sas_ata_task_done(struct sas_task *task)
94 {
95         struct ata_queued_cmd *qc = task->uldd_task;
96         struct domain_device *dev = task->dev;
97         struct task_status_struct *stat = &task->task_status;
98         struct ata_task_resp *resp = (struct ata_task_resp *)stat->buf;
99         struct sas_ha_struct *sas_ha = dev->port->ha;
100         enum ata_completion_errors ac;
101         unsigned long flags;
102         struct ata_link *link;
103         struct ata_port *ap;
104
105         spin_lock_irqsave(&dev->done_lock, flags);
106         if (test_bit(SAS_HA_FROZEN, &sas_ha->state))
107                 task = NULL;
108         else if (qc && qc->scsicmd)
109                 ASSIGN_SAS_TASK(qc->scsicmd, NULL);
110         spin_unlock_irqrestore(&dev->done_lock, flags);
111
112         /* check if libsas-eh got to the task before us */
113         if (unlikely(!task))
114                 return;
115
116         if (!qc)
117                 goto qc_already_gone;
118
119         ap = qc->ap;
120         link = &ap->link;
121
122         spin_lock_irqsave(ap->lock, flags);
123         /* check if we lost the race with libata/sas_ata_post_internal() */
124         if (unlikely(ap->pflags & ATA_PFLAG_FROZEN)) {
125                 spin_unlock_irqrestore(ap->lock, flags);
126                 if (qc->scsicmd)
127                         goto qc_already_gone;
128                 else {
129                         /* if eh is not involved and the port is frozen then the
130                          * ata internal abort process has taken responsibility
131                          * for this sas_task
132                          */
133                         return;
134                 }
135         }
136
137         if (stat->stat == SAS_PROTO_RESPONSE || stat->stat == SAM_STAT_GOOD ||
138             ((stat->stat == SAM_STAT_CHECK_CONDITION &&
139               dev->sata_dev.command_set == ATAPI_COMMAND_SET))) {
140                 ata_tf_from_fis(resp->ending_fis, &dev->sata_dev.tf);
141
142                 if (!link->sactive) {
143                         qc->err_mask |= ac_err_mask(dev->sata_dev.tf.command);
144                 } else {
145                         link->eh_info.err_mask |= ac_err_mask(dev->sata_dev.tf.command);
146                         if (unlikely(link->eh_info.err_mask))
147                                 qc->flags |= ATA_QCFLAG_FAILED;
148                 }
149         } else {
150                 ac = sas_to_ata_err(stat);
151                 if (ac) {
152                         SAS_DPRINTK("%s: SAS error %x\n", __func__,
153                                     stat->stat);
154                         /* We saw a SAS error. Send a vague error. */
155                         if (!link->sactive) {
156                                 qc->err_mask = ac;
157                         } else {
158                                 link->eh_info.err_mask |= AC_ERR_DEV;
159                                 qc->flags |= ATA_QCFLAG_FAILED;
160                         }
161
162                         dev->sata_dev.tf.feature = 0x04; /* status err */
163                         dev->sata_dev.tf.command = ATA_ERR;
164                 }
165         }
166
167         qc->lldd_task = NULL;
168         ata_qc_complete(qc);
169         spin_unlock_irqrestore(ap->lock, flags);
170
171 qc_already_gone:
172         list_del_init(&task->list);
173         sas_free_task(task);
174 }
175
176 static unsigned int sas_ata_qc_issue(struct ata_queued_cmd *qc)
177 {
178         unsigned long flags;
179         struct sas_task *task;
180         struct scatterlist *sg;
181         int ret = AC_ERR_SYSTEM;
182         unsigned int si, xfer = 0;
183         struct ata_port *ap = qc->ap;
184         struct domain_device *dev = ap->private_data;
185         struct sas_ha_struct *sas_ha = dev->port->ha;
186         struct Scsi_Host *host = sas_ha->core.shost;
187         struct sas_internal *i = to_sas_internal(host->transportt);
188
189         /* TODO: audit callers to ensure they are ready for qc_issue to
190          * unconditionally re-enable interrupts
191          */
192         local_irq_save(flags);
193         spin_unlock(ap->lock);
194
195         /* If the device fell off, no sense in issuing commands */
196         if (test_bit(SAS_DEV_GONE, &dev->state))
197                 goto out;
198
199         task = sas_alloc_task(GFP_ATOMIC);
200         if (!task)
201                 goto out;
202         task->dev = dev;
203         task->task_proto = SAS_PROTOCOL_STP;
204         task->task_done = sas_ata_task_done;
205
206         if (qc->tf.command == ATA_CMD_FPDMA_WRITE ||
207             qc->tf.command == ATA_CMD_FPDMA_READ) {
208                 /* Need to zero out the tag libata assigned us */
209                 qc->tf.nsect = 0;
210         }
211
212         ata_tf_to_fis(&qc->tf, 1, 0, (u8*)&task->ata_task.fis);
213         task->uldd_task = qc;
214         if (ata_is_atapi(qc->tf.protocol)) {
215                 memcpy(task->ata_task.atapi_packet, qc->cdb, qc->dev->cdb_len);
216                 task->total_xfer_len = qc->nbytes;
217                 task->num_scatter = qc->n_elem;
218         } else {
219                 for_each_sg(qc->sg, sg, qc->n_elem, si)
220                         xfer += sg->length;
221
222                 task->total_xfer_len = xfer;
223                 task->num_scatter = si;
224         }
225
226         task->data_dir = qc->dma_dir;
227         task->scatter = qc->sg;
228         task->ata_task.retry_count = 1;
229         task->task_state_flags = SAS_TASK_STATE_PENDING;
230         qc->lldd_task = task;
231
232         switch (qc->tf.protocol) {
233         case ATA_PROT_NCQ:
234                 task->ata_task.use_ncq = 1;
235                 /* fall through */
236         case ATAPI_PROT_DMA:
237         case ATA_PROT_DMA:
238                 task->ata_task.dma_xfer = 1;
239                 break;
240         }
241
242         if (qc->scsicmd)
243                 ASSIGN_SAS_TASK(qc->scsicmd, task);
244
245         if (sas_ha->lldd_max_execute_num < 2)
246                 ret = i->dft->lldd_execute_task(task, 1, GFP_ATOMIC);
247         else
248                 ret = sas_queue_up(task);
249
250         /* Examine */
251         if (ret) {
252                 SAS_DPRINTK("lldd_execute_task returned: %d\n", ret);
253
254                 if (qc->scsicmd)
255                         ASSIGN_SAS_TASK(qc->scsicmd, NULL);
256                 sas_free_task(task);
257                 ret = AC_ERR_SYSTEM;
258         }
259
260  out:
261         spin_lock(ap->lock);
262         local_irq_restore(flags);
263         return ret;
264 }
265
266 static bool sas_ata_qc_fill_rtf(struct ata_queued_cmd *qc)
267 {
268         struct domain_device *dev = qc->ap->private_data;
269
270         memcpy(&qc->result_tf, &dev->sata_dev.tf, sizeof(qc->result_tf));
271         return true;
272 }
273
274 static int sas_ata_hard_reset(struct ata_link *link, unsigned int *class,
275                                unsigned long deadline)
276 {
277         struct ata_port *ap = link->ap;
278         struct domain_device *dev = ap->private_data;
279         struct sas_internal *i =
280                 to_sas_internal(dev->port->ha->core.shost->transportt);
281         int res = TMF_RESP_FUNC_FAILED;
282         int ret = 0;
283
284         if (i->dft->lldd_I_T_nexus_reset)
285                 res = i->dft->lldd_I_T_nexus_reset(dev);
286
287         if (res != TMF_RESP_FUNC_COMPLETE) {
288                 SAS_DPRINTK("%s: Unable to reset I T nexus?\n", __func__);
289                 ret = -EAGAIN;
290         }
291
292         switch (dev->sata_dev.command_set) {
293                 case ATA_COMMAND_SET:
294                         SAS_DPRINTK("%s: Found ATA device.\n", __func__);
295                         *class = ATA_DEV_ATA;
296                         break;
297                 case ATAPI_COMMAND_SET:
298                         SAS_DPRINTK("%s: Found ATAPI device.\n", __func__);
299                         *class = ATA_DEV_ATAPI;
300                         break;
301                 default:
302                         SAS_DPRINTK("%s: Unknown SATA command set: %d.\n",
303                                     __func__,
304                                     dev->sata_dev.command_set);
305                         *class = ATA_DEV_UNKNOWN;
306                         break;
307         }
308
309         ap->cbl = ATA_CBL_SATA;
310         return ret;
311 }
312
313 static int sas_ata_soft_reset(struct ata_link *link, unsigned int *class,
314                                unsigned long deadline)
315 {
316         struct ata_port *ap = link->ap;
317         struct domain_device *dev = ap->private_data;
318         struct sas_internal *i =
319                 to_sas_internal(dev->port->ha->core.shost->transportt);
320         int res = TMF_RESP_FUNC_FAILED;
321         int ret = 0;
322
323         if (i->dft->lldd_ata_soft_reset)
324                 res = i->dft->lldd_ata_soft_reset(dev);
325
326         if (res != TMF_RESP_FUNC_COMPLETE) {
327                 SAS_DPRINTK("%s: Unable to soft reset\n", __func__);
328                 ret = -EAGAIN;
329         }
330
331         switch (dev->sata_dev.command_set) {
332         case ATA_COMMAND_SET:
333                 SAS_DPRINTK("%s: Found ATA device.\n", __func__);
334                 *class = ATA_DEV_ATA;
335                 break;
336         case ATAPI_COMMAND_SET:
337                 SAS_DPRINTK("%s: Found ATAPI device.\n", __func__);
338                 *class = ATA_DEV_ATAPI;
339                 break;
340         default:
341                 SAS_DPRINTK("%s: Unknown SATA command set: %d.\n",
342                             __func__, dev->sata_dev.command_set);
343                 *class = ATA_DEV_UNKNOWN;
344                 break;
345         }
346
347         ap->cbl = ATA_CBL_SATA;
348         return ret;
349 }
350
351 /*
352  * notify the lldd to forget the sas_task for this internal ata command
353  * that bypasses scsi-eh
354  */
355 static void sas_ata_internal_abort(struct sas_task *task)
356 {
357         struct sas_internal *si =
358                 to_sas_internal(task->dev->port->ha->core.shost->transportt);
359         unsigned long flags;
360         int res;
361
362         spin_lock_irqsave(&task->task_state_lock, flags);
363         if (task->task_state_flags & SAS_TASK_STATE_ABORTED ||
364             task->task_state_flags & SAS_TASK_STATE_DONE) {
365                 spin_unlock_irqrestore(&task->task_state_lock, flags);
366                 SAS_DPRINTK("%s: Task %p already finished.\n", __func__,
367                             task);
368                 goto out;
369         }
370         task->task_state_flags |= SAS_TASK_STATE_ABORTED;
371         spin_unlock_irqrestore(&task->task_state_lock, flags);
372
373         res = si->dft->lldd_abort_task(task);
374
375         spin_lock_irqsave(&task->task_state_lock, flags);
376         if (task->task_state_flags & SAS_TASK_STATE_DONE ||
377             res == TMF_RESP_FUNC_COMPLETE) {
378                 spin_unlock_irqrestore(&task->task_state_lock, flags);
379                 goto out;
380         }
381
382         /* XXX we are not prepared to deal with ->lldd_abort_task()
383          * failures.  TODO: lldds need to unconditionally forget about
384          * aborted ata tasks, otherwise we (likely) leak the sas task
385          * here
386          */
387         SAS_DPRINTK("%s: Task %p leaked.\n", __func__, task);
388
389         if (!(task->task_state_flags & SAS_TASK_STATE_DONE))
390                 task->task_state_flags &= ~SAS_TASK_STATE_ABORTED;
391         spin_unlock_irqrestore(&task->task_state_lock, flags);
392
393         return;
394  out:
395         list_del_init(&task->list);
396         sas_free_task(task);
397 }
398
399 static void sas_ata_post_internal(struct ata_queued_cmd *qc)
400 {
401         if (qc->flags & ATA_QCFLAG_FAILED)
402                 qc->err_mask |= AC_ERR_OTHER;
403
404         if (qc->err_mask) {
405                 /*
406                  * Find the sas_task and kill it.  By this point, libata
407                  * has decided to kill the qc and has frozen the port.
408                  * In this state sas_ata_task_done() will no longer free
409                  * the sas_task, so we need to notify the lldd (via
410                  * ->lldd_abort_task) that the task is dead and free it
411                  *  ourselves.
412                  */
413                 struct sas_task *task = qc->lldd_task;
414                 unsigned long flags;
415
416                 qc->lldd_task = NULL;
417                 if (task) {
418                         /* Should this be a AT(API) device reset? */
419                         spin_lock_irqsave(&task->task_state_lock, flags);
420                         task->task_state_flags |= SAS_TASK_NEED_DEV_RESET;
421                         spin_unlock_irqrestore(&task->task_state_lock, flags);
422
423                         task->uldd_task = NULL;
424                         sas_ata_internal_abort(task);
425                 }
426         }
427 }
428
429
430 static void sas_ata_set_dmamode(struct ata_port *ap, struct ata_device *ata_dev)
431 {
432         struct domain_device *dev = ap->private_data;
433         struct sas_internal *i =
434                 to_sas_internal(dev->port->ha->core.shost->transportt);
435
436         if (i->dft->lldd_ata_set_dmamode)
437                 i->dft->lldd_ata_set_dmamode(dev);
438 }
439
440 static struct ata_port_operations sas_sata_ops = {
441         .prereset               = ata_std_prereset,
442         .softreset              = sas_ata_soft_reset,
443         .hardreset              = sas_ata_hard_reset,
444         .postreset              = ata_std_postreset,
445         .error_handler          = ata_std_error_handler,
446         .post_internal_cmd      = sas_ata_post_internal,
447         .qc_defer               = ata_std_qc_defer,
448         .qc_prep                = ata_noop_qc_prep,
449         .qc_issue               = sas_ata_qc_issue,
450         .qc_fill_rtf            = sas_ata_qc_fill_rtf,
451         .port_start             = ata_sas_port_start,
452         .port_stop              = ata_sas_port_stop,
453         .set_dmamode            = sas_ata_set_dmamode,
454 };
455
456 static struct ata_port_info sata_port_info = {
457         .flags = ATA_FLAG_SATA | ATA_FLAG_PIO_DMA | ATA_FLAG_NCQ,
458         .pio_mask = ATA_PIO4,
459         .mwdma_mask = ATA_MWDMA2,
460         .udma_mask = ATA_UDMA6,
461         .port_ops = &sas_sata_ops
462 };
463
464 int sas_ata_init_host_and_port(struct domain_device *found_dev,
465                                struct scsi_target *starget)
466 {
467         struct Scsi_Host *shost = dev_to_shost(&starget->dev);
468         struct sas_ha_struct *ha = SHOST_TO_SAS_HA(shost);
469         struct ata_port *ap;
470
471         ata_host_init(&found_dev->sata_dev.ata_host,
472                       ha->dev,
473                       sata_port_info.flags,
474                       &sas_sata_ops);
475         ap = ata_sas_port_alloc(&found_dev->sata_dev.ata_host,
476                                 &sata_port_info,
477                                 shost);
478         if (!ap) {
479                 SAS_DPRINTK("ata_sas_port_alloc failed.\n");
480                 return -ENODEV;
481         }
482
483         ap->private_data = found_dev;
484         ap->cbl = ATA_CBL_SATA;
485         ap->scsi_host = shost;
486         found_dev->sata_dev.ap = ap;
487
488         return 0;
489 }
490
491 void sas_ata_task_abort(struct sas_task *task)
492 {
493         struct ata_queued_cmd *qc = task->uldd_task;
494         struct completion *waiting;
495
496         /* Bounce SCSI-initiated commands to the SCSI EH */
497         if (qc->scsicmd) {
498                 struct request_queue *q = qc->scsicmd->device->request_queue;
499                 unsigned long flags;
500
501                 spin_lock_irqsave(q->queue_lock, flags);
502                 blk_abort_request(qc->scsicmd->request);
503                 spin_unlock_irqrestore(q->queue_lock, flags);
504                 scsi_schedule_eh(qc->scsicmd->device->host);
505                 return;
506         }
507
508         /* Internal command, fake a timeout and complete. */
509         qc->flags &= ~ATA_QCFLAG_ACTIVE;
510         qc->flags |= ATA_QCFLAG_FAILED;
511         qc->err_mask |= AC_ERR_TIMEOUT;
512         waiting = qc->private_data;
513         complete(waiting);
514 }
515
516 static void sas_get_ata_command_set(struct domain_device *dev)
517 {
518         struct dev_to_host_fis *fis =
519                 (struct dev_to_host_fis *) dev->frame_rcvd;
520
521         if ((fis->sector_count == 1 && /* ATA */
522              fis->lbal         == 1 &&
523              fis->lbam         == 0 &&
524              fis->lbah         == 0 &&
525              fis->device       == 0)
526             ||
527             (fis->sector_count == 0 && /* CE-ATA (mATA) */
528              fis->lbal         == 0 &&
529              fis->lbam         == 0xCE &&
530              fis->lbah         == 0xAA &&
531              (fis->device & ~0x10) == 0))
532
533                 dev->sata_dev.command_set = ATA_COMMAND_SET;
534
535         else if ((fis->interrupt_reason == 1 && /* ATAPI */
536                   fis->lbal             == 1 &&
537                   fis->byte_count_low   == 0x14 &&
538                   fis->byte_count_high  == 0xEB &&
539                   (fis->device & ~0x10) == 0))
540
541                 dev->sata_dev.command_set = ATAPI_COMMAND_SET;
542
543         else if ((fis->sector_count == 1 && /* SEMB */
544                   fis->lbal         == 1 &&
545                   fis->lbam         == 0x3C &&
546                   fis->lbah         == 0xC3 &&
547                   fis->device       == 0)
548                 ||
549                  (fis->interrupt_reason == 1 && /* SATA PM */
550                   fis->lbal             == 1 &&
551                   fis->byte_count_low   == 0x69 &&
552                   fis->byte_count_high  == 0x96 &&
553                   (fis->device & ~0x10) == 0))
554
555                 /* Treat it as a superset? */
556                 dev->sata_dev.command_set = ATAPI_COMMAND_SET;
557 }
558
559 void sas_probe_sata(struct work_struct *work)
560 {
561         struct domain_device *dev, *n;
562         struct sas_discovery_event *ev =
563                 container_of(work, struct sas_discovery_event, work);
564         struct asd_sas_port *port = ev->port;
565
566         clear_bit(DISCE_PROBE, &port->disc.pending);
567
568         list_for_each_entry_safe(dev, n, &port->disco_list, disco_list_node) {
569                 int err;
570
571                 spin_lock_irq(&port->dev_list_lock);
572                 list_add_tail(&dev->dev_list_node, &port->dev_list);
573                 spin_unlock_irq(&port->dev_list_lock);
574
575                 err = sas_rphy_add(dev->rphy);
576
577                 if (err) {
578                         SAS_DPRINTK("%s: for %s device %16llx returned %d\n",
579                                     __func__, dev->parent ? "exp-attached" :
580                                                             "direct-attached",
581                                     SAS_ADDR(dev->sas_addr), err);
582                         sas_unregister_dev(port, dev);
583                 } else
584                         list_del_init(&dev->disco_list_node);
585         }
586 }
587
588 /**
589  * sas_discover_sata -- discover an STP/SATA domain device
590  * @dev: pointer to struct domain_device of interest
591  *
592  * Devices directly attached to a HA port, have no parents.  All other
593  * devices do, and should have their "parent" pointer set appropriately
594  * before calling this function.
595  */
596 int sas_discover_sata(struct domain_device *dev)
597 {
598         int res;
599
600         if (dev->dev_type == SATA_PM)
601                 return -ENODEV;
602
603         sas_get_ata_command_set(dev);
604         sas_fill_in_rphy(dev, dev->rphy);
605
606         res = sas_notify_lldd_dev_found(dev);
607         if (res)
608                 return res;
609
610         sas_discover_event(dev->port, DISCE_PROBE);
611         return 0;
612 }
613
614 void sas_ata_strategy_handler(struct Scsi_Host *shost)
615 {
616         struct scsi_device *sdev;
617         struct sas_ha_struct *sas_ha = SHOST_TO_SAS_HA(shost);
618
619         /* it's ok to defer revalidation events during ata eh, these
620          * disks are in one of three states:
621          * 1/ present for initial domain discovery, and these
622          *    resets will cause bcn flutters
623          * 2/ hot removed, we'll discover that after eh fails
624          * 3/ hot added after initial discovery, lost the race, and need
625          *    to catch the next train.
626          */
627         sas_disable_revalidation(sas_ha);
628
629         shost_for_each_device(sdev, shost) {
630                 struct domain_device *ddev = sdev_to_domain_dev(sdev);
631                 struct ata_port *ap = ddev->sata_dev.ap;
632
633                 if (!dev_is_sata(ddev))
634                         continue;
635
636                 ata_port_printk(ap, KERN_DEBUG, "sas eh calling libata port error handler");
637                 ata_scsi_port_error_handler(shost, ap);
638         }
639
640         sas_enable_revalidation(sas_ha);
641 }
642
643 int sas_ata_eh(struct Scsi_Host *shost, struct list_head *work_q,
644                struct list_head *done_q)
645 {
646         int rtn = 0;
647         struct scsi_cmnd *cmd, *n;
648         struct ata_port *ap;
649
650         do {
651                 LIST_HEAD(sata_q);
652
653                 ap = NULL;
654
655                 list_for_each_entry_safe(cmd, n, work_q, eh_entry) {
656                         struct domain_device *ddev = cmd_to_domain_dev(cmd);
657
658                         if (!dev_is_sata(ddev) || TO_SAS_TASK(cmd))
659                                 continue;
660                         if (ap && ap != ddev->sata_dev.ap)
661                                 continue;
662                         ap = ddev->sata_dev.ap;
663                         rtn = 1;
664                         list_move(&cmd->eh_entry, &sata_q);
665                 }
666
667                 if (!list_empty(&sata_q)) {
668                         ata_port_printk(ap, KERN_DEBUG, "sas eh calling libata cmd error handler\n");
669                         ata_scsi_cmd_error_handler(shost, ap, &sata_q);
670                         /*
671                          * ata's error handler may leave the cmd on the list
672                          * so make sure they don't remain on a stack list
673                          * about to go out of scope.
674                          *
675                          * This looks strange, since the commands are
676                          * now part of no list, but the next error
677                          * action will be ata_port_error_handler()
678                          * which takes no list and sweeps them up
679                          * anyway from the ata tag array.
680                          */
681                         while (!list_empty(&sata_q))
682                                 list_del_init(sata_q.next);
683                 }
684         } while (ap);
685
686         return rtn;
687 }