Bug #10812
closedTraffic graph shows 2X the actual traffic on VLAN interfaces.
Added by Nano Caiordo about 6 years ago. Updated almost 6 years ago.
100%
Description
#3314 is back again on, as of today, latest snapshot
2.5.0-DEVELOPMENT (amd64) built on Sun Aug 02 19:02:34 EDT 2020 FreeBSD 12.1-STABLE
→L Updated by → luckman212 about 6 years ago Actions #1
I notice #10852 also seems to be the same issue.
JP Updated by Jim Pingle about 6 years ago Actions #2
- Target version set to 2.5.0
Updated by Anonymous almost 6 years ago
Actions
#3
- Assignee set to Jim Pingle
JP Updated by Jim Pingle almost 6 years ago Actions #4
- Subject changed from Reopen - Traffic graph shows 2X the actual traffic on VLAN interfaces. to Traffic graph shows 2X the actual traffic on VLAN interfaces.
- Assignee changed from Jim Pingle to Renato Botelho
Last time this came up it was due to VLAN traffic counting twice on the parent. In https://redmine.pfsense.org/issues/3314#note-21 it was Renato that added the patch (or noted it was added, at least). Might be one we need to track down and reapply after the last stable merge.
MM Updated by Marcos M almost 6 years ago Actions #5
Jim Pingle wrote:
Last time this came up it was due to VLAN traffic counting twice on the parent. In https://redmine.pfsense.org/issues/3314#note-21 it was Renato that added the patch (or noted it was added, at least). Might be one we need to track down and reapply after the last stable merge.
Looked at commits during that time, but no luck finding it:
https://redmine.pfsense.org/projects/pfsense/repository/revisions?page=637
RB Updated by Renato Botelho almost 6 years ago Actions #6
- Assignee changed from Renato Botelho to Luiz Souza
Luiz enabled ALTQ on if_vlan.c few time before this ticket was opened. He is going to check if his commit re-introduced this issue
RB Updated by Renato Botelho almost 6 years ago Actions #7
It regressed before as we can see on #7751 and fix was committed to 2.4.4 branch on FreeBSD-src
commit 327d0a44358
Author: Luiz Souza <luiz@netgate.com>
Date: Thu Aug 17 23:31:03 2017 -0500
Fix the counter values for VLAN interfaces. The packet bytes are already counted in IFQ_HANDOFF().
Ticket #7751
diff --git a/sys/net/if_vlan.c b/sys/net/if_vlan.c
index e50b3ddebba..8ccd9c44453 100644
--- a/sys/net/if_vlan.c
+++ b/sys/net/if_vlan.c
@@ -1136,12 +1136,8 @@ vlan_start(struct ifnet *ifp)
* Send it, precisely as ether_output() would have.
*/
error = (p->if_transmit)(p, m);
- if (error == 0) {
- if_inc_counter(ifp, IFCOUNTER_OPACKETS, 1);
- if_inc_counter(ifp, IFCOUNTER_OBYTES, len);
- if_inc_counter(ifp, IFCOUNTER_OMCASTS, mcast);
- } else
- if_inc_counter(ifp, IFCOUNTER_OERRORS, 1);
+ if_inc_counter(ifp,
+ (error == 0) ? IFCOUNTER_OPACKETS : IFCOUNTER_OERRORS, 1);
}
}
LS Updated by Luiz Souza almost 6 years ago Actions #8
- Status changed from New to Feedback
- % Done changed from 0 to 100
Fix committed.
The fix is a bit different with the new code.
RB Updated by Renato Botelho almost 6 years ago Actions #9
Fixed by commit 2841d41b090 on branch devel-12 of FreeBSD-src repository
DZ Updated by Danilo Zrenjanin almost 6 years ago Actions #10
- Status changed from Feedback to Resolved
Tested on :
2.5.0-DEVELOPMENT (amd64) built on Sat Nov 14 07:01:37 EST 2020 FreeBSD 12.2-STABLE
The Traffic graph shows correct values for vlan interfaces. Ticket resolved.
Also available in: Atom