On Tue, Dec 1, 2015 at 3:49 PM, Jiri Pirko <[email protected]> wrote:
> Implement basic procedures for joining/leaving port to/from LAG. That
> includes HW setup of collector, core LAG mapping setup.
>
> Signed-off-by: Jiri Pirko <[email protected]>
> ---
> drivers/net/ethernet/mellanox/mlxsw/spectrum.c | 284
> +++++++++++++++++++++++--
> drivers/net/ethernet/mellanox/mlxsw/spectrum.h | 35 ++-
> 2 files changed, 302 insertions(+), 17 deletions(-)
>
> --- a/drivers/net/ethernet/mellanox/mlxsw/spectrum.c
> +++ b/drivers/net/ethernet/mellanox/mlxsw/spectrum.c
[...]
> +static int mlxsw_sp_port_lag_leave(struct mlxsw_sp_port *mlxsw_sp_port,
> + struct net_device *lag_dev)
> +{
> + struct mlxsw_sp *mlxsw_sp = mlxsw_sp_port->mlxsw_sp;
> + struct mlxsw_sp_upper *lag;
> + u16 lag_id = mlxsw_sp_port->lag_id;
> + int err;
> +
> + if (!mlxsw_sp_port->lagged)
> + return 0;
> + lag = mlxsw_sp_lag_get(mlxsw_sp, lag_id);
> + BUG_ON(lag->ref_count == 0);
is this a must to crash the system here? can't we do something less drastic?
> +
> + err = mlxsw_sp_lag_col_port_disable(mlxsw_sp_port, lag_id);
> + if (err)
> + return err;
> + mlxsw_sp_lag_col_port_remove(mlxsw_sp_port, lag_id);
> + if (err)
> + return err;
> +
> + if (lag->ref_count == 1) {
> + err = mlxsw_sp_lag_destroy(mlxsw_sp, lag_id);
> + if (err)
> + return err;
> + }
> +
> + mlxsw_core_lag_mapping_clear(mlxsw_sp->core, lag_id,
> + mlxsw_sp_port->local_port);
> + mlxsw_sp_port->lagged = 0;
> + lag->ref_count--;
> + return 0;
> +}
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
