Bug 248934 - e1000: hw.em.sbp default value change
Summary: e1000: hw.em.sbp default value change
Status: New
Alias: None
Product: Base System
Classification: Unclassified
Component: kern (show other bugs)
Version: 12.1-RELEASE
Hardware: Any Any
: --- Affects Only Me
Assignee: freebsd-net (Nobody)
URL:
Keywords: IntelNetworking
Depends on:
Blocks:
 
Reported: 2020-08-26 18:00 UTC by Franco Fichtner
Modified: 2020-09-24 06:40 UTC (History)
3 users (show)

See Also:


Attachments
hw.em.sbp off by default (1.08 KB, patch)
2020-08-26 18:00 UTC, Franco Fichtner
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Franco Fichtner 2020-08-26 18:00:22 UTC
Created attachment 217546 [details]
hw.em.sbp off by default

Hi,

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.


Cheers,
Franco
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.