All of lore.kernel.org
 help / color / mirror / Atom feed
From: Shung-Hsi Yu <shung-hsi.yu@suse.com>
To: Ihor Solodrai <ihor.solodrai@linux.dev>
Cc: bpf@vger.kernel.org, Andrii Nakryiko <andrii@kernel.org>,
	 Alexei Starovoitov <ast@kernel.org>,
	Daniel Borkmann <daniel@iogearbox.net>,
	 Eduard Zingerman <eddyz87@gmail.com>,
	Paul Chaignon <paul.chaignon@gmail.com>
Subject: Re: BPF CI for Stable
Date: Wed, 10 Jun 2026 16:18:02 +0800	[thread overview]
Message-ID: <aikYGA9To89q3Jd2@u94a> (raw)
In-Reply-To: <a45e6e99-fcb8-4a7b-8012-8c7e88ea7598@linux.dev>

On Fri, Jun 05, 2026 at 10:12:14AM -0700, Ihor Solodrai wrote:
> On 6/5/26 12:45 AM, Shung-Hsi Yu wrote:
[...]
> >   Is it possible to get BPF CI for stable in its own repository under a
> >   GitHub organization (e.g. libbpf/stable-bpf-ci)?
> 
> Hi Shung-Hsi,
> 
> I think a better home for the BPF CI targeting stable kernels is
> kernel-patches org [1] for a couple of reasons. First, repos in the
> kernel-patches org have access to hardware - generic runners provided
> by Meta and s390x runners provided by IBM. Second reason is that this
> org hosts source of truth repos for most of BPF CI code, even though a
> big chunk of it lives in libbpf/ci.
> 
> Let me know if this makes sense, and I'll create a new repository and
> grant you write permissions there.

Make sense, let's have it in kernel-patches org.

[...]
> > Technically BPF CI for stable can also simply be merged back to
> > libbpf/libbpf
> 
> No, that's a bad idea.
> 
> >, but that felt less ideal because:
> > - kbuilder-debian container image currently doesn't work when testing
> >   stable kernels (I haven't look at reason of failure, might not be hard
> >   to fix)
> 
> Technically, it's not a hard requirement to use the container.
> It makes the build jobs a little faster and a little more reliable, but
> it was crafted for bpf-next testing, which may not be ideal for stable.
> If necessary you could develop a separate "kbuilder-stable" container, but
> it's up to you to decide whether it's needed.

Understood, I'll stick with Ubuntu 24.04.

> > - actions that tests stable kernel will be mixed with actions targeting
> >   libbpf itself 
> > - BPF CI for stable will have to contain version-specific tweaks (e.g.
> >   [4])
> 
> This is fine and expected. It is of course great to be able to reuse code,
> but practically speaking, I think in this case it may be more work to
> keep the common parts compatible with both stable and bpf-next workflows,
> than maintaining two diverged CI trees.
> 
> The rule of thumb should be this: if you can use a libbpf/ci action, do
> it. If there is something very specific to the workflow, keep it in place.
> Don't be shy copy-pasting yaml fragments that work for you, it's fine.
[...]
> > OTOH we can also try kernel-patches/bpf-like approach and have a
> > kernel-patches/stable, providing proper, fully-fledged BPF CI for stable
> > that tests incoming patchset to stable (not sure if Daniel was asking
> > about this during the session). But I find this to be much more work.
> 
> We don't have to do this work right away (or ever).
> 
> First let's set up the kernel-patches/stable (or whatever the name) with
> the implementation you already have.
> 
> We can enhance and improve it when there are cycles and willingness to do
> so in the future, 

Sounds good to me, let's go with kernel-patches/stable. Thanks!

> > The current approach of applying queued patches from stable-queue[5],
> > while comes with quite some delay between the moment patch is proposed
> > to stable mailing vs the moment said patch gets tested, seem like a
> > workable compromise.
> 
> This can be automated by the way.
> Check out linux-next sync on current BPF CI, it's straightforward [2].
> There is a job periodically syncing a branch, and the CI pipeline is
> triggered on push.

I'll looking this, thanks for the pointer.

Shung-Hsi

> [1] https://github.com/kernel-patches/
> [2] https://github.com/kernel-patches/vmtest/blob/master/.github/workflows/linux-next-sync.yml
[...]

      reply	other threads:[~2026-06-10  8:18 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-06-05  7:45 BPF CI for Stable Shung-Hsi Yu
2026-06-05 17:12 ` Ihor Solodrai
2026-06-10  8:18   ` Shung-Hsi Yu [this message]

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=aikYGA9To89q3Jd2@u94a \
    --to=shung-hsi.yu@suse.com \
    --cc=andrii@kernel.org \
    --cc=ast@kernel.org \
    --cc=bpf@vger.kernel.org \
    --cc=daniel@iogearbox.net \
    --cc=eddyz87@gmail.com \
    --cc=ihor.solodrai@linux.dev \
    --cc=paul.chaignon@gmail.com \
    /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.