I think it's okay to remove it. Just make sure it doesn't break adhoc mode 11n rate "negotiation" stuff.

hi! see if those comment/commit updates help!

Take on comments from bz@

In D34920#791558, @bz wrote:

Was there a PR for this somewhere? If so, please mention that.

link

I'll +1 it to commit once they're moved to ieee80211_freebsd.c

update ids

update id

update id

address more things

address more things

address more things

n/a

n/a

Fix debug

update dates

update dates

update dates

Update dates

broke it up into three commits/reviews to make this easier

pull out the dts/include bits into a different commit in the stack

In D34096#771107, @manu wrote:

Do you expect people to review a change that big ?

fix debug flags macro expansion

What's left to land this?

.. if you're distributing workload with RSS and you set the numebr of RSS netisr contexts, that's where you can run your parallelism. The whole point here is to whack all the net processing in netisr contexts rather than keeping adding taskqueues and inventing new/fun ways to map cpus and figure out how many.

commit msg

remove old debugging

update from mmels' feedback:

In D33445#759706, @mmel wrote:

Sorry for the strong language, please don't take it the wrong way.

Fix commit msg

Address some feedback!

Updates from comments

going to update the patch in a sec with the above stuff updated

comment from manu

more updates from jrtc27

updates

In D33140#749351, @manu wrote:

I don't think that a new softc variable is needed.
Just validate that we have a regulator for v2 only and then if we have one just call the methods.

I think it's fine. This only /really/ works with non-VLAN packets (ask me how I know due to MIPS, sigh) as the VLAN header ends up making the ethernet header size 2 bytes larger. It brings it inline with the rest of the system behaviour in this case.