Page MenuHomeFreeBSD

ifnet: use ck_pr(3) store & load setting ifnet pointer in ifindex
ClosedPublic

Authored by glebius on Dec 4 2021, 9:15 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Oct 21, 2:51 AM
Unknown Object (File)
Thu, Oct 10, 5:00 PM
Unknown Object (File)
Tue, Oct 8, 12:02 AM
Unknown Object (File)
Oct 6 2024, 3:32 PM
Unknown Object (File)
Oct 3 2024, 1:55 PM
Unknown Object (File)
Oct 3 2024, 9:00 AM
Unknown Object (File)
Sep 28 2024, 11:39 PM
Unknown Object (File)
Sep 28 2024, 3:04 PM
Subscribers

Details

Summary

The lockless access to the array is protected by the network epoch.

Diff Detail

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

Event Timeline

I think there should only be one function for "store" pointer which is ifnet_setbyindex(); that'll be possible if you consider my comment from D33259; though in the longer term it probably won't matter to have the other one for ifindex_free() as we'd still guarantee that all accesses go through wrapper functions so we can keep changing the internals.

If you can explain the added WASSERT I am fine with this.

sys/net/if.c
416

Contrary to your description, why do you need to add this suddenly here?

sys/net/if.c
416

Can be removed, but it still removed when all index allocation bits are merged into one in D33262

This revision is now accepted and ready to land.Dec 6 2021, 10:48 AM