Page MenuHomeFreeBSD
Differential D43512

net80211: make sure calls to (*iv_update_bss)() are locked
ClosedPublic
Actions

Authored by bz on Jan 19 2024, 5:07 PM.
Tags
None
Referenced Files
F102657446: D43512.diff
Fri, Nov 15, 11:03 AM
Unknown Object (File)
Sun, Nov 10, 5:31 PM
Unknown Object (File)
Fri, Oct 25, 11:10 PM
Unknown Object (File)
Fri, Oct 18, 10:38 PM
Unknown Object (File)
Fri, Oct 18, 12:53 PM
Unknown Object (File)
Fri, Oct 18, 2:29 AM
Unknown Object (File)
Oct 3 2024, 11:33 AM
Unknown Object (File)
Oct 2 2024, 9:29 PM
View All 40 Files
Subscribers
glebius
imp
melifaro
wireless

Details

Reviewers
adrian
cc
Commits
rGeecf453c0e56: net80211: make sure calls to (*iv_update_bss)() are locked
rGf8ec03794357: net80211: make sure calls to (*iv_update_bss)() are locked
rG383cbcd21b73: net80211: make sure calls to (*iv_update_bss)() are locked
rG49619f73151a: net80211: make sure calls to (*iv_update_bss)() are locked
Summary

It turned out thare various calls into (*iv_update_bss)(), that is
direct changes to vap->iv_bss in the old days, happened without
synchronisation.

Use locking assertions to document the requirement or status quo
at some callers given ic locking will eventually have to be dealt
with.

MFC after: 3 days

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 55498
Build 52387: arc lint + arc unit

Event Timeline

bz created this revision.Jan 19 2024, 5:07 PM
Herald added subscribers: glebius, melifaro, imp. · View Herald TranscriptJan 19 2024, 5:07 PM
bz requested review of this revision.Jan 19 2024, 5:07 PM
Harbormaster completed remote builds in B55498: Diff 133037.Jan 19 2024, 5:07 PM
cc accepted this revision.Jan 23 2024, 5:22 PM
This revision is now accepted and ready to land.Jan 23 2024, 5:22 PM
cc added a comment.Jan 23 2024, 8:43 PM

By the way, I forgot to ask, is it also helpful to add IEEE80211_LOCK_ASSERT in lkpi_iv_update_bss()?

Closed by commit rG49619f73151a: net80211: make sure calls to (*iv_update_bss)() are locked (authored by bz). · Explain WhyFeb 3 2024, 1:57 PM
This revision was automatically updated to reflect the committed changes.
bz added a commit: rG49619f73151a: net80211: make sure calls to (*iv_update_bss)() are locked.
bz added a commit: rG383cbcd21b73: net80211: make sure calls to (*iv_update_bss)() are locked.Feb 18 2024, 9:12 PM
bz added a commit: rGf8ec03794357: net80211: make sure calls to (*iv_update_bss)() are locked.Feb 19 2024, 8:09 AM
bz added a commit: rGeecf453c0e56: net80211: make sure calls to (*iv_update_bss)() are locked.Feb 19 2024, 4:10 PM

Revision Contents
Changeset List

PathSize
sys/
net80211/
15 lines
2 lines

Diff 133037

View Options

sys/net80211/ieee80211_node.c

Loading...
View Options

sys/net80211/ieee80211_proto.c

Loading...