[PATCH net-next] selftests: drv-net: rss_ctx: Don't assume indirection table is present

[PATCH net-next] selftests: drv-net: rss_ctx: Don't assume indirection table is present

[Gal Pressman]

The test_rss_context_dump() test assumes the indirection table is always
supported, which is not true for all drivers, e.g., virtio_net when
VIRTIO_NET_F_RSS is disabled.

Skip the check if 'indir' is not present.

Reviewed-by: Nimrod Oren <noren@nvidia.com>
Signed-off-by: Gal Pressman <gal@nvidia.com>
---
 tools/testing/selftests/drivers/net/hw/rss_ctx.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/tools/testing/selftests/drivers/net/hw/rss_ctx.py b/tools/testing/selftests/drivers/net/hw/rss_ctx.py
index d6e69d7d5e43..ca60ae325c22 100755
--- a/tools/testing/selftests/drivers/net/hw/rss_ctx.py
+++ b/tools/testing/selftests/drivers/net/hw/rss_ctx.py
@@ -392,7 +392,7 @@ def test_rss_context_dump(cfg):
 
     # Sanity-check the results
     for data in ctxs:
-        ksft_ne(set(data['indir']), {0}, "indir table is all zero")
+        ksft_ne(set(data.get('indir', [1])), {0}, "indir table is all zero")
         ksft_ne(set(data.get('hkey', [1])), {0}, "key is all zero")
 
         # More specific checks
-- 
2.40.1

Re: [PATCH net-next] selftests: drv-net: rss_ctx: Don't assume indirection table is present

[Simon Horman]

On Tue, Mar 18, 2025 at 01:24:26PM +0200, Gal Pressman wrote:
> The test_rss_context_dump() test assumes the indirection table is always
> supported, which is not true for all drivers, e.g., virtio_net when
> VIRTIO_NET_F_RSS is disabled.
> 
> Skip the check if 'indir' is not present.
> 
> Reviewed-by: Nimrod Oren <noren@nvidia.com>
> Signed-off-by: Gal Pressman <gal@nvidia.com>

Reviewed-by: Simon Horman <horms@kernel.org>

Re: [PATCH net-next] selftests: drv-net: rss_ctx: Don't assume indirection table is present

[patchwork-bot+netdevbpf]

Hello:

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

On Tue, 18 Mar 2025 13:24:26 +0200 you wrote:
> The test_rss_context_dump() test assumes the indirection table is always
> supported, which is not true for all drivers, e.g., virtio_net when
> VIRTIO_NET_F_RSS is disabled.
> 
> Skip the check if 'indir' is not present.
> 
> Reviewed-by: Nimrod Oren <noren@nvidia.com>
> Signed-off-by: Gal Pressman <gal@nvidia.com>
> 
> [...]

Here is the summary with links:
  - [net-next] selftests: drv-net: rss_ctx: Don't assume indirection table is present
    https://git.kernel.org/netdev/net-next/c/c61209eeb0b3

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