Re: regression in ath10k dma allocation

2019-08-20 Thread Tobias Klausmann
On 20.08.19 09:12, Christoph Hellwig wrote: On Tue, Aug 20, 2019 at 02:58:33PM +0800, Hillf Danton wrote: On Tue, 20 Aug 2019 05:05:14 +0200 Christoph Hellwig wrote: Tobias, plase try this patch: New version below: --- From b8a805e93be5a5662323b8ac61fe686df839c4ac Mon Sep 17 00:00:00

Re: [PATCH 4/7] ath10k: disable TX complete indication of htt for sdio

2019-08-20 Thread Toke Høiland-Jørgensen
Wen Gong writes: > Tx complete message from firmware cost bus bandwidth of sdio, and bus > bandwidth is the bollteneck of throughput, it will effect the bandwidth > occupancy of data packet of TX and RX. > > This patch disable TX complete indication from firmware for htt data > packet, it

Re: [PATCH 2/7] ath10k: change max RX bundle size from 8 to 32 for sdio

2019-08-20 Thread Toke Høiland-Jørgensen
Wen Gong writes: > The max bundle size support by firmware is 32, change it from 8 to 32 > will help performance. This results in significant performance > improvement on RX path. What happens when the hardware doesn't have enough data to fill a bundle? Does it send a smaller one, or does it

[PATCH 5/7] ath10k: add htt TX bundle for sdio

2019-08-20 Thread Wen Gong
The transmission utilization ratio for sdio bus for small packet is slow, because the space and time cost for sdio bus is same for large length packet and small length packet. So the speed of data for large length packet is higher than small length. Test result of different length of data: data

[PATCH 7/7] ath10k: enable napi on RX path for sdio

2019-08-20 Thread Wen Gong
For tcp RX, the quantity of tcp acks to remote is 1/2 of the quantity of tcp data from remote, then it will have many small length packets on TX path of sdio bus, then it reduce the RX packets's bandwidth of tcp. This patch enable napi on RX path, then the RX packet of tcp will not feed to tcp

[PATCH 6/7] ath10k: enable alt data of TX path for sdio

2019-08-20 Thread Wen Gong
The default credit size is 1792 bytes, but the IP mtu is 1500 bytes, then it has about 290 bytes's waste for each data packet on sdio transfer path for TX bundle, it will reduce the transmission utilization ratio for data packet. This patch enable the small credit size in firmware, firmware will

[PATCH 1/7] ath10k: enable RX bundle receive for sdio

2019-08-20 Thread Wen Gong
From: Alagu Sankar The existing implementation of initiating multiple sdio transfers for receive bundling is slowing down the receive speed. Combining the transfers using a bundle method would be ideal. The transmission utilization ratio for sdio bus for small packet is slow, because the space

[PATCH 4/7] ath10k: disable TX complete indication of htt for sdio

2019-08-20 Thread Wen Gong
Tx complete message from firmware cost bus bandwidth of sdio, and bus bandwidth is the bollteneck of throughput, it will effect the bandwidth occupancy of data packet of TX and RX. This patch disable TX complete indication from firmware for htt data packet, it results in significant performance

[PATCH 2/7] ath10k: change max RX bundle size from 8 to 32 for sdio

2019-08-20 Thread Wen Gong
The max bundle size support by firmware is 32, change it from 8 to 32 will help performance. This results in significant performance improvement on RX path. Tested with QCA6174 SDIO with firmware WLAN.RMH.4.4.1-7-QCARMSWP-1. Signed-off-by: Wen Gong --- drivers/net/wireless/ath/ath10k/htc.h

[PATCH 0/7] ath10k: improve throughout of tcp/udp TX/RX of sdio

2019-08-20 Thread Wen Gong
The bottleneck of throughout on sdio chip is the bus bandwidth, to the patches are all to increase the use ratio of sdio bus. udp-rxudp-txtcp-rxtcp-tx without patches(Mbps) 320180 170 151 with patches(Mbps) 450410 400

[PATCH 3/7] ath10k: add workqueue for RX path of sdio

2019-08-20 Thread Wen Gong
The thread of read rx message by sdio bus from firmware is synchronous, it will cost much time for process the left part of rx message which includes indicate the rx packet to uppper net stack. It will reduce the time of read from sdio. This patch move the indication to a workqueue, it results in

Re: regression in ath10k dma allocation

2019-08-20 Thread Christoph Hellwig
On Tue, Aug 20, 2019 at 02:58:33PM +0800, Hillf Danton wrote: > > On Tue, 20 Aug 2019 05:05:14 +0200 Christoph Hellwig wrote: > > > > Tobias, plase try this patch: > > New version below: --- >From b8a805e93be5a5662323b8ac61fe686df839c4ac Mon Sep 17 00:00:00 2001 From: Christoph Hellwig Date:

Re: regression in ath10k dma allocation

2019-08-20 Thread Hillf Danton
On Tue, 20 Aug 2019 05:05:14 +0200 Christoph Hellwig wrote: > > Tobias, plase try this patch: > A minute! > -- > >From 88c590a2ecafc8279388f25bfbe1ead8ea3507a6 Mon Sep 17 00:00:00 2001 > From: Christoph Hellwig > Date: Tue, 20 Aug 2019 11:45:49 +0900 > Subject: dma-direct: fix zone selection