On Thu, Jul 23, 2009 at 12:01:46PM +0800, Jeff Garzik wrote: > Shaohua Li wrote: > > Hi, > > I did some tests about NCQ. It appears enabling NCQ sometimes will make > > disk slower than disabling it (well it does improve performance in > > random io case). Test using fio with attached script. > > With queue_depth 1 (so NCQ is disabled), the disk speed is about 27m/s. > > with queue_depth 2 - 31, the speed is about 24m/s. > > > > blktrace shows the Q2C time with NCQ enabled increases about 10%. > > Specifically the D2C time increases and time of other stages hasn't > > obvious changes. > > > > I added a 'udelay()' at the end of ahci_qc_issue(), as my test does > > 'submit a request, wait it and then submit another', so if the udelay > > time is shorter than the disk io time, the udelay should not be harmful. > > My test shows when the udelay time is smaller than 90us, the delay > > hasn't any impact to the ncq enabled case. On the other hand, the udelay > > time should be shorter than 30us otherwise the ncq disabled case has > > obvious performance downgrade. This test seems suggesting diskio is > > slower in ncq enabled. Is the single diskio slow in ncq case expected? > > You should test disks from different vendors and generations... > > NCQ is _usually_ a win, especially for multi-threaded applications. > Tests from years ago seemed to indicate that the advantage maxed out at > 4-8 queued commands, and added little value after that. > > In addition, some NCQ disks and some system workloads may underperform, > but this does not seem to be the general case. I just occasionally did the test and hadn't resource to do a full test. I tested two hard disks (one from WDC the other Seegate), both have the same behavior. If it's possible hardware will slower with NCQ in some cases, I'll not borther spending time on it any more. So can you guys suggest if the issue is related with hardware? Thanks, Shaohua -- 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