Re: [PATCH,v2][1/2] sata-mv: enable HDD led blinking when NCQ is active for GenIIe

[Mark Lord <liml@rtr.ca>]

Frans Pop wrote:
> I missed the fact that you now call mv_edma_cfg from mv_stop_edma, so the 
> simplification is now indeed possible.
> 
> Tested and everything still works.
> 
>> I have so far not tested the led bahavior when DMA is disabled (not
>> even sure if that's possible for a running system).
> 
> It seems that with libata-based drivers disabling dma is only possible at 
> boot time and not dynamically, so this isn't possible anyway.
> I don't know if access errors could cause the driver to fall back to PIO, 
> but I don't really want to test that :-)
..

Yeah, tricky to see that.  With sata_mv, though, DMA is disabled
for any irregular (non block layer R/W) commands.

So the code path is being exercised, just too quickly to see if
the LED actually blinks much in non-DMA modes.
I think you could test the non-DMA mode with this:

  #!/bin/bash
  i=0
  while ( sleep 0.3 ) ; do hdparm --read-sector $i --direct /dev/sda ; i=$((i + 1)) ; done

That should work.
you could even vary the sleep value to change the observed LED blinking rate.

I'll queue up the patch for 2.6.29 now, but if you could try the above
then that would also be cool.

Thanks