On Wed, 2026-05-20 at 11:34 +1000, Matthieu Baerts (NGI0) wrote:
> The run.sh script explicitly checks that CONFIG_MODULES is disabled.
>
> By default, this config option is enabled. Explicitly disable it to be
> able to run the RDS tests.
>
> Note that writing '# CONFIG_(...) is not set' is usually recommended to
> disable an option in the .config, but it looks like selftests usually
> set 'CONFIG_(...)=n', which looks clearer.
>
> Fixes: 0f5d68004780 ("selftests: rds: add
> tools/testing/selftests/net/rds/config")
> Signed-off-by: Matthieu Baerts (NGI0) <[email protected]>
> ---
> tools/testing/selftests/net/rds/config | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/tools/testing/selftests/net/rds/config
> b/tools/testing/selftests/net/rds/config
> index 97db7ecb892a..3d62d0c750a8 100644
> --- a/tools/testing/selftests/net/rds/config
> +++ b/tools/testing/selftests/net/rds/config
> @@ -1,3 +1,4 @@
> +CONFIG_MODULES=n
> CONFIG_NET_NS=y
> CONFIG_NET_SCH_NETEM=y
> CONFIG_RDS=y
>
> ---
> base-commit: 90fc1a393736063b2b4077115e215a2e2eebb797
> change-id: 20260520-net-rds-config-modules-cada926dc526
>
> Best regards,
> --
> Matthieu Baerts (NGI0) <[email protected]>
Hi Matthieu,
The fix looks fine to me. Thanks for catching this.
Reviewed-by: Allison Henderson <[email protected]>
Allison
>
