From owner-dev-commits-src-main@freebsd.org Tue Apr 27 11:38:44 2021 Return-Path: Delivered-To: dev-commits-src-main@mailman.nyi.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2610:1c1:1:606c::19:1]) by mailman.nyi.freebsd.org (Postfix) with ESMTP id 417D05E46DD; Tue, 27 Apr 2021 11:38:44 +0000 (UTC) (envelope-from kp@FreeBSD.org) Received: from smtp.freebsd.org (smtp.freebsd.org [IPv6:2610:1c1:1:606c::24b:4]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256 client-signature RSA-PSS (4096 bits) client-digest SHA256) (Client CN "smtp.freebsd.org", Issuer "R3" (verified OK)) by mx1.freebsd.org (Postfix) with ESMTPS id 4FV0C81QxMz3rmK; Tue, 27 Apr 2021 11:38:44 +0000 (UTC) (envelope-from kp@FreeBSD.org) Received: from venus.codepro.be (venus.codepro.be [5.9.86.228]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (Client CN "mx1.codepro.be", Issuer "R3" (verified OK)) (Authenticated sender: kp) by smtp.freebsd.org (Postfix) with ESMTPSA id 091FD2A7F7; Tue, 27 Apr 2021 11:38:44 +0000 (UTC) (envelope-from kp@FreeBSD.org) Received: by venus.codepro.be (Postfix, authenticated sender kp) id 90F1B478A3; Tue, 27 Apr 2021 13:38:42 +0200 (CEST) From: "Kristof Provost" To: "Alexander V. Chernikov" , "Li-Wen Hsu" Cc: src-committers@FreeBSD.org, dev-commits-src-all@FreeBSD.org, dev-commits-src-main@FreeBSD.org Subject: Re: git: 5d1403a79a3e - main - [rtsock] Enforce netmask/RTF_HOST consistency. Date: Tue, 27 Apr 2021 13:38:41 +0200 X-Mailer: MailMate (1.13.2r5673) Message-ID: In-Reply-To: <202104242303.13ON3w5l066307@gitrepo.freebsd.org> References: <202104242303.13ON3w5l066307@gitrepo.freebsd.org> MIME-Version: 1.0 Content-Type: text/plain; format=flowed Content-Transfer-Encoding: quoted-printable X-Content-Filtered-By: Mailman/MimeDel 2.1.34 X-BeenThere: dev-commits-src-main@freebsd.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: Commit messages for the main branch of the src repository List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 27 Apr 2021 11:38:44 -0000 On 25 Apr 2021, at 1:03, Alexander V. Chernikov wrote: > The branch main has been updated by melifaro: > > URL: = > https://cgit.FreeBSD.org/src/commit/?id=3D5d1403a79a3e56403fb63c062252a= 23fce81e5f1 > > commit 5d1403a79a3e56403fb63c062252a23fce81e5f1 > Author: Alexander V. Chernikov > AuthorDate: 2021-04-23 21:53:47 +0000 > Commit: Alexander V. Chernikov > CommitDate: 2021-04-24 22:41:27 +0000 > > [rtsock] Enforce netmask/RTF_HOST consistency. > > 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. > > 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. > > Continue normalization efforts from D29826 and D29826 and ensure = > that > RTF_HOST flag always reflects host/network data from netmask = > field. > > Differential Revision: https://reviews.freebsd.org/D29958 > MFC after: 2 days I believe this commit triggers a number of regression test failures: = 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.) Best regards, Kristof From owner-dev-commits-src-main@freebsd.org Tue Apr 27 12:42:13 2021 Return-Path: Delivered-To: dev-commits-src-main@mailman.nyi.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2610:1c1:1:606c::19:1]) by mailman.nyi.freebsd.org (Postfix) with ESMTP id 5A0C25E7F59; Tue, 27 Apr 2021 12:42:13 +0000 (UTC) (envelope-from melifaro@ipfw.ru) Received: from forward500j.mail.yandex.net (forward500j.mail.yandex.net [IPv6:2a02:6b8:0:801:2::110]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (Client did not present a certificate) by mx1.freebsd.org (Postfix) with ESMTPS id 4FV1cP0l91z3wNv; Tue, 27 Apr 2021 12:42:11 +0000 (UTC) (envelope-from melifaro@ipfw.ru) Received: from vla1-840b729f3759.qloud-c.yandex.net (vla1-840b729f3759.qloud-c.yandex.net [IPv6:2a02:6b8:c0d:4d18:0:640:840b:729f]) by forward500j.mail.yandex.net (Yandex) with ESMTP id 1E31B11C5E7D; Tue, 27 Apr 2021 15:26:55 +0300 (MSK) Received: from mail.yandex.ru (mail.yandex.ru [2a02:8084:d6bb:4a80:59f1:7c2c:a252:2fd0]) by vla1-840b729f3759.qloud-c.yandex.net (mxback/Yandex) with HTTP id rQagYZCIoiE1-QrIOTUO0; Tue, 27 Apr 2021 15:26:54 +0300 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=ipfw.ru; s=mail; t=1619526414; bh=kPXpkFBYuJo3WIBi7cLtD5BTWuBuvdgI3eIul4YR7ME=; h=Message-Id:Cc:Subject:In-Reply-To:Date:References:To:From; b=ME+5kY+IG1VOUXDfMt6UajWUbODzyLnMxXbgDjZs4q/7xSbHvjOk24+wZrBGbWRhx 5Icvs1MOm0/8DYaeo+CFqOXHgHR1/XUTmCyag+Y5L3XR1G+i1utQHECIN9e+8lK+43 BYpxU6PA8Gpaq4JPLthBrsRfK/VD08u8ohVw5ugQ= Received: by vla1-3f9fbb467d83.qloud-c.yandex.net with HTTP; Tue, 27 Apr 2021 15:26:53 +0300 From: Alexander V. Chernikov Envelope-From: melifaro@ipfw.ru To: Li-Wen Hsu , Kristof Provost Cc: "src-committers@FreeBSD.org" , "dev-commits-src-all@FreeBSD.org" , "dev-commits-src-main@FreeBSD.org" In-Reply-To: References: <202104242303.13ON3w5l066307@gitrepo.freebsd.org> Subject: Re: git: 5d1403a79a3e - main - [rtsock] Enforce netmask/RTF_HOST consistency. X-Mailer: Yamail [ http://yandex.ru ] 5.0 Date: Tue, 27 Apr 2021 13:26:53 +0100 Message-Id: <11556361619526413@vla1-3f9fbb467d83.qloud-c.yandex.net> X-Rspamd-Queue-Id: 4FV1cP0l91z3wNv X-Spamd-Bar: ---- Authentication-Results: mx1.freebsd.org; none X-Spamd-Result: default: False [-4.00 / 15.00]; REPLY(-4.00)[] MIME-Version: 1.0 Content-Type: text/plain X-Content-Filtered-By: Mailman/MimeDel 2.1.34 X-BeenThere: dev-commits-src-main@freebsd.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: Commit messages for the main branch of the src repository List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 27 Apr 2021 12:42:13 -0000