Regression: hinic performance degrades over time

Bug #1805248 reported by dann frazier on 2018-11-26
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
linux (Ubuntu)
Status tracked in Disco
Cosmic
Undecided
dann frazier
Disco
Undecided
dann frazier

Bug Description

[Impact]
Bug 1800664 introduced checksum offload and TSO support for HiNIC adapters. While that did improve performance in short iperf runs, longer runs were later found to show a significant regression in performance:
  https://bugs.launchpad.net/ubuntu/+source/linux/+bug/1800664/comments/3

[Test Case]
iperf -c 192.168.99.1 -P5

[Fix]
Revert the offending patch.

[Regression Risk]
This is reverting a patch, returning the driver to 18.10 GA behavior.

dann frazier (dannf) on 2018-11-26
Changed in linux (Ubuntu):
status: New → In Progress
Changed in linux (Ubuntu Cosmic):
status: New → In Progress
assignee: nobody → dann frazier (dannf)
dann frazier (dannf) on 2018-11-26
description: updated
Seth Forshee (sforshee) on 2018-11-27
Changed in linux (Ubuntu Disco):
status: In Progress → Fix Committed
Changed in linux (Ubuntu Cosmic):
status: In Progress → Fix Committed
Brad Figg (brad-figg) wrote :

This bug is awaiting verification that the kernel in -proposed solves the problem. Please test the kernel and update this bug with the results. If the problem is solved, change the tag 'verification-needed-cosmic' to 'verification-done-cosmic'. If the problem still exists, change the tag 'verification-needed-cosmic' to 'verification-failed-cosmic'.

If verification is not done by 5 working days from today, this fix will be dropped from the source code, and this bug will be closed.

See https://wiki.ubuntu.com/Testing/EnableProposed for documentation how to enable and use -proposed. Thank you!

tags: added: verification-needed-cosmic
dann frazier (dannf) wrote :

Testing shows performance inline with the 4.18.0-11-generic kernel.

tags: added: verification-done-cosmic
removed: verification-needed-cosmic
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers