On 22/06/2023 19:13, Greg KH wrote:
On Thu, Jun 22, 2023 at 04:06:22PM +0200, Matthieu Baerts wrote:
Hi Greg,
On 22/06/2023 11:19, Greg KH wrote:
On Thu, Jun 22, 2023 at 11:08:52AM +0200, Matthieu Baerts wrote:
commit dc97251bf0b70549c76ba261516c01b8096771c5 upstream.
Selftests are supposed to run on any kernels, including the old ones not supporting all MPTCP features.
One of them is the listen diag dump support introduced by commit 4fa39b701ce9 ("mptcp: listen diag dump support").
It looks like there is no good pre-check to do here, i.e. dedicated function available in kallsyms. Instead, we try to get info if nothing is returned, the test is marked as skipped.
That's not ideal because something could be wrong with the feature and instead of reporting an error, the test could be marked as skipped. If we know in advanced that the feature is supposed to be supported, the tester can set SELFTESTS_MPTCP_LIB_EXPECT_ALL_FEATURES env var to 1: in this case the test will report an error instead of marking the test as skipped if nothing is returned.
Link: https://github.com/multipath-tcp/mptcp_net-next/issues/368 Fixes: f2ae0fa68e28 ("selftests/mptcp: add diag listen tests") Cc: stable@vger.kernel.org Signed-off-by: Matthieu Baerts matthieu.baerts@tessares.net Signed-off-by: Jakub Kicinski kuba@kernel.org Signed-off-by: Matthieu Baerts matthieu.baerts@tessares.net
Applied on top of stable-rc/linux-6.1.y: 639ecee7e0d3 ("Linux 6.1.36-rc1") Conflicting with commit e04a30f78809 ("selftest: mptcp: add test for mptcp socket in use"): modifications around __chk_msk_nr() have been included here.
tools/testing/selftests/net/mptcp/diag.sh | 47 ++++++++++++----------- 1 file changed, 24 insertions(+), 23 deletions(-)
Now queued up, thanks.
Thank you for having already queued this patch and all the other ones from Linus' tree!
I just sent the last patches fixing conflicts in v5.10. I don't have any others linked to MPTCP and I replied to the ones that don't need to be backported to older versions than v6.1.
Thanks, I think I got them all now!
Yes, I think you did!
Thank you very much!
Cheers, Matt