nata: process interrupt even if busmaster active bit is set
authorSimon Schubert <corecode@dragonflybsd.org>
Thu, 29 Jan 2009 09:34:03 +0000 (10:34 +0100)
committerSimon Schubert <corecode@dragonflybsd.org>
Thu, 29 Jan 2009 09:47:21 +0000 (10:47 +0100)
commit9f2212d66176922cab4310c64dc12b991018c2f5
tree174b57bf834f3ec09285b210681bd9a684b21825
parentfbc26094424ad24410e9d0c758c2eec113edced4
nata: process interrupt even if busmaster active bit is set

The nata driver previously assumed that an interrupt which has the
interrupt and active bits set in the busmaster status register is not a
valid/completed interrupt.  In these situations if would return without
acknowledging the irq, leading to an interrupt livelock situation.

This condition of INTERRUPT|ACTIVE occured on two different machines
both equipped with different versions of an ICH SATA controller, when
accessing an attached SATA ATAPI (DVD) drive by TSScorp (Samsung
OEM/bulk).  Specifically, natapicam seems to trigger this condition on
attach.  It seems that the drive is unhappy with the INQUIRY command
sent by natapicam.  Related artifacts appear when trying to access a
video DVD (possibly due to missing region/css key).

Under the hood it seems that the drive reports an error condition
(visible in the ALT/STATUS register), but does not clear the ACTIVE bit
of the busmaster status.  As a result the nata driver could be waiting
for the completion of the transfer, while the SATA busmaster would keep
the interrupt line asserted.

We work around this issue by following the precedent of other operating
systems and ignoring the ACTIVE bit if the INTERRUPT bit is set.
Specifically, we wait a little bit to accomodate for possibly faulty
hardware which might assert the irq before the transfer is completely
finished.  Nevertheless, we do not reject interrupt proecessing if the
INTERRUPT bit is set in the busmaster status register.
sys/dev/disk/nata/ata-pci.c