2020-07-25 08:24:23

by Luo Jiaxing

[permalink] [raw]
Subject: [PATCH v2 0/2] scsi: libsas: An improvement on error handle and tidy-up

This patch introduces an improvement to reduce error handle time and a
tidy-up, including:
- postreset() is deleted from sas_sata_ops.
- Do not perform hard reset and delayed retry on a removed SATA disk. This
can effectively reduce the error handle duration of hot unplug a SATA disk
with traffic(reduce about 30s depending on the delay setting of libata).

Both John garry and Jason Yan participated in the review of the solution
and provided good suggestions during the development.

Change since v1:
- Removed an unnecessary tag from subject.

Luo Jiaxing (2):
scsi: libsas: delete postreset at sas_sata_ops
scsi: libsas: check link status at ATA prereset() ops

drivers/scsi/libsas/sas_ata.c | 17 +++++++++++++++--
1 file changed, 15 insertions(+), 2 deletions(-)

--
2.7.4


2020-07-25 09:00:40

by Luo Jiaxing

[permalink] [raw]
Subject: Re: [PATCH v2 0/2] scsi: libsas: An improvement on error handle and tidy-up

sorry,I missed Martin's e-mail "Re: [PATCH v1 0/2] scsi: libsas: An
improvement on error handle and tidy-up" here. Please ignore this set of
patches.

On 2020/7/25 16:18, Luo Jiaxing wrote:
> This patch introduces an improvement to reduce error handle time and a
> tidy-up, including:
> - postreset() is deleted from sas_sata_ops.
> - Do not perform hard reset and delayed retry on a removed SATA disk. This
> can effectively reduce the error handle duration of hot unplug a SATA disk
> with traffic(reduce about 30s depending on the delay setting of libata).
>
> Both John garry and Jason Yan participated in the review of the solution
> and provided good suggestions during the development.
>
> Change since v1:
> - Removed an unnecessary tag from subject.
>
> Luo Jiaxing (2):
> scsi: libsas: delete postreset at sas_sata_ops
> scsi: libsas: check link status at ATA prereset() ops
>
> drivers/scsi/libsas/sas_ata.c | 17 +++++++++++++++--
> 1 file changed, 15 insertions(+), 2 deletions(-)
>