All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Emil Tsalapatis" <emil@etsalapatis.com>
To: "Ricardo B. Marlière" <rbm@suse.com>,
	"Alexei Starovoitov" <ast@kernel.org>,
	"Daniel Borkmann" <daniel@iogearbox.net>,
	"Andrii Nakryiko" <andrii@kernel.org>,
	"Eduard Zingerman" <eddyz87@gmail.com>,
	"Kumar Kartikeya Dwivedi" <memxor@gmail.com>,
	"Martin KaFai Lau" <martin.lau@linux.dev>,
	"Song Liu" <song@kernel.org>,
	"Yonghong Song" <yonghong.song@linux.dev>,
	"Jiri Olsa" <jolsa@kernel.org>,
	"Emil Tsalapatis" <emil@etsalapatis.com>,
	"Jonathan Corbet" <corbet@lwn.net>,
	"Shuah Khan" <skhan@linuxfoundation.org>
Cc: <bpf@vger.kernel.org>, <linux-doc@vger.kernel.org>,
	<linux-kernel@vger.kernel.org>
Subject: Re: [PATCH] docs/bpf: Document BPF_STRICT_BUILD=0 to tolerate test build failures
Date: Mon, 06 Jul 2026 14:38:02 -0400	[thread overview]
Message-ID: <DJRPGA0KTH7U.WBW2Y6CVNBQR@etsalapatis.com> (raw)
In-Reply-To: <20260706-b4-bpf_strict_build_docs-v1-1-5324d605c7b0@suse.com>

On Mon Jul 6, 2026 at 11:28 AM EDT, Ricardo B. Marlière wrote:
> When the kernel config does not fully match the BPF selftest config
> fragment, some tests may fail to compile. BPF_STRICT_BUILD (defaulting to
> 1) makes any such failure fatal. Mention the option so that developers are
> aware they can set it to 0 to skip broken tests and keep the build going,
> which is particularly useful during bringup or when testing on constrained
> (e.g. distribution) configurations.
>
> Signed-off-by: Ricardo B. Marlière <rbm@suse.com>
> ---
>  Documentation/bpf/bpf_devel_QA.rst | 5 ++++-
>  1 file changed, 4 insertions(+), 1 deletion(-)
>
> diff --git a/Documentation/bpf/bpf_devel_QA.rst b/Documentation/bpf/bpf_devel_QA.rst
> index 45bc5c5cd793..edf8107a7beb 100644
> --- a/Documentation/bpf/bpf_devel_QA.rst
> +++ b/Documentation/bpf/bpf_devel_QA.rst
> @@ -479,7 +479,10 @@ for details.
>  
>  To maximize the number of tests passing, the .config of the kernel
>  under test should match the config file fragment in
> -tools/testing/selftests/bpf as closely as possible.
> +tools/testing/selftests/bpf as closely as possible. If not possible,
> +however, you can set ``BPF_STRICT_BUILD=0`` when invoking ``make``
> +to tolerate individual compilation failures and continue building
> +the remaining tests rather than treating each failure as fatal.
>  

Reviewed-by: Emil Tsalapatis <emil@etsalapatis.com>

Nit: You could add the caveat that anything in TRUNNER_EXTRA_SOURCES is
not covered, but it's pretty minor.

>  Finally to ensure support for latest BPF Type Format features -
>  discussed in Documentation/bpf/btf.rst - pahole version 1.16
>
> ---
> base-commit: 8cdeaa50eae8dad34885515f62559ee83e7e8dda
> change-id: 20260706-b4-bpf_strict_build_docs-a8aa7fa46e2f
>
> Best regards,
> --  
> Ricardo B. Marlière <rbm@suse.com>


  reply	other threads:[~2026-07-06 18:38 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-07-06 15:28 [PATCH] docs/bpf: Document BPF_STRICT_BUILD=0 to tolerate test build failures Ricardo B. Marlière
2026-07-06 18:38 ` Emil Tsalapatis [this message]
2026-07-06 22:30 ` patchwork-bot+netdevbpf

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=DJRPGA0KTH7U.WBW2Y6CVNBQR@etsalapatis.com \
    --to=emil@etsalapatis.com \
    --cc=andrii@kernel.org \
    --cc=ast@kernel.org \
    --cc=bpf@vger.kernel.org \
    --cc=corbet@lwn.net \
    --cc=daniel@iogearbox.net \
    --cc=eddyz87@gmail.com \
    --cc=jolsa@kernel.org \
    --cc=linux-doc@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=martin.lau@linux.dev \
    --cc=memxor@gmail.com \
    --cc=rbm@suse.com \
    --cc=skhan@linuxfoundation.org \
    --cc=song@kernel.org \
    --cc=yonghong.song@linux.dev \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.