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:[EMAIL PROTECTED] Sent: Wednesday, June 27, 2007 1:07 PM To: kuan luo Cc: linux-kernel@vger.kernel.org; [EMAIL PROTECTED]; [EMAIL PROTECTED]; 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 <[EMAIL PROTECTED]> > Signed-off-by: Peer Chen <[EMAIL PROTECTED]> > 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 [EMAIL PROTECTED] 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 [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/