only data need to be read here, it is enough
to use this technique. It is planned to use bpf_iter, when BPF programs
will be used to modify data from the different subflows.
mptcp_subflow_tcp_sock() and mptcp_for_each_stubflow() helpers have also
be imported.
Suggested-by: Martin KaFai Lau
Signed-
the MPTCP connection, while the
BPF program will see different options set per subflow of the same MPTCP
connection.
Closes: https://github.com/multipath-tcp/mptcp_net-next/issues/76
Signed-off-by: Geliang Tang
Reviewed-by: Mat Martineau
Signed-off-by: Matthieu Baerts (NGI0)
---
Notes:
- v2
MPTCP repo https://github.com/multipath-tcp/mptcp_net-next (the
"scripts" branch), and it has been adapted by Geliang.
Closes: https://github.com/multipath-tcp/mptcp_net-next/issues/76
Co-developed-by: Geliang Tang
Signed-off-by: Geliang Tang
Signed-off-by: Nicolas Rybowski
Reviewed
second patch.
Signed-off-by: Matthieu Baerts (NGI0)
---
Changes in v7:
- Patch 2/3: use 'can_loop' instead of 'cond_break'. (Martin)
- Patch 3/3: use bpf_program__attach_cgroup(). (Martin)
- Link to v6:
https://lore.kernel.org/r/20240911-upstream-bpf-next-20240506-mptcp-subflow
MPTCP repo https://github.com/multipath-tcp/mptcp_net-next (the
"scripts" branch), and it has been adapted by Geliang.
Closes: https://github.com/multipath-tcp/mptcp_net-next/issues/76
Co-developed-by: Geliang Tang
Signed-off-by: Geliang Tang
Signed-off-by: Nicolas Rybowski
Reviewed
the MPTCP connection, while the
BPF program will see different options set per subflow of the same MPTCP
connection.
Closes: https://github.com/multipath-tcp/mptcp_net-next/issues/76
Signed-off-by: Geliang Tang
Reviewed-by: Mat Martineau
Signed-off-by: Matthieu Baerts (NGI0)
---
Notes:
- v2
only data need to be read here,
it is enough to use this technique. It is planned to use bpf_iter, when
BPF programs will be used to modify data from the different subflows.
mptcp_subflow_tcp_sock() and mptcp_for_each_stubflow() helpers have also
be imported.
Suggested-by: Martin KaFai Lau
Signed-
second patch.
Signed-off-by: Matthieu Baerts (NGI0)
---
Changes in v6:
- Patch 3/3: use usleep() instead of sleep()
- Series: rebased on top of bpf-next/net
- Link to v5:
https://lore.kernel.org/r/20240910-upstream-bpf-next-20240506-mptcp-subflow-test-v5-0-2c664a7da...@kernel.org
Changes in v5
/testing/selftests \
TARGETS=net/mptcp \
install INSTALL_PATH=$KSFT_INSTALL_PATH
cd $KSFT_INSTALL_PATH
./run_kselftest.sh -c net/mptcp
Fixes: 1af3bc912eac ("selftests: mptcp: lib: use wait_local_port_listen helper")
Signed-off-by: Matthieu Baerts (NGI0)
---
too
=net/mptcp \
install INSTALL_PATH=$KSFT_INSTALL_PATH
cd $KSFT_INSTALL_PATH
./run_kselftest.sh -c net/mptcp
Fixes: f265d3119a29 ("selftests: mptcp: lib: use setup/cleanup_ns helpers")
Signed-off-by: Matthieu Baerts (NGI0)
---
tools/testing/selftests/net/mptcp/Makefile | 2
ut for the parent one which is linked anyway. So we can know in
advance what will be the expected behaviour, and add the new endpoint
only when it makes sense to do so.
Fixes: 4878f9f8421f ("selftests: mptcp: join: validate fullmesh endp on 1st sf")
Cc: sta...@vger.kernel.org
Signed-off-by: Ma
v6.11-rc1.
Signed-off-by: Matthieu Baerts (NGI0)
---
Matthieu Baerts (NGI0) (3):
selftests: mptcp: join: restrict fullmesh endp on 1st sf
selftests: mptcp: include lib.sh file
selftests: mptcp: include net_helper.sh file
tools/testing/selftests/net/mptcp/Makefile | 2
Hello,
On 10/09/2024 16:13, Matthieu Baerts (NGI0) wrote:
> From: Geliang Tang
>
> This patch adds a subtest named test_subflow in test_mptcp to load and
> verify the newly added MPTCP subflow BPF program. To goal is to make
> sure it is possible to set different socket optio
the MPTCP connection, while the
BPF program will see different options set per subflow of the same MPTCP
connection.
Closes: https://github.com/multipath-tcp/mptcp_net-next/issues/76
Signed-off-by: Geliang Tang
Reviewed-by: Mat Martineau
Signed-off-by: Matthieu Baerts (NGI0)
---
Notes:
- v2
only data need to be read here,
it is enough to use this technique. It is planned to use bpf_iter, when
BPF programs will be used to modify data from the different subflows.
mptcp_subflow_tcp_sock() and mptcp_for_each_stubflow() helpers have also
be imported.
Suggested-by: Martin KaFai Lau
Signed-
MPTCP repo https://github.com/multipath-tcp/mptcp_net-next (the
"scripts" branch), and it has been adapted by Geliang.
Closes: https://github.com/multipath-tcp/mptcp_net-next/issues/76
Co-developed-by: Geliang Tang
Signed-off-by: Geliang Tang
Signed-off-by: Nicolas Rybowski
Reviewed
second patch.
Signed-off-by: Matthieu Baerts (NGI0)
---
Changes in v5:
- See the individual changelog for more details about them
- Patch 1/3: set TCP on the 2nd subflow
- Patch 2/3: new
- Patch 3/3: use the BPF program from patch 2/3 to do the validation
instead of using ss.
- Link to
expecting
only one space around the directive delimiter (#).
While at it, change the variable name (result_msg) to something more
explicit.
Signed-off-by: Matthieu Baerts (NGI0)
---
tools/testing/selftests/net/mptcp/mptcp_connect.sh | 14 --
1 file changed, 8 insertions(+), 6 deletions
It doesn't need to be there, and it can cause some issues with TAP
parsers expecting only one space around the directive delimiter (#).
Signed-off-by: Matthieu Baerts (NGI0)
---
tools/testing/selftests/net/mptcp/diag.sh | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/
-by: Matthieu Baerts (NGI0)
---
tools/testing/selftests/net/mptcp/mptcp_connect.sh | 2 ++
tools/testing/selftests/net/mptcp/mptcp_join.sh| 3 ++-
tools/testing/selftests/net/mptcp/mptcp_sockopt.sh | 1 +
tools/testing/selftests/net/mptcp/pm_netlink.sh| 2 ++
tools/testing/selftests/net
It is now added by the MPTCP lib automatically, see the parent commit.
The time in the TAP output might be slightly different from the one
displayed before, but that's OK.
Reviewed-by: Mat Martineau
Signed-off-by: Matthieu Baerts (NGI0)
---
v2:
- Fixed typo in the commit message (
l.org/dev-tools/ktap.html [3]
Reviewed-by: Mat Martineau
Signed-off-by: Matthieu Baerts (NGI0)
---
tools/testing/selftests/net/mptcp/mptcp_lib.sh | 17 -
1 file changed, 16 insertions(+), 1 deletion(-)
diff --git a/tools/testing/selftests/net/mptcp/mptcp_lib.sh
b/tools/testing/self
proves the precision of the first subtests in all
MPTCP subtests.
Patches 4 and 5 remove duplicated spaces in TAP output, for the TAP
parsers that cannot handle them properly.
Link: https://github.com/linux-netdev/nipa/pull/36
Signed-off-by: Matthieu Baerts (NGI0)
---
Changes in v2:
- Typo in
Hi Jakub,
Thank you for your reply!
On 04/09/2024 21:40, Jakub Kicinski wrote:
> On Wed, 4 Sep 2024 18:15:09 +0200 Matthieu Baerts wrote:
>>> Best I could come up with is:
>>>
>>> diff --git a/contest/remote/vmksft-p.py b/contest/remote/vmksft-p.py
>>>
Hi Jakub,
On 04/09/2024 01:22, Jakub Kicinski wrote:
> On Mon, 02 Sep 2024 13:13:03 +0200 Matthieu Baerts (NGI0) wrote:
>> Patches here add 'time=ms' in the diagnostic data of the TAP output,
>> e.g.
>>
>> ok 1 - pm_netlink: defaults addr list # time=9ms
&
Hi David, Nico,
On 17/04/2021 06:24, David Gow wrote:
Hi Matt,
Like patch 1/6, I can apply it in MPTCP tree and send it later to
net-next with other patches.
Except if you guys prefer to apply it in KUnit tree and send it to
linux-next?
Given 1/6 is going to net-next, it makes sense to send
Hi David,
Thank you for your very clear reply!
On 15/04/2021 08:01, David Gow wrote:
On Wed, Apr 14, 2021 at 5:25 PM Matthieu Baerts
wrote:
Up to the KUnit maintainers to decide ;-)
To summarise my view: personally, I'd prefer things the way this patch
works: have everything e
Hi Nico,
On 14/04/2021 10:58, Nico Pache wrote:
Drop 'S' from end of CONFIG_MPTCP_KUNIT_TESTS inorder to adhear to the
KUNIT *_KUNIT_TEST config name format.
For MPTCP, we have multiple KUnit tests: crypto and token. That's why we
wrote TESTS with a S.
I'm fine without S if we need to stick
The following commit has been merged into the locking/core branch of tip:
Commit-ID: 7d95f22798ecea513f37b792b39fec4bcf20fec3
Gitweb:
https://git.kernel.org/tip/7d95f22798ecea513f37b792b39fec4bcf20fec3
Author:Matthieu Baerts
AuthorDate:Fri, 26 Mar 2021 11:50:23 +01:00
all: Align static_call_is_init() patching
condition")
Signed-off-by: Matthieu Baerts
---
Notes:
Feel free to modify this patch directly if you prefer. I can of course
send a new version if needed.
kernel/static_call.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(
Hi Chuck,
(+ cc: MPTCP list)
On 01/03/2021 15:52, Chuck Lever wrote:
On Mar 1, 2021, at 8:49 AM, syzbot
wrote:
(...)
syzbot found the following issue on:
(...)
Hi, thanks for the report.
Initial analysis:
c8e88e3aa738 ("NFSD: Replace READ* macros in nfsd4_decode_layoutget()"
changes
Hi Naresh,
On 19/11/2020 08:04, Naresh Kamboju wrote:
While running kselftest net/mptcp: mptcp_join.sh on x86_64 device running
linux next 20201118 tag the following warning was noticed.
Thank you for testing MPTCP and having reported this bug!
It looks like it is similar to what syzbot repor
Hi Randy,
On 16/11/2020 04:17, Randy Dunlap wrote:
The previous Kconfig patch led to some other build errors as
reported by the 0day bot and my own overnight build testing.
These are all in when KCOV is enabled but
SKB_EXTENSIONS is not enabled, so fix those by combining those conditions
in th
Hi Randy,
On 14/11/2020 02:11, Randy Dunlap wrote:
The previous Kconfig patch led to some other build errors as
reported by the 0day bot and my own overnight build testing.
Thank you for looking at that!
I had the same issue and I was going to propose a similar fix with one
small difference,
Hi Jakub,
09 Nov 2020 21:57:05 Jakub Kicinski :
> On Mon, 9 Nov 2020 17:28:54 +0100 Matthieu Baerts wrote:
>> A small detail (I think): the Signed-off-by of the sender (Geliang)
>> should be the last one in the list if I am not mistaken.
>> But I guess this is not blocki
Hi Geliang,
On 09/11/2020 14:59, Geliang Tang wrote:
This patch added NULL pointer check for mptcp_pm_alloc_anno_list, and
avoided similar static checker warnings in mptcp_pm_add_timer.
Signed-off-by: Geliang Tang
Reviewed-by: Dan Carpenter
I think Dan reviewed the v1 of your patch -- witho
detail (I think): the Signed-off-by of the sender (Geliang)
should be the last one in the list if I am not mistaken.
But I guess this is not blocking.
Reviewed-by: Matthieu Baerts
Cheers,
Matt
--
Tessares | Belgium | Hybrid Access Solutions
www.tessares.net
Hi Philip,
On 28/10/2020 00:00, Philip Li wrote:
On Tue, Oct 27, 2020 at 04:07:28PM +0100, Matthieu Baerts wrote:
FYI, this was already reported earlier:
https://github.com/multipath-tcp/mptcp_net-next/issues/102
Thanks for the info, we didn't notice this. We will take a look
of rep
Hi all,
On 27/10/2020 14:16, kernel test robot wrote:
Greeting,
FYI, we noticed the following commit (built with gcc-9):
commit: f2ff7f11f9a74842245db52d685bf9bc7ac2c4b1 ("selftests: mptcp: add ADD_ADDR
IPv6 test cases")
https://github.com/multipath-tcp/mptcp_net-next.git export
Thanks for
Hi Greg,
On 27/10/2020 14:53, Greg Kroah-Hartman wrote:
From: Matthieu Baerts
[ Upstream commit 287d35405989cfe0090e3059f7788dc531879a8d ]
Recently, CONFIG_MPTCP_IPV6 no longer selects CONFIG_IPV6. As a
consequence, if CONFIG_MPTCP_IPV6=y is added to the kconfig, it will no
longer ensure
on IPV6 instead of
selecting it")
Signed-off-by: Matthieu Baerts
---
tools/testing/selftests/net/mptcp/config | 1 +
1 file changed, 1 insertion(+)
diff --git a/tools/testing/selftests/net/mptcp/config
b/tools/testing/selftests/net/mptcp/config
index 8df5cb8f71ff..741a1c4f4ae8 100644
--
Hi Geert,
On 21/10/2020 11:52, Geert Uytterhoeven wrote:
Hi Matthieu,
On Wed, Oct 21, 2020 at 11:47 AM Matthieu Baerts
wrote:
On 21/10/2020 11:43, Geert Uytterhoeven wrote:
On Wed, Oct 21, 2020 at 5:56 AM Jakub Kicinski wrote:
On Tue, 20 Oct 2020 11:26:34 +0200 Matthieu Baerts wrote:
On
G_IPV6=m. But selecting IPV6 like we did
before was forcing it to be built-in while it was maybe not what the
user wants.
Reported-by: Geert Uytterhoeven
Fixes: 010b430d5df5 ("mptcp: MPTCP_IPV6 should depend on IPV6 instead of
selecting it")
Signed-off-by: Matthieu Baerts
---
Notes:
Fo
Hi Geert,
On 21/10/2020 11:43, Geert Uytterhoeven wrote:
Hi Jakub,
On Wed, Oct 21, 2020 at 5:56 AM Jakub Kicinski wrote:
On Tue, 20 Oct 2020 11:26:34 +0200 Matthieu Baerts wrote:
On 20/10/2020 09:38, Geert Uytterhoeven wrote:
MPTCP_IPV6 selects IPV6, thus enabling an optional feature the
Hi Jakub,
On 19/10/2020 22:40, Jakub Kicinski wrote:
On Mon, 19 Oct 2020 18:27:55 +0200 Matthieu Baerts wrote:
Hi Geliang,
On 19/10/2020 12:23, Geliang Tang wrote:
This patchset deals with initializations of mptcp_options_received's two
fields, ahmac and port.
Geliang Tang (2):
Hi Geert,
On 20/10/2020 09:40, Geert Uytterhoeven wrote:
On Mon, Oct 19, 2020 at 10:38 PM Geert Uytterhoeven
wrote:
On Mon, Oct 19, 2020 at 5:47 PM Matthieu Baerts
wrote:
On 19/10/2020 13:32, Geert Uytterhoeven wrote:
MPTCP_KUNIT_TESTS selects MPTCP, thus enabling an optional feature the
: Matthieu Baerts
Cheers,
Matt
--
Tessares | Belgium | Hybrid Access Solutions
www.tessares.net
ut a re-submit, here is my:
Reviewed-by: Matthieu Baerts
Also, if you don't mind and while I am here, I never know: is it OK for
you the maintainers to send one Acked/Reviewed-by for a whole series --
but then this is not reflected on patchwork -- or should we send one tag
for e
have an easy way to
enable all KUnit tests. We imitated what was and is still done in
fs/ext4/Kconfig.
But it probably makes sense to depend on MPTCP instead of selecting it.
So that's fine for me. But then please also send a patch to ext4
maintainer to do the same there.
Reviewed-by: Mat
Hi Rafael,
On 16/10/2020 19:15, Rafael J. Wysocki wrote:
On Friday, October 16, 2020 4:30:55 PM CEST Matthieu Baerts wrote:
By chance, do you already have a fix for that?
Can you please try the appended patch?
Thank you for the patch, this fixes the compilation warning I got.
Reported
Hi Rafael,
On 04/09/2020 19:24, Rafael J. Wysocki wrote:
From: "Rafael J. Wysocki"
The logical address of every GPE block in system memory must be
known before passing it to acpi_ev_initialize_gpe_block(), because
memory cannot be mapped on the fly from an interrupt handler.
Accordingly, the h
netns".
The commit 8b974778f998 ("selftests: mptcp: interpret \n as a new line")
is very similar to this one. But the modification in simult_flows.sh was
missed because this commit above was done in parallel to one here below.
Fixes: 1a418cb8e888 ("mptcp: simult flow self-tests&
(and read): the remaining bytes contained garbage.
Fixes: 6a6c05a8b016 ("mptcp: send out ADD_ADDR with echo flag")
Closes: https://github.com/multipath-tcp/mptcp_net-next/issues/95
Reported-and-tested-by: Davide Caratti
Acked-by: Geliang Tang
Signed-off-by: Matthieu Baerts
---
Hi David, Jakub,
On 21/09/2020 16:57, Matthieu Baerts wrote:
From: Mat Martineau
When receiving a DATA_FIN MPTCP option on a TCP FIN packet, the DATA_FIN
information would be stored but the MPTCP worker did not get
scheduled. In turn, the MPTCP socket state would remain in
TCP_ESTABLISHED and
://github.com/multipath-tcp/mptcp_net-next/issues/84
Fixes: 43b54c6ee382 ("mptcp: Use full MPTCP-level disconnect state machine")
Acked-by: Paolo Abeni
Signed-off-by: Mat Martineau
Signed-off-by: Matthieu Baerts
---
Notes:
This is a resend of v1 and v2 with the same code modification. The
8fbf241064 ("selftests: add test-cases for MPTCP MP_JOIN")
Fixes: 048d19d444be ("mptcp: add basic kselftest for mptcp")
Signed-off-by: Matthieu Baerts
---
Notes:
This commit improves the output in selftests in case of errors, mostly
seen when modifying MPTCP code.
Hi Colin,
On 15/09/2020 10:44, Colin King wrote:
From: Colin Ian King
There is a spelling mistake a pr_debug message. Fix it.
Thank you for the patch!
Reviewed-by: Matthieu Baerts
Now I don't know if it can be accepted if it was not sent to netdev ML.
You might have to re-submit i
dd netlink-based PM")
Fixes: ec3edaa7ca6ce ("mptcp: Add handling of outgoing MP_JOIN requests")
Fixes: f296234c98a8f ("mptcp: Add handling of incoming MP_JOIN requests")
Signed-off-by: Geliang Tang
Thank you for the v2!
Reviewed-by: Matthieu Baerts
Cheers,
Matt
--
we ignore this
address.
Fixes: 01cacb00b35cb ("mptcp: add netlink-based PM")
Signed-off-by: Geliang Tang
Thank you for the v2!
Reviewed-by: Matthieu Baerts
Cheers,
Matt
--
Tessares | Belgium | Hybrid Access Solutions
www.tessares.net
Hi Geliang,
On 07/09/2020 12:29, Geliang Tang wrote:
This patch set the init remote_id to zero, otherwise it will be a random
number.
Then it added the missing subflow's remote_id setting code both in
__mptcp_subflow_connect and in subflow_ulp_clone.
Thank you for this other patch!
Here as w
Hi Geliang,
On 07/09/2020 12:29, Geliang Tang wrote:
In mptcp_pm_nl_get_local_id, skc_local is the same as msk_local, so it
always return 0. Thus every subflow's local_id is 0. It's incorrect.
This patch fixed this issue.
Also, we need to ignore the zero address here, like 0.0.0.0 in IPv4. Whe
Hi Tim,
Thank you for having sent the patch!
On 06/08/2020 08:41, Tim Froidcoeur wrote:
Refactor the fastreuse update code in inet_csk_get_port into a small
helper function that can be called from other places.
(...)
diff --git a/net/ipv4/inet_connection_sock.c b/net/ipv4/inet_connection_so
other host didn't announce another one. This also
fixes inet_diag showing blank info about MPTCP sockets from unblocking
sockets doing a connect().
Fixes: 41be81a8d3d0 ("mptcp: fix unblocking connect()")
Signed-off-by: Matthieu Baerts
---
net/mptcp/protocol.c | 2 +-
1 file changed, 1
: Matthieu Baerts
Cheers,
Matt
--
Tessares | Belgium | Hybrid Access Solutions
www.tessares.net
Schmidt [ieee802154]
---
net/mptcp/protocol.c | 12 +++
Thank you for the v2!
For MPTCP-related code:
Acked-by: Matthieu Baerts
Cheers,
Matt
--
Tessares | Belgium | Hybrid Access Solutions
www.tessares.net
Hi Christoph,
On 20/07/2020 14:47, Christoph Hellwig wrote:
Rework the remaining setsockopt code to pass a sockptr_t instead of a
plain user pointer. This removes the last remaining set_fs(KERNEL_DS)
outside of architecture specific code.
Signed-off-by: Christoph Hellwig
---
...
diff --git
wig
---
include/linux/net.h | 6 --
include/net/sock.h | 4
net/core/sock.c | 30 ++
net/mptcp/protocol.c | 6 --
Thank you for looking at that!
For MPTCP-related code:
Acked-by: Matthieu Baerts
Cheers,
Matt
--
Tessares | Belg
79a56-X4O6gS-ns3-ns4-MPTCP-MPTCP-10.0.3.1-10030-listener.pcap
It was a connection from ns3 to ns4, better to start with ns3 then. The
port is also added, easier to find the trace we want.
Co-developed-by: Christoph Paasch
Signed-off-by: Christoph Paasch
Signed-off-by: Matthieu Baerts
---
ding to v1
spec")
Signed-off-by: Geliang Tang
Thank you for this v2. It looks good to me!
Reviewed-by: Matthieu Baerts
Cheers,
Matt
--
Tessares | Belgium | Hybrid Access Solutions
www.tessares.net
Hi Geliang,
On 18/06/2020 08:27, Geliang Tang wrote:
On Tue, Jun 16, 2020 at 05:18:56PM +0200, Matthieu Baerts wrote:
On 16/06/2020 08:47, Geliang Tang wrote:
>>
I would suggest you to discuss about that on MPTCP mailing list. We also
have meetings every Thursday. New devs are always w
Hi Geliang
On 16/06/2020 08:47, Geliang Tang wrote:
Add handling for sending and receiving the MP_PRIO, MP_FAIL, and
MP_FASTCLOSE suboptions.
Thank you for the patches!
Unfortunately, I don't think it would be wise to accept them now: for
the moment, these suboptions are ignored at the recep
ot;mptcp: unify MPTCP_PM_MAX_ADDR and MPTCP_PM_ADDR_MAX"
Thank you for this v2, it looks good to me!
Reviewed-by: Matthieu Baerts
Cheers,
Matt
--
Tessares | Belgium | Hybrid Access Solutions
www.tessares.net
Hi Geliang,
On 12/06/2020 07:27, Geliang Tang wrote:
Unify these two duplicate macros into 8.
Thank you for this new patch!
(...)
diff --git a/net/mptcp/protocol.h b/net/mptcp/protocol.h
index 809687d3f410..86d265500cf6 100644
--- a/net/mptcp/protocol.h
+++ b/net/mptcp/protocol.h
@@ -135,7
ang Tang
---
Changes in v2:
- Add "-net" subject and "Fixes" tag as Matt suggested.
Thanks for this v2! LGTM!
Reviewed-by: Matthieu Baerts
Cheers,
Matt
--
Matthieu Baerts | R&D Engineer
matthieu.bae...@tessares.net
Tessares SA | Hybrid Access Solutions
www.tessares.net
1 Avenue Jean Monnet, 1348 Louvain-la-Neuve, Belgium
Because this is a bug-fix for net, may you clearly indicate that in the
subject to help -net maintainers please? [PATCH net v2]
Also, may you add a "Fixes" tag as well as it is for -net ? I guess it
should be:
Fixes: 3df523ab582c ("mptcp: Add ADD_ADDR handling")
T
is to remove #ifdef CONFIG_NETDEVICES from net/core/filter.c
and rely on sock_bindtoindex() returning ENOPROTOOPT in the extreme
case of oddly configured kernels.
Fixes: 70c58997c1e8 ("bpf: Allow SO_BINDTODEVICE opt in bpf_setsockopt")
Suggested-by: Alexei Starovoitov
Signed-off-by:
Hi Alexei,
On 03/06/2020 20:14, Alexei Starovoitov wrote:
On Wed, Jun 03, 2020 at 11:12:01AM +0200, Matthieu Baerts wrote:
Hi Ferenc,
On 03/06/2020 10:56, Ferenc Fejes wrote:
Matthieu Baerts ezt írta (időpont:
2020. jún. 3., Sze, 10:11):
A recent commit added new variables only used if
Hi Ferenc,
On 03/06/2020 10:56, Ferenc Fejes wrote:
Matthieu Baerts ezt írta (időpont:
2020. jún. 3., Sze, 10:11):
A recent commit added new variables only used if CONFIG_NETDEVICES is
set.
Thank you for noticing and fixed this!
A simple fix is to only declare these variables if the same
these variables in the related "case" section.
Fixes: 70c58997c1e8 ("bpf: Allow SO_BINDTODEVICE opt in bpf_setsockopt")
Signed-off-by: Matthieu Baerts
---
Notes:
This fix currently applies on net-next and bpf-next only. Except that
net-next is now closed and -net w
"$err" is a variable pointing to a temp file. "$out" is not: only used
as a local variable in "check()" and representing the output of a
command line.
Fixes: eedbc685321b (selftests: add PM netlink functional tests)
Signed-off-by: Matthieu Baerts
---
too
to me, thank you for this!
Reviewed-by: Matthieu Baerts
--
Matthieu Baerts | R&D Engineer
matthieu.bae...@tessares.net
Tessares SA | Hybrid Access Solutions
www.tessares.net
1 Avenue Jean Monnet, 1348 Louvain-la-Neuve, Belgium
case a
subsubdir has the same name as another one in another subdirectory.
Fixes: 852c8cbf34d3 (selftests/kselftest/runner.sh: Add 45 second timeout per
test)
Signed-off-by: Matthieu Baerts
---
tools/testing/selftests/kselftest/runner.sh | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff -
ff.h:1406),
just before the BUG_ON() somehow suggests to add a check before using
it, what Tim did.
In master, this code path causing the issue will not be taken because
the implementation of tcp_rtx_queue_tail() is different:
tcp_fragment() → tcp_rtx_queue_tail() → tcp_write_queue_prev() →
)
> Signed-off-by: Eric Dumazet
> Reported-by: Andrew Prout
> Tested-by: Andrew Prout
> Tested-by: Jonathan Lemon
> Tested-by: Michal Kubecek
> Acked-by: Neal Cardwell
> Acked-by: Yuchung Cheng
> Acked-by: Christoph Paasch
> Cc: Jonathan Looney
> S
84 matches
Mail list logo