From: Tejun Heo <tj@kernel.org>
To: Jens Axboe <jens.axboe@oracle.com>
Cc: Alan Cox <alan@lxorguk.ukuu.org.uk>,
jeff@garzik.org, linux-ide@vger.kernel.org,
Mark Lord <mlord@pobox.com>, Kay Sievers <kay.sievers@vrfy.org>,
Gwendal Grignou <gwendal@google.com>
Subject: Re: libata bridge limits
Date: Tue, 26 Aug 2008 12:43:39 +0200 [thread overview]
Message-ID: <48B3DE5B.1020607@kernel.org> (raw)
In-Reply-To: <20080826101713.GW20055@kernel.dk>
(cc'ing Mark Lord, Kay Sievers and Gwendal Grignou)
Jens Axboe wrote:
>> d) Assume the bridge is ok but teach the SATA error handling code that if
>> there is a timeout immediately with such a bridge then to flip down to
>> UDMA5 and knobble the transfer length.
>
> That would be nice, assuming that we can rely on safe behaviour (eg not
> data corrupting badness).
Obstacles to such approaches are...
* The current IO timeouts are too long. It's not like reducing this is
difficult. The only reason why we haven't reduced it yet is because we
haven't been able to agree on what's the proper timeout value.
According to Mark, 8 secs should be fine (Windows uses it) for
read/writes but there seem to be some corner cases.
* Some rare controllers fail miserably after a timeout but this is
pretty rare and getting rarer. I don't think we need to consider this
the main deciding factor.
* Currently, the transfer speed setting reached by EH actions is not
persistent. On the next boot, the device would have to go through the
same thing all over again, which isn't too pleasant. It would be great
if we can make this setting persistent. Maybe this can be done libata
sysfs and udev?
Thanks.
--
tejun
next prev parent reply other threads:[~2008-08-26 10:47 UTC|newest]
Thread overview: 24+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-08-26 7:28 libata bridge limits Jens Axboe
2008-08-26 9:42 ` Alan Cox
2008-08-26 10:17 ` Jens Axboe
2008-08-26 10:43 ` Tejun Heo [this message]
2008-08-26 10:38 ` Alan Cox
2008-08-26 11:23 ` Tejun Heo
2008-08-26 12:25 ` Alan Cox
2008-08-26 12:45 ` Tejun Heo
2008-08-26 17:25 ` Gwendal Grignou
2008-08-26 17:45 ` James Bottomley
2008-08-26 19:25 ` Gwendal Grignou
2008-08-26 20:55 ` James Bottomley
2008-08-26 12:32 ` Brad Campbell
2008-08-26 12:48 ` Jens Axboe
2008-08-26 12:55 ` Tejun Heo
2008-08-26 13:06 ` Jens Axboe
2008-08-26 13:58 ` Jens Axboe
2008-08-26 14:20 ` Tejun Heo
2008-08-26 14:26 ` Jens Axboe
2008-08-26 14:25 ` Jens Axboe
2008-08-26 19:36 ` Jeff Garzik
2008-08-26 22:37 ` Mark Lord
2008-08-27 13:23 ` Jens Axboe
2008-10-31 5:45 ` Jeff Garzik
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=48B3DE5B.1020607@kernel.org \
--to=tj@kernel.org \
--cc=alan@lxorguk.ukuu.org.uk \
--cc=gwendal@google.com \
--cc=jeff@garzik.org \
--cc=jens.axboe@oracle.com \
--cc=kay.sievers@vrfy.org \
--cc=linux-ide@vger.kernel.org \
--cc=mlord@pobox.com \
/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;
as well as URLs for NNTP newsgroup(s).