* [PATCH net-next v5 0/2] net: ethernet: mediatek: add enhancements to RX path
@ 2016-09-03 9:59 sean.wang
2016-09-03 9:59 ` [PATCH net-next v5 1/2] net: ethernet: mediatek: enhance RX path by reducing the frequency of the memory barrier used sean.wang
` (2 more replies)
0 siblings, 3 replies; 6+ messages in thread
From: sean.wang @ 2016-09-03 9:59 UTC (permalink / raw)
To: john, davem; +Cc: nbd, netdev, linux-mediatek, keyhaede, objelf, Sean Wang
From: Sean Wang <sean.wang@mediatek.com>
Changes since v1:
- fix message typos and add coverletter
Changes since v2:
- split from the previous series for submitting add enhancements as
a series targeting 'net-next' and add indents before comments.
Changes since v3:
- merge the patch using PDMA RX path
- fixed the input of mtk_poll_rx is with the remaining budget
Changes since v4:
- save one wmb and register update when no packet is being handled
inside mtk_poll_rx call
- fixed incorrect return packet count from mtk_napi_rx
Sean Wang (2):
net: ethernet: mediatek: enhance RX path by reducing the frequency of
the memory barrier used
net: ethernet: mediatek: enhance RX path by aggregating more SKBs into
NAPI
drivers/net/ethernet/mediatek/mtk_eth_soc.c | 26 +++++++++++++++-----------
1 file changed, 15 insertions(+), 11 deletions(-)
--
1.9.1
^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH net-next v5 1/2] net: ethernet: mediatek: enhance RX path by reducing the frequency of the memory barrier used
2016-09-03 9:59 [PATCH net-next v5 0/2] net: ethernet: mediatek: add enhancements to RX path sean.wang
@ 2016-09-03 9:59 ` sean.wang
2016-09-07 6:35 ` John Crispin
2016-09-03 9:59 ` [PATCH net-next v5 2/2] net: ethernet: mediatek: enhance RX path by aggregating more SKBs into NAPI sean.wang
[not found] ` <1472896767-18352-1-git-send-email-sean.wang-NuS5LvNUpcJWk0Htik3J/w@public.gmane.org>
2 siblings, 1 reply; 6+ messages in thread
From: sean.wang @ 2016-09-03 9:59 UTC (permalink / raw)
To: john, davem; +Cc: nbd, netdev, linux-mediatek, keyhaede, objelf, Sean Wang
From: Sean Wang <sean.wang@mediatek.com>
The patch makes move wmb() to outside the loop that could help
RX path handling more faster although that RX descriptors aren't
freed for DMA to use as soon as possible, but based on my experiment
and the result shows it still can reach about 943Mbpis without
performance drop that is tested based on the setup with one port
using Giga PHY and 256 RX descriptors for DMA to move.
Signed-off-by: Sean Wang <sean.wang@mediatek.com>
---
drivers/net/ethernet/mediatek/mtk_eth_soc.c | 11 ++++++-----
1 file changed, 6 insertions(+), 5 deletions(-)
diff --git a/drivers/net/ethernet/mediatek/mtk_eth_soc.c b/drivers/net/ethernet/mediatek/mtk_eth_soc.c
index 431b851..a1bdb53 100644
--- a/drivers/net/ethernet/mediatek/mtk_eth_soc.c
+++ b/drivers/net/ethernet/mediatek/mtk_eth_soc.c
@@ -890,13 +890,14 @@ release_desc:
rxd->rxd2 = RX_DMA_PLEN0(ring->buf_size);
ring->calc_idx = idx;
- /* make sure that all changes to the dma ring are flushed before
- * we continue
- */
- wmb();
- mtk_w32(eth, ring->calc_idx, MTK_PRX_CRX_IDX0);
+
done++;
}
+ /* make sure that all changes to the dma ring are flushed before
+ * we continue
+ */
+ wmb();
+ mtk_w32(eth, ring->calc_idx, MTK_PRX_CRX_IDX0);
if (done < budget)
mtk_w32(eth, MTK_RX_DONE_INT, MTK_PDMA_INT_STATUS);
--
1.9.1
^ permalink raw reply related [flat|nested] 6+ messages in thread
* [PATCH net-next v5 2/2] net: ethernet: mediatek: enhance RX path by aggregating more SKBs into NAPI
2016-09-03 9:59 [PATCH net-next v5 0/2] net: ethernet: mediatek: add enhancements to RX path sean.wang
2016-09-03 9:59 ` [PATCH net-next v5 1/2] net: ethernet: mediatek: enhance RX path by reducing the frequency of the memory barrier used sean.wang
@ 2016-09-03 9:59 ` sean.wang
2016-09-07 6:38 ` John Crispin
[not found] ` <1472896767-18352-1-git-send-email-sean.wang-NuS5LvNUpcJWk0Htik3J/w@public.gmane.org>
2 siblings, 1 reply; 6+ messages in thread
From: sean.wang @ 2016-09-03 9:59 UTC (permalink / raw)
To: john, davem; +Cc: nbd, netdev, linux-mediatek, keyhaede, objelf, Sean Wang
From: Sean Wang <sean.wang@mediatek.com>
The patch adds support for aggregating more SKBs feed into NAPI in
order to get more benefits from generic receive offload (GRO) by
peeking at the RX ring status and moving more packets right before
returning from NAPI RX polling handler if NAPI budgets are still
available and some packets already present in hardware.
Signed-off-by: Sean Wang <sean.wang@mediatek.com>
---
drivers/net/ethernet/mediatek/mtk_eth_soc.c | 31 ++++++++++++++++-------------
1 file changed, 17 insertions(+), 14 deletions(-)
diff --git a/drivers/net/ethernet/mediatek/mtk_eth_soc.c b/drivers/net/ethernet/mediatek/mtk_eth_soc.c
index a1bdb53..66fd45a 100644
--- a/drivers/net/ethernet/mediatek/mtk_eth_soc.c
+++ b/drivers/net/ethernet/mediatek/mtk_eth_soc.c
@@ -893,14 +893,14 @@ release_desc:
done++;
}
- /* make sure that all changes to the dma ring are flushed before
- * we continue
- */
- wmb();
- mtk_w32(eth, ring->calc_idx, MTK_PRX_CRX_IDX0);
- if (done < budget)
- mtk_w32(eth, MTK_RX_DONE_INT, MTK_PDMA_INT_STATUS);
+ if (done) {
+ /* make sure that all changes to the dma ring are flushed before
+ * we continue
+ */
+ wmb();
+ mtk_w32(eth, ring->calc_idx, MTK_PRX_CRX_IDX0);
+ }
return done;
}
@@ -1020,10 +1020,13 @@ static int mtk_napi_rx(struct napi_struct *napi, int budget)
struct mtk_eth *eth = container_of(napi, struct mtk_eth, rx_napi);
u32 status, mask;
int rx_done = 0;
+ int remain_budget = budget;
mtk_handle_status_irq(eth);
+
+poll_again:
mtk_w32(eth, MTK_RX_DONE_INT, MTK_PDMA_INT_STATUS);
- rx_done = mtk_poll_rx(napi, budget, eth);
+ rx_done = mtk_poll_rx(napi, remain_budget, eth);
if (unlikely(netif_msg_intr(eth))) {
status = mtk_r32(eth, MTK_PDMA_INT_STATUS);
@@ -1032,18 +1035,18 @@ static int mtk_napi_rx(struct napi_struct *napi, int budget)
"done rx %d, intr 0x%08x/0x%x\n",
rx_done, status, mask);
}
-
- if (rx_done == budget)
+ if (rx_done == remain_budget)
return budget;
status = mtk_r32(eth, MTK_PDMA_INT_STATUS);
- if (status & MTK_RX_DONE_INT)
- return budget;
-
+ if (status & MTK_RX_DONE_INT) {
+ remain_budget -= rx_done;
+ goto poll_again;
+ }
napi_complete(napi);
mtk_irq_enable(eth, MTK_PDMA_INT_MASK, MTK_RX_DONE_INT);
- return rx_done;
+ return rx_done + budget - remain_budget;
}
static int mtk_tx_alloc(struct mtk_eth *eth)
--
1.9.1
^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH net-next v5 0/2] net: ethernet: mediatek: add enhancements to RX path
[not found] ` <1472896767-18352-1-git-send-email-sean.wang-NuS5LvNUpcJWk0Htik3J/w@public.gmane.org>
@ 2016-09-06 20:33 ` David Miller
0 siblings, 0 replies; 6+ messages in thread
From: David Miller @ 2016-09-06 20:33 UTC (permalink / raw)
To: sean.wang-NuS5LvNUpcJWk0Htik3J/w
Cc: nbd-p3rKhJxN3npAfugRpC6u6w, keyhaede-Re5JQEeQqe8AvxtiuMwx3w,
netdev-u79uwXL29TY76Z2rM5mHXA, john-Pj+rj9U5foFAfugRpC6u6w,
linux-mediatek-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
objelf-Re5JQEeQqe8AvxtiuMwx3w
From: <sean.wang-NuS5LvNUpcJWk0Htik3J/w@public.gmane.org>
Date: Sat, 3 Sep 2016 17:59:25 +0800
> Changes since v1:
> - fix message typos and add coverletter
>
> Changes since v2:
> - split from the previous series for submitting add enhancements as
> a series targeting 'net-next' and add indents before comments.
>
> Changes since v3:
> - merge the patch using PDMA RX path
> - fixed the input of mtk_poll_rx is with the remaining budget
>
> Changes since v4:
> - save one wmb and register update when no packet is being handled
> inside mtk_poll_rx call
> - fixed incorrect return packet count from mtk_napi_rx
Series applied.
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH net-next v5 1/2] net: ethernet: mediatek: enhance RX path by reducing the frequency of the memory barrier used
2016-09-03 9:59 ` [PATCH net-next v5 1/2] net: ethernet: mediatek: enhance RX path by reducing the frequency of the memory barrier used sean.wang
@ 2016-09-07 6:35 ` John Crispin
0 siblings, 0 replies; 6+ messages in thread
From: John Crispin @ 2016-09-07 6:35 UTC (permalink / raw)
To: sean.wang, davem; +Cc: nbd, netdev, linux-mediatek, keyhaede, objelf
On 03/09/2016 11:59, sean.wang@mediatek.com wrote:
> From: Sean Wang <sean.wang@mediatek.com>
>
> The patch makes move wmb() to outside the loop that could help
> RX path handling more faster although that RX descriptors aren't
> freed for DMA to use as soon as possible, but based on my experiment
> and the result shows it still can reach about 943Mbpis without
> performance drop that is tested based on the setup with one port
> using Giga PHY and 256 RX descriptors for DMA to move.
>
> Signed-off-by: Sean Wang <sean.wang@mediatek.com>
gave it a quick spin on my hardware and it works well even when using
more than 1 gmac.
Acked-by: John Crispin <john@phrozen.org>
> ---
> drivers/net/ethernet/mediatek/mtk_eth_soc.c | 11 ++++++-----
> 1 file changed, 6 insertions(+), 5 deletions(-)
>
> diff --git a/drivers/net/ethernet/mediatek/mtk_eth_soc.c b/drivers/net/ethernet/mediatek/mtk_eth_soc.c
> index 431b851..a1bdb53 100644
> --- a/drivers/net/ethernet/mediatek/mtk_eth_soc.c
> +++ b/drivers/net/ethernet/mediatek/mtk_eth_soc.c
> @@ -890,13 +890,14 @@ release_desc:
> rxd->rxd2 = RX_DMA_PLEN0(ring->buf_size);
>
> ring->calc_idx = idx;
> - /* make sure that all changes to the dma ring are flushed before
> - * we continue
> - */
> - wmb();
> - mtk_w32(eth, ring->calc_idx, MTK_PRX_CRX_IDX0);
> +
> done++;
> }
> + /* make sure that all changes to the dma ring are flushed before
> + * we continue
> + */
> + wmb();
> + mtk_w32(eth, ring->calc_idx, MTK_PRX_CRX_IDX0);
>
> if (done < budget)
> mtk_w32(eth, MTK_RX_DONE_INT, MTK_PDMA_INT_STATUS);
>
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH net-next v5 2/2] net: ethernet: mediatek: enhance RX path by aggregating more SKBs into NAPI
2016-09-03 9:59 ` [PATCH net-next v5 2/2] net: ethernet: mediatek: enhance RX path by aggregating more SKBs into NAPI sean.wang
@ 2016-09-07 6:38 ` John Crispin
0 siblings, 0 replies; 6+ messages in thread
From: John Crispin @ 2016-09-07 6:38 UTC (permalink / raw)
To: sean.wang, davem; +Cc: nbd, keyhaede, netdev, linux-mediatek, objelf
On 03/09/2016 11:59, sean.wang@mediatek.com wrote:
> From: Sean Wang <sean.wang@mediatek.com>
>
> The patch adds support for aggregating more SKBs feed into NAPI in
> order to get more benefits from generic receive offload (GRO) by
> peeking at the RX ring status and moving more packets right before
> returning from NAPI RX polling handler if NAPI budgets are still
> available and some packets already present in hardware.
>
> Signed-off-by: Sean Wang <sean.wang@mediatek.com>
this looks ok to me, gave it a quick try on my HW and it works well. i
am just not sure if this is something that is commonly done. someone
with more overview of how NAPI should be used might want to comment on this.
Tested-by: John Crispin <john@phrozen.org>
> ---
> drivers/net/ethernet/mediatek/mtk_eth_soc.c | 31 ++++++++++++++++-------------
> 1 file changed, 17 insertions(+), 14 deletions(-)
>
> diff --git a/drivers/net/ethernet/mediatek/mtk_eth_soc.c b/drivers/net/ethernet/mediatek/mtk_eth_soc.c
> index a1bdb53..66fd45a 100644
> --- a/drivers/net/ethernet/mediatek/mtk_eth_soc.c
> +++ b/drivers/net/ethernet/mediatek/mtk_eth_soc.c
> @@ -893,14 +893,14 @@ release_desc:
>
> done++;
> }
> - /* make sure that all changes to the dma ring are flushed before
> - * we continue
> - */
> - wmb();
> - mtk_w32(eth, ring->calc_idx, MTK_PRX_CRX_IDX0);
>
> - if (done < budget)
> - mtk_w32(eth, MTK_RX_DONE_INT, MTK_PDMA_INT_STATUS);
> + if (done) {
> + /* make sure that all changes to the dma ring are flushed before
> + * we continue
> + */
> + wmb();
> + mtk_w32(eth, ring->calc_idx, MTK_PRX_CRX_IDX0);
> + }
>
> return done;
> }
> @@ -1020,10 +1020,13 @@ static int mtk_napi_rx(struct napi_struct *napi, int budget)
> struct mtk_eth *eth = container_of(napi, struct mtk_eth, rx_napi);
> u32 status, mask;
> int rx_done = 0;
> + int remain_budget = budget;
>
> mtk_handle_status_irq(eth);
> +
> +poll_again:
> mtk_w32(eth, MTK_RX_DONE_INT, MTK_PDMA_INT_STATUS);
> - rx_done = mtk_poll_rx(napi, budget, eth);
> + rx_done = mtk_poll_rx(napi, remain_budget, eth);
>
> if (unlikely(netif_msg_intr(eth))) {
> status = mtk_r32(eth, MTK_PDMA_INT_STATUS);
> @@ -1032,18 +1035,18 @@ static int mtk_napi_rx(struct napi_struct *napi, int budget)
> "done rx %d, intr 0x%08x/0x%x\n",
> rx_done, status, mask);
> }
> -
> - if (rx_done == budget)
> + if (rx_done == remain_budget)
> return budget;
>
> status = mtk_r32(eth, MTK_PDMA_INT_STATUS);
> - if (status & MTK_RX_DONE_INT)
> - return budget;
> -
> + if (status & MTK_RX_DONE_INT) {
> + remain_budget -= rx_done;
> + goto poll_again;
> + }
> napi_complete(napi);
> mtk_irq_enable(eth, MTK_PDMA_INT_MASK, MTK_RX_DONE_INT);
>
> - return rx_done;
> + return rx_done + budget - remain_budget;
> }
>
> static int mtk_tx_alloc(struct mtk_eth *eth)
>
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2016-09-07 6:38 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-09-03 9:59 [PATCH net-next v5 0/2] net: ethernet: mediatek: add enhancements to RX path sean.wang
2016-09-03 9:59 ` [PATCH net-next v5 1/2] net: ethernet: mediatek: enhance RX path by reducing the frequency of the memory barrier used sean.wang
2016-09-07 6:35 ` John Crispin
2016-09-03 9:59 ` [PATCH net-next v5 2/2] net: ethernet: mediatek: enhance RX path by aggregating more SKBs into NAPI sean.wang
2016-09-07 6:38 ` John Crispin
[not found] ` <1472896767-18352-1-git-send-email-sean.wang-NuS5LvNUpcJWk0Htik3J/w@public.gmane.org>
2016-09-06 20:33 ` [PATCH net-next v5 0/2] net: ethernet: mediatek: add enhancements to RX path David Miller
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).