In D17646#379198, @linimon wrote:

I'm not sure that PORTEPOCH can start with zero.

In D17646#376890, @franco_opnsense.org wrote:

@skozlov feel free to take maintainership of net/intel-em-kmod and thanks for working on these modules! :)

Original patch committed in r481900
The revision wasn't automatically closed because I wasn't able to use the arcanist properly :(

net/intel-{ix,em}-kmod: freeze PORTREVISION and PORTEPOCH

I'll add -Wno-error to the master port, but because of the amount of testing it will require - I'd like to make it a separate review.

Add X5* controllers to supported list

In D17443#372344, @koobs wrote:

Looks good except for -Werror. This all matches the port WIP I have in my tree, except for the additions:

MANPAGENAME=        ixgbe
MANPAGELINKS=        ix if_ix if_ixgbe
​MAX_NETMAP_OSVERSION=        1199999  # Doesn't build w/NETMAP on 12

In D17443#372074, @sbruno wrote:

Does this require a LICENSE or some sort?

Adding port maintainer as a blocking reviewer.

The validation exposed a memory leak for a specific traffic pattern. After additional investigation the same leak was found in iflib igb/em driver as well.
As that's the only issue discovered during testing, and, most probably, it's not caused by the changes in the patch - it was decided to merge the patch.

In D13885#295301, @sbruno wrote:

Unrelated, is there a use case for FreeBSD Bhyve here? If we had PF support on FreeBSD, could we use pass through to use VFs?

Blocking until Intel internal validation is complete.

Smoke test passed. IMO good to go after Portlint issue is resolved.

Validation completed successfully.

This seems ugly to me, but that's the only way it works. ${CHMOD} -R doesn't do the right thing on 10.

I was able to reproduce the build issue as well using poudriere 3.2.
The inconsistency in behavior is caused by this commit, which is in 11, but not in 10: https://svnweb.freebsd.org/base?view=revision&revision=283997
After the distfile is unpacked - extract-fixup-modes target from bsd.port.mk tries to fix the permissions on the directories, but on 10 it does that only for the top level directory, so it can't look into the inner one.

Updated to a newer, silently updated version. Ready for review.

Updated to 1.8.1, unbroken. Ready for review.

Needs additional testing.

Needs additional testing.

Please let me know if you need another round of testing done to commit this one

Tested on 2 machines with 4Kn and 4Ke drives:

• Build the boot1.efifat from the patched HEAD
• cat the boot1.efifat onto /dev/gpt/efiboot0
• reboot

• Updated revision to 1.7.11
• Updated poudriere log. The Ports tree bug is fixed, no extra steps required to make poudriere like the patch
• Touch-tested the driver on 10.3 and 11.0

ixl-1.7.11 is out. Need to update the review.

Added vuln.xml diff

The change is outdated. Have to make a new one

Thank you for all the work you've done. Please go ahead and commit it.

Just drag them to the comment window or use the "Upload file" button

• Please add the logs separately, they must not be a part of the revision
• Please fix the poudriere warning: "USE_OPENSSL is deprecated, please use USES=ssl"

Could you please add some poudriere and portlint logs to the review?