diff options
author | 2024-04-11 12:22:31 -0700 | |
---|---|---|
committer | 2024-04-15 11:19:52 -0700 | |
commit | 721f076b62cb05108565adf17c27875ef5015307 (patch) | |
tree | 98ab8f19bf48dba680172ca5922c9e20bcd5b804 /include | |
parent | net: dql: Separate queue function responsibilities (diff) | |
download | linux-721f076b62cb05108565adf17c27875ef5015307.tar.gz linux-721f076b62cb05108565adf17c27875ef5015307.tar.bz2 linux-721f076b62cb05108565adf17c27875ef5015307.zip |
net: dql: Optimize stall information population
When Dynamic Queue Limit (DQL) is set, it always populate stall
information through dql_queue_stall(). However, this information is
only necessary if a stall threshold is set, stored in struct
dql->stall_thrs.
dql_queue_stall() is cheap, but not free, since it does have memory
barriers and so forth.
Do not call dql_queue_stall() if there is no stall threshold set, and
save some CPU cycles.
Signed-off-by: Breno Leitao <leitao@debian.org>
Link: https://lore.kernel.org/r/20240411192241.2498631-4-leitao@debian.org
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
Diffstat (limited to 'include')
-rw-r--r-- | include/linux/dynamic_queue_limits.h | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/include/linux/dynamic_queue_limits.h b/include/linux/dynamic_queue_limits.h index 9980df0b7247..869afb800ea1 100644 --- a/include/linux/dynamic_queue_limits.h +++ b/include/linux/dynamic_queue_limits.h @@ -137,7 +137,9 @@ static inline void dql_queued(struct dql *dql, unsigned int count) dql->num_queued += count; - dql_queue_stall(dql); + /* Only populate stall information if the threshold is set */ + if (READ_ONCE(dql->stall_thrs)) + dql_queue_stall(dql); } /* Returns how many objects can be queued, < 0 indicates over limit. */ |