public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Zoltan Boszormenyi <zboszor@dunaweb.hu>
To: kuan luo <kuanluo@gmail.com>
Cc: linux-kernel@vger.kernel.org, jeff@garzik.org,
	akpm@linux-foundation.org, hancockr@shaw.ca, prakash@punnoor.de,
	pchen@nvidia.com, kluo@nvidia.com
Subject: Re: [PATCH] ata: Add the SW NCQ support to sata_nv for MCP51/MCP55/MCP61
Date: Sat, 07 Jul 2007 09:33:26 +0200	[thread overview]
Message-ID: <468F41C6.3000109@dunaweb.hu> (raw)
In-Reply-To: <53f198c00707060131x1e1abdd7l4396c7fede360970@mail.gmail.com>

kuan luo írta:
> From: Kuan Luo <kluo@nvidia.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@nvidia.com>
> Signed-off-by: Peer Chen <pchen@nvidia.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



  reply	other threads:[~2007-07-07  7:33 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-07-06  8:31 [PATCH] ata: Add the SW NCQ support to sata_nv for MCP51/MCP55/MCP61 kuan luo
2007-07-07  7:33 ` Zoltan Boszormenyi [this message]
2007-07-09  2:48   ` Peer Chen
2007-07-09 16:28     ` Prakash Punnoor
  -- strict thread matches above, loose matches on Subject: below --
2007-06-27 10:51 Zoltan Boszormenyi
2007-07-03 22:31 ` Zoltan Boszormenyi
2007-07-05 18:53   ` Bill Davidsen
     [not found] <fa.5TBvnXzjCgAIfcuKOLBWeXi+y5I@ifi.uio.no>
2007-06-27  5:06 ` Robert Hancock
2007-06-27  8:15   ` Peer Chen
2007-06-27  8:34     ` Jeff Garzik
2007-06-27 16:09     ` Prakash Punnoor
2007-06-27  3:04 kuan luo
2007-06-27  5:26 ` Andrew Morton
2007-06-27  9:09   ` Kuan Luo
2007-06-27  9:21     ` Andrew Morton
2007-06-27 10:17       ` Kuan Luo

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=468F41C6.3000109@dunaweb.hu \
    --to=zboszor@dunaweb.hu \
    --cc=akpm@linux-foundation.org \
    --cc=hancockr@shaw.ca \
    --cc=jeff@garzik.org \
    --cc=kluo@nvidia.com \
    --cc=kuanluo@gmail.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=pchen@nvidia.com \
    --cc=prakash@punnoor.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body. 
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox