Re: [PATCH v12 07/10] selftests/bpf: Compile kernel with everything as built-in

[Daniel Müller]

On Thu, Aug 18, 2022 at 05:29:26PM +0200, roberto.sassu@xxxxxxxxxxxxxxx wrote:
> From: Roberto Sassu <roberto.sassu@xxxxxxxxxx>
> 
> Since the eBPF CI does not support kernel modules, change the kernel config
> to compile everything as built-in.
> 
> Signed-off-by: Roberto Sassu <roberto.sassu@xxxxxxxxxx>
> ---
>  tools/testing/selftests/bpf/config        | 26 +++++++++++------------
>  tools/testing/selftests/bpf/config.x86_64 |  2 +-
>  2 files changed, 14 insertions(+), 14 deletions(-)
> 
> diff --git a/tools/testing/selftests/bpf/config b/tools/testing/selftests/bpf/config
> index 3fc46f9cfb22..0fdd11e6b742 100644
> --- a/tools/testing/selftests/bpf/config
> +++ b/tools/testing/selftests/bpf/config
> @@ -7,9 +7,9 @@ CONFIG_BPF_LSM=y
>  CONFIG_BPF_STREAM_PARSER=y
>  CONFIG_BPF_SYSCALL=y
>  CONFIG_CGROUP_BPF=y
> -CONFIG_CRYPTO_HMAC=m
> -CONFIG_CRYPTO_SHA256=m
> -CONFIG_CRYPTO_USER_API_HASH=m
> +CONFIG_CRYPTO_HMAC=y
> +CONFIG_CRYPTO_SHA256=y
> +CONFIG_CRYPTO_USER_API_HASH=y
>  CONFIG_DYNAMIC_FTRACE=y
>  CONFIG_FPROBE=y
>  CONFIG_FTRACE_SYSCALLS=y
> @@ -24,30 +24,30 @@ CONFIG_IP_NF_FILTER=y
>  CONFIG_IP_NF_RAW=y
>  CONFIG_IP_NF_TARGET_SYNPROXY=y
>  CONFIG_IPV6=y
> -CONFIG_IPV6_FOU=m
> -CONFIG_IPV6_FOU_TUNNEL=m
> +CONFIG_IPV6_FOU=y
> +CONFIG_IPV6_FOU_TUNNEL=y
>  CONFIG_IPV6_GRE=y
>  CONFIG_IPV6_SEG6_BPF=y
> -CONFIG_IPV6_SIT=m
> +CONFIG_IPV6_SIT=y
>  CONFIG_IPV6_TUNNEL=y
>  CONFIG_LIRC=y
>  CONFIG_LWTUNNEL=y
>  CONFIG_MPLS=y
> -CONFIG_MPLS_IPTUNNEL=m
> -CONFIG_MPLS_ROUTING=m
> +CONFIG_MPLS_IPTUNNEL=y
> +CONFIG_MPLS_ROUTING=y
>  CONFIG_MPTCP=y
>  CONFIG_NET_CLS_ACT=y
>  CONFIG_NET_CLS_BPF=y
> -CONFIG_NET_CLS_FLOWER=m
> -CONFIG_NET_FOU=m
> +CONFIG_NET_CLS_FLOWER=y
> +CONFIG_NET_FOU=y
>  CONFIG_NET_FOU_IP_TUNNELS=y
>  CONFIG_NET_IPGRE=y
>  CONFIG_NET_IPGRE_DEMUX=y
>  CONFIG_NET_IPIP=y
> -CONFIG_NET_MPLS_GSO=m
> +CONFIG_NET_MPLS_GSO=y
>  CONFIG_NET_SCH_INGRESS=y
>  CONFIG_NET_SCHED=y
> -CONFIG_NETDEVSIM=m
> +CONFIG_NETDEVSIM=y
>  CONFIG_NETFILTER=y
>  CONFIG_NETFILTER_SYNPROXY=y
>  CONFIG_NETFILTER_XT_CONNMARK=y
> @@ -60,7 +60,7 @@ CONFIG_NF_DEFRAG_IPV6=y
>  CONFIG_RC_CORE=y
>  CONFIG_SECURITY=y
>  CONFIG_SECURITYFS=y
> -CONFIG_TEST_BPF=m
> +CONFIG_TEST_BPF=y
>  CONFIG_USERFAULTFD=y
>  CONFIG_VXLAN=y
>  CONFIG_XDP_SOCKETS=y
> diff --git a/tools/testing/selftests/bpf/config.x86_64 b/tools/testing/selftests/bpf/config.x86_64
> index f0859a1d37ab..ce70c9509204 100644
> --- a/tools/testing/selftests/bpf/config.x86_64
> +++ b/tools/testing/selftests/bpf/config.x86_64
> @@ -47,7 +47,7 @@ CONFIG_CPU_IDLE_GOV_LADDER=y
>  CONFIG_CPUSETS=y
>  CONFIG_CRC_T10DIF=y
>  CONFIG_CRYPTO_BLAKE2B=y
> -CONFIG_CRYPTO_DEV_VIRTIO=m
> +CONFIG_CRYPTO_DEV_VIRTIO=y
>  CONFIG_CRYPTO_SEQIV=y
>  CONFIG_CRYPTO_XXHASH=y
>  CONFIG_DCB=y
> -- 
> 2.25.1

The change looks good to me. Thanks for making it.

Acked-by: Daniel Müller <deso@xxxxxxxxxx>