linux-ide.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
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


  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).