igc: txrx function prototype cleanup

Drop variable names of function prototypes since the file is mixed in
listing them or not and they fall out of sync.

Sponsored by: BBOX.io

(cherry picked from

igc: txrx function prototype cleanup

Drop variable names of function prototypes since the file is mixed in
listing them or not and they fall out of sync.

Sponsored by: BBOX.io

(cherry picked from commit 7763b194d8de6fe8034f754064449a7510ca16b9)

show more ...

igc: Function prototype cleanup

Drop variable names of function prototypes since the file is mixed in
listing them or not and they fall out of sync.

Sponsored by: BBOX.io

(cherry picked from commi

igc: Function prototype cleanup

Drop variable names of function prototypes since the file is mixed in
listing them or not and they fall out of sync.

Sponsored by: BBOX.io

(cherry picked from commit 1b0e41ddffbf0ac84c88019df84dfd50ce9f7788)

show more ...

igc: Rename 'struct adapter' to 'struct igc_softc'

Rename the 'struct adapter' to 'struct igc_softc' to avoid type
ambiguity in things like kgdb and make sharing code with e1000 and
ixgbe easier.

S

igc: Rename 'struct adapter' to 'struct igc_softc'

Rename the 'struct adapter' to 'struct igc_softc' to avoid type
ambiguity in things like kgdb and make sharing code with e1000 and
ixgbe easier.

Sponsored by: BBOX.io

(cherry picked from commit 542f5d56317e94676cf19cb9d66207bf9a6b6758)

show more ...

igc: Add sysctls for some missing MAC stats

Sponsored by: BBOX.io

(cherry picked from commit 09526a771afc411e641a3d04c905824fe8afc627)

igc: Add sysctl for DMA Coalesce

This feature can increase efficiency at the expense of latency

It does not work well with the default interrupt delay, but expose
the otherwise unconnected code in

igc: Add sysctl for DMA Coalesce

This feature can increase efficiency at the expense of latency

It does not work well with the default interrupt delay, but expose
the otherwise unconnected code in the driver in case people want to
experiment.

See
https://www.intel.com/content/dam/support/us/en/documents/network/adapter/pro100/sb/466827_intel_r__dma_coalescing_white_paper_v003.pdf

Sponsored by: BBOX.io

(cherry picked from commit 68b1f5dc59368ce5175e3300bf95476ea5a560bc)

show more ...

igc: Want AIM at 2.5G

This should have been commited with bc9402a, need to account for
link_speed of 2500 as well on igc.

Sponsored by: Rubicon Communications, LLC ("Netgate")
Sponsored by: BBOX.io

igc: Want AIM at 2.5G

This should have been commited with bc9402a, need to account for
link_speed of 2500 as well on igc.

Sponsored by: Rubicon Communications, LLC ("Netgate")
Sponsored by: BBOX.io

(cherry picked from commit 669d26e5764f612cc60a4a61cdcf22ae46ddd360)

show more ...

igc: Improve a comment and update copyright dates

(cherry picked from commit 516d92304a7fa9b869c934436aa3e1c5202f03c9)

igc: Add AIM

igc is derived from igb and has never had an AIM implementation. The
same algorithm from e1000 is appropriate here.

Upon more detailed study of the Linux driver which has a newer AIM
i

igc: Add AIM

igc is derived from igb and has never had an AIM implementation. The
same algorithm from e1000 is appropriate here.

Upon more detailed study of the Linux driver which has a newer AIM
implementation, it finally became clear to me this is actually a
holdoff timer and not an interrupt limit as it is conventionally
(statically) programmed and displayed as an interrupt rate. The data
sheets also make this somewhat clear.

Thus, AIM accomplishes two beneficial things for a wide variety of
workloads[1]:

1. At low throughput/packet rates, it will significantly lower latency
(by counter-intuitively "increasing" the interrupt rate.. better
thought of as decreasing the holdoff timer because you will modulate
down before coming anywhere near these interrupt rates).
2. At bulk data rates, it is tuned to achieve a lower interrupt rate
(by increasing the holdoff timer) than the current static 8000/s. This
decreases processing overhead and yields more headroom for other work
such as packet filters or userland.

For a single NIC this might be worth a few sys% on common CPUs, but may
be meaningful when multiplied such as if_lagg, if_bridge and forwarding
setups.

The AIM algorithm was re-introduced from the older igb or out of tree
driver, and then modernized with permission to use Intel code from other
drivers.

[1]: http://iommu.com/datasheets/ethernet/controllers-nics/intel/e1000/gbe-controllers-interrupt-moderation-appl-note.pdf

Relnotes: yes
Sponsored by: Rubicon Communications, LLC ("Netgate")
Sponsored by: BBOX.io
Differential Revision: https://reviews.freebsd.org/D47053

(cherry picked from commit bc9402abdd11630ced33cbecb79b0d72f203f08a)

show more ...

igc: Add NVM/firmware prints and sysctl

This chipset suffered an (un)usual number of bugs and iterations. Let's
add our NVM/firmware code from e1000 and the similar igc_nvm function
from DPDK to kee

igc: Add NVM/firmware prints and sysctl

This chipset suffered an (un)usual number of bugs and iterations. Let's
add our NVM/firmware code from e1000 and the similar igc_nvm function
from DPDK to keep track of issues.

Sponsored by: BBOX.io

(cherry picked from commit 33ed9bdca307bedb3d66a50ed7d4d7b4bf4acf39)

show more ...

igc: Remove non-existent legacy absolute and packet timers

igc, derived from igb, does not use these registers. All interrupt
timing is governed by EITR or LLI and driven by write-back.

Sponsored b

igc: Remove non-existent legacy absolute and packet timers

igc, derived from igb, does not use these registers. All interrupt
timing is governed by EITR or LLI and driven by write-back.

Sponsored by: BBOX.io

(cherry picked from commit a40ecb6f740530fb7e6a95e702113aa3fe24d975)

show more ...