Page MenuHomeFreeBSD

iflib: Avoid double counting in rxeof
ClosedPublic

Authored by markj on Feb 23 2021, 3:51 PM.
Tags
None
Referenced Files
F131918534: D28900.id84609.diff
Sun, Oct 12, 5:17 AM
F131918533: D28900.id.diff
Sun, Oct 12, 5:17 AM
F131918527: D28900.id84556.diff
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
Subscribers

Details

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 requested review of this revision.Feb 23 2021, 3:51 PM
erj added inline comments.
sys/net/iflib.c
2926–2927

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

sys/net/iflib.c
2926–2927

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

This revision is now accepted and ready to land.Feb 23 2021, 8:16 PM
This revision was automatically updated to reflect the committed changes.