public inbox for netdev@vger.kernel.org
 help / color / mirror / Atom feed
* [PATCH net-next v3 0/4] amd-xgbe: add ARP offload and RSS self-tests
@ 2026-02-04 14:58 Raju Rangoju
  2026-02-06  3:09 ` Jakub Kicinski
  2026-02-06  3:20 ` patchwork-bot+netdevbpf
  0 siblings, 2 replies; 3+ messages in thread
From: Raju Rangoju @ 2026-02-04 14:58 UTC (permalink / raw)
  To: netdev
  Cc: pabeni, kuba, edumazet, davem, andrew+netdev, linux-kernel,
	Shyam-sundar.S-k, Raju Rangoju

This series adds ARP offload support and ethtool self-tests for the
AMD XGBE 10 Gigabit Ethernet driver.

Patch 1 is a build fix for randconfig coverage.

Patch 2 adds hardware ARP offload functionality that allows the MAC
to automatically respond to ARP requests without CPU intervention,
reducing system latency and power consumption during suspend/resume
and normal operation.

Patches 3 and 4 add ethtool self-tests for ARP offload and RSS
functionality respectively, allowing users to verify these hardware
features are working correctly via 'ethtool -t <interface>'.

The tests are integrated into the existing XGBE self-test framework
that already includes MAC loopback, PHY loopback, split header, and
jumbo frame tests.

Test results example:
  $ ethtool -t eth0
  The test result is PASS
  The test extra info:
  MAC Loopback       0
  PHY Loopback       0
  Split Header       0
  Jumbo Frame        0
  ARP Offload        0
  RSS                0

Changes since v2:
- Fix warnings about line length
- Use kfree_skb() instead of kfree() for skb cleanup in self-tests
- CC maintainer for patch 1

Changes since v1:
- Resubmitting to net-next as this was mistakenly sent to net previously.
- Fix INET=n build breakage reported by kernel test robot
- Ensure ARP self-test cleanup is safe on all error paths (no dangling
  packet handler).
- Use link-local IPs for self-tests to avoid interfering with real
  network configuration.

Raju Rangoju (4):
  amd-xgbe: do not select NET_SELFTESTS when INET is disabled
  amd-xgbe: add hardware ARP offload support
  amd-xgbe: add ARP offload ethtool self-test
  amd-xgbe: add RSS ethtool self-test

 drivers/net/ethernet/amd/Kconfig              |   2 +-
 drivers/net/ethernet/amd/xgbe/xgbe-common.h   |   3 +
 drivers/net/ethernet/amd/xgbe/xgbe-dev.c      |  24 +++
 drivers/net/ethernet/amd/xgbe/xgbe-selftest.c | 154 +++++++++++++++++-
 drivers/net/ethernet/amd/xgbe/xgbe.h          |   4 +
 include/net/selftests.h                       |   1 +
 6 files changed, 186 insertions(+), 2 deletions(-)

-- 
2.34.1


^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH net-next v3 0/4] amd-xgbe: add ARP offload and RSS self-tests
  2026-02-04 14:58 [PATCH net-next v3 0/4] amd-xgbe: add ARP offload and RSS self-tests Raju Rangoju
@ 2026-02-06  3:09 ` Jakub Kicinski
  2026-02-06  3:20 ` patchwork-bot+netdevbpf
  1 sibling, 0 replies; 3+ messages in thread
From: Jakub Kicinski @ 2026-02-06  3:09 UTC (permalink / raw)
  To: Raju Rangoju
  Cc: netdev, pabeni, edumazet, davem, andrew+netdev, linux-kernel,
	Shyam-sundar.S-k

On Wed, 4 Feb 2026 20:28:29 +0530 Raju Rangoju wrote:
>   amd-xgbe: do not select NET_SELFTESTS when INET is disabled

I'll apply this one

>   amd-xgbe: add hardware ARP offload support
>   amd-xgbe: add ARP offload ethtool self-test

These two I don't get. The feature is enabled during the test but it
can't be enabled by a normal user. So what's the point of testing it?

>   amd-xgbe: add RSS ethtool self-test

xgbe seems to implement Toeplitz hash, much better to run
tools/testing/selftests/drivers/net/hw/toeplitz.py
than just checking if the hash value != 0..

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH net-next v3 0/4] amd-xgbe: add ARP offload and RSS self-tests
  2026-02-04 14:58 [PATCH net-next v3 0/4] amd-xgbe: add ARP offload and RSS self-tests Raju Rangoju
  2026-02-06  3:09 ` Jakub Kicinski
@ 2026-02-06  3:20 ` patchwork-bot+netdevbpf
  1 sibling, 0 replies; 3+ messages in thread
From: patchwork-bot+netdevbpf @ 2026-02-06  3:20 UTC (permalink / raw)
  To: Raju Rangoju
  Cc: netdev, pabeni, kuba, edumazet, davem, andrew+netdev,
	linux-kernel, Shyam-sundar.S-k

Hello:

This series was applied to netdev/net.git (main)
by Jakub Kicinski <kuba@kernel.org>:

On Wed, 4 Feb 2026 20:28:29 +0530 you wrote:
> This series adds ARP offload support and ethtool self-tests for the
> AMD XGBE 10 Gigabit Ethernet driver.
> 
> Patch 1 is a build fix for randconfig coverage.
> 
> Patch 2 adds hardware ARP offload functionality that allows the MAC
> to automatically respond to ARP requests without CPU intervention,
> reducing system latency and power consumption during suspend/resume
> and normal operation.
> 
> [...]

Here is the summary with links:
  - [net-next,v3,1/4] amd-xgbe: do not select NET_SELFTESTS when INET is disabled
    https://git.kernel.org/netdev/net/c/ee9241524b46
  - [net-next,v3,2/4] amd-xgbe: add hardware ARP offload support
    (no matching commit)
  - [net-next,v3,3/4] amd-xgbe: add ARP offload ethtool self-test
    (no matching commit)
  - [net-next,v3,4/4] amd-xgbe: add RSS ethtool self-test
    (no matching commit)

You are awesome, thank you!
-- 
Deet-doot-dot, I am a bot.
https://korg.docs.kernel.org/patchwork/pwbot.html



^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2026-02-06  3:20 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-02-04 14:58 [PATCH net-next v3 0/4] amd-xgbe: add ARP offload and RSS self-tests Raju Rangoju
2026-02-06  3:09 ` Jakub Kicinski
2026-02-06  3:20 ` patchwork-bot+netdevbpf

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox