linux-ide.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Tejun Heo <htejun@gmail.com>
To: Mark Lord <liml@rtr.ca>
Cc: Brian & Chamaigne Scamman <scammanbc@earthlink.net>,
	jeff@garzik.org, linux-ide@vger.kernel.org
Subject: Re: [PATCHSET #upstream-fixes] git tree available
Date: Tue, 20 May 2008 13:28:11 +0900	[thread overview]
Message-ID: <4832535B.1080505@gmail.com> (raw)
In-Reply-To: <48323E4F.3050806@rtr.ca>

Mark Lord wrote:
> Brian & Chamaigne Scamman wrote:
>> Tejun-
>>
>> The hotplug fixes didn't solve the problem. After adding some "logic
>> monitoring" statements, I found that the successful discovery of the 
>> drives
>> depends on the response from sil24_exec_polled_cmd.
>>
>> If the call from ata_wait_register is 327680, the drives are 
>> recognized; if
>> the response is 262144 the drives have timed out.
>>
>> I've also noticed that the drives respond to the EH entering with DevExch
>> vs. PHY RDY changed.
>>
>> Bottom line, the drives seem to be recognized almost every time I attach
>> them to the PMP after the EH has completed processing the empty PMP.  
>> If the
>> drives are attached to the PMP before the PMP is attached to the 
>> controller,
>> the drives are almost never recognized (EH times out).

Can you please describe what you did exactly?  sil3726/4726 has some 
problems when its first fan out port goes online and offline while it's 
powered up.  You want to keep it occupied at all times.

Also, does putting sslep(5) right before followup-SRST help?

> hp-polling is beginning to look better and better..  again.  :)

hp-polling wouldn't really help here.  What's failing is not hotplug 
event detection, the reset protocol is failing.

-- 
tejun

  reply	other threads:[~2008-05-20  4:28 UTC|newest]

Thread overview: 30+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-05-18 16:15 [PATCHSET #upstream-fixes] libata: fix a bunch of PMP related problems Tejun Heo
2008-05-18 16:15 ` [PATCH 01/10] libata: fix sata_link_hardreset() @online out parameter handling Tejun Heo
2008-05-19 21:53   ` Jeff Garzik
2008-05-18 16:15 ` [PATCH 02/10] libata: reorganize ata_eh_reset() no reset method path Tejun Heo
2008-05-18 16:15 ` [PATCH 03/10] libata: move reset freeze/thaw handling into ata_eh_reset() Tejun Heo
2008-05-18 16:15 ` [PATCH 04/10] libata: kill hotplug related race condition Tejun Heo
2008-05-18 16:15 ` [PATCH 05/10] libata: ignore recovered PHY errors Tejun Heo
2008-05-19 21:50   ` Jeff Garzik
2008-05-18 16:15 ` [PATCH 06/10] libata: increase PMP register access timeout to 3s Tejun Heo
2008-05-18 16:15 ` [PATCH 07/10] libata: make sure PMP notification is turned off during recovery Tejun Heo
2008-05-18 16:15 ` [PATCH 08/10] libata: don't schedule LPM action seperately during probing Tejun Heo
2008-05-18 16:15 ` [PATCH 09/10] sata_sil24: don't use NCQ if marvell 4140 PMP is attached Tejun Heo
2008-05-18 21:14   ` Mark Lord
2008-05-18 16:15 ` [PATCH 10/10] libata: ignore SIMG4726 config pseudo device Tejun Heo
2008-05-18 16:29 ` [PATCHSET #upstream-fixes] git tree available Tejun Heo
2008-05-20  1:35   ` Brian & Chamaigne Scamman
2008-05-20  2:58     ` Mark Lord
2008-05-20  4:28       ` Tejun Heo [this message]
2008-05-20  4:43         ` Tejun Heo
2008-05-21  1:32           ` Brian & Chamaigne Scamman
2008-05-21  4:59             ` Tejun Heo
2008-05-21 11:14               ` Brian & Chamaigne Scamman
2008-05-21 19:42               ` Brian & Chamaigne Scamman
2008-05-22  0:40                 ` Tejun Heo
2008-05-23  0:49                   ` Brian & Chamaigne Scamman
2008-05-23  1:04                     ` Tejun Heo
2008-05-29  3:06                       ` Tejun Heo
2008-05-29  3:11                         ` Brian & Chamaigne Scamman
2008-05-20 12:08         ` Brian & Chamaigne Scamman
2008-05-20 14:50           ` 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=4832535B.1080505@gmail.com \
    --to=htejun@gmail.com \
    --cc=jeff@garzik.org \
    --cc=liml@rtr.ca \
    --cc=linux-ide@vger.kernel.org \
    --cc=scammanbc@earthlink.net \
    /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).