Page MenuHomeFreeBSD
Differential D28900

iflib: Avoid double counting in rxeof
ClosedPublic
Actions

Authored by markj on Feb 23 2021, 3:51 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Oct 12, 5:17 AM
Unknown Object (File)
Sun, Oct 12, 5:17 AM
Unknown Object (File)
Sun, Oct 12, 5:17 AM
Unknown Object (File)
Sat, Oct 11, 7:14 PM
Unknown Object (File)
Tue, Oct 7, 6:55 PM
Unknown Object (File)
Sat, Sep 20, 9:23 AM
Unknown Object (File)
Sep 12 2025, 4:54 PM
Unknown Object (File)
Sep 11 2025, 4:07 PM
View All Files
Subscribers
ae
erj
imp
melifaro

Details

Reviewers
shurd
gallatin
erj
Group Reviewers
iflib
Commits
rG18b6283014a2: iflib: Avoid double counting in rxeof
rG76294b65df64: iflib: Avoid double counting in rxeof
rGb6999635b195: iflib: Avoid double counting in rxeof
Summary

iflib_rxeof() was counting everything twice. This was introduced when
pfil hooks were added to iflib. I believe we want to count rx
packets/bytes before the pfil hooks are executed, so remove the counter
adjustments that are executed after.

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

markj created this revision.Feb 23 2021, 3:51 PM
Herald added a reviewer: shurd. · View Herald TranscriptFeb 23 2021, 3:51 PM
Herald added a reviewer: iflib. · View Herald Transcript
Herald added subscribers: melifaro, ae, imp. · View Herald Transcript
markj requested review of this revision.Feb 23 2021, 3:51 PM
Harbormaster completed remote builds in B37317: Diff 84556.Feb 23 2021, 3:51 PM
markj added a reviewer: gallatin.Feb 23 2021, 3:51 PM
erj added a subscriber: erj.Feb 23 2021, 5:43 PM
erj added inline comments.
sys/net/iflib.c
2926–2927

So then, this is the one place that the stats should be incremented

markj added inline comments.Feb 23 2021, 5:47 PM
sys/net/iflib.c
2926–2927

Correct. The deleted lines run after pfil hooks run, and so won't count dropped (recycled) packets.

gallatin accepted this revision.Feb 23 2021, 8:16 PM
This revision is now accepted and ready to land.Feb 23 2021, 8:16 PM
erj accepted this revision.Feb 23 2021, 10:43 PM
Closed by commit rGb6999635b195: iflib: Avoid double counting in rxeof (authored by markj). · Explain WhyFeb 24 2021, 3:13 PM
This revision was automatically updated to reflect the committed changes.
markj added a commit: rGb6999635b195: iflib: Avoid double counting in rxeof.
markj added a commit: rG76294b65df64: iflib: Avoid double counting in rxeof.Feb 27 2021, 4:09 PM
markj added a commit: rG18b6283014a2: iflib: Avoid double counting in rxeof.Feb 28 2021, 8:51 PM

Revision Contents
Changeset List

PathSize
sys/
net/
2 lines

Diff 84609

View Options

sys/net/iflib.c

Loading...