Messages in this thread | | | From | "Drewek, Wojciech" <> | Subject | RE: selftests: net: l2tp.sh regression starting with 6.1-rc1 | Date | Wed, 29 Mar 2023 14:16:37 +0000 |
| |
Hi,
Modifying UAPI was not a good idea although the patch should not break userspace (related discussion [1]). We could revert this patch with one additional change (include l2tp.h in net/sched/cls_flower.c) but then again, modifying UAPI. This patch was mostly cosmetic anyway. Second option is to try to fix the automatic load. I'm not an expert but I think MODULE_ALIAS_NET_PF_PROTO macro is somehow responsible for that. I noticed some comments saying that "__stringify doesn't like enums" (this macro is using _stringify) and my patch defined IPPROTO_L2TP in enum. We can just replace IPPROTO_L2TP with 115 (where this macro is used) in order to fix this. I'm going to give it a try and will let you know.
Regards, Wojtek
[1] https://lore.kernel.org/netdev/Ywis3PYhKiATnzXB@nanopsycho/
> -----Original Message----- > From: Andrea Righi <andrea.righi@canonical.com> > Sent: środa, 29 marca 2023 14:24 > To: Drewek, Wojciech <wojciech.drewek@intel.com> > Cc: David S. Miller <davem@davemloft.net>; Eric Dumazet <edumazet@google.com>; Jakub Kicinski <kuba@kernel.org>; Paolo Abeni > <pabeni@redhat.com>; Shuah Khan <shuah@kernel.org>; netdev@vger.kernel.org; linux-kselftest@vger.kernel.org; linux- > kernel@vger.kernel.org > Subject: selftests: net: l2tp.sh regression starting with 6.1-rc1 > > I noticed that l2tp.sh net selftest is failing in recent kernels with > the following error: > > RTNETLINK answers: Protocol not supported > > See also: https://bugs.launchpad.net/bugs/2013014 > > Apprently the module lt2p_ipv6 is not automatically loaded when the test > is trying to create an l2tp ipv6 tunnel. > > I did a bisect and found that the offending commit is this one: > > 65b32f801bfb ("uapi: move IPPROTO_L2TP to in.h") > > I've temporarily reverted this commit for now, any suggestion on how to > fix this properly? > > Thanks, > -Andrea
| |