public inbox for linux-scsi@vger.kernel.org
 help / color / mirror / Atom feed
From: James Bottomley <James.Bottomley@HansenPartnership.com>
To: Hannes Reinecke <hare@suse.de>
Cc: "linux-scsi@vger.kernel.org" <linux-scsi@vger.kernel.org>,
	Christoph Hellwig <hch@infradead.org>,
	James Smart <james.smart@emulex.com>,
	Andrew Vasquez <andrew.vasquez@qlogic.com>
Subject: Re: Is there any plan to support 64bit lun in mainline?
Date: Tue, 15 Oct 2013 15:58:33 +0100	[thread overview]
Message-ID: <3aeace08-b1f4-45b0-a026-960cc93cef55@email.android.com> (raw)
In-Reply-To: <525CDA2A.2060308@suse.de>

Ok may be brief: on top of cliff with fleeting mobile signal.

Hannes Reinecke <hare@suse.de> wrote:
>On 10/14/2013 08:30 PM, James Bottomley wrote:
>> On Mon, 2013-10-14 at 12:32 +0200, Hannes Reinecke wrote:
>[ .. ]
>>>
>>> However, there are two other patchsets pending (EH Deadline and
>>> asynchronous command aborts), both of which have been tested
>>> thoroughly _and_ have acked-by from various other parties.
>>> None of these patchset had received any feedback from James
>>> Bottomley, let alone any indication if or when they'll be merged.
>>>
>>> So I stopped sending further patchsets, sensing some communication
>>> breakdown on the line.
>> 
>> There's no communications breakdown, I've just been very busy.  EH
>> deadline looks OK, but let me look again to make sure.  Async aborts
>I'm
>> not necessarily keen on because I know a lot of devices where
>aborting
>> just confuses the firmware more, so I think skipping aborts and
>moving
>> to LUN reset might be a better form of acceleration.
>> 
>
>Hmm. I've made exactly the opposite experience with FC devices.
>A faulty SFP or wiring might cause the occasional frame drop, which
>will be happily corrected with an ABTS.
>And HBAs like lpfc or qla2xxx even have a fast command abort built
>into the firmware, where the firmware will not even wait for a
>command abort to hit the wire but rather just disable the exchange
>internally and return.
>_Not_ sending an abort but rather escalating directly to LUN reset
>will only make matters worse here.
>Plus a LUN reset won't be able to fix anything here.

Its mostly for*device* firmware problems. The while problem with our reset escalations is that they don't fit with a transport model. Its a bit accidental that abort works for the fc transport because its designed as a device message.

>But sure, if there are some devices which are confused by aborts,
>sure we should be doing something about it.
>Just out of curiosity, which devices are we talking about?

Lots of older drives. Even some arrays (but also only older ones). For them when they start to timeout they've already begun losing track of commands so aborts make this worse.

>I would very much prefer to have inlined command aborts at least for
>FC, as it has proven to be really beneficial there.

Ok, so how about this. I looked over
The running abort series. It looks fine apart from some coding bits. For the deadline, we can't waste all the deadline with the tiny hammer and then fail to use the bigone. Should have internet on Wednesday will expand then.

James

>Cheers,
>
>Hannes

-- 
Sent from my Android device with K-9 Mail. Please excuse my brevity.

  reply	other threads:[~2013-10-15 14:59 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-10-12  6:35 Is there any plan to support 64bit lun in mainline? Wangshen (C)
2013-10-14 10:32 ` Hannes Reinecke
2013-10-14 15:29   ` Christoph Hellwig
2013-10-20 16:20     ` Douglas Gilbert
2013-10-14 18:30   ` James Bottomley
2013-10-15  6:01     ` Hannes Reinecke
2013-10-15 14:58       ` James Bottomley [this message]
2013-10-16  4:11       ` Vladislav Bolkhovitin

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=3aeace08-b1f4-45b0-a026-960cc93cef55@email.android.com \
    --to=james.bottomley@hansenpartnership.com \
    --cc=andrew.vasquez@qlogic.com \
    --cc=hare@suse.de \
    --cc=hch@infradead.org \
    --cc=james.smart@emulex.com \
    --cc=linux-scsi@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox