Page MenuHomeFreeBSD

net80211: refactor out ifp->if_broadcastaddr into ieee80211_freebsd.c
ClosedPublic

Authored by adrian on May 18 2025, 4:45 AM.
Referenced Files
Unknown Object (File)
Thu, Jul 10, 3:31 PM
Unknown Object (File)
Mon, Jul 7, 7:58 PM
Unknown Object (File)
Mon, Jul 7, 2:06 PM
Unknown Object (File)
Mon, Jul 7, 8:58 AM
Unknown Object (File)
Sun, Jul 6, 9:11 PM
Unknown Object (File)
Sun, Jul 6, 9:08 AM
Unknown Object (File)
Sun, Jul 6, 12:05 AM
Unknown Object (File)
Sat, Jul 5, 11:18 AM

Details

Summary
  • create ieee80211_vap_get_broadcast_address() to fetch the broadcast MAC address for the given VAP
  • refactor references to ifp->if_broadcastaddr -> ieee80211_vap_get_broadcast_address()

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

bz added a subscriber: bz.

Scrolling through apart from the local variable this looks good.

sys/net80211/ieee80211_freebsd.c
1328

Could pass vap->if_ifp here and save the local variable.

This revision is now accepted and ready to land.May 18 2025, 8:16 PM
This revision now requires review to proceed.May 19 2025, 4:11 AM
bz requested changes to this revision.May 24 2025, 11:27 PM

Just style.

sys/net80211/ieee80211_freebsd.c
1326

return (...)

sys/net80211/ieee80211_hostap.c
563

Can't say in Phabricator if indentation is right..

sys/net80211/ieee80211_wds.c
491

again indent unclear in Phabricator

This revision now requires changes to proceed.May 24 2025, 11:27 PM
This revision is now accepted and ready to land.May 25 2025, 1:38 PM