ixgbe: There is no need to update num_rx_pools in L2 fwd offload
The num_rx_pools value is overwritten when we reinitialize the queue configuration. In reality we shouldn't need to be updating the value since it is redone every time we call into ixgbe_setup_tc so for now just drop the spots where we were incrementing or decrementing the value. Signed-off-by: Alexander Duyck <alexander.h.duyck@intel.com> Tested-by: Andrew Bowers <andrewx.bowers@intel.com> Signed-off-by: Jeff Kirsher <jeffrey.t.kirsher@intel.com>
This commit is contained in:
parent
2af62c5614
commit
ff815fb2cf
@ -701,7 +701,7 @@ static void ixgbe_set_num_queues(struct ixgbe_adapter *adapter)
|
||||
adapter->num_rx_queues = 1;
|
||||
adapter->num_tx_queues = 1;
|
||||
adapter->num_xdp_queues = 0;
|
||||
adapter->num_rx_pools = adapter->num_rx_queues;
|
||||
adapter->num_rx_pools = 1;
|
||||
adapter->num_rx_queues_per_pool = 1;
|
||||
|
||||
#ifdef CONFIG_IXGBE_DCB
|
||||
|
@ -9843,7 +9843,6 @@ static void *ixgbe_fwd_add(struct net_device *pdev, struct net_device *vdev)
|
||||
return ERR_PTR(-ENOMEM);
|
||||
|
||||
pool = find_first_zero_bit(&adapter->fwd_bitmask, 32);
|
||||
adapter->num_rx_pools++;
|
||||
set_bit(pool, &adapter->fwd_bitmask);
|
||||
limit = find_last_bit(&adapter->fwd_bitmask, 32);
|
||||
|
||||
@ -9872,7 +9871,6 @@ static void *ixgbe_fwd_add(struct net_device *pdev, struct net_device *vdev)
|
||||
netdev_info(pdev,
|
||||
"%s: dfwd hardware acceleration failed\n", vdev->name);
|
||||
clear_bit(pool, &adapter->fwd_bitmask);
|
||||
adapter->num_rx_pools--;
|
||||
kfree(fwd_adapter);
|
||||
return ERR_PTR(err);
|
||||
}
|
||||
@ -9884,7 +9882,6 @@ static void ixgbe_fwd_del(struct net_device *pdev, void *priv)
|
||||
unsigned int limit;
|
||||
|
||||
clear_bit(fwd_adapter->pool, &adapter->fwd_bitmask);
|
||||
adapter->num_rx_pools--;
|
||||
|
||||
limit = find_last_bit(&adapter->fwd_bitmask, 32);
|
||||
adapter->ring_feature[RING_F_VMDQ].limit = limit + 1;
|
||||
|
Loading…
Reference in New Issue
Block a user