All of lore.kernel.org
 help / color / mirror / Atom feed
From: Yonghong Song <yonghong.song@linux.dev>
To: Daniel Borkmann <daniel@iogearbox.net>, bpf@vger.kernel.org
Cc: malin89@huawei.com, ast@kernel.org,
	Rongzhen Cui <cuirongzhen@huawei.com>,
	Jingguo Tan <tanjingguo@huawei.com>
Subject: Re: [PATCH bpf-next 2/2] selftests/bpf: Cover tail-call cgroup storage prog-array checks
Date: Wed, 10 Jun 2026 13:15:00 -0700	[thread overview]
Message-ID: <23d0b8c8-71bd-4c60-846c-d4abc70b9858@linux.dev> (raw)
In-Reply-To: <20260610105539.705887-2-daniel@iogearbox.net>



On 6/10/26 3:55 AM, Daniel Borkmann wrote:
> From: Lin Ma <malin89@huawei.com>
>
> Add tail-call selftests for prog-array ownership when cgroup storage
> is in use. Verify that loading succeeds when callers and callees reuse
> the owner's cgroup storage map, and that loading fails for a different
> storage map and for the A(storage) -> B(no storage) -> C(storage)
> bridge case addressed in the previous commit.
>
> Also verify that a storage-less leaf program which cannot perform tail
> calls itself is still allowed to join a storage-owned prog array, while
> a storage-less tail-caller is rejected also at map update time.
>
>    # LDLIBS=-static PKG_CONFIG='pkg-config --static' ./vmtest.sh -- ./test_progs -t tailcalls
>    [...]
>    #475/25  tailcalls/tailcall_freplace:OK
>    #475/26  tailcalls/tailcall_bpf2bpf_freplace:OK
>    #475/27  tailcalls/tailcall_failure:OK
>    #475/28  tailcalls/reject_tail_call_spin_lock:OK
>    #475/29  tailcalls/reject_tail_call_rcu_lock:OK
>    #475/30  tailcalls/reject_tail_call_preempt_lock:OK
>    #475/31  tailcalls/reject_tail_call_ref:OK
>    #475/32  tailcalls/tailcall_sleepable:OK
>    #475/33  tailcalls/tailcall_cgrp_storage:OK
>    #475/34  tailcalls/tailcall_cgrp_storage_diff_storage:OK
>    #475/35  tailcalls/tailcall_cgrp_storage_no_storage:OK
>    #475/36  tailcalls/tailcall_cgrp_storage_no_storage_leaf:OK
>    #475/37  tailcalls/tailcall_cgrp_storage_no_storage_bridge:OK
>    #475     tailcalls:OK
>    Summary: 1/37 PASSED, 0 SKIPPED, 0 FAILED
>
> Signed-off-by: Lin Ma <malin89@huawei.com>
> Signed-off-by: Rongzhen Cui <cuirongzhen@huawei.com>
> Signed-off-by: Jingguo Tan <tanjingguo@huawei.com>
> Signed-off-by: Daniel Borkmann <daniel@iogearbox.net>

Acked-by: Yonghong Song <yonghong.song@linux.dev>


  reply	other threads:[~2026-06-10 20:15 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-06-10 10:55 [PATCH bpf-next 1/2] bpf: Tighten cgroup storage cookie checks for prog arrays Daniel Borkmann
2026-06-10 10:55 ` [PATCH bpf-next 2/2] selftests/bpf: Cover tail-call cgroup storage prog-array checks Daniel Borkmann
2026-06-10 20:15   ` Yonghong Song [this message]
2026-06-10 20:14 ` [PATCH bpf-next 1/2] bpf: Tighten cgroup storage cookie checks for prog arrays Yonghong Song
2026-06-10 23:20 ` 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=23d0b8c8-71bd-4c60-846c-d4abc70b9858@linux.dev \
    --to=yonghong.song@linux.dev \
    --cc=ast@kernel.org \
    --cc=bpf@vger.kernel.org \
    --cc=cuirongzhen@huawei.com \
    --cc=daniel@iogearbox.net \
    --cc=malin89@huawei.com \
    --cc=tanjingguo@huawei.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.