Skip to content

dpdk: move RSS configuration into port configuration#319

Merged
SiskaPavel merged 4 commits intomasterfrom
dpdk-rss-fix
Feb 4, 2026
Merged

dpdk: move RSS configuration into port configuration#319
SiskaPavel merged 4 commits intomasterfrom
dpdk-rss-fix

Conversation

@SiskaPavel
Copy link
Collaborator

@SiskaPavel SiskaPavel commented Dec 16, 2025

Summary

Fix RSS configuration with i40e and multiple RX queues

Details

Issue #313 was reproduced on an identical NIC in our lab. When multiple RX queues are configured with the i40e driver, the default RSS configuration (RTE_ETH_RSS_IP) does not distribute packets correctly, sometimes resulting in Errno:-22.
By default, the plugin now falls back to using NIC/driver-provided RSS (rss_offload: null). Users can explicitly override RSS using the rss_offload parameter, though this is experimental and may lead to uneven RX queue distribution.

Additional notes
When only a single RX queue is used, RSS is effectively disabled, and all packets are received on that single queue.

✅ Checks

  • Relevant documentation was updated (if needed)
  • CI pipeline passes (build, tests, lint, static analysis)
  • New code follows clang-tidy rules
  • The change is clearly related to a specific issue / requirement
  • Documentation (e.g. Doxygen) is updated if needed
  • The PR is rebased on the latest master
  • No unrelated changes are included

fix RSS configuration with i40e and multiple RX queues
@SiskaPavel SiskaPavel self-assigned this Dec 16, 2025
@SiskaPavel SiskaPavel closed this Feb 4, 2026
@SiskaPavel SiskaPavel deleted the dpdk-rss-fix branch February 4, 2026 12:48
@SiskaPavel SiskaPavel restored the dpdk-rss-fix branch February 4, 2026 14:20
@SiskaPavel SiskaPavel reopened this Feb 4, 2026
@SiskaPavel SiskaPavel force-pushed the dpdk-rss-fix branch 2 times, most recently from 7289c74 to e7015b7 Compare February 4, 2026 15:15
@SiskaPavel SiskaPavel merged commit da427df into master Feb 4, 2026
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant