From: erik quanstrom <quanstro@quanstro.net>
To: 9fans@9fans.net
Subject: [9fans] inaccurate etheroq accounting
Date: Tue, 11 Aug 2009 16:21:19 -0400 [thread overview]
Message-ID: <b0bc581700dd993447738444398bf038@quanstro.net> (raw)
i was looking to add bytes transmited to
/net/ether?/stats accounting but was derailed
because the existing accounting seems
inaccurate.
i'm pretty sure that etheroq can be called
concurrently on the same ether, so doesn't
that make the outpackets inaccurate at
high packet rates on an mp machine?
even if outpackets were not a vlong, it
would still be a seperate load and store
and meanwhile another processor could
have stored.
static int
etheroq(Ether* ether, Block* bp)
{
int len, loopback, s;
Etherpkt *pkt;
ether->outpackets++;
for my public-facing 8169 chip operating
at a pitiful 10mbps, i do see evidence that
this really happens:
out: 16703370 from stats (e.g. ether->outpackets)
TxOk: 16607827 from the chip's accounting (ifstats)
(i'm pretty sure that lockstats.inglare and
friends in taslock.c have the same issue.)
in any event, it seems like there's got to
be a better way than adding a big fat lock
in the middle of the ether tx and rx paths.
seems like in the ether-specific functions
are going to lock things down at some point,
so maybe that accounting would be better
there?
- erik
reply other threads:[~2009-08-11 20:21 UTC|newest]
Thread overview: [no followups] expand[flat|nested] mbox.gz Atom feed
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=b0bc581700dd993447738444398bf038@quanstro.net \
--to=quanstro@quanstro.net \
--cc=9fans@9fans.net \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).