Received-SPF: pass (google.com: best guess record for domain of linux-wireless-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67;
From:   Toke =?utf-8?Q?H=C3=B8iland-J=C3=B8rgensen?= <toke@redhat.com>
To:     Johannes Berg <johannes@sipsolutions.net>
Cc:     linux-wireless@vger.kernel.org, Kan Yan <kyan@google.com>
Subject: Re: [PATCH] mac80211: Add new sta_info getter by sta/vif addrs
In-Reply-To: <20191112130835.382062-1-toke@redhat.com>
References: <20191112130835.382062-1-toke@redhat.com>
Date:   Tue, 12 Nov 2019 14:17:36 +0100
Message-ID: <87sgmtp72n.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

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

> In ieee80211_tx_status() we don't have an sdata struct when looking up th=
e
> destination sta. Instead, we just do a lookup by the vif addr that is the
> source of the packet being completed. Factor this out into a new sta_info
> getter helper, since we need to use it for accounting AQL as well.
>
> Signed-off-by: Toke H=C3=B8iland-J=C3=B8rgensen <toke@redhat.com>

This was another self-contained change that is a pre-requisite for the
AQL patch, but I figured I could just send on its own while we iterate
on the main patch.

Johannes, do feel free to tell me if it is annoying to do it this way;
I'm just trying to speed things up a bit (while eyeing the fast
approaching merge window :)).

-Toke