On Wed, 1 Jul 2020 15:53:15 +0100 Edward Cree wrote:
> Signed-off-by: Edward Cree <ec...@solarflare.com>
> ---
>  drivers/net/ethernet/sfc/efx.c          | 3 ---
>  drivers/net/ethernet/sfc/efx_channels.c | 4 ++++
>  2 files changed, 4 insertions(+), 3 deletions(-)
> 
> diff --git a/drivers/net/ethernet/sfc/efx.c b/drivers/net/ethernet/sfc/efx.c
> index f4173f855438..9f659cc34252 100644
> --- a/drivers/net/ethernet/sfc/efx.c
> +++ b/drivers/net/ethernet/sfc/efx.c
> @@ -336,9 +336,6 @@ static int efx_probe_nic(struct efx_nic *efx)
>                                   sizeof(efx->rss_context.rx_hash_key));
>       efx_set_default_rx_indir_table(efx, &efx->rss_context);
>  
> -     netif_set_real_num_tx_queues(efx->net_dev, efx->n_tx_channels);
> -     netif_set_real_num_rx_queues(efx->net_dev, efx->n_rx_channels);
> -
>       /* Initialise the interrupt moderation settings */
>       efx->irq_mod_step_us = DIV_ROUND_UP(efx->timer_quantum_ns, 1000);
>       efx_init_irq_moderation(efx, tx_irq_mod_usec, rx_irq_mod_usec, true,
> diff --git a/drivers/net/ethernet/sfc/efx_channels.c 
> b/drivers/net/ethernet/sfc/efx_channels.c
> index 41bf18f2b081..dd6ee60b66a0 100644
> --- a/drivers/net/ethernet/sfc/efx_channels.c
> +++ b/drivers/net/ethernet/sfc/efx_channels.c
> @@ -894,6 +894,10 @@ int efx_set_channels(struct efx_nic *efx)
>                       }
>               }
>       }
> +
> +     netif_set_real_num_tx_queues(efx->net_dev, efx->n_tx_channels);
> +     netif_set_real_num_rx_queues(efx->net_dev, efx->n_rx_channels);

For a third time in last 7 days - these can fail :)

I know you're just moving them, but perhaps worth cleaning this up..

Reply via email to