Page MenuHomeFreeBSD

Add support for cloning pseudo interfaces
ClosedPublic

Authored by mmacy on May 8 2018, 4:00 AM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Jan 17, 4:52 PM
Unknown Object (File)
Dec 8 2024, 10:48 PM
Unknown Object (File)
Nov 29 2024, 5:00 AM
Unknown Object (File)
Nov 23 2024, 1:56 AM
Unknown Object (File)
Nov 3 2024, 7:53 PM
Unknown Object (File)
Oct 20 2024, 1:19 PM
Unknown Object (File)
Oct 14 2024, 5:28 PM
Unknown Object (File)
Oct 2 2024, 5:37 AM

Details

Summary

Part 3 of many ...
The VPC framework relies heavily on cloning pseudo interfaces (vmnics, vpc switch, vcpswitch port, hostif, vxlan if, etc).

This pulls in that bit. Largely self explanatory. Some ancillary changes get pulled in as a side effect.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

sys/net/iflib.c
4547 ↗(On Diff #42258)

This appears to leak ctx and sc.

sys/net/iflib.h
360 ↗(On Diff #42258)

I don't think this value should be renumbered. It's in stable/11 now as 0x100.

sys/net/iflib.h
360 ↗(On Diff #42258)

Maybe I'm naive, but this value seems to be only consumed by kernel space. That being said, if someone is trying to use an out of tree KMOD, then there would be a problem right?

sys/net/iflib.h
360 ↗(On Diff #42258)

Yeah, the main concern is that if someone doesn't rebuild a module across this change, it will behave unexpectedly. Since there's no technical reason to renumber it, it may as well be retained.

  • Fix leak on iflib_register fail
  • don't renumber IFLIB_NEED_ETHER_PAD
This revision is now accepted and ready to land.May 11 2018, 7:26 PM
This revision was automatically updated to reflect the committed changes.