Messages in this thread | | | Subject | RE: [PATCH] ata: Add the SW NCQ support to sata_nv for MCP51/MCP55/MCP61 | Date | Wed, 27 Jun 2007 16:15:41 +0800 | From | "Peer Chen" <> |
| |
We did the many test with the new version driver and didn't encounter that problem, but in certain cases, DMASETUP command packets from drive to the controller are corrupted, and the controller issues an R_ERR to the drive. Drives that comply with SATA spec will re-transmit the corrupted packet and normal operation continues. However, some Maxtor NCQ drives do not re-transmit the DMASETUP command packet, resulting in software timeout for this command. So if you are using the Maxtor HD and meet this problem,please don't enable the NCQ function.
BRs Peer Chen
-----Original Message----- From: Robert Hancock [mailto:hancockr@shaw.ca] Sent: Wednesday, June 27, 2007 1:07 PM To: kuan luo Cc: linux-kernel@vger.kernel.org; jeff@garzik.org; akpm@linux-foundation.org; Peer Chen; Kuan Luo Subject: Re: [PATCH] ata: Add the SW NCQ support to sata_nv for MCP51/MCP55/MCP61
kuan luo wrote: > Add the Software NCQ support to sata_nv.c for MCP51/MCP55/MCP61 SATA > controller. > NCQ function is disable by default, you can enable it with 'swncq=1' > > Signed-off-by: Kuan Luo <kluo@nvidia.com> > Signed-off-by: Peer Chen <pchen@nvidia.com> >
Haven't reviewed in detail, but does look cleaner than the previous version. Some people reported seeing some unrecognized FIS, etc. errors with the previous version, have those been looked into/fixed?
-- Robert Hancock Saskatoon, SK, Canada To email, remove "nospam" from hancockr@nospamshaw.ca Home Page: http://www.roberthancock.com/
----------------------------------------------------------------------------------- This email message is for the sole use of the intended recipient(s) and may contain confidential information. Any unauthorized review, use, disclosure or distribution is prohibited. If you are not the intended recipient, please contact the sender by reply email and destroy all copies of the original message. ----------------------------------------------------------------------------------- - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |