Summary: | igb stats double counting in 12-CURRENT | ||
---|---|---|---|
Product: | Base System | Reporter: | Ben Woods <woodsb02> |
Component: | kern | Assignee: | freebsd-net (Nobody) <net> |
Status: | Closed FIXED | ||
Severity: | Affects Some People | CC: | amigan, kaho, sbruno |
Priority: | --- | Keywords: | IntelNetworking |
Version: | CURRENT | ||
Hardware: | Any | ||
OS: | Any |
Description
Ben Woods
2017-06-21 23:24:27 UTC
A commit references this bug: Author: sbruno Date: Thu Aug 10 03:11:05 UTC 2017 New revision: 322337 URL: https://svnweb.freebsd.org/changeset/base/322337 Log: Export IFCAP_HWSTATS so that we don't experience double stats counting on iflib enabled devices. PR: 220198 Submitted by: Matt Macy <matt@mattmacy.io> Reported by: Ben Woods <woodsb02@freebsd.org> Sponsored by: Limelight Networks Changes: head/sys/net/iflib.c Hi Sean, I can confirm this has fixed the error. Thanks for your help! -Ben (In reply to commit-hook from comment #1) I think the bug is fixed by this commit, but it is not a good way. 1. Every driver using iflib should set IFCAP_HWSTATS flag within the driver. For example, in the em(4) driver, add IFCAP_HWSTATS flag into #define EM_CAPS and #define IGB_CAPS lines of if_em.c. 2. The IFCAP_FLAGS in iflib.c seems to hold flags which can be changed by a ioctl function. Who want to set or unset the IFCAP_HWSTATS flag by a function? I have cherry-picked this into my local stable/13 and can also confirm that this fixes the issue. Oops, disregard. |