Re: [PATCH 0/4] cfg80211/mac80211: Add support for TID specific configuration

From: Sergey Matyukevich <sergey.matyukevich.os@quantenna.com>
To: Johannes Berg <johannes@sipsolutions.net>
Cc: Tamizh chelvam <tamizhr@codeaurora.org>,
	"ath10k@lists.infradead.org" <ath10k@lists.infradead.org>,
	"linux-wireless@vger.kernel.org" <linux-wireless@vger.kernel.org>,
	Igor Mitsyanko <igor.mitsyanko.os@quantenna.com>,
	Ben Greear <greearb@candelatech.com>
Subject: Re: [PATCH 0/4] cfg80211/mac80211: Add support for TID specific configuration
Date: Tue, 6 Nov 2018 10:45:53 +0000	[thread overview]
Message-ID: <20181106104547.5mnbymn4peuhgng3@bars> (raw)
In-Reply-To: <b833d8c0987e838759dbf00d97d4989a267a10fb.camel@sipsolutions.net>

> On Mon, 2018-10-22 at 23:25 +0530, Tamizh chelvam wrote:
> > Add infrastructure for per TID aggregation/retry count configurations
> > such as retry count and AMPDU aggregation control(disable/enable).
> > In some scenario reducing the number of retry count for a specific data
> > traffic can reduce the latency by proceeding with the next packet
> > instead of retrying the same packet more time. This will be useful
> > where the next packet can resume the operation without an issue.
> 
> Not sure I understand this, how can you expect to control something on a
> per-packet basis using this?
> 
> Sergey, looks like your A-MPDU control is already in here per RA/TID,
> and A-MSDU could be added easily?

Hello Johannes,

Thanks for pointing me at this patch series. Indeed, it looks like an
exact match for proper RA/TID aware implementation of AMPDU control.
AMSDU can be added following the same approach.

Regards,
Sergey