Project

General

Profile

Bug #12933

Update i40e core code to support NVM v1.7

Added by Paul Winder about 1 month ago. Updated 26 days ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
-
Start date:
Due date:
% Done:

100%

Estimated time:
Difficulty:
Medium
Tags:
Gerrit CR:

Description

There are enough differences between NVM 1.5 and 1.7 that even though the driver attaches to a v1.7 device, it does not perform well.

The core code originally came from FreeBSD. There is an update in there which is labelled as 1.9.9-k at:
https://github.com/freebsd/freebsd/commit/1a64c55b96734a3f1c3e35e95fcfa4ef00d5182c#diff-1cf0dee4e7e10b686ebfe7004e2f2933

The changes to "core" should be pulled from there


Related issues

Related to illumos gate - Bug #12298: i40e on XL710 is out of date, port has DLPI errorsNew

Actions

History

#1

Updated by Paul Winder about 1 month ago

I have tested this on a v1.7 XXV710 (pciex8086,158b) and a v1.5 X722 (pciex8086,37d2).

Ran iperf, created aggrs, changed MTUs ....

#2

Updated by Electric Monk about 1 month ago

  • Gerrit CR set to 780
#3

Updated by Paul Winder 29 days ago

  • Related to Bug #12298: i40e on XL710 is out of date, port has DLPI errors added
#4

Updated by Paul Winder 27 days ago

This code was the basis of #12957 and #12958 and has been through the test cycle included with those.

I also got the following feedback from Ryan Zezesk:

Okay, I ran my mac-tests and lso-tests successfully [1].

Ethernet Controller X710 for 10GbE SFP+
API Version: 1.4
Firmware Version: 4.40

1: https://github.com/rzezeski/rpz-misc/tree/master/illumos/net-tests

-Ryan

#5

Updated by Electric Monk 26 days ago

  • Status changed from In Progress to Closed
  • % Done changed from 0 to 100

git commit 93f1cac532b220b8db1e7715dbb2ae7ec5fe8fe7

commit  93f1cac532b220b8db1e7715dbb2ae7ec5fe8fe7
Author: Paul Winder <paul@winder.uk.net>
Date:   2020-07-17T15:39:13.000Z

    12933 Update i40e core code to support NVM v1.7
    Portions contributed by: Krzysztof Galazka <krzysztof.galazka@intel.com>
    Reviewed by: Robert Mustacchi <rm@fingolfin.org>
    Reviewed by: Randy Fishel <randyf@sibernet.com>
    Approved by: Garrett D'Amore <garrett@damore.org>

Also available in: Atom PDF