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
Unknown Object (File)
Fri, May 22, 4:45 PM
Unknown Object (File)
Fri, May 22, 1:44 AM
Unknown Object (File)
Fri, May 22, 1:43 AM
Unknown Object (File)
Fri, May 22, 1:43 AM
Unknown Object (File)
Mon, May 18, 3:26 PM
Unknown Object (File)
Mon, May 18, 3:22 PM
Unknown Object (File)
Sun, May 17, 10:10 PM
Unknown Object (File)
Sun, May 17, 10:05 PM
View All 53 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 Not Applicable
Unit
Tests Not Applicable

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
1608

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
1608

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
1556
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 175799

View Options

sys/netinet6/nd6_rtr.c

Loading...