From: Jakub Kicinski <kuba@kernel.org>
To: Jeffrey Ji <jeffreyjilinux@gmail.com>
Cc: Ido Schimmel <idosch@idosch.org>, David Ahern <dsahern@gmail.com>,
Stephen Hemminger <stephen@networkplumber.org>,
Eric Dumazet <edumazet@google.com>,
Brian Vazquez <brianvv@google.com>,
netdev@vger.kernel.org, Jeffrey Ji <jeffreyji@google.com>
Subject: Re: [PATCH net-next] show rx_otherhost_dropped stat in ip link show
Date: Wed, 18 May 2022 12:39:47 -0700 [thread overview]
Message-ID: <20220518123947.44a23904@kernel.org> (raw)
In-Reply-To: <CACB8nPkQkH3fJt29kNQ_YqikP8eKPSuBJvh-_cFO_zqie2rw0A@mail.gmail.com>
On Wed, 18 May 2022 07:29:08 -1000 Jeffrey Ji wrote:
> I thought we wanted to avoid otherhost_dropped being counted as an
> error, I recall Jakub saying something about not wanting users to call
> for help when they see error in the counter name.
You should probably set up a normal email client if you want to
continue working on the kernel, you replied to the wrong message :S
Yes, displaying the otherhost as an error could be too alarming.
That said the split between the main RX: line and RX errors: in
ip -s -s is somewhat arbitrary so no strong preference here.
next prev parent reply other threads:[~2022-05-18 19:39 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-05-09 19:18 [PATCH net-next] show rx_otherhost_dropped stat in ip link show Jeffrey Ji
2022-05-09 20:10 ` Brian Vazquez
2022-05-10 7:09 ` Ido Schimmel
2022-05-18 17:29 ` Jeffrey Ji
2022-05-18 19:39 ` Jakub Kicinski [this message]
2022-05-18 20:00 ` Stephen Hemminger
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20220518123947.44a23904@kernel.org \
--to=kuba@kernel.org \
--cc=brianvv@google.com \
--cc=dsahern@gmail.com \
--cc=edumazet@google.com \
--cc=idosch@idosch.org \
--cc=jeffreyji@google.com \
--cc=jeffreyjilinux@gmail.com \
--cc=netdev@vger.kernel.org \
--cc=stephen@networkplumber.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.