All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Alexis Lothoré" <alexis.lothore@bootlin.com>
To: "Alexei Starovoitov" <alexei.starovoitov@gmail.com>,
	"Alexis Lothoré (eBPF Foundation)" <alexis.lothore@bootlin.com>
Cc: "Andrii Nakryiko" <andrii@kernel.org>,
	"Eduard Zingerman" <eddyz87@gmail.com>,
	"Alexei Starovoitov" <ast@kernel.org>,
	"Daniel Borkmann" <daniel@iogearbox.net>,
	"Martin KaFai Lau" <martin.lau@linux.dev>,
	"Song Liu" <song@kernel.org>,
	"Yonghong Song" <yonghong.song@linux.dev>,
	"John Fastabend" <john.fastabend@gmail.com>,
	"KP Singh" <kpsingh@kernel.org>,
	"Stanislav Fomichev" <sdf@fomichev.me>,
	"Hao Luo" <haoluo@google.com>, "Jiri Olsa" <jolsa@kernel.org>,
	"Shuah Khan" <shuah@kernel.org>,
	"Quentin Monnet" <qmo@kernel.org>, <ebpf@linuxfoundation.org>,
	"Bastien Curutchet" <bastien.curutchet@bootlin.com>,
	"Thomas Petazzoni" <thomas.petazzoni@bootlin.com>,
	"LKML" <linux-kernel@vger.kernel.org>,
	"bpf" <bpf@vger.kernel.org>,
	"open list:KERNEL SELFTEST FRAMEWORK"
	<linux-kselftest@vger.kernel.org>
Subject: Re: [PATCH bpf-next v2 1/4] selftests/bpf: reorder test_progs sources alphabetically
Date: Wed, 21 Jan 2026 20:30:55 +0100	[thread overview]
Message-ID: <DFUIMCAAYRYZ.3RH0727Y7TOS0@bootlin.com> (raw)
In-Reply-To: <CAADnVQK0kxfjkORv8=a8W8peiBTtpzB91FCh=6X+N0iA+OCzJA@mail.gmail.com>

Hi Alexei,

On Wed Jan 21, 2026 at 6:26 PM CET, Alexei Starovoitov wrote:
> On Wed, Jan 21, 2026 at 7:18 AM Alexis Lothoré (eBPF Foundation)
> <alexis.lothore@bootlin.com> wrote:
>>
>> Update test_progs sources listing to make those sorted alphabetically
> ...
>
>> -TRUNNER_EXTRA_SOURCES := test_progs.c          \
>> -                        cgroup_helpers.c       \
>> -                        trace_helpers.c        \
>> -                        network_helpers.c      \
>> -                        testing_helpers.c      \
>> -                        btf_helpers.c          \
>> -                        cap_helpers.c          \
>> -                        unpriv_helpers.c       \
>> -                        netlink_helpers.c      \
>> -                        jit_disasm_helpers.c   \
>> -                        io_helpers.c           \
>> -                        test_loader.c          \
>> -                        xsk.c                  \
>> -                        disasm.c               \
>> -                        disasm_helpers.c       \
>> -                        json_writer.c          \
>> +TRUNNER_EXTRA_SOURCES := btf_helpers.c                 \
>> +                        cap_helpers.c                  \
>> +                        cgroup_helpers.c               \
>
>
> no need. Even if we sort it now it will not stick.
> Files might get renamed, new files added to the end, etc.
> Just add whatever you need to the end.

Ok, I'll remove it and just add bpftool_helpers at the end.

> Also see that there is a merge conflict. Pls respin.

ACK, will do.

Thanks,

Alexis




-- 
Alexis Lothoré, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com


  reply	other threads:[~2026-01-21 19:31 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-01-21 15:18 [PATCH bpf-next v2 0/4] selftests/bpf: migrate a few bpftool testing scripts Alexis Lothoré (eBPF Foundation)
2026-01-21 15:18 ` [PATCH bpf-next v2 1/4] selftests/bpf: reorder test_progs sources alphabetically Alexis Lothoré (eBPF Foundation)
2026-01-21 17:26   ` Alexei Starovoitov
2026-01-21 19:30     ` Alexis Lothoré [this message]
2026-01-21 15:18 ` [PATCH bpf-next v2 2/4] bpf/selftests: add a few helpers for bpftool testing Alexis Lothoré (eBPF Foundation)
2026-01-21 15:18 ` [PATCH bpf-next v2 3/4] selftests/bpf: convert test_bpftool_metadata.sh into test_progs framework Alexis Lothoré (eBPF Foundation)
2026-01-21 15:18 ` [PATCH bpf-next v2 4/4] selftests/bpf: convert test_bpftool_map_access.sh " Alexis Lothoré (eBPF Foundation)

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=DFUIMCAAYRYZ.3RH0727Y7TOS0@bootlin.com \
    --to=alexis.lothore@bootlin.com \
    --cc=alexei.starovoitov@gmail.com \
    --cc=andrii@kernel.org \
    --cc=ast@kernel.org \
    --cc=bastien.curutchet@bootlin.com \
    --cc=bpf@vger.kernel.org \
    --cc=daniel@iogearbox.net \
    --cc=ebpf@linuxfoundation.org \
    --cc=eddyz87@gmail.com \
    --cc=haoluo@google.com \
    --cc=john.fastabend@gmail.com \
    --cc=jolsa@kernel.org \
    --cc=kpsingh@kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-kselftest@vger.kernel.org \
    --cc=martin.lau@linux.dev \
    --cc=qmo@kernel.org \
    --cc=sdf@fomichev.me \
    --cc=shuah@kernel.org \
    --cc=song@kernel.org \
    --cc=thomas.petazzoni@bootlin.com \
    --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.