linux-ide.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Justin Maggard <jmaggard10@gmail.com>
To: Tejun Heo <tj@kernel.org>
Cc: Robert Hancock <hancockrwd@gmail.com>, linux-ide@vger.kernel.org
Subject: Re: Hotplug issue with Marvell 6121
Date: Wed, 10 Mar 2010 18:32:31 -0800	[thread overview]
Message-ID: <150c16851003101832o358a8b4aj760384020658d1fc@mail.gmail.com> (raw)
In-Reply-To: <4B9840D3.7050109@kernel.org>

On Wed, Mar 10, 2010 at 5:01 PM, Tejun Heo <tj@kernel.org> wrote:
> Hello,
>
> On 03/11/2010 09:53 AM, Robert Hancock wrote:
>> That doesn't sound like a great solution. I'd think that the speed
>> limiting should be reset upon disabling the device so that we start over
>> at 3Gbps again when you reconnect..
>
> libata EH already does that.  It resets the speed limit mask after the
> device is disabled and when the *next* reset happens on device
> hotplug, the highest speed will be applied.  It's just that there
> usually is no need to do an extra reset right after disabling a
> device.
>
> Does 1.5Gbps work at all on those marvell controllers?  It's *very*
> strange to miss hotplug under 1.5Gbps.  Those signals are OOB and
> don't even follow the usual signal frequency.

It's not the 1.5Gbps that is causing the problem.  It appears to be
the act of changing speeds.  I added "libata.force=3:1.5Gbps" to my
kernel command line, and hotplugs seemed to work fine on that port
(presumably due to the fact that libata EH won't change the link speed
limit if it's already at 1.5Gbps).  The other port on the controller
still had hotplug problems.  So everything seems to be pointing to the
Marvell controller having problems when you change speeds.

-Justin

  reply	other threads:[~2010-03-11  2:32 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-03-10  0:51 Hotplug issue with Marvell 6121 Justin Maggard
2010-03-10 22:46 ` Justin Maggard
2010-03-11  0:53   ` Robert Hancock
2010-03-11  1:01     ` Tejun Heo
2010-03-11  2:32       ` Justin Maggard [this message]
2010-03-11  3:33         ` Mark Lord
2010-03-11  3:45           ` Tejun Heo
2010-03-11 18:42             ` Justin Maggard
2010-03-11 22:28               ` 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=150c16851003101832o358a8b4aj760384020658d1fc@mail.gmail.com \
    --to=jmaggard10@gmail.com \
    --cc=hancockrwd@gmail.com \
    --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).