[Intel-wired-lan] [PATCH net v2] iavf: Fix promiscuous mode configuration flow messages

[Nguyen, Anthony L]

On Fri, 2021-10-29 at 14:20 +0200, Karen Sornek wrote:
> Currently when configuring promiscuous mode on the AVF we detect a
> change in the netdev->flags. We use IFF_PROMISC and IFF_ALLMULTI to
> determine whether or not we need to request/release promiscuous mode
> and/or multicast promiscuous mode. The problem is that the AQ calls
> for
> setting/clearing promiscuous/multicast mode are treated separately.
> This
> leads to a case where we can trigger two promiscuous mode AQ calls in
> a row with the incorrect state. To fix this make a few changes.
> 
> Use IAVF_FLAG_AQ_CONFIGURE_PROMISC_MODE instead of the previous
> IAVF_FLAG_AQ_[REQUEST|RELEASE]_[PROMISC|ALLMULTI] flags.
> 
> In iavf_set_rx_mode() detect if there is a change in the
> netdev->flags in comparison with adapter->flags and set the
> IAVF_FLAG_AQ_CONFIGURE_PROMISC_MODE aq_required bit. Then in
> iavf_process_aq_command() only check for
> IAVF_FLAG_CONFIGURE_PROMISC_MODE
> and call iavf_set_promiscuous() if it's set.
> 
> In iavf_set_promiscuous() check again to see which (if any)
> promiscuous mode bits have changed when comparing the netdev->flags
> with
> the adapter->flags. Use this to set the flags which get sent to the
> PF
> driver.
> 
> Add a spinlock that is used for updating current_netdev_promisc_flags
> and only allows one promiscuous mode AQ at a time.
> 
> [1] Fixes the fact that we will only have one AQ call in the
> aq_required
> queue at any one time.
> 
> [2] Streamlines the change in promiscuous mode to only set one AQ
> required bit.
> 
> [3] This allows us to keep track of the current state of the flags
> and
> also makes it so we can take the most recent netdev->flags
> promiscuous
> mode state.
> 
> [4] This fixes the problem where a change in the netdev->flags can
> cause
> IAVF_FLAG_AQ_CONFIGURE_PROMISC_MODE to be set in iavf_set_rx_mode(),
> but cleared in iavf_set_promiscuous() before the change is ever made
> via
> AQ call.
> 
> Fixes: 129cf89e5856 ("iavf: rename functions and structs to new
> name")
> Title: iavf: Fix promiscuous mode configuration flow messages
> Change-type: DefectResolution
> HSDES-Number: 18017803943
> HSDES-Tenant: server_platf_lan.bug

Please remove these unneccessary lines.

> Signed-off-by: Brett Creeley <brett.creeley at intel.com>
> Signed-off-by: Karen Sornek <karen.sornek at intel.com>
> ---
> v2 - previous version had applying issue

This still does not apply. Please be sure you are using the right tree.
This should be using net-queue since you want it going to net.

> ---
>  drivers/net/ethernet/intel/iavf/iavf.h        | 16 ++--
>  drivers/net/ethernet/intel/iavf/iavf_main.c   | 44 +++++------
>  .../net/ethernet/intel/iavf/iavf_virtchnl.c   | 74 ++++++++++++-----
> --
>  3 files changed, 75 insertions(+), 59 deletions(-)