Page MenuHomeFreeBSD

routing: Remove unused rib_head members
ClosedPublic

Authored by pouria on Jul 29 2026, 5:57 PM.
Tags
None
Referenced Files
F173552920: D58537.diff
Sat, Sep 26, 7:36 PM
F173528645: D58537.id183067.diff
Sat, Sep 26, 3:16 PM
Unknown Object (File)
Sat, Sep 26, 9:30 AM
Unknown Object (File)
Sat, Sep 26, 9:29 AM
Unknown Object (File)
Sat, Sep 26, 9:29 AM
Unknown Object (File)
Mon, Sep 21, 3:07 AM
Unknown Object (File)
Fri, Sep 18, 9:24 PM
Unknown Object (File)
Thu, Sep 17, 11:19 PM
Subscribers

Details

Summary

Remove unused rnh_multipath and rib_algo_fixed members.
While here, convert rib_dying and rib_algo_init from uint32_t to bool.

Diff Detail

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

Event Timeline

sys/net/route/route_var.h
69–71

Practically a bool occupies one byte. Not sure if there is a standard about that, though. So change as is will take 2 bytes. But if you code it as bit fields, they are getting packed. With suggested change two bools will take only one byte and later you can add 6 more bools in the same manner without growing the structure.

I actually really would like us officially to promote this style instead of using pre-processor defines for flags.

pouria marked an inline comment as done.

Address @glebius comment

This revision is now accepted and ready to land.Jul 29 2026, 11:10 PM
This revision was automatically updated to reflect the committed changes.