Page MenuHomeFreeBSD

netlink: some refactoring of NETLINK_GENERIC layer
ClosedPublic

Authored by glebius on Jan 3 2025, 10:35 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Jan 13, 12:12 PM
Unknown Object (File)
Sat, Jan 10, 4:50 AM
Unknown Object (File)
Sat, Jan 10, 4:37 AM
Unknown Object (File)
Sat, Jan 10, 4:31 AM
Unknown Object (File)
Sat, Jan 10, 4:27 AM
Unknown Object (File)
Fri, Jan 9, 6:58 PM
Unknown Object (File)
Fri, Jan 9, 5:24 AM
Unknown Object (File)
Wed, Dec 24, 10:13 PM
Subscribers

Details

Summary
  • Statically initialize control family/group. This removes extra startup

code and provides a strong guarantee that they reside at the 0 index of
the respective arrays. Before a genl_register_family() with a higher
SYSINIT order could try to hijack index 0.

  • Remove the family_id field completely. Now the family ID as well as

group ID are array indices and there is basically no place for a mistake.
Previous code had a bug where a KPI user could induce an ID mismatch.

  • Merge netlink_generic_kpi.c to netlink_generic.c. Both files are small

and now there is more dependency between the control family and the family
allocator. Ok'ed by melifaro@.

Diff Detail

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