Thread (5 messages) flat view 5 messages, 3 authors, 2021-09-20

Re: [PATCH] tcp: tcp_drop adds `SNMP` and `reason` parameter for tracing

From: Cong Wang <hidden>
Date: 2021-09-20 20:45:13
Also in: lkml

On Mon, Sep 20, 2021 at 10:15 AM Steven Rostedt [off-list ref] wrote:
On Mon, 20 Sep 2021 09:54:02 -0700
Cong Wang [off-list ref] wrote:
quoted
Also, kernel does not have to explain it in strings, those SNMP
counters are already available for user-space, so kernel could
just use SNMP enums and let user-space interpret them. In many
cases, you are just adding strings for those SNMP enums.
The strings were requested by the networking maintainers.

  https://lore.kernel.org/all/CANn89iJO8jzjFWvJ610TPmKDE8WKi8ojTr_HWXLz5g=4pdQHEA@mail.gmail.com/ (local)
I think you misunderstand my point. Eric's point is hex address
vs. string, which I never disagree. With SNMP enum, user-space
can easily interpret it to string too, so at the end you still get strings
but not from kernel. This would at least save a handful of strings
from vmlinux, especially if we expand it beyond TCP.

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