From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-pf1-f177.google.com (mail-pf1-f177.google.com [209.85.210.177]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 0AB073D7A15 for ; Wed, 17 Jun 2026 09:36:52 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.210.177 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1781689016; cv=none; b=bZ6qWc1So08m0N43KK1NsFCCofzY6gSn9yf504VEqf8/0Av2bIqLUh5ApaCjRWXkPuBahv1ivwhbQWZhi0Hu7yzhqvxzlkaw5Uoc1a2SVEDhXf8vdO6Xgo2ufhVz8vDcFL5wSdhbgQkhe8nSbrAdqYG6j5DLTH4UrBWGlkSg0TE= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1781689016; c=relaxed/simple; bh=PdBtwKgFqyMGtomh7hTUPsscrriY96d4wUo+kESMV6s=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=XP4/SykK/Huxi1WscXOSM8eA8k+MA52usSoCrCIC04CcT6Wi3yeArnsc5OlfaEGFIyLsWT7VUM7+qZfcQ9x2abfD+nRUmRuoyzOhfxhlzNZdZO0MgQTBIVG+cRbLAXkIuGNQErHHyOAklDx670IpW8oCA9Pst28y5h96Uige0PE= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com; spf=pass smtp.mailfrom=gmail.com; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b=P2uObVUW; arc=none smtp.client-ip=209.85.210.177 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="P2uObVUW" Received: by mail-pf1-f177.google.com with SMTP id d2e1a72fcca58-8423f869421so4342614b3a.3 for ; Wed, 17 Jun 2026 02:36:52 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1781689012; x=1782293812; darn=vger.kernel.org; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date :message-id:reply-to; bh=5HGsJuBJyPxuBJWOZpUxJqFkb8helZpMElVirEJAoUU=; b=P2uObVUWQDzWZwNk/QWV/rrp23bBAIo0SmfraD9noK8JbCi5hI7Xfwfx3xgUlTr3FM EV+wjcIPCWnstK6iSe5US6+Ocy92HHT8Bu5M2Xw3Id1pFJkj2gAZG05WwZZ2OLBX9xmn rK0yINbI+f30A3e5dUgnRcOYh8VRa+mQtivTpOsD/2QHcwhP3bpXABdzj90MvpQ2l/5V cvwzccJKCr+kF00nPKW8Dms9R44mUHW4cc7oTaSaVUOxD2W0zMfcnEQ6lfND6w4PIrlT GbE+KdD/CwjzAONSCaYe8Q+VHYKIsy2q0ybc3hqoSdyxJkfAlPQEZkv1APeNFRIPkgu6 Hw+A== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1781689012; x=1782293812; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-gg:x-gm-message-state:from :to:cc:subject:date:message-id:reply-to; bh=5HGsJuBJyPxuBJWOZpUxJqFkb8helZpMElVirEJAoUU=; b=K9oAF/3FUucy+zaaTefaWa39iyv5QCdGXx/2tKQPBrViCW2Mm0BZ/rhksLdvQxJrBh NauhG8xJk8QS2A3ocInxa1HdZbiyVch8S0oS4H8hyiRGPYSGR3K6sYOqs43LT4D0BaMv ohl92yANrmEWlBs1XyCp4WJXkmEDnlz4FPtOA1TjWEle7OJPPO4IEtlPToXEHcJ4nomm kboBPVSPqVGNZSmicjS0eLhTEgQ897yr7jt4bUbs75Xj1xIdUTuY6+65leFx+NK16QRB soV08Zij3JKqndXEgtXdHJdDA5vjO1PS5snNOcAVkTIFLHLHey1kvDlkWWMc58QdorF2 X65A== X-Forwarded-Encrypted: i=1; AFNElJ/Bv0qbpqnBFCqwqI6ikBo8XaVKWBgDq63//m1f/5KtkxiEZ6gqvFweEmX4pXssj5kUfkzGVdjccdsFo+I=@vger.kernel.org X-Gm-Message-State: AOJu0YxSWqDVSrOiOisrokRd4OdGqroa3ynsyw9/lt5uVIdQLY7vNhWe jVC6mdKFE8yhz9Jc71wbu78iWAeh99M3f2R8KxPjTViseiF6ClWomDbo X-Gm-Gg: Acq92OEBvs9TZe/7UfVbo165+3DYeNN/lld4Ru6osHIMZFpw4BHd5SPj4gMo7PEvvlr U/BEjczTOJWp6hoEv9A5QSzU80eYAgQHYAxUnoWUBBHLWnC0CcgCkvwqjTJmLvNymLsUEFALo4+ 1G/gg7f55XHMeHKtSdzyAyp1OVN9uWlXRWfT9zWA6qaar4n7sW4r4WdvZMo4k9zS77wVlzv5FNS RloIQPffAojUbq3R5D/YZuORxkJncT99SGWW9Q5fqfbMzEhnUxbaqfKxh3CD/cSPtMFzLdb14Se UvMwpR5SRQ2XzbKvlHL0I4IRTVq/11+W4xLZhTABDxkhExEwlMPAdD7yrZjW3W9YUfpe9ihdS2M kP2IZgHvqxreRhSGQD2fBh6FY8IsXRK2QXYxPxlkWyA64LBeMgm0adJJHJwnlPRp4WoC5bLjZ/H TYXtN+bPegWHIeAO+t4lajQmsX59z1nu719/FLCZGu3ftp3EhxzXBT+C5yNV3N6Yk= X-Received: by 2002:a05:6a00:3697:b0:83f:250d:59c with SMTP id d2e1a72fcca58-845245b79e6mr3179862b3a.39.1781689012085; Wed, 17 Jun 2026 02:36:52 -0700 (PDT) Received: from localhost.localdomain ([2a12:a304:100::206b]) by smtp.gmail.com with ESMTPSA id d2e1a72fcca58-8434b0226acsm15521653b3a.48.2026.06.17.02.36.41 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 17 Jun 2026 02:36:51 -0700 (PDT) From: Sun Jian To: bpf@vger.kernel.org Cc: netdev@vger.kernel.org, linux-kselftest@vger.kernel.org, linux-kernel@vger.kernel.org, ast@kernel.org, daniel@iogearbox.net, andrii@kernel.org, martin.lau@linux.dev, eddyz87@gmail.com, memxor@gmail.com, song@kernel.org, yonghong.song@linux.dev, jolsa@kernel.org, davem@davemloft.net, edumazet@google.com, kuba@kernel.org, pabeni@redhat.com, horms@kernel.org, shuah@kernel.org, hawk@kernel.org, john.fastabend@gmail.com, sdf@fomichev.me, toke@redhat.com, lorenzo@kernel.org, paul.chaignon@gmail.com, Sun Jian Subject: [PATCH bpf v3 2/2] selftests/bpf: Cover partial copy of non-linear test_run output Date: Wed, 17 Jun 2026 17:35:57 +0800 Message-ID: <20260617093557.63880-3-sun.jian.kdev@gmail.com> X-Mailer: git-send-email 2.43.0 In-Reply-To: <20260617093557.63880-1-sun.jian.kdev@gmail.com> References: <20260617093557.63880-1-sun.jian.kdev@gmail.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: 8bit prog_run_opts already verifies that BPF_PROG_TEST_RUN returns -ENOSPC for a short data_out buffer while still reporting the full output size through data_size_out. Add the same coverage for non-linear test_run output. Use pass-through TC and XDP programs with a 9000-byte packet, a 64-byte linear data area, and a 100-byte data_out buffer. The expected output spans both the linear data and the first fragment. Verify that test_run returns -ENOSPC, reports the full packet length through data_size_out, and copies the packet prefix into data_out for both non-linear skb and XDP frags paths. Signed-off-by: Sun Jian --- .../selftests/bpf/prog_tests/prog_run_opts.c | 70 +++++++++++++++++++ .../selftests/bpf/progs/test_pkt_access.c | 12 ++++ 2 files changed, 82 insertions(+) diff --git a/tools/testing/selftests/bpf/prog_tests/prog_run_opts.c b/tools/testing/selftests/bpf/prog_tests/prog_run_opts.c index 01f1d1b6715a..9cc898e6a9f7 100644 --- a/tools/testing/selftests/bpf/prog_tests/prog_run_opts.c +++ b/tools/testing/selftests/bpf/prog_tests/prog_run_opts.c @@ -4,6 +4,10 @@ #include "test_pkt_access.skel.h" +#define NONLINEAR_PKT_LEN 9000 +#define NONLINEAR_LINEAR_DATA_LEN 64 +#define SHORT_OUT_LEN 100 + static const __u32 duration; static void check_run_cnt(int prog_fd, __u64 run_cnt) @@ -20,6 +24,69 @@ static void check_run_cnt(int prog_fd, __u64 run_cnt) "incorrect number of repetitions, want %llu have %llu\n", run_cnt, info.run_cnt); } +static void init_pkt(__u8 *pkt, size_t len) +{ + size_t i; + + for (i = 0; i < len; i++) + pkt[i] = i & 0xff; +} + +static void test_skb_nonlinear_data_out_partial(struct test_pkt_access *skel) +{ + LIBBPF_OPTS(bpf_test_run_opts, topts); + __u8 pkt[NONLINEAR_PKT_LEN]; + __u8 out[SHORT_OUT_LEN]; + struct __sk_buff skb = {}; + int prog_fd, err; + + init_pkt(pkt, sizeof(pkt)); + + skb.data_end = NONLINEAR_LINEAR_DATA_LEN; + + topts.data_in = pkt; + topts.data_size_in = sizeof(pkt); + topts.data_out = out; + topts.data_size_out = sizeof(out); + topts.ctx_in = &skb; + topts.ctx_size_in = sizeof(skb); + + prog_fd = bpf_program__fd(skel->progs.tc_pass_prog); + err = bpf_prog_test_run_opts(prog_fd, &topts); + + ASSERT_EQ(err, -ENOSPC, "skb_nonlinear_partial_err"); + ASSERT_EQ(topts.data_size_out, sizeof(pkt), "skb_nonlinear_partial_data_size_out"); + ASSERT_OK(memcmp(out, pkt, sizeof(out)), "skb_nonlinear_partial_data_out"); +} + +static void test_xdp_nonlinear_data_out_partial(struct test_pkt_access *skel) +{ + LIBBPF_OPTS(bpf_test_run_opts, topts); + __u8 pkt[NONLINEAR_PKT_LEN]; + __u8 out[SHORT_OUT_LEN]; + struct xdp_md ctx = {}; + int prog_fd, err; + + init_pkt(pkt, sizeof(pkt)); + + ctx.data = 0; + ctx.data_end = NONLINEAR_LINEAR_DATA_LEN; + + topts.data_in = pkt; + topts.data_size_in = sizeof(pkt); + topts.data_out = out; + topts.data_size_out = sizeof(out); + topts.ctx_in = &ctx; + topts.ctx_size_in = sizeof(ctx); + + prog_fd = bpf_program__fd(skel->progs.xdp_frags_pass_prog); + err = bpf_prog_test_run_opts(prog_fd, &topts); + + ASSERT_EQ(err, -ENOSPC, "xdp_nonlinear_partial_err"); + ASSERT_EQ(topts.data_size_out, sizeof(pkt), "xdp_nonlinear_partial_data_size_out"); + ASSERT_OK(memcmp(out, pkt, sizeof(out)), "xdp_nonlinear_partial_data_out"); +} + void test_prog_run_opts(void) { struct test_pkt_access *skel; @@ -69,6 +136,9 @@ void test_prog_run_opts(void) run_cnt += topts.repeat; check_run_cnt(prog_fd, run_cnt); + test_skb_nonlinear_data_out_partial(skel); + test_xdp_nonlinear_data_out_partial(skel); + cleanup: if (skel) test_pkt_access__destroy(skel); diff --git a/tools/testing/selftests/bpf/progs/test_pkt_access.c b/tools/testing/selftests/bpf/progs/test_pkt_access.c index bce7173152c6..cd284401eebd 100644 --- a/tools/testing/selftests/bpf/progs/test_pkt_access.c +++ b/tools/testing/selftests/bpf/progs/test_pkt_access.c @@ -150,3 +150,15 @@ int test_pkt_access(struct __sk_buff *skb) return TC_ACT_UNSPEC; } + +SEC("tc") +int tc_pass_prog(struct __sk_buff *skb) +{ + return TC_ACT_OK; +} + +SEC("xdp.frags") +int xdp_frags_pass_prog(struct xdp_md *ctx) +{ + return XDP_PASS; +} -- 2.43.0