From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from 69-171-232-181.mail-mxout.facebook.com (69-171-232-181.mail-mxout.facebook.com [69.171.232.181]) (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 E3609288C05 for ; Fri, 6 Jun 2025 17:41:58 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=69.171.232.181 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1749231721; cv=none; b=qv98ChIfM8+5VJ3UzVK/Ukq8qSBa4tKgl1D9BUkrr1CHZQ3JrgE71suitoUommHCQThRay5JR6UUch6Jl8rYdyDsDW47aIflG1ekfJcHO6T+806xcTmwVanaNwdeT1dP2PAA7k+8C4EXh7oh6mfH2HjaouJdJVJIpUKI5SGfpxw= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1749231721; c=relaxed/simple; bh=hHLYhil1lEHMtJ4WsLcI2QiQyZas2+PY8aHCTmpv/Lg=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=FjHTE6GpMLLaMIyP8Iz8hlauBTwLwkU4qV+cJgkjVB/yQLG1VoNfH+8fGrClRPDBt0A8nSIxmd30JO1PLfT4z9jrrIXnR7pnj0dtekBVbDv10pzpKNIGW3wBnAEM/I/J0WXhKvBueGPgf26I/XRFnsMkFZqkT6BZ61zfcJH4il0= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=fail (p=none dis=none) header.from=linux.dev; spf=fail smtp.mailfrom=linux.dev; arc=none smtp.client-ip=69.171.232.181 Authentication-Results: smtp.subspace.kernel.org; dmarc=fail (p=none dis=none) header.from=linux.dev Authentication-Results: smtp.subspace.kernel.org; spf=fail smtp.mailfrom=linux.dev Received: by devvm16039.vll0.facebook.com (Postfix, from userid 128203) id 1A0179046CF5; Fri, 6 Jun 2025 10:41:45 -0700 (PDT) From: Yonghong Song To: bpf@vger.kernel.org Cc: Alexei Starovoitov , Andrii Nakryiko , Daniel Borkmann , kernel-team@fb.com, Martin KaFai Lau Subject: [PATCH bpf-next v2 1/4] selftests/bpf: Reduce test_xdp_adjust_frags_tail_grow logs Date: Fri, 6 Jun 2025 10:41:45 -0700 Message-ID: <20250606174145.3036925-1-yonghong.song@linux.dev> X-Mailer: git-send-email 2.47.1 In-Reply-To: <20250606174139.3036576-1-yonghong.song@linux.dev> References: <20250606174139.3036576-1-yonghong.song@linux.dev> Precedence: bulk X-Mailing-List: bpf@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable For selftest xdp_adjust_tail/xdp_adjust_frags_tail_grow, if tested failur= e, I see a long list of log output like ... test_xdp_adjust_frags_tail_grow:PASS:9Kb+10b-untouched 0 nsec test_xdp_adjust_frags_tail_grow:PASS:9Kb+10b-untouched 0 nsec test_xdp_adjust_frags_tail_grow:PASS:9Kb+10b-untouched 0 nsec test_xdp_adjust_frags_tail_grow:PASS:9Kb+10b-untouched 0 nsec ... There are total 7374 lines of the above which is too much. Let us only issue such logs when it is an assert failure. Signed-off-by: Yonghong Song --- .../selftests/bpf/prog_tests/xdp_adjust_tail.c | 18 ++++++++++++------ 1 file changed, 12 insertions(+), 6 deletions(-) diff --git a/tools/testing/selftests/bpf/prog_tests/xdp_adjust_tail.c b/t= ools/testing/selftests/bpf/prog_tests/xdp_adjust_tail.c index b2b2d85dbb1b..e361129402a1 100644 --- a/tools/testing/selftests/bpf/prog_tests/xdp_adjust_tail.c +++ b/tools/testing/selftests/bpf/prog_tests/xdp_adjust_tail.c @@ -246,14 +246,20 @@ static void test_xdp_adjust_frags_tail_grow(void) ASSERT_EQ(topts.retval, XDP_TX, "9Kb+10b retval"); ASSERT_EQ(topts.data_size_out, exp_size, "9Kb+10b size"); =20 - for (i =3D 0; i < 9000; i++) - ASSERT_EQ(buf[i], 1, "9Kb+10b-old"); + for (i =3D 0; i < 9000; i++) { + if (buf[i] !=3D 1) + ASSERT_EQ(buf[i], 1, "9Kb+10b-old"); + } =20 - for (i =3D 9000; i < 9010; i++) - ASSERT_EQ(buf[i], 0, "9Kb+10b-new"); + for (i =3D 9000; i < 9010; i++) { + if (buf[i] !=3D 0) + ASSERT_EQ(buf[i], 0, "9Kb+10b-new"); + } =20 - for (i =3D 9010; i < 16384; i++) - ASSERT_EQ(buf[i], 1, "9Kb+10b-untouched"); + for (i =3D 9010; i < 16384; i++) { + if (buf[i] !=3D 1) + ASSERT_EQ(buf[i], 1, "9Kb+10b-untouched"); + } =20 /* Test a too large grow */ memset(buf, 1, 16384); --=20 2.47.1