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
F153667225: D37524.id113579.diff
Wed, Apr 22, 6:59 PM
Unknown Object (File)
Tue, Apr 21, 6:44 PM
Unknown Object (File)
Tue, Apr 21, 3:15 PM
Unknown Object (File)
Tue, Apr 21, 3:11 PM
Unknown Object (File)
Sun, Apr 19, 11:30 PM
Unknown Object (File)
Sun, Apr 19, 11:26 PM
Unknown Object (File)
Wed, Apr 15, 8:17 AM
Unknown Object (File)
Sun, Apr 5, 5:32 PM
View All 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...