On 23-05-18 11:49:45, Frank Li wrote:
> At iMX8QM platform, enable NCM gadget and run 'iperf3 -s'.
> At host, run 'iperf3 -V -c fe80::6863:98ff:feef:3e0%enxc6e147509498'
>
> [ 5] 0.00-1.00 sec 1.55 MBytes 13.0 Mbits/sec 90 4.18 KBytes
> [ 5] 1.00-2.00 sec 1.44 MBytes 12.0 Mbits/sec 75 4.18 KBytes
> [ 5] 2.00-3.00 sec 1.48 MBytes 12.4 Mbits/sec 75 4.18 KBytes
>
> Expected speed should be bigger than 300Mbits/sec.
>
> The root cause of this performance drop was found to be data corruption
> happening at 4K borders in some Ethernet packets, leading to TCP
> checksum errors. This corruption occurs from the position
> (4K - (address & 0x7F)) to 4K. The u_ether function's allocation of
> skb_buff reserves 64B, meaning all RX addresses resemble 0xXXXX0040.
>
> Force trb_burst_size to 16 can fix this problem.
>
> Cc: stable@xxxxxxxxxxxxxxx
> Fixes: 7733f6c32e36 ("usb: cdns3: Add Cadence USB3 DRD Driver")
> Signed-off-by: Frank Li <Frank.Li@xxxxxxx>
> ---
> drivers/usb/cdns3/cdns3-gadget.c | 13 +++++++++++++
> 1 file changed, 13 insertions(+)
>
> diff --git a/drivers/usb/cdns3/cdns3-gadget.c b/drivers/usb/cdns3/cdns3-gadget.c
> index ccfaebca6faa..1dcadef933e3 100644
> --- a/drivers/usb/cdns3/cdns3-gadget.c
> +++ b/drivers/usb/cdns3/cdns3-gadget.c
> @@ -2097,6 +2097,19 @@ int cdns3_ep_config(struct cdns3_endpoint *priv_ep, bool enable)
> else
> priv_ep->trb_burst_size = 16;
>
> + /*
> + * In versions preceding DEV_VER_V2, for example, iMX8QM, there exit the bugs
> + * in the DMA. These bugs occur when the trb_burst_size exceeds 16 and the
> + * address is not aligned to 128 Bytes (which is a product of the 64-bit AXI
> + * and AXI maximum burst length of 16 or 0xF+1, dma_axi_ctrl0[3:0]). This
> + * results in data corruption when it crosses the 4K border. The corruption
> + * specifically occurs from the position (4K - (address & 0x7F)) to 4K.
> + *
> + * So force trb_burst_size to 16 at such platform.
> + */
> + if (priv_dev->dev_ver < DEV_VER_V2)
> + priv_ep->trb_burst_size = 16;
> +
> mult = min_t(u8, mult, EP_CFG_MULT_MAX);
> buffering = min_t(u8, buffering, EP_CFG_BUFFERING_MAX);
> maxburst = min_t(u8, maxburst, EP_CFG_MAXBURST_MAX);
> --
> 2.34.1
>
Acked-by: Peter Chen <peter.chen@xxxxxxxxxx>
--
Thanks,
Peter Chen
