From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-wm2-f13.google.com (mail-wm2-f13.google.com [74.125.225.141]) (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 6B0D2414425 for ; Wed, 23 Sep 2026 22:11:47 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=74.125.225.141 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1790201509; cv=none; b=Pp23U54SFCJTw7QIrBW1UPaNaa5k69j4H5W+1GJgR0+OPclpldoRJlVjFoIngdOkcWWoVehy+FEtDtUUGg8X3qSMJ5x+rqLIh4dN62aUDE2I4CInwPdNYvQBkmIJwlfMxrLl9+qInHx71Rzt+jZ5g7Kmxlq4ruDApBdFG7uCIyw= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1790201509; c=relaxed/simple; bh=G+sXLb5QQACGJWFw2GYB/k46SIVKIpCEAPVvzKkNo84=; h=Mime-Version:Content-Type:Date:Message-Id:Cc:Subject:From:To: References:In-Reply-To; b=IjhMrnjTO4Wmx4Rmpz7dy8ge+EpsJR9/l9lB0bk/1iTT7uCPwnE4IhTSLoe+GO/j5XQo6leT8nkIxL38ibVJqmhhyQUeFTuVWTtIGf9S0t8tC/VaRAvcBC5QlCMiEIPH7BWkqXniDTmnd6YXxvMC0w120xvfpPSbyQJDcuOwYsA= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=etsalapatis.com; spf=pass smtp.mailfrom=etsalapatis.com; dkim=pass (2048-bit key) header.d=etsalapatis-com.20251104.gappssmtp.com header.i=@etsalapatis-com.20251104.gappssmtp.com header.b=taa0RHq7; arc=none smtp.client-ip=74.125.225.141 Authentication-Results: smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=etsalapatis.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=etsalapatis.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=etsalapatis-com.20251104.gappssmtp.com header.i=@etsalapatis-com.20251104.gappssmtp.com header.b="taa0RHq7" Received: by mail-wm2-f13.google.com with SMTP id 5b1f17b1804b1-49cd38e0f79so7835665e9.3 for ; Wed, 23 Sep 2026 15:11:47 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=etsalapatis-com.20251104.gappssmtp.com; s=20251104; t=1790201506; x=1790806306; darn=vger.kernel.org; h=in-reply-to:references:to:from:subject:cc:message-id:date :content-type:content-transfer-encoding:mime-version:from:to:cc :subject:date:message-id:reply-to:content-type; bh=GAy3js82zkC6BZGrspDqEhBjsPWiuHyN9je/HHFqVI8=; b=taa0RHq7gcEylNl6iCnAL35RW7qHxoCrus8sPHrquAghyLiXANtiWbQZ6LwZYLiwxP LQEqXLzN8r41H1c98UGdhqkxQMSrQlS+cR6MS0iS/7FQ+V37Rilg4opgBVs6lC8mqJUY mhGvsmJfxxgiuswQH99sxcAmzay6rKO9cJP4tGk1/EZKo+es3iOQiVVvcHV0rzYkRi5Q F4LJwiLGVeWQaXihqs4r1ZWNrz5Z1BeHTmM9krFQ0suHLNTWFmx7ikcFySfsi4QGG2fH zHcyHQeoykjQ0RGMe/Kcg28bjyY46n/j1Eq0yx1gW16atbiRFt0LuI6a4ry5ERn2o2R/ hSOw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20260707; t=1790201506; x=1790806306; h=in-reply-to:references:to:from:subject:cc:message-id:date :content-type:content-transfer-encoding:mime-version:x-gm-gg :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to :content-type; bh=GAy3js82zkC6BZGrspDqEhBjsPWiuHyN9je/HHFqVI8=; b=vADOPz9EtZAFEbcsP+G0QGu6727dXNrvOmBMoQm2IYRGIbke561zOMemnDHPewXTPn hcboHRisDqdIDjDJefr4eUoovB1L4o2bAD4zQsJ40WHH35ByZO8GeCfq0gjqcbkkOb+4 wJQf9Wz+YgFCov0o5Jjb6xQtPbW6O8cMQ453PjzimGf+queeR5j0bsAIWcxQUS90FXoo Wt+6WhoxPmTQNyfcCsosTz/GqdrWAQNjUslkuhcktxsyJwjZ8oQJRfwr/LTA7I2GO+P1 BHgO/5+ResL4DivWvfxPTwGjo7EJ8JtDKjFd7FK75RRy81F86xsX6bWJ1+bJB1c0Jzh2 nPyQ== X-Forwarded-Encrypted: i=1; AKwUvBwTHqohtGIICeivmuoSjb01Q2Hfw6rC7Y57tcI/ZmdrSWf4mT1ye28/wA625hFsbceA45I=@vger.kernel.org X-Gm-Message-State: AFuF++kOs0Dus1GcKSVCEyt+ahEYhkSG/1c8T1gDQyAnK3N/ie1h5gxa Xamdw0r5Gqso0CoPFS+1sNFUA7KjEVtKYdZRqkHIQMl3GY8ZRaFc8SSkM0sdvZL3VFM= X-Gm-Gg: AYBFou28tYH94JrgbWfLdhLZihcqxyUUF3tR/qKy0jOrzSGJWXjyu2EdWZg6stAkcva 2jQMTHz7SyfetP6qRV3dcsQtQKdO1bqo/2Hi1ZCmRNPl9OvYEd1m2oShtS0ej55kKY3D9J6Fe9B x0JgJJMgw/58oYaTH8ZMudb/z+EZpK7cHQ6bVWz4TiBat0c3eBr55S7rkPE440kkwbb5RsF3yvh 8mdB7L8B4mEVK6UchpTxphzWXHMgydJquOB2pshAi4uwVfapsfgdMN0eCwbVYjLZjaV8ASrz3lX 9n4XWYvBq7AHQFP+JPylbQhblopBO8HRp7qk9ieVj2cm+VnMNz/nh0iTpdCA7cUg/iLgE2SETSK BlbGEzIbmBP2QCQ5d2Pr80y8fHfqI7oHjtU7K8WLMgpakC9g0an4U2qqOwF0TyHZOZa3nDJ50w5 FQtv6KUfA5fUuKIunJX5rmjr4ydh5HZZ3kV1bzYUKLxPEaydR34+X/oBWSXkVytHzZTQ== X-Received: by 2002:a05:600c:a013:b0:49d:2555:1a6d with SMTP id 5b1f17b1804b1-49fe66f1ac3mr7346185e9.16.1790201505557; Wed, 23 Sep 2026 15:11:45 -0700 (PDT) Received: from localhost ([2620:10d:c090:600::1:b92a]) by smtp.gmail.com with ESMTPSA id 5b1f17b1804b1-49fe5dfa6a7sm26956515e9.12.2026.09.23.15.11.38 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Wed, 23 Sep 2026 15:11:44 -0700 (PDT) Precedence: bulk X-Mailing-List: bpf@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: Mime-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset=UTF-8 Date: Wed, 23 Sep 2026 22:11:37 +0000 Message-Id: Cc: "Yonghong Song" , "John Fastabend" , "Stanislav Fomichev" , "Eric Dumazet" , "Neal Cardwell" , "Willem de Bruijn" , "Tenzin Ukyab" , =?utf-8?q?Cl=C3=A9ment_L=C3=A9ger?= , "Kuniyuki Iwashima" , , Subject: Re: [PATCH v2 bpf-next 2/8] selftest: bpf: Use BPF_SOCK_OPS_ALL_CB_FLAGS + 1 for bad_cb_test_rv. From: "Emil Tsalapatis" To: "Kuniyuki Iwashima" , "Alexei Starovoitov" , "Daniel Borkmann" , "Andrii Nakryiko" , "Martin KaFai Lau" , "Eduard Zingerman" , "Kumar Kartikeya Dwivedi" X-Mailer: aerc 0.21.0 References: <20260923213719.224838-1-kuniyu@google.com> <20260923213719.224838-3-kuniyu@google.com> In-Reply-To: <20260923213719.224838-3-kuniyu@google.com> On Wed Sep 23, 2026 at 9:35 PM UTC, Kuniyuki Iwashima wrote: > Once bpf_sock_ops_cb_flags_set() supports a new flag, > tcpbpf_user.c fails due to the hard-coded max value, 0x80. > > Let's replace 0x80 with BPF_SOCK_OPS_ALL_CB_FLAGS + 1. > > Signed-off-by: Kuniyuki Iwashima > Acked-by: Stanislav Fomichev Reviewed-by: Emil Tsalapatis > --- > tools/testing/selftests/bpf/prog_tests/tcpbpf_user.c | 3 ++- > tools/testing/selftests/bpf/progs/test_tcpbpf_kern.c | 3 ++- > 2 files changed, 4 insertions(+), 2 deletions(-) > > diff --git a/tools/testing/selftests/bpf/prog_tests/tcpbpf_user.c b/tools= /testing/selftests/bpf/prog_tests/tcpbpf_user.c > index 7e8fe1bad03f..e4849d2a2956 100644 > --- a/tools/testing/selftests/bpf/prog_tests/tcpbpf_user.c > +++ b/tools/testing/selftests/bpf/prog_tests/tcpbpf_user.c > @@ -26,7 +26,8 @@ static void verify_result(struct tcpbpf_globals *result= ) > ASSERT_EQ(result->bytes_acked, 1002, "bytes_acked"); > ASSERT_EQ(result->data_segs_in, 1, "data_segs_in"); > ASSERT_EQ(result->data_segs_out, 1, "data_segs_out"); > - ASSERT_EQ(result->bad_cb_test_rv, 0x80, "bad_cb_test_rv"); > + ASSERT_EQ(result->bad_cb_test_rv, BPF_SOCK_OPS_ALL_CB_FLAGS + 1, > + "bad_cb_test_rv"); > ASSERT_EQ(result->good_cb_test_rv, 0, "good_cb_test_rv"); > ASSERT_EQ(result->num_listen, 1, "num_listen"); > =20 > diff --git a/tools/testing/selftests/bpf/progs/test_tcpbpf_kern.c b/tools= /testing/selftests/bpf/progs/test_tcpbpf_kern.c > index 6935f32eeb8f..e30cb1fab079 100644 > --- a/tools/testing/selftests/bpf/progs/test_tcpbpf_kern.c > +++ b/tools/testing/selftests/bpf/progs/test_tcpbpf_kern.c > @@ -92,7 +92,8 @@ int bpf_testcb(struct bpf_sock_ops *skops) > break; > case BPF_SOCK_OPS_ACTIVE_ESTABLISHED_CB: > /* Test failure to set largest cb flag (assumes not defined) */ > - global.bad_cb_test_rv =3D bpf_sock_ops_cb_flags_set(skops, 0x80); > + global.bad_cb_test_rv =3D bpf_sock_ops_cb_flags_set(skops, > + BPF_SOCK_OPS_ALL_CB_FLAGS + 1); > /* Set callback */ > global.good_cb_test_rv =3D bpf_sock_ops_cb_flags_set(skops, > BPF_SOCK_OPS_STATE_CB_FLAG);