Page MenuHomeFreeBSD
Differential D45195

if_vxlan(4): Exclude ETHER_CRC_LEN while calculating VXLAN_MAX_MTU
ClosedPublic
Actions

Authored by zlei on May 14 2024, 4:46 AM.
Tags
None
Referenced Files
F165290656: D45195.id138518.diff
Fri, Aug 7, 11:07 AM
F165208662: D45195.diff
Thu, Aug 6, 8:15 PM
Unknown Object (File)
Thu, Aug 6, 7:55 AM
Unknown Object (File)
Wed, Aug 5, 12:01 PM
Unknown Object (File)
Sat, Aug 1, 1:00 PM
Unknown Object (File)
Sat, Aug 1, 6:53 AM
Unknown Object (File)
Wed, Jul 22, 9:22 PM
Unknown Object (File)
Mon, Jul 20, 11:45 PM
View All Files
Subscribers
ae
afedorov
glebius
imp
melifaro

Details

Reviewers
jhb
bryanv
Group Reviewers
network
Commits
rG44aca0397cc8: if_vxlan(4): Exclude ETHER_CRC_LEN from macro VXLAN_MAX_MTU
rG7de38cbb5dc4: if_vxlan(4): Exclude ETHER_CRC_LEN from macro VXLAN_MAX_MTU
rGd6963b9ed328: if_vxlan(4): Exclude ETHER_CRC_LEN from macro VXLAN_MAX_MTU
Summary

The encapsulated (original) frame does not count in FCS as per Section 5
of RFC 7348.

Fixes: b7592822d5de Allow set MTU more than 1500 bytes
MFC after: 1 week

Diff Detail

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

Event Timeline

zlei created this revision.May 14 2024, 4:46 AM
Herald added subscribers: glebius, melifaro, ae, imp. · View Herald TranscriptMay 14 2024, 4:46 AM
zlei requested review of this revision.May 14 2024, 4:46 AM
zlei added a comment.Jul 1 2024, 1:11 PM

Ping .

afedorov added a subscriber: afedorov.Jul 1 2024, 4:14 PM

As the author of the original patch about increasing MTU, I consider this change to be correct.

bryanv accepted this revision.Jul 2 2024, 1:31 AM
This revision is now accepted and ready to land.Jul 2 2024, 1:31 AM
zlei added a comment.Jul 2 2024, 4:59 AM
In D45195#1044804, @afedorov wrote:

As the author of the original patch about increasing MTU, I consider this change to be correct.

Punished you a Reviewed by: afedorov :)

Closed by commit rGd6963b9ed328: if_vxlan(4): Exclude ETHER_CRC_LEN from macro VXLAN_MAX_MTU (authored by zlei). · Explain WhyJul 2 2024, 4:59 AM
This revision was automatically updated to reflect the committed changes.
zlei added a commit: rGd6963b9ed328: if_vxlan(4): Exclude ETHER_CRC_LEN from macro VXLAN_MAX_MTU.
zlei added a commit: rG7de38cbb5dc4: if_vxlan(4): Exclude ETHER_CRC_LEN from macro VXLAN_MAX_MTU.Jul 10 2024, 10:18 AM
zlei added a commit: rG44aca0397cc8: if_vxlan(4): Exclude ETHER_CRC_LEN from macro VXLAN_MAX_MTU.Jul 10 2024, 11:45 AM

Revision Contents
Changeset List

PathSize
sys/
net/
2 lines

Diff 140451

View Options

sys/net/if_vxlan.c

Loading...