* [PATCH net-next] ipv6: lower "link become ready"'s level message
@ 2023-06-02 9:36 Matthieu Baerts
2023-06-02 11:19 ` ipv6: lower "link become ready"'s level message: Tests Results MPTCP CI
` (3 more replies)
0 siblings, 4 replies; 5+ messages in thread
From: Matthieu Baerts @ 2023-06-02 9:36 UTC (permalink / raw)
To: mptcp, David S. Miller, Eric Dumazet, Jakub Kicinski,
Paolo Abeni, David Ahern, Mat Martineau, Stephen Hemminger
Cc: Hideaki YOSHIFUJI, netdev, linux-kernel, Matthieu Baerts
This following message is printed in the console each time a network
device configured with an IPv6 addresses is ready to be used:
ADDRCONF(NETDEV_CHANGE): <iface>: link becomes ready
When netns are being extensively used -- e.g. by re-creating netns' with
veth to discuss with each others for testing purposes like mptcp_join.sh
selftest does -- it generates a lot of messages like that: more than 700
when executing mptcp_join.sh with the latest version.
It looks like this message is not that helpful after all: maybe it can
be used as a sign to know if there is something wrong, e.g. if a device
is being regularly reconfigured by accident? But even then, there are
better ways to monitor and diagnose such issues.
When looking at commit 3c21edbd1137 ("[IPV6]: Defer IPv6 device
initialization until the link becomes ready.") which introduces this new
message, it seems it had been added to verify that the new feature was
working as expected. It could have then used a lower level than "info"
from the beginning but it was fine like that back then: 17 years ago.
It seems then OK today to simply lower its level, similar to commit
7c62b8dd5ca8 ("net/ipv6: lower the level of "link is not ready" messages")
and as suggested by Mat [1], Stephen and David [2].
Link: https://lore.kernel.org/mptcp/614e76ac-184e-c553-af72-084f792e60b0@kernel.org/T/ [1]
Link: https://lore.kernel.org/netdev/68035bad-b53e-91cb-0e4a-007f27d62b05@tessares.net/T/ [2]
Suggested-by: Mat Martineau <martineau@kernel.org>
Suggested-by: Stephen Hemminger <stephen@networkplumber.org>
Suggested-by: David Ahern <dsahern@kernel.org>
Signed-off-by: Matthieu Baerts <matthieu.baerts@tessares.net>
---
RFC to v1:
- Thanks to Stephen and David's feedback, this new version is smaller
and simpler: only the level of the problematic message is changed, no
more sysctl.
- Link to RFC: https://lore.kernel.org/r/20230601-net-next-skip_print_link_becomes_ready-v1-1-c13e64c14095@tessares.net
---
net/ipv6/addrconf.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/net/ipv6/addrconf.c b/net/ipv6/addrconf.c
index 3797917237d0..5479da08ef40 100644
--- a/net/ipv6/addrconf.c
+++ b/net/ipv6/addrconf.c
@@ -3633,8 +3633,8 @@ static int addrconf_notify(struct notifier_block *this, unsigned long event,
idev->if_flags |= IF_READY;
}
- pr_info("ADDRCONF(NETDEV_CHANGE): %s: link becomes ready\n",
- dev->name);
+ pr_debug("ADDRCONF(NETDEV_CHANGE): %s: link becomes ready\n",
+ dev->name);
run_pending = 1;
}
---
base-commit: a395b8d1c7c3a074bfa83b9759a4a11901a295c5
change-id: 20230601-net-next-skip_print_link_becomes_ready-5bc2655daa24
Best regards,
--
Matthieu Baerts <matthieu.baerts@tessares.net>
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: ipv6: lower "link become ready"'s level message: Tests Results
2023-06-02 9:36 [PATCH net-next] ipv6: lower "link become ready"'s level message Matthieu Baerts
@ 2023-06-02 11:19 ` MPTCP CI
2023-06-02 12:43 ` [PATCH net-next] ipv6: lower "link become ready"'s level message Eric Dumazet
` (2 subsequent siblings)
3 siblings, 0 replies; 5+ messages in thread
From: MPTCP CI @ 2023-06-02 11:19 UTC (permalink / raw)
To: Matthieu Baerts; +Cc: mptcp
Hi Matthieu,
Thank you for your modifications, that's great!
Our CI did some validations and here is its report:
- KVM Validation: normal (except selftest_mptcp_join):
- Unstable: 1 failed test(s): packetdrill_fastopen 🔴:
- Task: https://cirrus-ci.com/task/4819198161977344
- Summary: https://api.cirrus-ci.com/v1/artifact/task/4819198161977344/summary/summary.txt
- KVM Validation: debug (except selftest_mptcp_join):
- Unstable: 4 failed test(s): packetdrill_fastopen packetdrill_mp_join packetdrill_sockopts selftest_diag 🔴:
- Task: https://cirrus-ci.com/task/5382148115398656
- Summary: https://api.cirrus-ci.com/v1/artifact/task/5382148115398656/summary/summary.txt
- KVM Validation: debug (only selftest_mptcp_join):
- Success! ✅:
- Task: https://cirrus-ci.com/task/6508048022241280
- Summary: https://api.cirrus-ci.com/v1/artifact/task/6508048022241280/summary/summary.txt
- KVM Validation: normal (only selftest_mptcp_join):
- Success! ✅:
- Task: https://cirrus-ci.com/task/5945098068819968
- Summary: https://api.cirrus-ci.com/v1/artifact/task/5945098068819968/summary/summary.txt
Initiator: Patchew Applier
Commits: https://github.com/multipath-tcp/mptcp_net-next/commits/29577f44821e
If there are some issues, you can reproduce them using the same environment as
the one used by the CI thanks to a docker image, e.g.:
$ cd [kernel source code]
$ docker run -v "${PWD}:${PWD}:rw" -w "${PWD}" --privileged --rm -it \
--pull always mptcp/mptcp-upstream-virtme-docker:latest \
auto-debug
For more details:
https://github.com/multipath-tcp/mptcp-upstream-virtme-docker
Please note that despite all the efforts that have been already done to have a
stable tests suite when executed on a public CI like here, it is possible some
reported issues are not due to your modifications. Still, do not hesitate to
help us improve that ;-)
Cheers,
MPTCP GH Action bot
Bot operated by Matthieu Baerts (Tessares)
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH net-next] ipv6: lower "link become ready"'s level message
2023-06-02 9:36 [PATCH net-next] ipv6: lower "link become ready"'s level message Matthieu Baerts
2023-06-02 11:19 ` ipv6: lower "link become ready"'s level message: Tests Results MPTCP CI
@ 2023-06-02 12:43 ` Eric Dumazet
2023-06-02 14:43 ` ipv6: lower "link become ready"'s level message: Tests Results MPTCP CI
2023-06-05 23:16 ` [PATCH net-next] ipv6: lower "link become ready"'s level message Jakub Kicinski
3 siblings, 0 replies; 5+ messages in thread
From: Eric Dumazet @ 2023-06-02 12:43 UTC (permalink / raw)
To: Matthieu Baerts
Cc: mptcp, David S. Miller, Jakub Kicinski, Paolo Abeni, David Ahern,
Mat Martineau, Stephen Hemminger, Hideaki YOSHIFUJI, netdev,
linux-kernel
On Fri, Jun 2, 2023 at 11:36 AM Matthieu Baerts
<matthieu.baerts@tessares.net> wrote:
>
> This following message is printed in the console each time a network
> device configured with an IPv6 addresses is ready to be used:
>
> ADDRCONF(NETDEV_CHANGE): <iface>: link becomes ready
>
> When netns are being extensively used -- e.g. by re-creating netns' with
> veth to discuss with each others for testing purposes like mptcp_join.sh
> selftest does -- it generates a lot of messages like that: more than 700
> when executing mptcp_join.sh with the latest version.
>
> It looks like this message is not that helpful after all: maybe it can
> be used as a sign to know if there is something wrong, e.g. if a device
> is being regularly reconfigured by accident? But even then, there are
> better ways to monitor and diagnose such issues.
>
> When looking at commit 3c21edbd1137 ("[IPV6]: Defer IPv6 device
> initialization until the link becomes ready.") which introduces this new
> message, it seems it had been added to verify that the new feature was
> working as expected. It could have then used a lower level than "info"
> from the beginning but it was fine like that back then: 17 years ago.
>
> It seems then OK today to simply lower its level, similar to commit
> 7c62b8dd5ca8 ("net/ipv6: lower the level of "link is not ready" messages")
> and as suggested by Mat [1], Stephen and David [2].
>
> Link: https://lore.kernel.org/mptcp/614e76ac-184e-c553-af72-084f792e60b0@kernel.org/T/ [1]
> Link: https://lore.kernel.org/netdev/68035bad-b53e-91cb-0e4a-007f27d62b05@tessares.net/T/ [2]
> Suggested-by: Mat Martineau <martineau@kernel.org>
> Suggested-by: Stephen Hemminger <stephen@networkplumber.org>
> Suggested-by: David Ahern <dsahern@kernel.org>
> Signed-off-by: Matthieu Baerts <matthieu.baerts@tessares.net>
> ---
Reviewed-by: Eric Dumazet <edumazet@google.com>
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: ipv6: lower "link become ready"'s level message: Tests Results
2023-06-02 9:36 [PATCH net-next] ipv6: lower "link become ready"'s level message Matthieu Baerts
2023-06-02 11:19 ` ipv6: lower "link become ready"'s level message: Tests Results MPTCP CI
2023-06-02 12:43 ` [PATCH net-next] ipv6: lower "link become ready"'s level message Eric Dumazet
@ 2023-06-02 14:43 ` MPTCP CI
2023-06-05 23:16 ` [PATCH net-next] ipv6: lower "link become ready"'s level message Jakub Kicinski
3 siblings, 0 replies; 5+ messages in thread
From: MPTCP CI @ 2023-06-02 14:43 UTC (permalink / raw)
To: Matthieu Baerts; +Cc: mptcp
Hi Matthieu,
Thank you for your modifications, that's great!
Our CI did some validations and here is its report:
- KVM Validation: normal (except selftest_mptcp_join):
- Success! ✅:
- Task: https://cirrus-ci.com/task/5975182569897984
- Summary: https://api.cirrus-ci.com/v1/artifact/task/5975182569897984/summary/summary.txt
- KVM Validation: debug (except selftest_mptcp_join):
- Unstable: 3 failed test(s): packetdrill_fastopen packetdrill_mp_capable selftest_diag 🔴:
- Task: https://cirrus-ci.com/task/6538132523319296
- Summary: https://api.cirrus-ci.com/v1/artifact/task/6538132523319296/summary/summary.txt
- KVM Validation: normal (only selftest_mptcp_join):
- Success! ✅:
- Task: https://cirrus-ci.com/task/5412232616476672
- Summary: https://api.cirrus-ci.com/v1/artifact/task/5412232616476672/summary/summary.txt
- KVM Validation: debug (only selftest_mptcp_join):
- Unstable: 1 failed test(s): selftest_mptcp_join 🔴:
- Task: https://cirrus-ci.com/task/5941386143334400
- Summary: https://api.cirrus-ci.com/v1/artifact/task/5941386143334400/summary/summary.txt
Initiator: Patchew Applier
Commits: https://github.com/multipath-tcp/mptcp_net-next/commits/7662725df26d
If there are some issues, you can reproduce them using the same environment as
the one used by the CI thanks to a docker image, e.g.:
$ cd [kernel source code]
$ docker run -v "${PWD}:${PWD}:rw" -w "${PWD}" --privileged --rm -it \
--pull always mptcp/mptcp-upstream-virtme-docker:latest \
auto-debug
For more details:
https://github.com/multipath-tcp/mptcp-upstream-virtme-docker
Please note that despite all the efforts that have been already done to have a
stable tests suite when executed on a public CI like here, it is possible some
reported issues are not due to your modifications. Still, do not hesitate to
help us improve that ;-)
Cheers,
MPTCP GH Action bot
Bot operated by Matthieu Baerts (Tessares)
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH net-next] ipv6: lower "link become ready"'s level message
2023-06-02 9:36 [PATCH net-next] ipv6: lower "link become ready"'s level message Matthieu Baerts
` (2 preceding siblings ...)
2023-06-02 14:43 ` ipv6: lower "link become ready"'s level message: Tests Results MPTCP CI
@ 2023-06-05 23:16 ` Jakub Kicinski
3 siblings, 0 replies; 5+ messages in thread
From: Jakub Kicinski @ 2023-06-05 23:16 UTC (permalink / raw)
To: Matthieu Baerts
Cc: mptcp, David S. Miller, Eric Dumazet, Paolo Abeni, David Ahern,
Mat Martineau, Stephen Hemminger, Hideaki YOSHIFUJI, netdev,
linux-kernel
On Fri, 02 Jun 2023 11:36:07 +0200 Matthieu Baerts wrote:
> This following message is printed in the console each time a network
> device configured with an IPv6 addresses is ready to be used:
>
> ADDRCONF(NETDEV_CHANGE): <iface>: link becomes ready
>
> When netns are being extensively used -- e.g. by re-creating netns' with
> veth to discuss with each others for testing purposes like mptcp_join.sh
> selftest does -- it generates a lot of messages like that: more than 700
> when executing mptcp_join.sh with the latest version.
>
> It looks like this message is not that helpful after all: maybe it can
> be used as a sign to know if there is something wrong, e.g. if a device
> is being regularly reconfigured by accident? But even then, there are
> better ways to monitor and diagnose such issues.
>
> When looking at commit 3c21edbd1137 ("[IPV6]: Defer IPv6 device
> initialization until the link becomes ready.") which introduces this new
> message, it seems it had been added to verify that the new feature was
> working as expected. It could have then used a lower level than "info"
> from the beginning but it was fine like that back then: 17 years ago.
>
> It seems then OK today to simply lower its level, similar to commit
> 7c62b8dd5ca8 ("net/ipv6: lower the level of "link is not ready" messages")
> and as suggested by Mat [1], Stephen and David [2].
>
> Link: https://lore.kernel.org/mptcp/614e76ac-184e-c553-af72-084f792e60b0@kernel.org/T/ [1]
> Link: https://lore.kernel.org/netdev/68035bad-b53e-91cb-0e4a-007f27d62b05@tessares.net/T/ [2]
> Suggested-by: Mat Martineau <martineau@kernel.org>
> Suggested-by: Stephen Hemminger <stephen@networkplumber.org>
> Suggested-by: David Ahern <dsahern@kernel.org>
> Signed-off-by: Matthieu Baerts <matthieu.baerts@tessares.net>
This appears to have been applied as commit f69de8aa4752 ("ipv6: lower
"link become ready"'s level message") in net-next, thank you!
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2023-06-05 23:16 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-06-02 9:36 [PATCH net-next] ipv6: lower "link become ready"'s level message Matthieu Baerts
2023-06-02 11:19 ` ipv6: lower "link become ready"'s level message: Tests Results MPTCP CI
2023-06-02 12:43 ` [PATCH net-next] ipv6: lower "link become ready"'s level message Eric Dumazet
2023-06-02 14:43 ` ipv6: lower "link become ready"'s level message: Tests Results MPTCP CI
2023-06-05 23:16 ` [PATCH net-next] ipv6: lower "link become ready"'s level message Jakub Kicinski
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).