linux-wireless.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Kalle Valo <kvalo@codeaurora.org>
To: Loic Poulain <loic.poulain@linaro.org>
Cc: wcn36xx@lists.infradead.org, linux-wireless@vger.kernel.org,
	Loic Poulain <loic.poulain@linaro.org>
Subject: Re: [PATCH v2 1/6] wcn36xx: Fix multiple AMPDU sessions support
Date: Fri, 14 Aug 2020 15:02:49 +0000 (UTC)	[thread overview]
Message-ID: <20200814150249.A9779C43395@smtp.codeaurora.org> (raw)
In-Reply-To: <1595586052-16081-2-git-send-email-loic.poulain@linaro.org>

Loic Poulain <loic.poulain@linaro.org> wrote:

> Several AMPDU sessions can be started, e.g. for different TIDs.
> Currently the driver does not take care of the session ID when
> requesting block-ack (statically set to 0), which leads to never
> block-acked packet with sessions other than 0.
> 
> Fix this by saving the session id when creating the ba session and
> use it in subsequent ba operations.
> 
> This issue can be reproduced with iperf in two steps (tid 0 strem
> then tid 6 stream).
> 
> 1.0 iperf -s                                # wcn36xx side
> 1.1 iperf -c ${IP_ADDR}                     # host side
> 
> Then
> 
> 2.0 iperf -s -u -S 0xC0                     # wcn36xx side
> 2.1 iperf -c ${IP_ADDR} -u -S 0xC0 -l 2000  # host side
> 
> Signed-off-by: Loic Poulain <loic.poulain@linaro.org>
> Signed-off-by: Kalle Valo <kvalo@codeaurora.org>

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

ffe835aa5bdb wcn36xx: Fix multiple AMPDU sessions support
fdf21cc37149 wcn36xx: Add TX ack support
1c20560607e6 wcn36xx: Increase number of TX retries
512b191d9652 wcn36xx: Fix TX data path
84aff52e4f57 wcn36xx: Use sequence number allocated by mac80211

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

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


  reply	other threads:[~2020-08-14 15:03 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-07-24 10:20 [PATCH v2 0/6] wcn36xx misc fixes Loic Poulain
2020-07-24 10:20 ` [PATCH v2 1/6] wcn36xx: Fix multiple AMPDU sessions support Loic Poulain
2020-08-14 15:02   ` Kalle Valo [this message]
2020-07-24 10:20 ` [PATCH v2 2/6] wcn36xx: Add TX ack support Loic Poulain
2020-07-24 10:20 ` [PATCH v2 3/6] wcn36xx: Increase number of TX retries Loic Poulain
     [not found]   ` <CA+Kvs9nZm+uoRJfjq33s1_nTbPt82NnEKmgYCWk0q5vH+-ca9w@mail.gmail.com>
2020-07-24 13:12     ` Kalle Valo
2020-07-24 13:43       ` Ramon Fried
2020-07-24 10:20 ` [PATCH v2 4/6] wcn36xx: Fix TX data path Loic Poulain
2020-07-24 10:20 ` [PATCH v2 5/6] wcn36xx: Fix software-driven scan Loic Poulain
2020-07-29 23:03   ` Bryan O'Donoghue
2020-08-14 14:50   ` Kalle Valo
2020-07-24 10:20 ` [PATCH v2 6/6] wcn36xx: Use sequence number allocated by mac80211 Loic Poulain
2020-07-24 13:47 ` [PATCH v2 0/6] wcn36xx misc fixes Ramon Fried
2020-07-24 14:02   ` Loic Poulain
2020-07-24 18:51     ` Ramon Fried
2020-07-28 10:13       ` Loic Poulain

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20200814150249.A9779C43395@smtp.codeaurora.org \
    --to=kvalo@codeaurora.org \
    --cc=linux-wireless@vger.kernel.org \
    --cc=loic.poulain@linaro.org \
    --cc=wcn36xx@lists.infradead.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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).