]> pilppa.com Git - linux-2.6-omap-h63xx.git/commitdiff
sata_nv: revert use of notifiers for now
authorRobert Hancock <hancockr@shaw.ca>
Fri, 9 Mar 2007 00:02:18 +0000 (18:02 -0600)
committerJeff Garzik <jeff@garzik.org>
Fri, 9 Mar 2007 13:41:16 +0000 (08:41 -0500)
Commit 721449bf0d51213fe3abf0ac3e3561ef9ea7827a added support for using the
ADMA notifier bits to determine which commands to check for completion.
However there have been reports that this causes command timeouts in certain
cases. This is still being investigated. In addition, apparently the notifiers
won't work if ADMA is disabled on the other port as a result of an ATAPI device
being connected, and we don't handle this case properly.

For now, just restore the previous behavior of checking all active commands
to see if they are complete, without relying on the notifiers.

Signed-off-by: Robert Hancock <hancockr@shaw.ca>
Signed-off-by: Jeff Garzik <jeff@garzik.org>
drivers/ata/sata_nv.c

index 388d07fab5f7dcc465d1d962db09ac8a65909b24..9d9670a9b117e67d3689b8f5b2a1aa4b586f9ac6 100644 (file)
@@ -874,8 +874,14 @@ static irqreturn_t nv_adma_interrupt(int irq, void *dev_instance)
 
                        if (status & (NV_ADMA_STAT_DONE |
                                      NV_ADMA_STAT_CPBERR)) {
-                               u32 check_commands = notifier | notifier_error;
+                               u32 check_commands;
                                int pos, error = 0;
+
+                               if(ata_tag_valid(ap->active_tag))
+                                       check_commands = 1 << ap->active_tag;
+                               else
+                                       check_commands = ap->sactive;
+
                                /** Check CPBs for completed commands */
                                while ((pos = ffs(check_commands)) && !error) {
                                        pos--;