mbox series

[net-next,v2,0/8] net: bcmgenet: Turn on offloads by default

Message ID 1576630275-17591-1-git-send-email-opendmb@gmail.com
Headers show
Series net: bcmgenet: Turn on offloads by default | expand

Message

Doug Berger Dec. 18, 2019, 12:51 a.m. UTC 
This commit stack is based on Florian's commit 4e8aedfe78c7 ("net: 
systemport: Turn on offloads by default") and enables the offloads for
the bcmgenet driver by default.

The first commit adds support for the HIGHDMA feature to the driver.

The second converts the Tx checksum implementation to use the generic
hardware logic rather than the deprecated IP centric methods.

The third modifies the Rx checksum implementation to use the hardware
offload to compute the complete checksum rather than filtering out bad
packets detected by the hardware's IP centric implementation. This may
increase processing load by passing bad packets to the network stack,
but it provides for more flexible handling of packets by the network
stack without requiring software computation of the checksum.

The remaining commits mirror the extensions Florian made to the sysport
driver to retain symmetry with that driver and to make the benefits of
the hardware offloads more ubiquitous.

Doug Berger (8):
  net: bcmgenet: enable NETIF_F_HIGHDMA flag
  net: bcmgenet: enable NETIF_F_HW_CSUM feature
  net: bcmgenet: use CHECKSUM_COMPLETE for NETIF_F_RXCSUM
  net: bcmgenet: Refactor bcmgenet_set_features()
  net: bcmgenet: Utilize bcmgenet_set_features() during resume/open
  net: bcmgenet: Turn on offloads by default
  net: bcmgenet: Be drop monitor friendly while re-allocating headroom
  net: bcmgenet: Add software counters to track reallocations

 drivers/net/ethernet/broadcom/genet/bcmgenet.c | 113 ++++++++++++++-----------
 drivers/net/ethernet/broadcom/genet/bcmgenet.h |   4 +-
 2 files changed, 67 insertions(+), 50 deletions(-)

Comments

David Miller Dec. 20, 2019, 2:13 a.m. UTC | #1 
From: Doug Berger <opendmb@gmail.com>
Date: Tue, 17 Dec 2019 16:51:07 -0800

> This commit stack is based on Florian's commit 4e8aedfe78c7 ("net: 
> systemport: Turn on offloads by default") and enables the offloads for
> the bcmgenet driver by default.
> 
> The first commit adds support for the HIGHDMA feature to the driver.
> 
> The second converts the Tx checksum implementation to use the generic
> hardware logic rather than the deprecated IP centric methods.
> 
> The third modifies the Rx checksum implementation to use the hardware
> offload to compute the complete checksum rather than filtering out bad
> packets detected by the hardware's IP centric implementation. This may
> increase processing load by passing bad packets to the network stack,
> but it provides for more flexible handling of packets by the network
> stack without requiring software computation of the checksum.
> 
> The remaining commits mirror the extensions Florian made to the sysport
> driver to retain symmetry with that driver and to make the benefits of
> the hardware offloads more ubiquitous.

Series applied, thanks.