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;
From:   Toke =?utf-8?Q?H=C3=B8iland-J=C3=B8rgensen?= <toke@redhat.com>
To:     Felix Fietkau <nbd@nbd.name>, linux-wireless@vger.kernel.org
Cc:     johannes@sipsolutions.net
Subject: Re: [PATCH 5.9 v2 1/2] mac80211: extend AQL aggregation estimation
 to HE and fix unit mismatch
In-Reply-To: <20200915085945.3782-1-nbd@nbd.name>
References: <20200915085945.3782-1-nbd@nbd.name>
Date:   Tue, 15 Sep 2020 11:24:30 +0200
Message-ID: <87d02nz92p.fsf@toke.dk>
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: quoted-printable
Sender: linux-wireless-owner@vger.kernel.org
Precedence: bulk

Felix Fietkau <nbd@nbd.name> writes:

> The unit of the return value of ieee80211_get_rate_duration is nanosecond=
s, not
> microseconds. Adjust the duration checks to account for that.
> For higher data rates, allow larger estimated aggregation sizes, and add =
some
> values for HE as well, which can use much larger aggregates.
> Since small packets with high data rates can now lead to duration values =
too
> small for info->tx_time_est, return a minimum of 4us.
>
> Fixes: f01cfbaf9b29 ("mac80211: improve AQL aggregation estimation for lo=
w data rates")
> Signed-off-by: Felix Fietkau <nbd@nbd.name>

Acked-by: Toke H=C3=B8iland-J=C3=B8rgensen <toke@redhat.com>