Message ID | 20220803164045.3585187-1-adel.abushaev@gmail.com |
---|---|
Headers | show |
Series | net: support QUIC crypto | expand |
Looking at https://github.com/shemminger/iproute2/blob/main/misc/ss.c#L589 the ss.c still uses proc/. Adel. On 8/4/22 8:29 AM, Andrew Lunn wrote: > On Wed, Aug 03, 2022 at 11:51:59AM -0700, Adel Abouchaev wrote: >> Andrew, >> >> Could you add more to your comment? The /proc was used similarly to kTLS. >> Netlink is better, though, unsure how ULP stats would fit in it. > How do tools like ss(1) retrieve the protocol summary statistics? Do > they still use /proc, or netlink? > > Andrew
On Thu, Aug 4, 2022 at 9:58 AM Adel Abouchaev <adel.abushaev@gmail.com> wrote: > > Looking at > https://github.com/shemminger/iproute2/blob/main/misc/ss.c#L589 the ss.c > still uses proc/. > Only for legacy reasons. ss -t for sure will use netlink first, then fallback to /proc New counters should use netlink, please. > Adel. > > On 8/4/22 8:29 AM, Andrew Lunn wrote: > > On Wed, Aug 03, 2022 at 11:51:59AM -0700, Adel Abouchaev wrote: > >> Andrew, > >> > >> Could you add more to your comment? The /proc was used similarly to kTLS. > >> Netlink is better, though, unsure how ULP stats would fit in it. > > How do tools like ss(1) retrieve the protocol summary statistics? Do > > they still use /proc, or netlink? > > > > Andrew
On Thu, 4 Aug 2022 10:00:37 -0700 Eric Dumazet wrote: > On Thu, Aug 4, 2022 at 9:58 AM Adel Abouchaev <adel.abushaev@gmail.com> wrote: > > Looking at > > https://github.com/shemminger/iproute2/blob/main/misc/ss.c#L589 the ss.c > > still uses proc/. > > Only for legacy reasons. That but in all honesty also the fact that a proc file is pretty easy and self-describing while the historic netlink families are undocumented code salads. > ss -t for sure will use netlink first, then fallback to /proc > > New counters should use netlink, please. Just to be sure I'm not missing anything - we're talking about some new netlink, right? Is there an existing place for "overall prot family stats" over netlink today?
On Thu, Aug 4, 2022 at 11:09 AM Jakub Kicinski <kuba@kernel.org> wrote: > > On Thu, 4 Aug 2022 10:00:37 -0700 Eric Dumazet wrote: > > On Thu, Aug 4, 2022 at 9:58 AM Adel Abouchaev <adel.abushaev@gmail.com> wrote: > > > Looking at > > > https://github.com/shemminger/iproute2/blob/main/misc/ss.c#L589 the ss.c > > > still uses proc/. > > > > Only for legacy reasons. > > That but in all honesty also the fact that a proc file is pretty easy > and self-describing while the historic netlink families are undocumented > code salads. > > > ss -t for sure will use netlink first, then fallback to /proc > > > > New counters should use netlink, please. > > Just to be sure I'm not missing anything - we're talking about some > new netlink, right? Is there an existing place for "overall prot family > stats" over netlink today? I thought we were speaking of dumping ULP info on a per UDP socket basis. If this is about new SNMP counters, then sure, /proc is fine I guess.