Page MenuHomeFreeBSD
Differential D40397

if_epair: do not transmit packets that exceed the interface MTU
ClosedPublic
Actions

Authored by kp on Jun 2 2023, 7:29 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Sep 24, 4:07 AM
Unknown Object (File)
Mon, Sep 23, 6:34 PM
Unknown Object (File)
Wed, Sep 18, 2:46 PM
Unknown Object (File)
Tue, Sep 17, 9:15 PM
Unknown Object (File)
Tue, Sep 17, 10:56 AM
Unknown Object (File)
Wed, Sep 11, 4:53 PM
Unknown Object (File)
Sat, Sep 7, 8:05 AM
Unknown Object (File)
Wed, Sep 4, 11:53 PM
View All 66 Files
Subscribers
ae
bz
glebius
imp
melifaro

Details

Reviewers
None
Group Reviewers
network
pfsense
Commits
rGa9bfd080d09a: if_epair: do not transmit packets that exceed the interface MTU
Summary

While if_epair has no issues doing this we should drop those packets
anyway, because it improves the fidelity of the automated tests.

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

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 51868
Build 48759: arc lint + arc unit

Event Timeline

kp created this revision.Jun 2 2023, 7:29 PM
Herald added subscribers: glebius, melifaro, ae, imp. · View Herald TranscriptJun 2 2023, 7:29 PM
kp requested review of this revision.Jun 2 2023, 7:29 PM
Harbormaster completed remote builds in B51868: Diff 122770.Jun 2 2023, 7:29 PM
bz added a subscriber: bz.Jun 2 2023, 10:35 PM

What is the default MTU for epair these days?

sys/net/if_epair.c
339

Probably wants more vertical space for that comment /*\n...\n*/ ?

kp added a comment.Jun 3 2023, 9:53 AM
In D40397#919783, @bz wrote:

What is the default MTU for epair these days?

We ether_ifattach(), which defaults to ETHERMTU, 1500.

if_epair doesn't need to care, of course, but the lack of this validation actually prevents the test in D40396 (fix for that problem is D40395) from failing when it should.
SIOCSIFMTU still works, and and will let users set whatever.

kp updated this revision to Diff 122792.Jun 3 2023, 11:06 AM
  • reformat comment block
Harbormaster completed remote builds in B51881: Diff 122792.Jun 3 2023, 11:06 AM
This revision was not accepted when it landed; it landed in state Needs Review.Jun 6 2023, 8:53 AM
Closed by commit rGa9bfd080d09a: if_epair: do not transmit packets that exceed the interface MTU (authored by kp). · Explain Why
This revision was automatically updated to reflect the committed changes.
kp added a commit: rGa9bfd080d09a: if_epair: do not transmit packets that exceed the interface MTU.

Revision Contents
Changeset List

PathSize
sys/
net/
9 lines

Diff 122770

View Options

sys/net/if_epair.c

Loading...