Page MenuHomeFreeBSD
Differential D56130

nd6: break nd6_prefix_update out of prelist_update
ClosedPublic
Actions

Authored by pouria on Mar 28 2026, 8:13 PM.
Tags
None
Referenced Files
F157640305: D56130.id175794.diff
Sat, May 23, 3:46 PM
Unknown Object (File)
Fri, May 22, 12:06 AM
Unknown Object (File)
Fri, May 22, 12:05 AM
Unknown Object (File)
Thu, May 14, 8:14 AM
Unknown Object (File)
Thu, May 14, 5:29 AM
Unknown Object (File)
Wed, May 13, 4:27 PM
Unknown Object (File)
Mon, May 11, 9:04 AM
Unknown Object (File)
Sat, May 9, 7:47 AM
View All 28 Files
Subscribers
ae
imp
melifaro

Details

Reviewers
markj
glebius
bz
Group Reviewers
network
Commits
rGd96e8cb4e728: nd6: Break nd6_prefix_update out of prelist_update
Summary

if PI exists, call prefix_update, instead of doing it inside the prelist_update.
no functional change intended.

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:13 PM
Herald added subscribers: melifaro, ae, imp. · View Herald TranscriptMar 28 2026, 8:13 PM
pouria requested review of this revision.Mar 28 2026, 8:13 PM
Harbormaster completed remote builds in B71778: Diff 174454.Mar 28 2026, 8:13 PM
pouria added a child revision: D56131: nd6: break pfxrtr_add out of nd6_prelist_add.Mar 28 2026, 8:13 PM
pouria added a comment.Mar 28 2026, 8:27 PM

For reviewers:
The whole revision stack will reduce PI code complexity and fix multiple issues along the way.
I also wrote additional tests in D56128 to make sure everything works as intended.

markj added inline comments.Mar 29 2026, 7:14 AM
sys/netinet6/nd6_rtr.c
1552

Before, prelist_update() would return the value of error in this case. Now it does not. Is it intentional?

markj accepted this revision.Mar 29 2026, 7:16 AM
markj added inline comments.
sys/netinet6/nd6_rtr.c
1552

After D54132 it does not matter, okay.

This revision is now accepted and ready to land.Mar 29 2026, 7:16 AM
pouria marked 2 inline comments as done.Mar 29 2026, 10:52 AM
pouria added inline comments.
sys/netinet6/nd6_rtr.c
1552

*D56132. Thanks!

ae added a comment.Mar 29 2026, 3:28 PM
In D56130#1283954, @pouria wrote:

For reviewers:
The whole revision stack will reduce PI code complexity and fix multiple issues along the way.
I also wrote additional tests in D56128 to make sure everything works as intended.

Can you show what issues you are talking about?
I also have the patch related to prefixes and CARP.

pouria marked an inline comment as done.Mar 31 2026, 8:45 PM
In D56130#1284183, @ae wrote:

Can you show what issues you are talking about?
I also have the patch related to prefixes and CARP.

I've mostly done cleanups and preparation for my other patches:

  • removing several unused conditions
  • broke down of a large function with ~350 lines.
  • fixes some DRY issues.

I took a quick look at your patch, I'll have multiple conflicts with it, especially because you used some reserved RA flags.

pouria retitled this revision from ndp: break nd6_prefix_update out of prelist_update to nd6: break nd6_prefix_update out of prelist_update.Apr 1 2026, 4:06 PM
Closed by commit rGd96e8cb4e728: nd6: Break nd6_prefix_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: rGd96e8cb4e728: nd6: Break nd6_prefix_update out of prelist_update.

Revision Contents
Changeset List

PathSize
sys/
netinet6/
78 lines

Diff 175794

View Options

sys/netinet6/nd6_rtr.c

Loading...