Page MenuHomeFreeBSD

if_gif: Use the MTU on the gif interface for the outgoing packets
Needs RevisionPublic

Authored by meta on Sep 17 2025, 3:46 PM.
Tags
None
Referenced Files
F147493900: D52594.id.diff
Wed, Mar 11, 10:18 AM
Unknown Object (File)
Mon, Mar 9, 4:52 PM
Unknown Object (File)
Sun, Mar 1, 3:01 AM
Unknown Object (File)
Tue, Feb 17, 5:04 PM
Unknown Object (File)
Tue, Feb 17, 5:04 PM
Unknown Object (File)
Feb 7 2026, 4:43 PM
Unknown Object (File)
Jan 30 2026, 9:33 PM
Unknown Object (File)
Dec 23 2025, 12:21 AM

Details

Reviewers
zlei
ae
hrs
pouria
Group Reviewers
network
Summary

This is a revised version of D51866 to address review comments
and resolve conflicts after D51835.

As I mentioned in D51866 [1], I would like this changes to be included in
15.0-RELEASE. I worked on D51297 to ensure that the fix for bug 280736 would
be included in the release, as there has been no progress on D45854 for over
a year. Since this is a better approach for bug 280736, it should be included
as well.

Ongoing work can be seen on GitHub:
https://github.com/metalefty/freebsd-src/commits/gif/

[1] https://reviews.freebsd.org/D51866#1196328

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 67125
Build 64008: arc lint + arc unit

Event Timeline

meta requested review of this revision.Sep 17 2025, 3:46 PM

This worked very well for me, both for EtherIP (Ethernet-ovet-IPv6) and IPIP (v6-over-v6).

This worked very well for me, both for EtherIP (Ethernet-ovet-IPv6) and IPIP (v6-over-v6).

I'm preparing the tests. It appears good.

I'm going to firstly integrate part of this change, to be precisely, the if_bridge's MTU change behavior of adding if_gif(4) as a member interface.

hrs requested changes to this revision.Sep 28 2025, 2:18 AM
hrs added a subscriber: hrs.

Please stop duplicating the existing review to obtain approval from someone else by excluding the original author.

This revision now requires changes to proceed.Sep 28 2025, 2:18 AM

Nothing had happened for over a year since D45854 was submitted. I just tried to get things moving to avoid waiting another year. Because we didn't get feedback from you after the reviewers left review comments, and the deadline was approaching.

To be clear, I did not intend to exclude you, just forgot to add.

@meta
IMHO, You may proceed again with 'Co-Authored-By: full name <email>' tag if and only if D51866 stalled again for another month.
I pinged the original revision to see if author is available to address @zlei and your comments or not.

@meta
IMHO, You may proceed again with 'Co-Authored-By: full name <email>' tag if and only if D51866 stalled again for another month.
I pinged the original revision to see if author is available to address @zlei and your comments or not.

I'd love to proceed. He tends to propose changes and respond to review comments a year or more later. Because of that, I could not manage to get this included in 15.0-RELEASE. It's been over a year since the original changes were proposed, so perhaps we should wait a shorter period.