Page MenuHomeFreeBSD
Differential D56135

nd6: Break nd6_prefix_lifetime_update out of prelist_update
ClosedPublic
Actions

Authored by pouria on Mar 28 2026, 8:14 PM.
Tags
None
Referenced Files
F167129395: D56135.diff
Wed, Aug 19, 8:33 AM
Unknown Object (File)
Tue, Aug 18, 8:24 AM
Unknown Object (File)
Tue, Aug 18, 2:57 AM
Unknown Object (File)
Thu, Aug 13, 7:00 AM
Unknown Object (File)
Sun, Aug 9, 11:01 AM
Unknown Object (File)
Sat, Aug 8, 4:20 PM
Unknown Object (File)
Thu, Aug 6, 4:13 PM
Unknown Object (File)
Thu, Aug 6, 7:38 AM
View All 91 Files
Subscribers
ae
imp
melifaro

Details

Reviewers
markj
glebius
bz
Group Reviewers
network
Commits
rG5f27592e149f: nd6: Break nd6_prefix_lifetime_update out of prelist_update
Summary

Logic of updating prefix lifetime is big enough that deserves
its own function.
While here, fix style.

Diff Detail

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

Event Timeline

pouria created this revision.Mar 28 2026, 8:14 PM
Herald added subscribers: melifaro, ae, imp. · View Herald TranscriptMar 28 2026, 8:14 PM
pouria requested review of this revision.Mar 28 2026, 8:14 PM
Harbormaster completed remote builds in B71783: Diff 174459.Mar 28 2026, 8:14 PM
pouria added a parent revision: D56134: nd6: Remove anycast check in prelist_update.Mar 28 2026, 8:14 PM
pouria added a child revision: D56136: nd6: Remove goto and unused condition in prelist_update.
pouria retitled this revision from ndp: Break nd6_prefix_lifetime_update out of prelist_update to nd6: Break nd6_prefix_lifetime_update out of prelist_update.Apr 1 2026, 4:36 PM
markj added inline comments.Apr 2 2026, 9:46 AM
sys/netinet6/nd6_rtr.c
1589

These lines look too long. I think you can simplify the code a bit by storing time_uptime - ia6->ia6_createtime in a local variable.

pouria updated this revision to Diff 174803.Apr 2 2026, 11:27 AM
pouria marked an inline comment as done.

Address @markj comment.
Thank you

Harbormaster completed remote builds in B71914: Diff 174803.Apr 2 2026, 11:27 AM
markj added inline comments.Apr 3 2026, 7:29 AM
sys/netinet6/nd6_rtr.c
1589

Further simplification: set vltime = time_uptime - ia6->ia6_createtime + V_ip6_desync_factor.

pouria updated this revision to Diff 174851.Apr 3 2026, 1:09 PM

Address @markj comment

Harbormaster completed remote builds in B71931: Diff 174851.Apr 3 2026, 1:09 PM
pouria marked an inline comment as done.Apr 3 2026, 1:09 PM
pouria added a comment.Apr 16 2026, 9:49 PM

kind ping: @markj

markj accepted this revision.Apr 17 2026, 12:45 PM
markj added inline comments.
sys/netinet6/nd6_rtr.c
1537
This revision is now accepted and ready to land.Apr 17 2026, 12:45 PM
Closed by commit rG5f27592e149f: nd6: Break nd6_prefix_lifetime_update out of prelist_update (authored by pouria). · Explain WhyApr 17 2026, 10:59 PM
This revision was automatically updated to reflect the committed changes.
pouria added a commit: rG5f27592e149f: nd6: Break nd6_prefix_lifetime_update out of prelist_update.

Revision Contents
Changeset List

PathSize
sys/
netinet6/
207 lines

Diff 174851

View Options

sys/netinet6/nd6_rtr.c

Loading...