linux-kernel - Re: [PATCH 0/5] scsi: Allow fast io fail without waiting through timeout

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <519A4716.7000001@emulex.com>
Date:	Mon, 20 May 2013 11:53:58 -0400
From:	James Smart <James.Smart@...lex.com>
To:	Ren Mingxin <renmx@...fujitsu.com>
CC:	<James.Bottomley@...senPartnership.com>,
	<Nagalakshmi.Nandigama@....com>, <Sreekanth.Reddy@....com>,
	<michaelc@...wisc.edu>, <robert.w.love@...el.com>,
	<dgilbert@...erlog.com>, <hare@...e.de>, <bprakash@...adcom.com>,
	<agk@...hat.com>, <neilb@...e.de>, <gregkh@...uxfoundation.org>,
	<axboe@...nel.dk>, <tj@...nel.org>, <linux-kernel@...r.kernel.org>,
	<linux-scsi@...r.kernel.org>
Subject: Re: [PATCH 0/5] scsi: Allow fast io fail without waiting through
 timeout

Ren,

Based on the discussion recently held at LSF 2013, we are reworking the 
error recovery path to address all the issues you are mentioning.   That 
work contradicts these patches. So for now, these should be held off.

-- james s



On 5/20/2013 3:14 AM, Ren Mingxin wrote:
> When there is a scsi command timed-out or failed, the scsi eh
> tries a thorugh recovery, which is necessary for non-redundant
> systems. However, the thorugh recovery usually takes much time,
> which is not acceptable for misson critical systems. To improve
> this latency, if we are working on a redundant system, we should
> avoid the scsi eh for its long time failing recovery, and quick
> failover to another path.
>
> This set of patches is trying to implement above.
>
> NOTE: the userland tools need to eusure the environment
> restriction, which will be implemented later.
>
> Thanks,
> Ren
>
> Ren Mingxin (5):
>    scsi: rename return code FAST_IO_FAIL to FAST_IO
>    FC transport: Add interface to specify fast io level for timed-out cmds
>    SAS transport: Add interface to specify fast io level for timed-out cmds
>    lpfc: Allow fast timed-out io recovery
>    mptfusion: Allow fast timed-out io recovery
>
>   drivers/message/fusion/mptscsih.c   |   29 ++++++++-
>   drivers/scsi/lpfc/lpfc_scsi.c       |   34 ++++++++++
>   drivers/scsi/scsi_error.c           |   18 ++---
>   drivers/scsi/scsi_sas_internal.h    |    4 -
>   drivers/scsi/scsi_transport_fc.c    |  112 ++++++++++++++++++++++++++++++++++--
>   drivers/scsi/scsi_transport_iscsi.c |    6 -
>   drivers/scsi/scsi_transport_sas.c   |  103 ++++++++++++++++++++++++++++++++-
>   include/scsi/scsi.h                 |    2
>   include/scsi/scsi_transport_fc.h    |   11 +++
>   include/scsi/scsi_transport_sas.h   |    8 ++
>   10 files changed, 303 insertions(+), 24 deletions(-)
>
>

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/