From: Tejun Heo <htejun@gmail.com>
To: Rusty Conover <rconover@infogears.com>
Cc: linux-ide@vger.kernel.org
Subject: Re: Resets on sil3124 & sil3726 PMP
Date: Sat, 25 Aug 2007 11:19:38 +0900 [thread overview]
Message-ID: <46CF91BA.7030608@gmail.com> (raw)
In-Reply-To: <7644C0A3-8AEF-4C00-837A-429F73DD616D@infogears.com>
Hello, Rusty.
Rusty Conover wrote:
> I have some interesting results.
>
> I had a pair of Seagate 250 GB SATA disks (models below) and tried those
> out rather then the WD's. At the 1.5 gbps rather they appear to work
> just fine both being on the same PMP, at 3.0 gbps they timeout just like
> the other disks did. Possibly, the code isn't detecting the max rate of
> the disks correctly since these drives only do 1.5 and it attempted to
> do 3.0 gbps?
Hardware PHY layer is fully responsible of 1.5/3.0 negotiation. The
driver can limit the maximum it can go but doesn't have much say in the
actual negotiation.
Unfortunately, some SATA PHY combinations have problems at 3.0bps. I
saw quite a few weird cases. e.g. marvell PMP can't detect older
Seagate drives on 3Gbps and can't talk to a first gen Maxtor even at
1.5. I'm still not quite sure how to solve this. Once detected, libata
EH will do the right thing and speed down to 1.5 after a few
transmission errors but it's a bit trickier before detection. It's
probably not worth driving fan-out ports at 3.0Ghz in the first place so
maybe limiting PMP fan-out ports to 1.5 by default can alleviate most
situations. Oh well, users won't be happy tho.
> Have you found that drives can be picky if they will work or not with
> PMP ports?
As I said above, yeap. Not only PMPs some host PHYs do that too.
> I'll be trying to get a bunch of different drives to replace the ones
> that don't work with PMP.
Yeah, that will be the easiest way out for the moment.
Thanks.
--
tejun
next prev parent reply other threads:[~2007-08-26 10:09 UTC|newest]
Thread overview: 33+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-08-15 13:58 Resets on sil3124 & sil3726 PMP Rusty Conover
2007-08-17 18:06 ` Tejun Heo
2007-08-20 19:56 ` Rusty Conover
2007-08-21 2:42 ` Rusty Conover
2007-08-22 3:03 ` Tejun Heo
2007-08-22 5:00 ` Rusty Conover
2007-08-22 5:43 ` Tejun Heo
2007-08-22 6:11 ` Rusty Conover
2007-08-22 6:39 ` Tejun Heo
2007-08-22 6:56 ` Rusty Conover
2007-08-22 7:02 ` Tejun Heo
2007-08-22 7:49 ` Rusty Conover
2007-08-25 2:19 ` Tejun Heo [this message]
2007-08-26 16:37 ` Rusty Conover
2007-08-27 1:18 ` Tejun Heo
-- strict thread matches above, loose matches on Subject: below --
2007-09-03 19:39 Richard Scobie
2007-09-03 20:34 ` Robin H. Johnson
2007-09-03 21:21 ` Richard Scobie
2007-08-27 8:08 Richard Scobie
2007-08-27 22:49 ` Robin H. Johnson
2007-08-27 4:55 Richard Scobie
2007-08-27 7:11 ` Petr Vandrovec
2007-09-03 8:59 ` Tejun Heo
2007-09-03 9:57 ` Petr Vandrovec
2007-09-03 12:50 ` Tejun Heo
2007-09-04 1:38 ` Petr Vandrovec
2007-09-05 12:08 ` Petr Vandrovec
2007-09-05 21:30 ` Robin H. Johnson
2007-09-06 9:52 ` Petr Vandrovec
2007-09-06 18:02 ` Tejun Heo
2007-09-06 17:59 ` Tejun Heo
2007-08-14 23:39 Rusty Conover
2007-08-15 6:10 ` Tejun Heo
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=46CF91BA.7030608@gmail.com \
--to=htejun@gmail.com \
--cc=linux-ide@vger.kernel.org \
--cc=rconover@infogears.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).