Page MenuHomeFreeBSD

net80211: migrate FC0_TYPE_MASK / FC0_SUBTYPE_MASK frame type checks to macros
ClosedPublic

Authored by adrian on Sat, Nov 9, 9:48 PM.
Referenced Files
Unknown Object (File)
Sun, Nov 17, 6:00 PM
Unknown Object (File)
Sun, Nov 17, 7:53 AM
Unknown Object (File)
Sun, Nov 17, 3:28 AM
Unknown Object (File)
Sun, Nov 17, 2:58 AM
Unknown Object (File)
Sat, Nov 16, 7:23 AM
Unknown Object (File)
Thu, Nov 14, 6:50 AM
Unknown Object (File)
Thu, Nov 14, 6:45 AM
Unknown Object (File)
Thu, Nov 14, 5:53 AM

Details

Summary
  • Add macros for the management and control frame type checks that I've come across in the drivers.
  • Delete some now old code (eg ath's ieee80211_is_action()) as there's now a macro for it.

Local testing:

  • not yet, I have a lot of wifi devices to find and test against

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 60474
Build 57358: arc lint + arc unit

Event Timeline

adrian requested review of this revision.Sat, Nov 9, 9:48 PM
bz added a subscriber: bz.

If you want please address the comments but given none is a logic problem I'll leave it to you.

sys/dev/ath/if_ath_tx.c
1136

superfluous ()

sys/dev/ipw/if_ipw.c
1122

I think style in this file doesn't need the space after !

sys/dev/otus/if_otus.c
1689

Space after !

sys/dev/usb/wlan/if_urtw.c
1726–1727

Those two could be macros as well.

This revision is now accepted and ready to land.Sat, Nov 9, 11:22 PM
This revision now requires review to proceed.Sun, Nov 10, 4:08 AM

updates, thanks!

This revision is now accepted and ready to land.Sun, Nov 10, 2:53 PM

FYI if Phab doesn't add the commit automatically (either because of a typo or misformatted commit message or because Phab is just broken) you can add it later via Edit Related Objects > Edit Commits and then paste the commit hash.