Thread (8 messages) flat view 8 messages, 2 authors, 2022-08-30

Re: [PATCH net-next 0/3] net: sched: add other statistics when calling qdisc_drop()

From: shaozhengchao <hidden>
Date: 2022-08-27 03:17:09
Also in: lkml

On 2022/8/27 10:40, Jakub Kicinski wrote:
On Thu, 25 Aug 2022 11:29:40 +0800 Zhengchao Shao wrote:
quoted
According to the description, "other" should be added when calling
qdisc_drop() to discard packets.
The fact that an old copy & pasted comment says something is not
in itself a sufficient justification to make code changes.

qdisc_drop() already counts drops, duplicating the same information
in another place seems like a waste of CPU cycles.
Hi Jakub:
	Thank you for your reply. It seems more appropriate to delete the other 
variable, if it is unused?

Zhengchao Shao
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help