All of lore.kernel.org
 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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.