diff options
| author |   Ben Hutchings <ben.hutchings@codethink.co.uk> | 2015-01-27 00:49:47 +0000 |
|---|---|---|
| committer |   David S. Miller <davem@davemloft.net> | 2015-01-27 00:18:53 -0800 |
| commit | aa3933b87309c61b4fa4bb93c1ad4c3f08afb1b8 (patch) | |
| tree | 57d6b303dfed3458372679a1105ae903a12181bb /drivers/net/ethernet/renesas/sh_eth.c | |
| parent | sh_eth: Ensure DMA engines are stopped before freeing buffers (diff) | |
| download | linux-dev-aa3933b87309c61b4fa4bb93c1ad4c3f08afb1b8.tar.xz linux-dev-aa3933b87309c61b4fa4bb93c1ad4c3f08afb1b8.zip | |
sh_eth: Check for DMA mapping errors on transmit
dma_map_single() may fail if an IOMMU or swiotlb is in use, so
we need to check for this.
Signed-off-by: Ben Hutchings <ben.hutchings@codethink.co.uk>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to '')
| -rw-r--r-- | drivers/net/ethernet/renesas/sh_eth.c | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/drivers/net/ethernet/renesas/sh_eth.c b/drivers/net/ethernet/renesas/sh_eth.c index 245166bbccfd..4a085da57188 100644 --- a/drivers/net/ethernet/renesas/sh_eth.c +++ b/drivers/net/ethernet/renesas/sh_eth.c @@ -2174,6 +2174,10 @@ static int sh_eth_start_xmit(struct sk_buff *skb, struct net_device *ndev) skb->len + 2); txdesc->addr = dma_map_single(&ndev->dev, skb->data, skb->len, DMA_TO_DEVICE); + if (dma_mapping_error(&ndev->dev, txdesc->addr)) { + kfree_skb(skb); + return NETDEV_TX_OK; + } txdesc->buffer_length = skb->len; if (entry >= mdp->num_tx_ring - 1) |