Re: [REPOST PATCH] sata_promise: use TF interface for polling NODATA commands

[Jeff Garzik <jeff@garzik.org>]

On Thu, Jun 07, 2007 at 01:25:07PM -0400, Jeff Garzik wrote:
> On Thu, Jun 07, 2007 at 01:20:13PM -0400, Chuck Ebbert wrote:
> > On 06/06/2007 12:05 PM, Jeff Garzik wrote:
> > > FYI to all -- 
> > > 
> > > As a reminder. the Promise hardware programs registers when it receives
> > > a SET FEATURES - XFER MODE.
> > > 
> > > If data transfer is occurring on OTHER ports at the time this is issued,
> > > then data corruption is guaranteed to occur.  Polling will not fix this
> > > problem -- all ports need to be inactive, when a SET FEATURES - XFER
> > > MODE command is issued for any port.
> 
> > So is this patch OK but yet more work needs to be done, or does
> > this patch cause new problems?
> 
> Causes no /new/ problems...  :)  The existing problem described above
> remains.

Just to expand...  this problem doesn't really affect a lot of users in
the majority case, since we do speed tuning before data transfer starts.

The main problem that remains is the rare (but no less important) case
where libata-EH will tune speed during operation in respond to certain
classes of errors.

	Jeff