Received-SPF: pass (google.com: domain of linux-wireless-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
DMARC-Filter: OpenDMARC Filter v1.3.2 smtp.codeaurora.org 8FF83C433CB
Content-Type: text/plain; charset="utf-8"
MIME-Version: 1.0
Content-Transfer-Encoding: 7bit
Subject: Re: [PATCH v10 1/2] ath10k: add htt TX bundle for sdio
From:   Kalle Valo <kvalo@codeaurora.org>
In-Reply-To: <20200410061400.14231-2-wgong@codeaurora.org>
References: <20200410061400.14231-2-wgong@codeaurora.org>
To:     Wen Gong <wgong@codeaurora.org>
Cc:     ath10k@lists.infradead.org, linux-wireless@vger.kernel.org,
        wgong@codeaurora.org
User-Agent: pwcli/0.0.0-git (https://github.com/kvalo/pwcli/) Python/3.5.2
Message-Id: <20200422064402.EC4D4C433BA@smtp.codeaurora.org>
Date:   Wed, 22 Apr 2020 06:44:02 +0000 (UTC)
Sender: linux-wireless-owner@vger.kernel.org
Precedence: bulk

Wen Gong <wgong@codeaurora.org> wrote:

> 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 packet(byte)   cost time(us)   calculated rate(Mbps)
>       256               28                73
>       512               33               124
>      1024               35               234
>      1792               45               318
>     14336              168               682
>     28672              333               688
>     57344              660               695
> 
> This patch change the TX packet from single packet to a large length
> bundle packet, max size is 32, it results in significant performance
> improvement on TX path.
> 
> Also there's a fourth thread "ath10k_tx_complete_wq" added to ath10k as it
> improves TCP RX throughput (values in Mbps):
> 
>                                        TCP-RX    TCP-TX    UDP-RX      UDP-TX
> use workqueue_tx_complete              423       357       448         412
> change it to ar->workqueue             410       360       449         414
> change it to ar->workqueue_aux         405       339       446         401
> 
> This patch only effect sdio chip, it will not effect PCI, SNOC etc.
> It only enable bundle for sdio chip.
> 
> Tested with QCA6174 SDIO with firmware
> WLAN.RMH.4.4.1-00017-QCARMSWP-1.
> 
> Signed-off-by: Wen Gong <wgong@codeaurora.org>
> Signed-off-by: Kalle Valo <kvalo@codeaurora.org>

2 patches applied to ath-next branch of ath.git, thanks.

c8334512f3dd ath10k: add htt TX bundle for sdio
2f918ea98606 ath10k: enable alt data of TX path for sdio

-- 
https://patchwork.kernel.org/patch/11482769/

https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches