Page MenuHomeFreeBSD
Differential D37524

if_ovpn: remove peer limit
ClosedPublic
Actions

Authored by kp on Nov 28 2022, 9:21 PM.
Tags
None
Referenced Files
F111556017: D37524.diff
Wed, Mar 5, 5:33 AM
Unknown Object (File)
Tue, Feb 18, 8:44 AM
Unknown Object (File)
Sun, Feb 16, 8:29 AM
Unknown Object (File)
Sat, Feb 8, 5:04 PM
Unknown Object (File)
Jan 21 2025, 1:17 AM
Unknown Object (File)
Jan 20 2025, 10:12 PM
Unknown Object (File)
Dec 27 2024, 1:08 AM
Unknown Object (File)
Dec 26 2024, 1:12 AM
View All 63 Files
Subscribers
ae
glebius
imp
melifaro
zlei

Details

Reviewers
zlei
Group Reviewers
network
pfsense
Commits
rG97c802923e20: if_ovpn: remove peer limit
Summary

Replace the fixed-sized array by an RB_TREE. This should both speed up
lookups and remove the 128 peer limit.

We may wish to consider adding additional trees to also speed up the
lookup by IPv4/IPv6 addresses.

Sponsored by: Rubicon Communications, LLC ("Netgate")

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

kp created this revision.Nov 28 2022, 9:21 PM
Herald added subscribers: glebius, melifaro, ae, imp. · View Herald TranscriptNov 28 2022, 9:21 PM
kp requested review of this revision.Nov 28 2022, 9:21 PM
Harbormaster completed remote builds in B48525: Diff 113579.Nov 28 2022, 9:21 PM
zlei added a subscriber: zlei.Nov 29 2022, 12:56 AM
zlei added inline comments.Nov 29 2022, 3:54 AM
sys/net/if_ovpn.c
269

(struct ovpn_kpeer *)&p

The cast looks redundant. p is declared as struct ovpn_kpeer at line 278

1620–1623

IPv6

kp updated this revision to Diff 113595.Nov 29 2022, 9:31 AM
  • review remarks
Harbormaster completed remote builds in B48530: Diff 113595.Nov 29 2022, 9:31 AM
kp marked 2 inline comments as done.Nov 29 2022, 9:32 AM

Good catch.

zlei accepted this revision.Dec 1 2022, 10:10 AM

Looks good to me.

And you may also want to retire OVPN_MAX_PEERS ;)

This revision is now accepted and ready to land.Dec 1 2022, 10:10 AM
kp added a comment.Dec 1 2022, 10:24 AM
In D37524#854386, @zlei wrote:

Looks good to me.

And you may also want to retire OVPN_MAX_PEERS ;)

Thanks. I spotted that one eventually and fixed it, but figured it wasn't worth updating the review for that.

Closed by commit rG97c802923e20: if_ovpn: remove peer limit (authored by kp). · Explain WhyDec 1 2022, 4:43 PM
This revision was automatically updated to reflect the committed changes.
kp added a commit: rG97c802923e20: if_ovpn: remove peer limit.

Revision Contents
Changeset List

PathSize
sys/
net/
136 lines

Diff 113758

View Options

sys/net/if_ovpn.c

Loading...