Page MenuHomeFreeBSD
Differential D56135

nd6: Break nd6_prefix_lifetime_update out of prelist_update
ClosedPublic
Actions

Authored by pouria on Sat, Mar 28, 8:14 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Apr 18, 11:33 AM
Unknown Object (File)
Fri, Apr 17, 1:06 PM
Unknown Object (File)
Fri, Apr 17, 12:39 PM
Unknown Object (File)
Fri, Apr 17, 12:37 PM
Unknown Object (File)
Fri, Apr 17, 12:36 PM
Unknown Object (File)
Fri, Apr 17, 10:26 AM
Unknown Object (File)
Wed, Apr 15, 12:12 PM
Unknown Object (File)
Wed, Apr 15, 11:16 AM
View All 29 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.Sat, Mar 28, 8:14 PM
Herald added subscribers: melifaro, ae, imp. · View Herald TranscriptSat, Mar 28, 8:14 PM
pouria requested review of this revision.Sat, Mar 28, 8:14 PM
Harbormaster completed remote builds in B71783: Diff 174459.Sat, Mar 28, 8:14 PM
pouria added a parent revision: D56134: nd6: Remove anycast check in prelist_update.Sat, Mar 28, 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.Wed, Apr 1, 4:36 PM
markj added inline comments.Thu, Apr 2, 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.Thu, Apr 2, 11:27 AM
pouria marked an inline comment as done.

Address @markj comment.
Thank you

Harbormaster completed remote builds in B71914: Diff 174803.Thu, Apr 2, 11:27 AM
markj added inline comments.Fri, Apr 3, 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.Fri, Apr 3, 1:09 PM

Address @markj comment

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

kind ping: @markj

markj accepted this revision.Fri, Apr 17, 12:45 PM
markj added inline comments.
sys/netinet6/nd6_rtr.c
1537
This revision is now accepted and ready to land.Fri, Apr 17, 12:45 PM
Closed by commit rG5f27592e149f: nd6: Break nd6_prefix_lifetime_update out of prelist_update (authored by pouria). · Explain WhyFri, Apr 17, 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...