qlge: Fix bug in firmware event handler.
authorRon Mercer <ron.mercer@qlogic.com>
Sun, 7 Jun 2009 13:58:26 +0000 (13:58 +0000)
committerDavid S. Miller <davem@davemloft.net>
Mon, 8 Jun 2009 07:09:45 +0000 (00:09 -0700)
Each firmware mailbox command can have a different number of
valid data elements. When waiting for a mailbox command to
complete it the process passes it's element count and
waits for the completion.  It is possible that while waiting
an unrelated firmware async event (AE) can arrive.  When this
happens, the handler will over write the element count
with the value for the newly arrived AE. This can cause
the mailbox command to not get all of it's data.
This patch restores original mailbox count at the end
of the handler.

Signed-off-by: Ron Mercer <ron.mercer@qlogic.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/qlge/qlge_mpi.c

index 7cb30fdc9bac10700b413c75ef371cee38366732..ff8424330a9cf4d5c5691a5c6dbd827c8bc8225c 100644 (file)
@@ -453,6 +453,13 @@ static int ql_mpi_handler(struct ql_adapter *qdev, struct mbox_params *mbcp)
        }
 end:
        ql_write32(qdev, CSR, CSR_CMD_CLR_R2PCI_INT);
+       /* Restore the original mailbox count to
+        * what the caller asked for.  This can get
+        * changed when a mailbox command is waiting
+        * for a response and an AEN arrives and
+        * is handled.
+        * */
+       mbcp->out_count = orig_count;
        return status;
 }