diff options
author | Leslie Monis <lesliemonis@gmail.com> | 2020-03-05 00:26:01 +0530 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2020-03-04 13:25:55 -0800 |
commit | 105e808c1da2a2827a4a374ae6e3003249729eec (patch) | |
tree | e6045369559f0fb426929d5fee0e22303a7d95e7 /net/sched/act_skbmod.c | |
parent | 220d4ac74ed691033b6fe2bd98dc07d6bdece046 (diff) | |
download | linux-105e808c1da2a2827a4a374ae6e3003249729eec.tar.gz linux-105e808c1da2a2827a4a374ae6e3003249729eec.tar.bz2 linux-105e808c1da2a2827a4a374ae6e3003249729eec.zip |
pie: remove pie_vars->accu_prob_overflows
The variable pie_vars->accu_prob is used as an accumulator for
probability values. Since probabilty values are scaled using the
MAX_PROB macro denoting (2^64 - 1), pie_vars->accu_prob is
likely to overflow as it is of type u64.
The variable pie_vars->accu_prob_overflows counts the number of
times the variable pie_vars->accu_prob overflows.
The MAX_PROB macro needs to be equal to at least (2^39 - 1) in
order to do precise calculations without any underflow. Thus
MAX_PROB can be reduced to (2^56 - 1) without affecting the
precision in calculations drastically. Doing so will eliminate
the need for the variable pie_vars->accu_prob_overflows as the
variable pie_vars->accu_prob will never overflow.
Removing the variable pie_vars->accu_prob_overflows also reduces
the size of the structure pie_vars to exactly 64 bytes.
Signed-off-by: Mohit P. Tahiliani <tahiliani@nitk.edu.in>
Signed-off-by: Gautam Ramakrishnan <gautamramk@gmail.com>
Signed-off-by: Leslie Monis <lesliemonis@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/sched/act_skbmod.c')
0 files changed, 0 insertions, 0 deletions