niansa-misc/linux
1
0
Fork 0
mirror of synced 2025-03-06 20:59:54 +01:00
Code Issues Projects Releases Packages Wiki Activity

net: rswitch: Add unmap_addrs instead of dma address in each desc

Browse source 
If the driver would like to transmit a jumbo frame like 2KiB or more,
it should be split into multiple queues. In the near future, to support
this, add unmap_addrs array to unmap dma mapping address instead of dma
address in each TX descriptor because the descriptors may not have
the top dma address.

Signed-off-by: Yoshihiro Shimoda <yoshihiro.shimoda.uh@renesas.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
This commit is contained in:
Yoshihiro Shimoda 2023-12-08 13:10:25 +09:00 • committed by David S. Miller
parent 6a203cb516
commit 271e015b91
2 changed files with 12 additions and 8 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

19
drivers/net/ethernet/renesas/rswitch.c
View file

@ -284,6 +284,8 @@ static void rswitch_gwca_queue_free(struct net_device *ndev,
gq->tx_ring = NULL; gq->tx_ring = NULL;
kfree(gq->skbs); kfree(gq->skbs);
gq->skbs = NULL; gq->skbs = NULL;
kfree(gq->unmap_addrs);
gq->unmap_addrs = NULL;
} }
} }
@ -322,6 +324,9 @@ static int rswitch_gwca_queue_alloc(struct net_device *ndev,
gq->skbs = kcalloc(gq->ring_size, sizeof(*gq->skbs), GFP_KERNEL); gq->skbs = kcalloc(gq->ring_size, sizeof(*gq->skbs), GFP_KERNEL);
if (!gq->skbs) if (!gq->skbs)
return -ENOMEM; return -ENOMEM;
gq->unmap_addrs = kcalloc(gq->ring_size, sizeof(*gq->unmap_addrs), GFP_KERNEL);
if (!gq->unmap_addrs)
goto out;
gq->tx_ring = dma_alloc_coherent(ndev->dev.parent, gq->tx_ring = dma_alloc_coherent(ndev->dev.parent,
sizeof(struct rswitch_ext_desc) * sizeof(struct rswitch_ext_desc) *
(gq->ring_size + 1), &gq->ring_dma, GFP_KERNEL); (gq->ring_size + 1), &gq->ring_dma, GFP_KERNEL);
@ -787,9 +792,7 @@ static void rswitch_tx_free(struct net_device *ndev)
struct rswitch_device *rdev = netdev_priv(ndev); struct rswitch_device *rdev = netdev_priv(ndev);
struct rswitch_gwca_queue *gq = rdev->tx_queue; struct rswitch_gwca_queue *gq = rdev->tx_queue;
struct rswitch_ext_desc *desc; struct rswitch_ext_desc *desc;
dma_addr_t dma_addr;
struct sk_buff *skb; struct sk_buff *skb;
unsigned int size;
for (; rswitch_get_num_cur_queues(gq) > 0; for (; rswitch_get_num_cur_queues(gq) > 0;
gq->dirty = rswitch_next_queue_index(gq, false, 1)) { gq->dirty = rswitch_next_queue_index(gq, false, 1)) {
@ -798,18 +801,17 @@ static void rswitch_tx_free(struct net_device *ndev)
break; break;
dma_rmb(); dma_rmb();
size = le16_to_cpu(desc->desc.info_ds) & TX_DS;
skb = gq->skbs[gq->dirty]; skb = gq->skbs[gq->dirty];
if (skb) { if (skb) {
dma_addr = rswitch_desc_get_dptr(&desc->desc); dma_unmap_single(ndev->dev.parent,
dma_unmap_single(ndev->dev.parent, dma_addr, gq->unmap_addrs[gq->dirty],
size, DMA_TO_DEVICE); skb->len, DMA_TO_DEVICE);
dev_kfree_skb_any(gq->skbs[gq->dirty]); dev_kfree_skb_any(gq->skbs[gq->dirty]);
gq->skbs[gq->dirty] = NULL; gq->skbs[gq->dirty] = NULL;
rdev->ndev->stats.tx_packets++;
rdev->ndev->stats.tx_bytes += skb->len;
} }
desc->desc.die_dt = DT_EEMPTY; desc->desc.die_dt = DT_EEMPTY;
rdev->ndev->stats.tx_packets++;
rdev->ndev->stats.tx_bytes += size;
} }
} }
@ -1538,6 +1540,7 @@ static netdev_tx_t rswitch_start_xmit(struct sk_buff *skb, struct net_device *nd
goto err_kfree; goto err_kfree;
gq->skbs[gq->cur] = skb; gq->skbs[gq->cur] = skb;
gq->unmap_addrs[gq->cur] = dma_addr;
desc = &gq->tx_ring[gq->cur]; desc = &gq->tx_ring[gq->cur];
rswitch_desc_set_dptr(&desc->desc, dma_addr); rswitch_desc_set_dptr(&desc->desc, dma_addr);
desc->desc.info_ds = cpu_to_le16(skb->len); desc->desc.info_ds = cpu_to_le16(skb->len);

1
drivers/net/ethernet/renesas/rswitch.h
View file

@ -956,6 +956,7 @@ struct rswitch_gwca_queue {
/* For TX */ /* For TX */
struct { struct {
struct sk_buff **skbs; struct sk_buff **skbs;
dma_addr_t *unmap_addrs;
}; };
/* For RX */ /* For RX */
struct { struct {