]> git.baikalelectronics.ru Git - kernel.git/commit
i40e: Fix state flags for bit set and clean operations of PF
authorMauro S. M. Rodrigues <maurosr@linux.vnet.ibm.com>
Sat, 13 May 2017 02:26:56 +0000 (23:26 -0300)
committerJeff Kirsher <jeffrey.t.kirsher@intel.com>
Tue, 6 Jun 2017 09:45:32 +0000 (02:45 -0700)
commit3f231ac20c20a65ef2508d41bcaed1cf35c3bb4c
tree84543952579f0682f319d7f59efde85c5f6984a2
parent8b5fe2030439bdbf69ae1044d99dfeb1c40e39dd
i40e: Fix state flags for bit set and clean operations of PF

Commit 7a8b3f0b72c5 ("i40e: use DECLARE_BITMAP for state fields")
introduced changes in the way i40e works with state flags converting
them to bitmaps using kernel bitmap API. This change introduced a
regression due to a mistaken substitution using __I40E_VSI_DOWN instead
of __I40E_DOWN when testing state of a PF at i40e_reset_subtask()
function. This caused a flood in the kernel log with the follow message:

[49.013] i40e 0002:01:00.0: bad reset request 0x00000020

Commit 9e58ebc1f34e ("i40e: separate PF and VSI state flags")
also introduced some misuse of the VSI and PF flags, so both could be
considered as the offenders.

This patch simply fixes the flags where it makes sense by changing
__I40E_VSI_DOWN to __I40E_DOWN.

Fixes: 7a8b3f0b72c5 ("i40e: use DECLARE_BITMAP for state fields")
Fixes: 9e58ebc1f34e ("i40e: separate PF and VSI state flags")
Reviewed-by: "Guilherme G. Piccoli" <gpiccoli@linux.vnet.ibm.com>
Signed-off-by: "Mauro S. M. Rodrigues" <maurosr@linux.vnet.ibm.com>
Tested-by: Andrew Bowers <andrewx.bowers@intel.com>
Signed-off-by: Jeff Kirsher <jeffrey.t.kirsher@intel.com>
drivers/net/ethernet/intel/i40e/i40e_main.c