Page MenuHomeFreeBSD
Differential D27675

Use more light-weight versions of routing lookup functions in ng_netflow.
ClosedPublic
Actions

Authored by melifaro on Dec 18 2020, 8:57 PM.
Tags
None
Referenced Files
F164555477: D27675.id80919.diff
Sun, Aug 2, 1:58 AM
Unknown Object (File)
Fri, Jul 31, 2:05 AM
Unknown Object (File)
Thu, Jul 30, 10:44 AM
Unknown Object (File)
Sun, Jul 26, 4:25 PM
Unknown Object (File)
Sun, Jul 26, 9:49 AM
Unknown Object (File)
Fri, Jul 24, 7:39 PM
Unknown Object (File)
Wed, Jul 22, 12:33 PM
Unknown Object (File)
Tue, Jul 21, 10:18 AM
View All Files
Subscribers
ae
donner
imp

Details

Reviewers
None
Group Reviewers
network
Commits
rG4e19e0d92ac6: Use light-weight versions of routing lookup functions in ng_netflow.
Summary

Use recently-added combination of fib[46]_lookup_rt() which returns rtentry & raw nexthop with
rt_get_inet[6]_plen() which returns address/prefix length of prefix inside rt instead of rib_lookup_info().

New calls does not require reference counting objects and reduce the amount of copied/processed rtentry data.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 35523
Build 32425: arc lint + arc unit

Event Timeline

melifaro created this revision.Dec 18 2020, 8:57 PM
Herald added subscribers: donner, imp. · View Herald TranscriptDec 18 2020, 8:57 PM
Harbormaster completed remote builds in B35523: Diff 80916.Dec 18 2020, 8:57 PM
melifaro requested review of this revision.Dec 18 2020, 8:57 PM
melifaro updated this revision to Diff 80918.Dec 18 2020, 9:05 PM

Add nhop_select_func().

Herald added a subscriber: ae. · View Herald TranscriptDec 18 2020, 9:05 PM
Harbormaster completed remote builds in B35525: Diff 80918.Dec 18 2020, 9:06 PM
melifaro edited the summary of this revision. (Show Details)Dec 18 2020, 9:08 PM
melifaro added a reviewer: network.
melifaro updated this revision to Diff 80919.Dec 18 2020, 9:09 PM

.

Harbormaster completed remote builds in B35526: Diff 80919.Dec 18 2020, 9:10 PM
donner added a comment.Dec 18 2020, 9:23 PM

Excellent example how to apply the new API.

This revision was not accepted when it landed; it landed in state Needs Review.Dec 26 2020, 11:33 AM
Closed by commit rG4e19e0d92ac6: Use light-weight versions of routing lookup functions in ng_netflow. (authored by melifaro). · Explain Why
This revision was automatically updated to reflect the committed changes.
melifaro added a commit: rG4e19e0d92ac6: Use light-weight versions of routing lookup functions in ng_netflow..

Revision Contents
Changeset List

PathSize
sys/
netgraph/
netflow/
145 lines

Diff 80916

View Options

sys/netgraph/netflow/netflow.c

Loading...