RE: [PATCH] libata-pmp: add schedule timeout to support some PMP cards

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




Thanks and Best Regards
Jerry Huang

> -----Original Message-----
> From: Tejun Heo [mailto:htejun@xxxxxxxxx] On Behalf Of Tejun Heo
> Sent: Thursday, November 10, 2011 11:21 PM
> To: Huang Changming-R66093
> Cc: linux-ide@xxxxxxxxxxxxxxx
> Subject: Re: [PATCH] libata-pmp: add schedule timeout to support some
> PMP cards
> 
> Hello,
> 
> On Thu, Nov 10, 2011 at 08:07:47AM +0000, Huang Changming-R66093 wrote:
> > > Ummm... We can't really issue commands after failure without going
> > > through recovery.  For ahci, it probably works.  For other
> > > controllers, it may not.  Where does this delay come from?  Is
> there
> > > any other way to wait for device readiness?
> > >
> > It is very bad.  I have two PMP cards, one (5 ports) can work well
> > without the delay time before reading PMP SCR, but the other (15
> > ports) must delay enough time before reading the PMP SCR.  It seems
> > there is no other way to detect the device readiness.
> >
> > Maybe I will just delay some time before send SCR command, if it
> > failed, then we will report this error to driver?
> 
> Hmmm... I'm curious what's causing the problem.  Which port multiplier
> are we talking about?  If it's specific to that PMP, we can add quirk
> to support it.  Does the problem also happen when the PMP is connected
> to a different controller?
> 
Hi,
The PMP card we are talking about is JMB393:
Port Multiplier 1.2, 0x197b:0x0325 r0, 15 ports, feat 0x5/0xf.

This issue is observed on Freescale's Sata controller, including p1020e and mpc837xe.

--
To unsubscribe from this list: send the line "unsubscribe linux-ide" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux Filesystems]     [Linux SCSI]     [Linux RAID]     [Git]     [Kernel Newbies]     [Linux Newbie]     [Security]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Samba]     [Device Mapper]

  Powered by Linux