On 9/27/2023 12:00 AM, Johannes Berg wrote:
On Tue, 2023-09-26 at 13:35 -0700, Jeff Johnson wrote:
On 9/26/2023 7:09 AM, Johannes Berg wrote:
From: Johannes Berg <johannes.berg@xxxxxxxxx>
There's no user for this, so remove the support.
Signed-off-by: Johannes Berg <johannes.berg@xxxxxxxxx>
if you are going to remove it, why not just revert e306784a8de0
("cfg80211: AP mode driver offload for FILS association crypto") to make
sure you get all of the artifacts? for example, I believe your patch
will leave behind an unused struct cfg80211_fils_aad and unused
FILS_AAD_ASSIGN() trace helper macro.
Hah, good point, I didn't do this well.
the caveat to reverting is that it should only be a partial revert; the
UAPI definitions would need to be retained (and should be documented as
obsolete).
Yeah, that's why I didn't do it as a revert.
however, let me check to make sure there is no plan to actually utilize
this interface upstream. as i've indicated earlier, we are in the
process of trying to transition to an "upstream first" mentality, but
this is not going to happen overnight, but instead will take years. that
said, i'd hate to rip out an interface now just to need to add it back
in the future.
Sure. I don't mind keeping something around that really _has_ a future,
but it's been two years and nobody showed up ... but yeah, I also think
that "has a future" means upstream.
And clearly the old "other people can use it" argument doesn't work any
more either, the only other vendors who are doing something in AP mode
are Mediatek and maybe to some extent Realtek, and they all work on top
of mac80211 with thinner firmware. Broadcom has disappeared as far as I
can tell, with the occasional patch like recently that I'm also
suspecting serves pure out-of-tree driver purposes...
Based on internal discussion I don't see a path forward to using this
interface upstream, so please continue with the extraction,
incorporating my prior comments.
Thanks!
/jeff
