Date: Tue, 27 Apr 2021 14:53:19 +0100 From: "Alexander V. Chernikov" <melifaro@ipfw.ru> To: "Alexander V. Chernikov" <melifaro@freebsd.org> Cc: Li-Wen Hsu <lwhsu@freebsd.org>, Kristof Provost <kp@freebsd.org>, "src-committers@FreeBSD.org" <src-committers@freebsd.org>, "dev-commits-src-all@FreeBSD.org" <dev-commits-src-all@freebsd.org>, "dev-commits-src-main@FreeBSD.org" <dev-commits-src-main@freebsd.org> Subject: Re: git: 5d1403a79a3e - main - [rtsock] Enforce netmask/RTF_HOST consistency. Message-ID: <2F6FDB4D-37D8-4161-98B1-95BB6AE924C0@ipfw.ru> In-Reply-To: <11556361619526413@vla1-3f9fbb467d83.qloud-c.yandex.net> References: <E9696582-EC46-4C3A-B339-F6421564161E@FreeBSD.org> <202104242303.13ON3w5l066307@gitrepo.freebsd.org> <11556361619526413@vla1-3f9fbb467d83.qloud-c.yandex.net>
next in thread | previous in thread | raw e-mail | index | archive | help
> On 27 Apr 2021, at 13:26, Alexander V. Chernikov = <melifaro@freebsd.org> wrote: >=20 > Ack. Interesting. Will look later today. Thank you for bisecting it - = thought the issue was caused by a different change. Looks like there is no more failures in = https://ci.freebsd.org/job/FreeBSD-main-amd64-test/18039/#showFailuresLink= . In fact, I think = https://cgit.freebsd.org/src/commit/?id=3D439d087d0b55574db81f4a2799a411c1= 236d95e3 is the reason. >=20 > -- > Sent from Yandex.Mail for mobile >=20 > 27.04.2021, 12:38, "Kristof Provost" <kp@freebsd.org>: > On 25 Apr 2021, at 1:03, Alexander V. Chernikov wrote: >=20 > The branch main has been updated by melifaro: >=20 > URL: = https://cgit.FreeBSD.org/src/commit/?id=3D5d1403a79a3e56403fb63c062252a23f= ce81e5f1 = <https://cgit.freebsd.org/src/commit/?id=3D5d1403a79a3e56403fb63c062252a23= fce81e5f1> >=20 > commit 5d1403a79a3e56403fb63c062252a23fce81e5f1 > Author: Alexander V. Chernikov <melifaro@FreeBSD.org = <mailto:melifaro@FreeBSD.org>> > AuthorDate: 2021-04-23 21:53:47 +0000 > Commit: Alexander V. Chernikov <melifaro@FreeBSD.org = <mailto:melifaro@FreeBSD.org>> > CommitDate: 2021-04-24 22:41:27 +0000 >=20 > [rtsock] Enforce netmask/RTF_HOST consistency. >=20 > Traditionally we had 2 sources of information whether the > added/delete route request targets network or a host route: > netmask (RTA_NETMASK) and RTF_HOST flag. >=20 > The former one is tricky: netmask can be empty or can explicitly > specify the host netmask. Parsing netmask sockaddr requires per-family > parsing and that's what rtsock code traditionally avoided. As a = result, > consistency was not enforced and it was possible to specify network = with > the RTF_HOST flag and vice versa. >=20 > Continue normalization efforts from D29826 and D29826 and ensure that > RTF_HOST flag always reflects host/network data from netmask field. >=20 > Differential Revision: https://reviews.freebsd.org/D29958 = <https://reviews.freebsd.org/D29958>; > MFC after: 2 days >=20 > I believe this commit triggers a number of regression test failures: = https://ci.freebsd.org/job/FreeBSD-main-amd64-test/18011/#showFailuresLink= = <https://ci.freebsd.org/job/FreeBSD-main-amd64-test/18011/#showFailuresLin= k> > The easiest way to reproduce the problem is to kldload ipsec ; cd = /usr/tests/sys/netipsec ; kyua test. > Reverting this patch allows the tests to pass again. (There are a few = others that also fail, but the ipsec tests appear to be the most = reliable way to demonstrate the problem.) >=20 > Best regards, > Kristof >=20
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?2F6FDB4D-37D8-4161-98B1-95BB6AE924C0>