Page MenuHomeFreeBSD
Differential D26896

ixl: honor IPI_TX_INTR
Needs ReviewPublic
Actions

Authored by vmaffione on Oct 21 2020, 8:14 PM.
Tags
None
Referenced Files
F103686066: D26896.diff
Thu, Nov 28, 1:15 AM
F103686061: D26896.diff
Thu, Nov 28, 1:15 AM
F103686045: D26896.diff
Thu, Nov 28, 1:15 AM
F103686043: D26896.diff
Thu, Nov 28, 1:15 AM
F103686040: D26896.diff
Thu, Nov 28, 1:15 AM
Unknown Object (File)
Wed, Nov 27, 1:15 AM
Unknown Object (File)
Wed, Nov 27, 1:15 AM
Unknown Object (File)
Wed, Nov 27, 1:15 AM
View All Files
Subscribers
ae
erj
imp

Details

Reviewers
krzysztof.galazka_intel.com
Group Reviewers
Intel Networking
Summary

The ixl_isc_txd_encap() iflib callback was not handling the IPI_TX_INTR flag properly.
When this flag is not set, the callback is not supposed to set the report flag in the EOP descriptor, and therefore do not ask the NIC for a descriptor writeback (and possibly a TX interrupt).

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 34310

Event Timeline

vmaffione created this revision.Oct 21 2020, 8:14 PM
Herald added subscribers: ae, imp. · View Herald TranscriptOct 21 2020, 8:14 PM
vmaffione requested review of this revision.Oct 21 2020, 8:14 PM
Harbormaster completed remote builds in B34310: Diff 78565.Oct 21 2020, 8:14 PM
lwhsu added a reviewer: Intel Networking.Oct 22 2020, 2:10 AM
vmaffione added a comment.Oct 22 2020, 8:11 PM

Although the change seems legit, it has been reported to cause hangs with netmap pkt-gen.
See:
https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=248652 (from comment 29)
Any idea why this may be the case?
In stable/11, the same logic (two report flags set for every ring) does not cause hangs.

erj added a subscriber: erj.Mar 3 2021, 1:56 AM

@krzysztof.galazka_intel.com, do you have any update on an answer to @vmaffione's question ?

Revision Contents
Changeset List

PathSize
sys/
dev/
ixl/
24 lines

Diff 78565

View Options

sys/dev/ixl/ixl_txrx.c

Loading...