From mboxrd@z Thu Jan 1 00:00:00 1970 Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.remote-ip=91.218.175.243 ARC-Seal: i=1; d=subspace.kernel.org; s=arc20261015; a=rsa-sha256; cv=none; t=1794233378; b=t8ehEts4tKXtfMzLg1NlkaBgJTdcvt+TCgT8gIFnEiQdXaD817QllV3d6nYOvpNz9H8L +Z4+pRcIC/gGlAnUJkJGP9r2fkyeTIlTkCYYiIaOgigIZSCkBJLawTsgv1PMlQhpqHSgC aLkm2RoAGpxl9VU88OmHUV9iaPz1hhLwQsapDhOqovsqAnl0cBd4u2vMb7iHEk0t4PKTU qCA6l4eVoVVNI7G92DOHrT9WDpbEg6LQnFwsmQkktVPmqVVbdwpcGilAxILTydgx5bTal plsQh68D6zT8w66kbNbmbFV9srnM4kvdcx0M353lmSAaEODzE+2kM6fgSCy5f6W5vAA== ARC-Message-Signature: i=1; d=subspace.kernel.org; s=arc20261015; a=rsa-sha256; c=relaxed/relaxed; t=1794233378; h=DMARC-Filter:X-Envelope-To:DKIM-Signature:X-Envelope-To:Received: X-Mizu-Trace-ID:X-Migadu-Flow:From:To:Cc:Subject:Date:Message-ID: X-Mailer:In-Reply-To:References:MIME-Version: Content-Transfer-Encoding; bh=hIbh6EbfhDOncQjZvGHUe6etv0nEZkX8eOE42PTaiP4=; b=pm9/DM4r4eo0d8wFZQqep1WWf3HelRQa8S1Bx8qMDJ2og5kUKiH9criLisiPtyEnxuwL fN/CjXSAd+/SS0fjddpTuCBa8OukgYjWRGtlWqhx9w+R4IcNDA0gvrHk+HDC2gYz37d5r /PUyeHf/Rf94p3MP6bZAjLJ9FxKq6mwQC2SYw7Yma7lWypgeNh3bbTDe3QV9y5/rjD0Mw rKT2hpSsGYRpURnKAHr4T5WVigQAcGwwLfxQYgWgoDuOV2JnhYZE7tP/OCMlpJqiiO5x+ /DQDOlQ6EsTcIh0g0WWWd9cxFXkvvJaQvvefDKoC3BMJEeQ7SNNcIAx/qlrNfb5vGdQ== ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass header.from=linux.dev; spf=pass smtp.mailfrom=linux.dev; dkim=pass header.d=linux.dev header.i=@linux.dev header.a=rsa-sha256 header.s=key1 header.b=u1lfTtpc; arc=none smtp.remote-ip=91.218.175.243 Received: from mta0.migadu.com (out-243.mta0.migadu.com [91.218.175.243]) (using TLSv1.3 with cipher TLS_AES_128_GCM_SHA256 (128/128 bits) key-exchange X25519 server-signature ECDSA (prime256v1) server-digest SHA256) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id A22EE606EEB3 for ; Mon, 9 Nov 2026 14:09:37 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.4.2 smtp.subspace.kernel.org A22EE606EEB3 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linux.dev Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=linux.dev Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key, unprotected) header.d=linux.dev header.i=@linux.dev header.a=rsa-sha256 header.s=key1 header.b=u1lfTtpc X-Envelope-To: linux-kselftest@vger.kernel.org DKIM-Signature: a=rsa-sha256; bh=j4YuKKOawnaz1xc9cWV5lDbUelCd51dMJdfYALRo3So=; c=simple/simple; d=linux.dev; h=from:to:subject:date:message-id:mime-version:content-type; s=key1; t=1794233375; v=1; x=1794838175; b=u1lfTtpcQ6A3guqyElUO6mUrGw9iVxp5czQctA0rOMuHljl15WgOK6bY6GjsOtHzwfzh0K0N Xh6SCfmT3rBXYybYJJ35cFeR4cAJa8S0mvDquVP0NaAu0kGCSjZrFXahNt1i136EJjYsZPd9u4V SDQpxQ5bqE5zm4nvgi5M87BY= X-Envelope-To: linux-kselftest@vger.kernel.org Received: by smtp.migadu.com with ESMTPS id 85ad056a3d11807c; Mon, 09 Nov 2026 14:09:35 +0000 X-Mizu-Trace-ID: 85ad056a3d11807c X-Migadu-Flow: FLOW_OUT From: Leon Hwang To: bpf@vger.kernel.org Cc: Alexei Starovoitov , Daniel Borkmann , John Fastabend , Andrii Nakryiko , Eduard Zingerman , Kumar Kartikeya Dwivedi , Martin KaFai Lau , Song Liu , Yonghong Song , Jiri Olsa , Emil Tsalapatis , Ihor Solodrai , Steven Rostedt , Masami Hiramatsu , Mathieu Desnoyers , KP Singh , Matt Bobrowski , Shuah Khan , Leon Hwang , linux-kernel@vger.kernel.org, linux-trace-kernel@vger.kernel.org, linux-kselftest@vger.kernel.org, netdev@vger.kernel.org Subject: [PATCH bpf-next 8/9] selftests/bpf: Verify iter link_create logs Date: Mon, 9 Nov 2026 22:08:15 +0800 Message-ID: <20261109140816.5196-9-leon.hwang@linux.dev> X-Mailer: git-send-email 2.55.0 In-Reply-To: <20261109140816.5196-1-leon.hwang@linux.dev> References: <20261109140816.5196-1-leon.hwang@linux.dev> X-Mailing-List: linux-kselftest@vger.kernel.org List-Id: List-Post: List-Help: List-Subscribe: List-Unsubscribe: List-Owner: Precedence: list MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Check the error code, log message, and reported log size for invalid target FDs, flags, and iterator information. Also cover attaching a sleepable program to an iterator that does not support it. Signed-off-by: Leon Hwang --- .../selftests/bpf/prog_tests/iter_link.c | 132 ++++++++++++++++++ 1 file changed, 132 insertions(+) create mode 100644 tools/testing/selftests/bpf/prog_tests/iter_link.c diff --git a/tools/testing/selftests/bpf/prog_tests/iter_link.c b/tools/testing/selftests/bpf/prog_tests/iter_link.c new file mode 100644 index 000000000000..c684a109091a --- /dev/null +++ b/tools/testing/selftests/bpf/prog_tests/iter_link.c @@ -0,0 +1,132 @@ +// SPDX-License-Identifier: GPL-2.0 +#include +#include "bpf_iter_bpf_map.skel.h" + +static void test_iter_link_create_log(int target_fd, struct bpf_link_create_opts *opts, + __u64 flags, int err_exp, const char *msg) +{ + struct bpf_iter_bpf_map *skel; + char log_buf[128] = {}; + int err, fd, prog_fd; + LIBBPF_OPTS(bpf_log_opts, log_opts, + .buf = log_buf, + .size = sizeof(log_buf), + .level = 1, + ); + + skel = bpf_iter_bpf_map__open(); + if (!ASSERT_OK_PTR(skel, "bpf_iter_bpf_map__open")) + return; + + err = bpf_program__set_flags(skel->progs.dump_bpf_map, flags); + if (!ASSERT_OK(err, "bpf_program__set_flags")) + goto out; + + err = bpf_iter_bpf_map__load(skel); + if (!ASSERT_OK(err, "bpf_iter_bpf_map__load")) + goto out; + + opts->log_opts = &log_opts; + prog_fd = bpf_program__fd(skel->progs.dump_bpf_map); + fd = bpf_link_create(prog_fd, target_fd, BPF_TRACE_ITER, opts); + if (!ASSERT_LT(fd, 0, "bpf_link_create")) { + close(fd); + goto out; + } + + ASSERT_EQ(fd, err_exp, "err"); + ASSERT_STREQ(log_buf, msg, "log_buf"); + ASSERT_EQ(log_opts.true_size, strlen(msg) + 1, "log_size"); + +out: + bpf_iter_bpf_map__destroy(skel); +} + +static void test_log_invalid_target_fd(void) +{ + const char *msg = "Invalid target_fd.\n"; + LIBBPF_OPTS(bpf_link_create_opts, opts); + + test_iter_link_create_log(-1, &opts, 0, -EINVAL, msg); +} + +static void test_log_invalid_flags(void) +{ + const char *msg = "Invalid flags.\n"; + LIBBPF_OPTS(bpf_link_create_opts, opts, + .flags = 1, + ); + + test_iter_link_create_log(0, &opts, 0, -EINVAL, msg); +} + +static void test_log_iter_info_only(void) +{ + const char *msg = "iter_info and iter_info_len must be both set or both unset.\n"; + union bpf_iter_link_info info = {}; + LIBBPF_OPTS(bpf_link_create_opts, opts, + .iter.info = &info, + ); + + test_iter_link_create_log(0, &opts, 0, -EINVAL, msg); +} + +static void test_log_iter_info_len_only(void) +{ + const char *msg = "iter_info and iter_info_len must be both set or both unset.\n"; + LIBBPF_OPTS(bpf_link_create_opts, opts, + .iter.len = sizeof(union bpf_iter_link_info), + ); + + test_iter_link_create_log(0, &opts, 0, -EINVAL, msg); +} + +static void test_log_invalid_iter_info(void) +{ + unsigned char info[sizeof(union bpf_iter_link_info) + 1] = {}; + const char *msg = "Invalid iter_info.\n"; + LIBBPF_OPTS(bpf_link_create_opts, opts, + .iter.info = (union bpf_iter_link_info *)info, + .iter.len = sizeof(info), + ); + + info[sizeof(info) - 1] = 1; + test_iter_link_create_log(0, &opts, 0, -E2BIG, msg); +} + +static void test_log_invalid_iter_info_len(void) +{ + const char *msg = "Invalid iter_info_len.\n"; + LIBBPF_OPTS(bpf_link_create_opts, opts, + .iter.info = (union bpf_iter_link_info *)1, + .iter.len = sizeof(union bpf_iter_link_info), + ); + + test_iter_link_create_log(0, &opts, 0, -EFAULT, msg); +} + +static void test_log_invalid_prog(void) +{ + const char *msg = "The iter target does not support the bpf prog.\n"; + LIBBPF_OPTS(bpf_link_create_opts, opts); + + test_iter_link_create_log(0, &opts, BPF_F_SLEEPABLE, -EINVAL, msg); +} + +void test_iter_link_log(void) +{ + if (test__start_subtest("invalid_target_fd")) + test_log_invalid_target_fd(); + if (test__start_subtest("invalid_flags")) + test_log_invalid_flags(); + if (test__start_subtest("iter_info_only")) + test_log_iter_info_only(); + if (test__start_subtest("iter_info_len_only")) + test_log_iter_info_len_only(); + if (test__start_subtest("invalid_iter_info")) + test_log_invalid_iter_info(); + if (test__start_subtest("invalid_iter_info_len")) + test_log_invalid_iter_info_len(); + if (test__start_subtest("invalid_prog")) + test_log_invalid_prog(); +} -- 2.55.0