lkml.org 
[lkml]   [2023]   [Mar]   [21]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
SubjectRe: [PATCH] scsi: libsas: Add end eh callback
From
Sorry, please ignore this patch and I have resend a new one.

On 2023/3/21 15:22, Xingui Yang wrote:
> If an error occurs while the disk is processing an NCQ command and the host
> received the abnormal SDB FIS, let libata EH to analyze the NCQ error, and
> it is not necessary to reset the target to recover.
>
> Then the hisi_sas has some special process to set dev_status to normal when
> end the eh for NCQ error without reset the target, so add a callback and
> fill it in for the hisi_sas driver.
>
> Signed-off-by: Xingui Yang <yangxingui@huawei.com>
> ---
> drivers/scsi/hisi_sas/hisi_sas_main.c | 12 +++++++++---
> drivers/scsi/hisi_sas/hisi_sas_v3_hw.c | 7 +++++--
> drivers/scsi/libsas/sas_ata.c | 4 ++++
> include/scsi/libsas.h | 2 ++
> 4 files changed, 20 insertions(+), 5 deletions(-)
>
> diff --git a/drivers/scsi/hisi_sas/hisi_sas_main.c b/drivers/scsi/hisi_sas/hisi_sas_main.c
> index 325d6d6a21c3..61686ead0027 100644
> --- a/drivers/scsi/hisi_sas/hisi_sas_main.c
> +++ b/drivers/scsi/hisi_sas/hisi_sas_main.c
> @@ -1777,9 +1777,6 @@ static int hisi_sas_I_T_nexus_reset(struct domain_device *device)
> struct device *dev = hisi_hba->dev;
> int rc;
>
> - if (sas_dev->dev_status == HISI_SAS_DEV_NCQ_ERR)
> - sas_dev->dev_status = HISI_SAS_DEV_NORMAL;
> -
> rc = hisi_sas_internal_task_abort_dev(sas_dev, false);
> if (rc < 0) {
> dev_err(dev, "I_T nexus reset: internal abort (%d)\n", rc);
> @@ -1967,6 +1964,14 @@ static bool hisi_sas_internal_abort_timeout(struct sas_task *task,
> return false;
> }
>
> +static void hisi_sas_end_eh(struct domain_device *dev)
> +{
> + struct hisi_sas_device *sas_dev = dev->lldd_dev;
> +
> + if (sas_dev->dev_status == HISI_SAS_DEV_NCQ_ERR)
> + sas_dev->dev_status = HISI_SAS_DEV_NORMAL;
> +}
> +
> static void hisi_sas_port_formed(struct asd_sas_phy *sas_phy)
> {
> hisi_sas_port_notify_formed(sas_phy);
> @@ -2083,6 +2088,7 @@ static struct sas_domain_function_template hisi_sas_transport_ops = {
> .lldd_write_gpio = hisi_sas_write_gpio,
> .lldd_tmf_aborted = hisi_sas_tmf_aborted,
> .lldd_abort_timeout = hisi_sas_internal_abort_timeout,
> + .lldd_end_eh = hisi_sas_end_eh,
> };
>
> void hisi_sas_init_mem(struct hisi_hba *hisi_hba)
> diff --git a/drivers/scsi/hisi_sas/hisi_sas_v3_hw.c b/drivers/scsi/hisi_sas/hisi_sas_v3_hw.c
> index 66fcb340b98e..abad57de4aee 100644
> --- a/drivers/scsi/hisi_sas/hisi_sas_v3_hw.c
> +++ b/drivers/scsi/hisi_sas/hisi_sas_v3_hw.c
> @@ -2433,15 +2433,18 @@ static int complete_v3_hw(struct hisi_sas_cq *cq)
> struct hisi_sas_device *sas_dev =
> &hisi_hba->devices[device_id];
> struct domain_device *device = sas_dev->sas_device;
> + bool force_reset = true;
>
> dev_err(dev, "erroneous completion disk err dev id=%d sas_addr=0x%llx CQ hdr: 0x%x 0x%x 0x%x 0x%x\n",
> device_id, itct->sas_addr, dw0, dw1,
> complete_hdr->act, dw3);
>
> - if (is_ncq_err_v3_hw(complete_hdr))
> + if (is_ncq_err_v3_hw(complete_hdr)) {
> sas_dev->dev_status = HISI_SAS_DEV_NCQ_ERR;
> + force_reset = false;
> + }
>
> - sas_ata_device_link_abort(device, true);
> + sas_ata_device_link_abort(device, force_reset);
> } else if (likely(iptt < HISI_SAS_COMMAND_ENTRIES_V3_HW)) {
> slot = &hisi_hba->slot_info[iptt];
> slot->cmplt_queue_slot = rd_point;
> diff --git a/drivers/scsi/libsas/sas_ata.c b/drivers/scsi/libsas/sas_ata.c
> index 77714a495cbb..2d48643a08cf 100644
> --- a/drivers/scsi/libsas/sas_ata.c
> +++ b/drivers/scsi/libsas/sas_ata.c
> @@ -539,6 +539,10 @@ void sas_ata_end_eh(struct ata_port *ap)
> spin_lock_irqsave(&ha->lock, flags);
> if (test_and_clear_bit(SAS_DEV_EH_PENDING, &dev->state))
> ha->eh_active--;
> +
> + if (i->dft->lldd_end_eh)
> + i->dft->lldd_end_eh(device);
> +
> spin_unlock_irqrestore(&ha->lock, flags);
> }
>
> diff --git a/include/scsi/libsas.h b/include/scsi/libsas.h
> index 159823e0afbf..659395ef616e 100644
> --- a/include/scsi/libsas.h
> +++ b/include/scsi/libsas.h
> @@ -683,6 +683,8 @@ struct sas_domain_function_template {
> int (*lldd_lu_reset)(struct domain_device *, u8 *lun);
> int (*lldd_query_task)(struct sas_task *);
>
> + void (*lldd_end_eh)(struct domain_device *dev);
> +
> /* Special TMF callbacks */
> void (*lldd_tmf_exec_complete)(struct domain_device *dev);
> void (*lldd_tmf_aborted)(struct sas_task *task);
>

\
 
 \ /
  Last update: 2023-03-27 01:10    [W:0.068 / U:0.172 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site