Commit 7beb0146 authored by Alexander Duyck's avatar Alexander Duyck Committed by David S. Miller

igb/igbvf: set rx csum always enabled in hw, disable via sw

An issue was found in which rx checksum could not be enabled without
resetting the interface.  The issue was the hardware enable was not being
done via ethtool.  To resolve this issue and prevent conflicts with VF
configuration we will leave the feature always enabled in hardware, and
then in software we will choose to ignore the results via a sw flag.
Signed-off-by: default avatarAlexander Duyck <alexander.h.duyck@intel.com>
Signed-off-by: default avatarJeff Kirsher <jeffrey.t.kirsher@intel.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent 0364d6fd
...@@ -238,7 +238,6 @@ struct igb_adapter { ...@@ -238,7 +238,6 @@ struct igb_adapter {
u64 hw_csum_err; u64 hw_csum_err;
u64 hw_csum_good; u64 hw_csum_good;
u32 alloc_rx_buff_failed; u32 alloc_rx_buff_failed;
bool rx_csum;
u32 gorc; u32 gorc;
u64 gorc_old; u64 gorc_old;
u16 rx_ps_hdr_size; u16 rx_ps_hdr_size;
...@@ -286,6 +285,7 @@ struct igb_adapter { ...@@ -286,6 +285,7 @@ struct igb_adapter {
#define IGB_FLAG_DCA_ENABLED (1 << 1) #define IGB_FLAG_DCA_ENABLED (1 << 1)
#define IGB_FLAG_QUAD_PORT_A (1 << 2) #define IGB_FLAG_QUAD_PORT_A (1 << 2)
#define IGB_FLAG_NEED_CTX_IDX (1 << 3) #define IGB_FLAG_NEED_CTX_IDX (1 << 3)
#define IGB_FLAG_RX_CSUM_DISABLED (1 << 4)
enum e1000_state_t { enum e1000_state_t {
__IGB_TESTING, __IGB_TESTING,
......
...@@ -275,13 +275,17 @@ static int igb_set_pauseparam(struct net_device *netdev, ...@@ -275,13 +275,17 @@ static int igb_set_pauseparam(struct net_device *netdev,
static u32 igb_get_rx_csum(struct net_device *netdev) static u32 igb_get_rx_csum(struct net_device *netdev)
{ {
struct igb_adapter *adapter = netdev_priv(netdev); struct igb_adapter *adapter = netdev_priv(netdev);
return adapter->rx_csum; return !(adapter->flags & IGB_FLAG_RX_CSUM_DISABLED);
} }
static int igb_set_rx_csum(struct net_device *netdev, u32 data) static int igb_set_rx_csum(struct net_device *netdev, u32 data)
{ {
struct igb_adapter *adapter = netdev_priv(netdev); struct igb_adapter *adapter = netdev_priv(netdev);
adapter->rx_csum = data;
if (data)
adapter->flags &= ~IGB_FLAG_RX_CSUM_DISABLED;
else
adapter->flags |= IGB_FLAG_RX_CSUM_DISABLED;
return 0; return 0;
} }
......
...@@ -1395,8 +1395,6 @@ static int __devinit igb_probe(struct pci_dev *pdev, ...@@ -1395,8 +1395,6 @@ static int __devinit igb_probe(struct pci_dev *pdev,
igb_validate_mdi_setting(hw); igb_validate_mdi_setting(hw);
adapter->rx_csum = 1;
/* Initial Wake on LAN setting If APM wake is enabled in the EEPROM, /* Initial Wake on LAN setting If APM wake is enabled in the EEPROM,
* enable the ACPI Magic Packet filter * enable the ACPI Magic Packet filter
*/ */
...@@ -2249,13 +2247,12 @@ static void igb_configure_rx(struct igb_adapter *adapter) ...@@ -2249,13 +2247,12 @@ static void igb_configure_rx(struct igb_adapter *adapter)
rxcsum = rd32(E1000_RXCSUM); rxcsum = rd32(E1000_RXCSUM);
/* Disable raw packet checksumming */ /* Disable raw packet checksumming */
rxcsum |= E1000_RXCSUM_PCSD; rxcsum |= E1000_RXCSUM_PCSD;
/* Don't need to set TUOFL or IPOFL, they default to 1 */
if (!adapter->rx_csum) if (adapter->hw.mac.type == e1000_82576)
rxcsum &= ~(E1000_RXCSUM_TUOFL | E1000_RXCSUM_IPOFL);
else if (adapter->hw.mac.type == e1000_82576)
/* Enable Receive Checksum Offload for SCTP */ /* Enable Receive Checksum Offload for SCTP */
rxcsum |= E1000_RXCSUM_CRCOFL; rxcsum |= E1000_RXCSUM_CRCOFL;
/* Don't need to set TUOFL or IPOFL, they default to 1 */
wr32(E1000_RXCSUM, rxcsum); wr32(E1000_RXCSUM, rxcsum);
/* Set the default pool for the PF's first queue */ /* Set the default pool for the PF's first queue */
...@@ -4455,7 +4452,8 @@ static inline void igb_rx_checksum_adv(struct igb_adapter *adapter, ...@@ -4455,7 +4452,8 @@ static inline void igb_rx_checksum_adv(struct igb_adapter *adapter,
skb->ip_summed = CHECKSUM_NONE; skb->ip_summed = CHECKSUM_NONE;
/* Ignore Checksum bit is set or checksum is disabled through ethtool */ /* Ignore Checksum bit is set or checksum is disabled through ethtool */
if ((status_err & E1000_RXD_STAT_IXSM) || !adapter->rx_csum) if ((status_err & E1000_RXD_STAT_IXSM) ||
(adapter->flags & IGB_FLAG_RX_CSUM_DISABLED))
return; return;
/* TCP/UDP checksum error bit is set */ /* TCP/UDP checksum error bit is set */
if (status_err & if (status_err &
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment