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

Daniel Müller
2022-08-18T17:36:05Z

On Thu, Aug 18, 2022 at 05:29:26PM +0200, roberto.sassu@huaweicloud.com wrote:
> From: Roberto Sassu <roberto.sassu@huawei.com>
>
> 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@huawei.com>
> ---
> 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@posteo.net>