Page MenuHomeFreeBSD

sysutils/intel-nvmupdate: NVM Update Utility for Intel(R) Ethernet Adapters

Authored by skozlov on Apr 4 2016, 4:14 PM.



New port for Intel's nvmupdate utility.

The tool requires the latest version of if_ixl driver to be installed on the system, which is provided by intel-ixl-kmod package (D5761), and should not be committed before it. The requirement is mentioned in pkg-message of the port.


  • The port doesn't respect hier(7) as all the files of the tool must be in the same directory for it to function properly.
  • Wrapper script is installed to ${PREFIX}/sbin which changes the dir to the ${PREFIX}/intel-nvmupdate before running the tool as this is a tool's requirement.
  • If the user choses to have a backup copy of the old NVM while using the tool it's placed in the ${PREFIX}/intel-nvmupdate directory. If that happens - the backup and the intel-nvmupdate dir are preserved when the package is deinstalled.

Known bugs:

  • The tool doesn't work (coredumps) on latest 11-CURRENT before the upgrade process, no harm done to the hardware.

Putting my Intel hat on for this port.

Sponsored by: Intel Corporation

Test Plan

Poudriere log (portlint log embeded):

Used driver from D5761 and the tool to update NVM image from stock to the latest on XL710-QDA2, log:

Diff Detail

rP FreeBSD ports repository
Automatic diff as part of commit; lint not applicable.
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

skozlov retitled this revision from to sysutils/intel-nvmupdate: NVM Update Utility for Intel(R) Ethernet Adapters.
skozlov updated this object.
skozlov edited the test plan for this revision. (Show Details)
skozlov added reviewers: sbruno, Intel Networking.
skozlov set the repository for this revision to rP FreeBSD ports repository.
11 ↗(On Diff #14869)

set DISTVERSION= 5_02 instead of setting PORTVERSION, and use it here.

27–28 ↗(On Diff #14869)

Don't use both, just set

63 ↗(On Diff #14869)

I'm not sure the whole @(..) is needed here, you are installing the file with INSTALL_PROGRAM, it should already be 755.

skozlov edited the test plan for this revision. (Show Details)

Implements suggestions from mat.

This revision is now accepted and ready to land.Apr 4 2016, 11:48 PM
sbruno added a reviewer: portmgr.
sbruno edited edge metadata.

Need a "LGTM" from portmgr, and we can commit it. (I'll probably do it).

bapt added a reviewer: bapt.
bapt added a subscriber: bapt.

This LGTM, but also not that portmgr does not have to be involved in this review

This revision was automatically updated to reflect the committed changes.