From: Albert Lee <albertcc@tw.ibm.com>
To: Doug Maxey <dwm@falcon30.maxeymade.com>
Cc: Tejun Heo <htejun@gmail.com>, Jeff Garzik <jeff@garzik.org>,
albertl@mail.com, Carlos Pardo <Carlos.Pardo@siliconimage.com>,
Jonathan Blake Benson <airbatica@verizon.net>,
linux-ide@vger.kernel.org
Subject: Re: libata machine check on Alpha
Date: Thu, 06 Apr 2006 15:14:59 +0800 [thread overview]
Message-ID: <4434BFF3.3080005@tw.ibm.com> (raw)
In-Reply-To: <200604060644.k366i0O6007713@falcon30.maxeymade.com>
Doug Maxey wrote:
> On Thu, 06 Apr 2006 14:38:39 +0800, Albert Lee wrote:
>
>>Tejun Heo wrote:
>>
>>>Jeff Garzik wrote:
>>>
>>>>DMADIR is in my copy of ATA-7...
>>>>
>>>
>>>Here's the relavant part from ATA8 draft. It seems that the bridge is
>>>supposed to mangle the IDENTIFY PACKET DEVICE result. It's supposed to
>>>nuke all DMA transfer mode information reported from the original device
>>>and report supported modes from the view point of the bridge in bits
>>>10:1 (which, BTW, should be all 1's).
>>>
>>>The question is whether or not bridges implement this properly. I think
>>>we can add a printk to word 62 and ask Jonathan to test it.
>>>
>>
>>Ah, thanks for the info. The DMADIR is in my copy ATA-7, too.
>>I just checked word 50 and overlooked it. :(
>>I should have checked other part of the spec...
>>
>>Anyway, will try to make a patch based on the ATA-7 spec.
>
>
> Do we have any devices to test your patch with? Perhaps in Rochester?
>
Yes, we have some Marvell bridges in Rochester to test.
I'll also test the JMicron bridge, once the auto enable DMADIR patch is ready.
We have no SiI 3611 at hand, so, need Jonathan's help.
--
Albert
next prev parent reply other threads:[~2006-04-06 7:15 UTC|newest]
Thread overview: 19+ messages / expand[flat|nested] mbox.gz Atom feed top
2006-04-04 17:12 libata machine check on Alpha Carlos Pardo
2006-04-06 2:04 ` Albert Lee
2006-04-06 2:17 ` Jeff Garzik
2006-04-06 2:31 ` Tejun Heo
2006-04-06 6:38 ` Albert Lee
2006-04-06 6:44 ` Doug Maxey
2006-04-06 7:14 ` Albert Lee [this message]
-- strict thread matches above, loose matches on Subject: below --
2006-04-07 2:01 Jonathan Benson
2006-04-07 6:18 ` Albert Lee
2006-04-05 0:14 Jonathan Benson
2006-04-03 18:56 Jonathan Blake Benson
2006-04-04 1:43 ` Tejun Heo
2006-04-04 2:12 ` Albert Lee
2006-04-04 2:15 ` Tejun Heo
2006-04-04 2:49 ` Albert Lee
2006-04-04 3:05 ` Tejun Heo
2006-04-04 8:24 ` Jeff Garzik
2006-04-05 15:48 ` Jonathan Blake Benson
2006-04-06 9:17 ` Albert Lee
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=4434BFF3.3080005@tw.ibm.com \
--to=albertcc@tw.ibm.com \
--cc=Carlos.Pardo@siliconimage.com \
--cc=airbatica@verizon.net \
--cc=albertl@mail.com \
--cc=dwm@falcon30.maxeymade.com \
--cc=htejun@gmail.com \
--cc=jeff@garzik.org \
--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 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).