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)
Fri, Nov 28, 6:44 PM
Unknown Object (File)
Fri, Nov 28, 7:32 AM
Unknown Object (File)
Fri, Nov 28, 6:21 AM
Unknown Object (File)
Fri, Nov 28, 5:34 AM
Unknown Object (File)
Fri, Nov 28, 5:33 AM
Unknown Object (File)
Thu, Nov 27, 1:21 PM
Unknown Object (File)
Thu, Nov 27, 10:35 AM
Unknown Object (File)
Mon, Nov 24, 10:09 PM

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