]> www.infradead.org Git - users/jedix/linux-maple.git/commit
firewire: ohci: fix too-early completion of IR multichannel buffers
authorClemens Ladisch <clemens@ladisch.de>
Mon, 12 Mar 2012 20:45:47 +0000 (21:45 +0100)
committerMaxim Uvarov <maxim.uvarov@oracle.com>
Thu, 12 Apr 2012 22:14:37 +0000 (15:14 -0700)
commite1edb533724a1d502ee70f0a2b34b5f2e688f286
treec22e9d6fa80fc76831e55b905e282071bcddcb97
parentde6e61dc9cd1bfa94ec9356b85fc723eb219769a
firewire: ohci: fix too-early completion of IR multichannel buffers

commit 0c0efbacab8d70700d13301e0ae7975783c0cb0a upstream.

handle_ir_buffer_fill() assumed that a completed descriptor would be
indicated by a non-zero transfer_status (as in most other descriptors).
However, this field is written by the controller as soon as (the end of)
the first packet has been written into the buffer.  As a consequence, if
we happen to run into such a descriptor when the interrupt handler is
executed after such a packet has completed, the descriptor would be
taken out of the list of active descriptors as soon as the buffer had
been partially filled, so the event for the buffer being completely
filled would never be sent.

To fix this, handle descriptors only when they have been completely
filled, i.e., when res_count == 0.  (This also matches the condition
that is reported by the controller with an interrupt.)

Signed-off-by: Clemens Ladisch <clemens@ladisch.de>
Signed-off-by: Stefan Richter <stefanr@s5r6.in-berlin.de>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/firewire/ohci.c