Bug 248934

Summary: e1000: hw.em.sbp default value change
Product: Base System Reporter: Franco Fichtner <franco>
Component: kernAssignee: freebsd-net (Nobody) <net>
Status: New ---    
Severity: Affects Only Me CC: emaste, erj, raul.munoz
Priority: --- Keywords: IntelNetworking
Version: 12.1-RELEASE   
Hardware: Any   
OS: Any   
Description Flags
hw.em.sbp off by default none

Description Franco Fichtner 2020-08-26 18:00:22 UTC
Created attachment 217546 [details]
hw.em.sbp off by default


The iflib rewrite changed sysctl hw.em.sbp default to on but it is still documented as off by default. This coupled with #248869 caused error counters for the affected NICs to report lots of errors as compared to FreeBSD 11 and below.

This also affects CURRENT and is present since 12.0, but marking this as a 12.1-RELEASE issue.

Patch is attached to restore the previous behaviour, but maybe the documentation should be changed instead.

Comment 1 Eric Joyner freebsd_committer 2020-09-15 21:13:14 UTC
I'm not sure about this one; I agree that either the code or the documentation should change, but I don't know if we should change the default back, for 13.0 or 12.2.

I also don't know why it changed from 11, either. We at Intel didn't do the 1G driver conversion to iflib. (and I'm not even supposed to be maintaining it!)
Comment 2 Franco Fichtner 2020-09-24 06:40:15 UTC
I am unsure too. The only issue I see is that some applications VLANs with Netmap only work in device promisc mode and having this enabled by default might decrease performance which is hard to track down to this "debug" sysctl.