On Thu, 2015-08-27 at 17:59 +0200, Jiri Pirko wrote:
> When transmit fails, it is an error, not a warning.
[]
> diff --git a/drivers/net/ethernet/mellanox/mlxsw/core.c 
> b/drivers/net/ethernet/mellanox/mlxsw/core.c
[]
> @@ -376,8 +376,8 @@ static int __mlxsw_emad_transmit(struct mlxsw_core 
> *mlxsw_core,
>  
>       err = mlxsw_core_skb_transmit(mlxsw_core->driver_priv, skb, tx_info);
>       if (err) {
> -             dev_warn(mlxsw_core->bus_info->dev, "Failed to transmit EMAD 
> (tid=%llx)\n",
> -                      mlxsw_core->emad.tid);
> +             dev_err(mlxsw_core->bus_info->dev, "Failed to transmit EMAD 
> (tid=%llx)\n",
> +                     mlxsw_core->emad.tid);

dev_err_ratelimited?


--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to