Page MenuHomeFreeBSD

ixl(4): Fix two important RSS bugs.
ClosedPublic

Authored by erj on Feb 5 2016, 10:14 PM.
Tags
None
Referenced Files
F84951274: D5203.diff
Thu, May 30, 3:38 PM
Unknown Object (File)
Sun, May 19, 1:33 PM
Unknown Object (File)
Sat, May 4, 4:05 AM
Unknown Object (File)
Thu, May 2, 10:02 PM
Unknown Object (File)
Apr 26 2024, 5:37 AM
Unknown Object (File)
Apr 25 2024, 10:43 AM
Unknown Object (File)
Feb 23 2024, 3:09 PM
Unknown Object (File)
Jan 9 2024, 5:44 AM
Subscribers

Details

Summary
  • Change tc_mapping field from 0x8000 to 0x0c00; this allows RSS to hash to 64 queues instead of 16.
  • Set hash_lut_size to 512 before calling filter control; the default size is 128 if that bit is not set.

Related bugzilla: https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=203445

Test Plan

Test using 8-64 queues (64 threads of 32 byte UDP)

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

erj retitled this revision from to ixl(4): Fix two important RSS bugs..
erj updated this object.
sys/dev/ixl/if_ixl.c
2763 ↗(On Diff #13069)

hiya,

can you comment what this mapping means? Otherwise it's just a magic constant. :)

sys/dev/ixl/if_ixl.c
2763 ↗(On Diff #13069)

I guess I could clarify what the 3 bits that are set here are for. But honestly, you should be reading the datasheet to find out what it does.

Until then, it means the number of queues assigned to the VSI for traffic class 0; 2^n of the value bytes/bits 1.1-1.3 is that number. I'll look to see if there's a constant defined in the shared code that I can use, or otherwise create one.

Update with no magic number and a better comment.

This revision is now accepted and ready to land.Feb 11 2016, 11:01 PM
gallatin edited edge metadata.
This revision was automatically updated to reflect the committed changes.