diff options
author | David S. Miller <davem@davemloft.net> | 2017-10-19 07:49:16 -0400 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2017-10-19 07:49:16 -0400 |
commit | 3fd3b03b4359852914b0a504cc87d1c1170c5d7c (patch) | |
tree | a5c968c36b127bc732c83f28584dccf076db9c42 | |
parent | 28e33f9d78eefe98ea86673ab31e988b37a9a738 (diff) | |
parent | a59df396768a7e37c6ddafeb9666a30c8ac07854 (diff) |
Merge branch 'ena-fixes'
Netanel Belgazal says:
====================
ENA ethernet driver bug fixes
Some fixes for ENA ethernet driver
====================
Signed-off-by: David S. Miller <davem@davemloft.net>
-rw-r--r-- | drivers/net/ethernet/amazon/ena/ena_ethtool.c | 4 | ||||
-rw-r--r-- | drivers/net/ethernet/amazon/ena/ena_netdev.c | 7 |
2 files changed, 6 insertions, 5 deletions
diff --git a/drivers/net/ethernet/amazon/ena/ena_ethtool.c b/drivers/net/ethernet/amazon/ena/ena_ethtool.c index b1212debc2e1..967020fb26ee 100644 --- a/drivers/net/ethernet/amazon/ena/ena_ethtool.c +++ b/drivers/net/ethernet/amazon/ena/ena_ethtool.c | |||
@@ -742,8 +742,8 @@ static void ena_get_channels(struct net_device *netdev, | |||
742 | { | 742 | { |
743 | struct ena_adapter *adapter = netdev_priv(netdev); | 743 | struct ena_adapter *adapter = netdev_priv(netdev); |
744 | 744 | ||
745 | channels->max_rx = ENA_MAX_NUM_IO_QUEUES; | 745 | channels->max_rx = adapter->num_queues; |
746 | channels->max_tx = ENA_MAX_NUM_IO_QUEUES; | 746 | channels->max_tx = adapter->num_queues; |
747 | channels->max_other = 0; | 747 | channels->max_other = 0; |
748 | channels->max_combined = 0; | 748 | channels->max_combined = 0; |
749 | channels->rx_count = adapter->num_queues; | 749 | channels->rx_count = adapter->num_queues; |
diff --git a/drivers/net/ethernet/amazon/ena/ena_netdev.c b/drivers/net/ethernet/amazon/ena/ena_netdev.c index f7dc22f65d9f..c6bd5e24005d 100644 --- a/drivers/net/ethernet/amazon/ena/ena_netdev.c +++ b/drivers/net/ethernet/amazon/ena/ena_netdev.c | |||
@@ -966,7 +966,7 @@ static inline void ena_rx_checksum(struct ena_ring *rx_ring, | |||
966 | u64_stats_update_begin(&rx_ring->syncp); | 966 | u64_stats_update_begin(&rx_ring->syncp); |
967 | rx_ring->rx_stats.bad_csum++; | 967 | rx_ring->rx_stats.bad_csum++; |
968 | u64_stats_update_end(&rx_ring->syncp); | 968 | u64_stats_update_end(&rx_ring->syncp); |
969 | netif_err(rx_ring->adapter, rx_err, rx_ring->netdev, | 969 | netif_dbg(rx_ring->adapter, rx_err, rx_ring->netdev, |
970 | "RX IPv4 header checksum error\n"); | 970 | "RX IPv4 header checksum error\n"); |
971 | return; | 971 | return; |
972 | } | 972 | } |
@@ -979,7 +979,7 @@ static inline void ena_rx_checksum(struct ena_ring *rx_ring, | |||
979 | u64_stats_update_begin(&rx_ring->syncp); | 979 | u64_stats_update_begin(&rx_ring->syncp); |
980 | rx_ring->rx_stats.bad_csum++; | 980 | rx_ring->rx_stats.bad_csum++; |
981 | u64_stats_update_end(&rx_ring->syncp); | 981 | u64_stats_update_end(&rx_ring->syncp); |
982 | netif_err(rx_ring->adapter, rx_err, rx_ring->netdev, | 982 | netif_dbg(rx_ring->adapter, rx_err, rx_ring->netdev, |
983 | "RX L4 checksum error\n"); | 983 | "RX L4 checksum error\n"); |
984 | skb->ip_summed = CHECKSUM_NONE; | 984 | skb->ip_summed = CHECKSUM_NONE; |
985 | return; | 985 | return; |
@@ -3064,7 +3064,8 @@ static void ena_release_bars(struct ena_com_dev *ena_dev, struct pci_dev *pdev) | |||
3064 | if (ena_dev->mem_bar) | 3064 | if (ena_dev->mem_bar) |
3065 | devm_iounmap(&pdev->dev, ena_dev->mem_bar); | 3065 | devm_iounmap(&pdev->dev, ena_dev->mem_bar); |
3066 | 3066 | ||
3067 | devm_iounmap(&pdev->dev, ena_dev->reg_bar); | 3067 | if (ena_dev->reg_bar) |
3068 | devm_iounmap(&pdev->dev, ena_dev->reg_bar); | ||
3068 | 3069 | ||
3069 | release_bars = pci_select_bars(pdev, IORESOURCE_MEM) & ENA_BAR_MASK; | 3070 | release_bars = pci_select_bars(pdev, IORESOURCE_MEM) & ENA_BAR_MASK; |
3070 | pci_release_selected_regions(pdev, release_bars); | 3071 | pci_release_selected_regions(pdev, release_bars); |