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
--
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
