Mark Lord wrote:
Jeff Garzik wrote:
Mark Lord wrote:
Meanwhile, no further action required here.
ACK :)
And thanks for rounding out the NCQ work. sata_mv has needed love and
attention for a while (well, really, its entire life).
..
Well, it's going to be getting plenty of TLC over the next few months.
In the short term, my plan is to submit further small patches to fix
the IRQ and error-handling in sata_mv, as bug fixes for 2.6.25.
Note that hot plug/unplug will begin to work correctly once the IRQ/EH
code gets fixed (it sort of works already, but sometimes kills the
machine).
There are also some errata that need to be addressed in the 2.6.25
timeframe.
In particular, there's an NCQ EH errata for the 60x1 chips,
and a tricky issue about HOB access not working correctly on
most versions of the chips.
Bigger stuff that I'm deferring for 2.6.26:
-- Port multiplier support (though this does look rather simple..)
-- power management support
-- ATAPI
I'm interested to see this :)
-- IRQ Coalescing
Most "modern" SATA has some form of this, but I've yet to see any
benefit. I've dealt with interrupt (packet) rates of well over 500k/sec
in network land, and IMO the overhead in storage, even with tiny
operations, is really small in comparison.
So, I'm not sure its worth the latency penalty... at least as turned on
by default.
-- Target Mode support (interfaces yet to be defined)
I would assume this would be along the lines of the SCSI target mode stuff.
-- TCQ support: would be good in general for libata on smart hosts,
but I'm not sure of the impact on libata EH processing.
Agreed, it would be nice to support host queueing controllers.
However, specifically for TCQ, it was rather poorly conceived. For most
controllers (mv, broadcom/svw, others) an error will stop the DMA
engine, and you perform recovery in software. All well and good, but
figuring out all the states possible during recovery is non-trivial (I
looked into it years ago). Its just messy.
Jeff
-
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
