From: Ido Schimmel <[email protected]>
Date: Sun, 17 Feb 2019 07:18:41 +0000

> From: Petr Machata <[email protected]>
> 
> The function-local variable "delay" enters the loop interpreted as delay
> in bits. However, inside the loop it gets overwritten by the result of
> mlxsw_sp_pg_buf_delay_get(), and thus leaves the loop as quantity in
> cells. Thus on second and further loop iterations, the headroom for a
> given priority is configured with a wrong size.
> 
> Fix by introducing a loop-local variable, delay_cells. Rename thres to
> thres_cells for consistency.
> 
> Fixes: f417f04da589 ("mlxsw: spectrum: Refactor port buffer configuration")
> Signed-off-by: Petr Machata <[email protected]>
> Acked-by: Jiri Pirko <[email protected]>
> Signed-off-by: Ido Schimmel <[email protected]>

Applied and queued up for -stable, thanks Ido.

Reply via email to