Commit 2b0c2e2d authored by Sujith Sankar's avatar Sujith Sankar Committed by David S. Miller

enic: do notify_check before returning credits

We should complete notify_check before returning the credits. Once we return the
credits, adaptor may access the notify data.
Signed-off-by: default avatarSujith Sankar <ssujith@cisco.com>
Signed-off-by: default avatarGovindarajulu Varadarajan <_govind@gmx.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent 31639b94
...@@ -272,8 +272,8 @@ static irqreturn_t enic_isr_legacy(int irq, void *data) ...@@ -272,8 +272,8 @@ static irqreturn_t enic_isr_legacy(int irq, void *data)
} }
if (ENIC_TEST_INTR(pba, notify_intr)) { if (ENIC_TEST_INTR(pba, notify_intr)) {
vnic_intr_return_all_credits(&enic->intr[notify_intr]);
enic_notify_check(enic); enic_notify_check(enic);
vnic_intr_return_all_credits(&enic->intr[notify_intr]);
} }
if (ENIC_TEST_INTR(pba, err_intr)) { if (ENIC_TEST_INTR(pba, err_intr)) {
...@@ -346,8 +346,8 @@ static irqreturn_t enic_isr_msix_notify(int irq, void *data) ...@@ -346,8 +346,8 @@ static irqreturn_t enic_isr_msix_notify(int irq, void *data)
struct enic *enic = data; struct enic *enic = data;
unsigned int intr = enic_msix_notify_intr(enic); unsigned int intr = enic_msix_notify_intr(enic);
vnic_intr_return_all_credits(&enic->intr[intr]);
enic_notify_check(enic); enic_notify_check(enic);
vnic_intr_return_all_credits(&enic->intr[intr]);
return IRQ_HANDLED; return IRQ_HANDLED;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment