Similar to "mptcp: sched: check both directions for backup": it didn't
matter before because the two flags were very likely set by accident.
But that's no longer the case now.
Signed-off-by: Matthieu Baerts (NGI0) <matttbe@kernel.org>
---
tools/testing/selftests/bpf/progs/mptcp_bpf_bkup.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/tools/testing/selftests/bpf/progs/mptcp_bpf_bkup.c b/tools/testing/selftests/bpf/progs/mptcp_bpf_bkup.c
index 14091bcc4cde..296f0318d843 100644
--- a/tools/testing/selftests/bpf/progs/mptcp_bpf_bkup.c
+++ b/tools/testing/selftests/bpf/progs/mptcp_bpf_bkup.c
@@ -29,7 +29,8 @@ int BPF_PROG(bpf_bkup_get_subflow, struct mptcp_sock *msk,
if (!subflow)
break;
- if (!BPF_CORE_READ_BITFIELD_PROBED(subflow, backup)) {
+ if (!BPF_CORE_READ_BITFIELD_PROBED(subflow, backup) ||
+ !BPF_CORE_READ_BITFIELD_PROBED(subflow, request_bkup)) {
nr = i;
break;
}
--
2.45.2