linux-kernel - RE: [PATCH] ata: Add the SW NCQ support to sata

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <15F501D1A78BD343BE8F4D8DB854566B15F9A3F5@hkemmail01.nvidia.com>
Date:	Mon, 9 Jul 2007 10:48:33 +0800
From:	"Peer Chen" <pchen@...dia.com>
To:	"Zoltan Boszormenyi" <zboszor@...aweb.hu>,
	"kuan luo" <kuanluo@...il.com>
Cc:	<linux-kernel@...r.kernel.org>, <jeff@...zik.org>,
	<akpm@...ux-foundation.org>, <hancockr@...w.ca>,
	<prakash@...noor.de>, "Kuan Luo" <kluo@...dia.com>
Subject: RE: [PATCH] ata: Add the SW NCQ support to sata_nv for MCP51/MCP55/MCP61

It's a rule for all our drivers not just for linux, also if we put the Maxtor drive to the blacklist so that its NCQ function won't be enable for all controllers of other vendors,but we don't have the strong evidence that those Maxtor HDDs are also broken for other controllers.


BRs
Peer Chen

-----Original Message-----
From: Zoltan Boszormenyi [mailto:zboszor@...aweb.hu] 
Sent: Saturday, July 07, 2007 3:33 PM
To: kuan luo
Cc: linux-kernel@...r.kernel.org; jeff@...zik.org; akpm@...ux-foundation.org; hancockr@...w.ca; prakash@...noor.de; Peer Chen; Kuan Luo
Subject: Re: [PATCH] ata: Add the SW NCQ support to sata_nv for MCP51/MCP55/MCP61

kuan luo írta:
> From: Kuan Luo <kluo@...dia.com>
>
> 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'. NCQ will be turned off if the drive is Maxtor on MCP51 
> or
> MCP55 rev 0xa2  platform.
>
> Signed-off-by: Kuan Luo <kluo@...dia.com>
> Signed-off-by: Peer Chen <pchen@...dia.com>
> ---

Thanks, I am using it on 2.6.22-rc7-git5, have run a stress test yesterday night.
It seems to be as stable as the previous version. I guess it's safe to turn it on by default when it gets into Linus' kernels.

I have a question though. Why is the blanket needed for Maxtor drives?
Can't it be narrowed down to certain models? Maybe those models should be put into libata blacklist...
Not that my current machine is affected, I am just curious.

Best regards,
Zoltán Böszörményi


-----------------------------------------------------------------------------------
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@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/