From: Jens Axboe <jens.axboe@oracle.com>
To: jeff@garzik.org
Cc: tj@kernel.org, linux-ide@vger.kernel.org
Subject: libata bridge limits
Date: Tue, 26 Aug 2008 09:28:41 +0200 [thread overview]
Message-ID: <20080826072841.GS20055@kernel.dk> (raw)
Hi,
Jeff, a long time ago we talked about the bridge limits in libata core.
Back then it was my T43 notebook, but now I'm running into the same
problem. The Mtron 70xx series SSD drives sit behind a PATA-SATA bridge
and thus runs into the bridging limits in libata. To put things into
perspective, the difference between UDMA5 and UDMA6 modes on this drive
is 85 vs 115Mb/sec or 77 vs 106Mb/sec (depends on the host controller I
tested), so it's quite a lot.
There are several ways to solve this:
a) Why was this limit put in there? It limits both transfer speed and
request size. If it's due to some dodgy drive/bridge, perhaps we
should just check for that and only apply the transfer limits when
detected (or blacklisted). On the bridge setups I've seen, I've never
had problems with killing the limit.
b) Put in a whitelist, easy to do for these Mtron drives.
c) Add a parameter to turn it on (or off, depending on the default) for
a specific drive.
I'm in favor of a) personally, but I'd like to hear why the check was
added originally first. Dropping 20-30% of the throughput performance on
the floor without option seems like a really bad choice.
--
Jens Axboe
next reply other threads:[~2008-08-26 7:28 UTC|newest]
Thread overview: 24+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-08-26 7:28 Jens Axboe [this message]
2008-08-26 9:42 ` libata bridge limits Alan Cox
2008-08-26 10:17 ` Jens Axboe
2008-08-26 10:43 ` Tejun Heo
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=20080826072841.GS20055@kernel.dk \
--to=jens.axboe@oracle.com \
--cc=jeff@garzik.org \
--cc=linux-ide@vger.kernel.org \
--cc=tj@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;
as well as URLs for NNTP newsgroup(s).