[Intel-wired-lan] [PATCH net 09/13] ice: remove DCBNL_DEVRESET bit from PF state
Brelinski, TonyX
tonyx.brelinski at intel.com
Mon Mar 22 23:58:34 UTC 2021
> -----Original Message-----
> From: Intel-wired-lan <intel-wired-lan-bounces at osuosl.org> On Behalf Of
> Tony Nguyen
> Sent: Friday, February 26, 2021 1:19 PM
> To: intel-wired-lan at lists.osuosl.org
> Subject: [Intel-wired-lan] [PATCH net 09/13] ice: remove DCBNL_DEVRESET
> bit from PF state
>
> From: Dave Ertman <david.m.ertman at intel.com>
>
> The original purpose of the ICE_DCBNL_DEVRESET was to protect the driver
> during DCBNL device resets. But, the flow for DCBNL device resets now
> consists of only calls up the stack such as dev_close() and dev_open() that
> will result in NDO calls to the driver. These will be handled with state changes
> from the stack. Also, there is a problem of the dev_close and dev_open
> being blocked by checks for reset in progress also using the
> ICE_DCBNL_DEVRESET bit.
>
> Since the ICE_DCBNL_DEVRESET bit is not necessary for protecting the driver
> from DCBNL device resets and it is actually blocking changes coming form the
> DCBNL interface, remove the bit from the PF state and don't block driver
> function based on DCBNL reset in progress.
>
> Fixes: b94b013eb626 ("ice: Implement DCBNL support")
> Signed-off-by: Dave Ertman <david.m.ertman at intel.com>
> ---
> drivers/net/ethernet/intel/ice/ice.h | 1 -
> drivers/net/ethernet/intel/ice/ice_dcb_nl.c | 2 --
> drivers/net/ethernet/intel/ice/ice_lib.c | 1 -
> 3 files changed, 4 deletions(-)
Tested-by: Tony Brelinski <tonyx.brelinski at intel.com> A Contingent Worker at Intel
More information about the Intel-wired-lan
mailing list