Re: [PATCH net-next v14 0/7] r8169: add RSS support for RTL8127

On 9/18/26 08:19, javen wrote:

From: Javen Xu <redacted>

This patch series adds RSS (Receive Side Scaling) support for the r8169
ethernet driver, specifically for RTL8127 (RTL_GIGA_MAC_VER_80).

RSS enables packet distribution across multiple receive queues, which can
significantly improve network throughput on multi-core systems by allowing
parallel processing of incoming packets.

Key features:
- Multi-queue RX support (up to 8 queues)
- MSI-X interrupt with vector mapping
- Dynamic queue configuration via ethtool (-L)
- RSS hash computation for flow classification

Experiments:
Platform: AMD Ryzen Embedded R2514 with Radeon Graphics(4 Cores/8 Threads)
Arch: x86_64
Test command:
   Server: iperf3 -s
   Client: iperf3 -c 192.168.2.1 -P 20 -t 3600
Monitor: mpstat -P ALL 1

Before this patch (Without RSS):
   Throughput: Unstable, fluctuating between 3.76 Gbits/sec and
   8.2 Gbits/sec.
   CPU Usage: A single CPU core is fully occupied with softirq reaching
   up to 96%.

After this patch (With RSS enabled):
   Throughput: Stable at 9.42 Gbits/sec.
   CPU Usage: The traffic load is evenly distributed across multiple CPU
   cores. The maximum softirq on a single core dropped to 63%.
   
Other Experiments:
Link: https://lore.kernel.org/netdev/0A5279953D81BB9C+f50c9b49-3e5d-467f-b69a-7e49ed223383@radxa.com/ (local)

The API usage changes suggested by clashiko on patch 3/7 should be IMHO
be addressed even if marked 'low prio' (IMHO clashiko mislabeled them).

The claim about the fixes tag on patch 5/7 can instead IMHO be ignored,
but on the same patch clashiko reports more low prio issues that again
should be addressed.

/P