From: Tejun Heo <htejun@gmail.com>
To: Mark Lord <liml@rtr.ca>
Cc: Jeff Garzik <jgarzik@pobox.com>,
IDE/ATA development list <linux-ide@vger.kernel.org>,
Alan Cox <alan@redhat.com>
Subject: Re: ata_std_qc_defer not good enough for FIS-based switching ?
Date: Thu, 15 May 2008 16:12:13 +0900 [thread overview]
Message-ID: <482BE24D.8020900@gmail.com> (raw)
In-Reply-To: <482B0C5D.5080207@rtr.ca>
Mark Lord wrote:
> Mark Lord wrote:
>> Mark Lord wrote:
>>> Mark Lord wrote:
>>>> ..
>>>> At the moment, I have the Sil PM plugged in, and it all behaves nicely.
>>>> The previous failure was with the Marvell PM, so I'll hook that up next
>>>> and retest with it.
>>> ..
>>>
>>> It went bonkers again. This is with 2.6.25.
>>> I'll build 2.6.26-git and try again with that shortly.
>> ..
>>
>> Okay, I have today's 2.6.26-rc2-git installed now.
>> My PCIe notebook has an ExpressCard with this on it:
>>
>> 0d:00.0 Mass storage controller: Silicon Image, Inc. SiI 3132 Serial
>> ATA Raid II Controller (rev 01)
>>
>> That's the controller I use with sata_sil24 here.
>>
>> I have a Marvell 4140 port-multiplier (standalone card),
>> and have a Seagate 750GB drive (pmp=0) and a Hitachi 500GB drive (pmp=2).
>>
>> When I simply do "hdparm -t" simultaneously on both drives,
>> something gets very confused somewhere. Neither drive has any media
>> errors,
>> and this exact same test works fine with sata_mv in my other system.
>> The same test also works fine with both controllers
>> when using a Silicon Image PM instead of the Marvell.
>>
>> When I move the second drive to pmp=1 (instead of pmp=2),
>> then sata_sil24 no longer has problems. Very peculiar.
>>
>> Here's the log from sata_sil24, ports 0,2, as it fails:
> ..
>
> And here is the log from sata_sil24, ports 0,1, as it works:
Heh.. detection doesn't work too well either. I have to tell you mv
4140 is my favorite pmp. It just behaved right. Ironically, all later
works were focused on simg 3726/4726's because they didn't behave. I'll
give well-deserved love to my 4140 and get back to you soon.
Thanks.
--
tejun
next prev parent reply other threads:[~2008-05-15 7:12 UTC|newest]
Thread overview: 17+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-04-28 21:06 ata_std_qc_defer not good enough for FIS-based switching ? Mark Lord
2008-04-28 22:08 ` Mark Lord
2008-04-28 23:11 ` Tejun Heo
2008-04-28 23:42 ` Mark Lord
2008-04-29 1:18 ` Tejun Heo
2008-04-29 2:26 ` Mark Lord
2008-05-14 14:11 ` Tejun Heo
2008-05-14 14:45 ` Mark Lord
2008-05-14 14:53 ` Mark Lord
2008-05-14 15:50 ` Mark Lord
2008-05-14 15:59 ` Mark Lord
2008-05-15 7:12 ` Tejun Heo [this message]
2008-05-15 6:51 ` Tejun Heo
2008-04-29 17:09 ` Mark Lord
2008-04-29 17:17 ` Mark Lord
2008-04-30 9:38 ` Tejun Heo
2008-04-30 14:02 ` Mark Lord
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=482BE24D.8020900@gmail.com \
--to=htejun@gmail.com \
--cc=alan@redhat.com \
--cc=jgarzik@pobox.com \
--cc=liml@rtr.ca \
--cc=linux-ide@vger.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 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.