diff options
| author | Jacob Keller <[email protected]> | 2017-09-07 12:05:49 +0000 |
|---|---|---|
| committer | Jeff Kirsher <[email protected]> | 2017-10-09 21:21:26 +0000 |
| commit | dbadbbe235f82f13224c85d29e65cf859afaa18c (patch) | |
| tree | 0d1bb9ea2adb11908f18685f4608d03566dfd2f0 /drivers/net/ethernet/intel/i40e/i40e_virtchnl_pf.c | |
| parent | i40e/i40evf: fix incorrect default ITR values on driver load (diff) | |
| download | kernel-dbadbbe235f82f13224c85d29e65cf859afaa18c.tar.gz kernel-dbadbbe235f82f13224c85d29e65cf859afaa18c.zip | |
i40e/i40evf: always set the CLEARPBA flag when re-enabling interrupts
In the past we changed driver behavior to not clear the PBA when
re-enabling interrupts. This change was motivated by the flawed belief
that clearing the PBA would cause a lost interrupt if a receive
interrupt occurred while interrupts were disabled.
According to empirical testing this isn't the case. Additionally, the
data sheet specifically says that we should set the CLEARPBA bit when
re-enabling interrupts in a polling setup.
This reverts commit 40d72a509862 ("i40e/i40evf: don't lose interrupts")
Signed-off-by: Jacob Keller <[email protected]>
Tested-by: Andrew Bowers <[email protected]>
Signed-off-by: Jeff Kirsher <[email protected]>
Diffstat (limited to 'drivers/net/ethernet/intel/i40e/i40e_virtchnl_pf.c')
| -rw-r--r-- | drivers/net/ethernet/intel/i40e/i40e_virtchnl_pf.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/net/ethernet/intel/i40e/i40e_virtchnl_pf.c b/drivers/net/ethernet/intel/i40e/i40e_virtchnl_pf.c index c062d74d21f3..10298956a81b 100644 --- a/drivers/net/ethernet/intel/i40e/i40e_virtchnl_pf.c +++ b/drivers/net/ethernet/intel/i40e/i40e_virtchnl_pf.c @@ -1358,7 +1358,7 @@ err_alloc: i40e_free_vfs(pf); err_iov: /* Re-enable interrupt 0. */ - i40e_irq_dynamic_enable_icr0(pf, false); + i40e_irq_dynamic_enable_icr0(pf); return ret; } |