ACK: [X/B][PATCH] bnx2x: Disable multi-cos feature.

Stefan Bader stefan.bader at canonical.com
Tue Aug 27 14:35:47 UTC 2019


On 22.08.19 15:19, Mauricio Faria de Oliveira wrote:
> From: Sudarsana Reddy Kalluru <skalluru at marvell.com>
> 
> BugLink: https://bugs.launchpad.net/bugs/1840789
> 
> Commit 3968d38917eb ("bnx2x: Fix Multi-Cos.") which enabled multi-cos
> feature after prolonged time in driver added some regression causing
> numerous issues (sudden reboots, tx timeout etc.) reported by customers.
> We plan to backout this commit and submit proper fix once we have root
> cause of issues reported with this feature enabled.
> 
> Fixes: 3968d38917eb ("bnx2x: Fix Multi-Cos.")
> Signed-off-by: Sudarsana Reddy Kalluru <skalluru at marvell.com>
> Signed-off-by: Manish Chopra <manishc at marvell.com>
> Signed-off-by: David S. Miller <davem at davemloft.net>
> (backported from commit d1f0b5dce8fda09a7f5f04c1878f181d548e42f5)
> [mfo: backport: essentially revert the mentioned commit;
>  upstream more recently removed the fallback() function
>  and moved to netdev_pick_tx(), and has more arguments.
>  the parentheses are left in as done in this commit.]
> Signed-off-by: Mauricio Faria de Oliveira <mfo at canonical.com>
Acked-by: Stefan Bader <stefan.bader at canonical.com>
> ---
>  drivers/net/ethernet/broadcom/bnx2x/bnx2x_cmn.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/net/ethernet/broadcom/bnx2x/bnx2x_cmn.c b/drivers/net/ethernet/broadcom/bnx2x/bnx2x_cmn.c
> index 5d1796ab3403..641caef6ce31 100644
> --- a/drivers/net/ethernet/broadcom/bnx2x/bnx2x_cmn.c
> +++ b/drivers/net/ethernet/broadcom/bnx2x/bnx2x_cmn.c
> @@ -1954,7 +1954,7 @@ u16 bnx2x_select_queue(struct net_device *dev, struct sk_buff *skb,
>  	}
>  
>  	/* select a non-FCoE queue */
> -	return fallback(dev, skb) % (BNX2X_NUM_ETH_QUEUES(bp) * bp->max_cos);
> +	return fallback(dev, skb) % (BNX2X_NUM_ETH_QUEUES(bp));
>  }
>  
>  void bnx2x_set_num_queues(struct bnx2x *bp)
> 


-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: OpenPGP digital signature
URL: <https://lists.ubuntu.com/archives/kernel-team/attachments/20190827/a02be627/attachment.sig>


More information about the kernel-team mailing list