> Moving the code to handle the LAG part of a VF reset to helper functions
> deprecated the function ice_lag_move_new_vf_nodes().
> The cleanup missed a call to this function in the error path of
> ice_vc_cfg_qs_msg().
> 
> In the case that would end in the error path, a NULL pointer would be
> encountered due to the empty list of netdevs for members of the aggregate.
> 
> Remove the unnecessary call to ice_lag_move_new_vf_nodes(), and since
> this is the only call to this function, remove the function as well.
> 
> Fixes: 351d8d8ab6af ("ice: breakout common LAG code into helpers")
> Reviewed-by: Przemek Kitszel <[email protected]>
> Reviewed-by: Aleksandr Loktionov <[email protected]>
> Reviewed-by: Paul Menzel <[email protected]>
> Signed-off-by: Dave Ertman <[email protected]>
> ---
> This goes to -next as the code is not yet present in -net
> ---
>  drivers/net/ethernet/intel/ice/ice_lag.c     | 55 --------------------
>  drivers/net/ethernet/intel/ice/ice_lag.h     |  1 -
>  drivers/net/ethernet/intel/ice/virt/queues.c |  2 -
>  3 files changed, 58 deletions(-)

Tested-by: Priya Singh <[email protected]>

Reply via email to