From nobody Wed Sep 07 18:16:07 2022 X-Original-To: dev-commits-src-main@mlmmj.nyi.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2610:1c1:1:606c::19:1]) by mlmmj.nyi.freebsd.org (Postfix) with ESMTP id 4MN9Rt71YKz4bnkW; Wed, 7 Sep 2022 18:16:10 +0000 (UTC) (envelope-from bz@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 4MN9Rt6Tzbz3CTW; Wed, 7 Sep 2022 18:16:10 +0000 (UTC) (envelope-from bz@freebsd.org) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=freebsd.org; s=dkim; t=1662574570; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: in-reply-to:in-reply-to:references:references; bh=HVEm6vCVAIO4+zJ7072YvhNIyp5Fkpi3e2ZT3e3NvX0=; b=qg/4n/873D5BcVFx4sl4RM6DvPRpfAayIQCdh2VGNbJk5Y90142l/fdbkXJIplvsZ9LjKv QiwX9c0ZpcvXKuCEO6f7xF0r3xvLJRGOoUUp0ElycnCwfl1oaHQAps93KLAWdNorvP8w3u nO0hefwBA779zYpE2/vA+OqNIHRkM57ujMtd3OOuzlh8l0PivHXyLk2+Wlgnhfocl9b/u0 QW6pz1lyNAV7zlPs7gNHz2K6YBOssVLQkDYJrdFjU1yMTO6EAszgn3ZVHCs9nXfNEW5Vtg 7q4mimQ26QTs8vGi2uLhl6bp0eX7GXZhnHvylA3B0GZBmKKHnHEwiqpU3dGLqQ== Received: from mx1.sbone.de (cross.sbone.de [195.201.62.131]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (Client CN "mx1.sbone.de", Issuer "SBone.DE" (not verified)) (Authenticated sender: bz/mail) by smtp.freebsd.org (Postfix) with ESMTPSA id 4MN9Rt4LBHz15Kk; Wed, 7 Sep 2022 18:16:10 +0000 (UTC) (envelope-from bz@freebsd.org) Received: from mail.sbone.de (mail.sbone.de [IPv6:fde9:577b:c1a9:4902:0:7404:2:1025]) (using TLSv1.2 with cipher ADH-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mx1.sbone.de (Postfix) with ESMTPS id 58CB28D4A345; Wed, 7 Sep 2022 18:16:09 +0000 (UTC) Received: from content-filter.t4-02.sbone.de (content-filter.t4-02.sbone.de [IPv6:fde9:577b:c1a9:4902:0:7404:2:2742]) (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) (No client certificate requested) by mail.sbone.de (Postfix) with ESMTPS id B93385C3A832; Wed, 7 Sep 2022 18:16:08 +0000 (UTC) X-Virus-Scanned: amavisd-new at sbone.de Received: from mail.sbone.de ([IPv6:fde9:577b:c1a9:4902:0:7404:2:1025]) by content-filter.t4-02.sbone.de (content-filter.t4-02.sbone.de [IPv6:fde9:577b:c1a9:4902:0:7404:2:2742]) (amavisd-new, port 10024) with ESMTP id UbZEuofaiFLB; Wed, 7 Sep 2022 18:16:07 +0000 (UTC) Received: from strong-iwl0.sbone.de (strong-iwl0.sbone.de [IPv6:fde9:577b:c1a9:4902:b66b:fcff:fef3:e3d2]) (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) (No client certificate requested) by mail.sbone.de (Postfix) with ESMTPSA id A772C5C3A82F; Wed, 7 Sep 2022 18:16:07 +0000 (UTC) Date: Wed, 7 Sep 2022 18:16:07 +0000 (UTC) From: "Bjoern A. Zeeb" To: Gleb Smirnoff cc: src-committers@freebsd.org, dev-commits-src-all@freebsd.org, dev-commits-src-main@freebsd.org Subject: Re: git: e72c522858cb - main - divert(4): make it compilable and working without INET In-Reply-To: Message-ID: References: <202208302249.27UMnSwI022109@gitrepo.freebsd.org> <6q6n8553-2143-n24p-4816-406549997s@mnoonqbm.arg> <15on1poo-pnpq-3pos-qs7s-673pp1q0s83@serrofq.bet> <6r5o5nqn-9srp-79qr-n33-q4992936q51q@serrofq.bet> X-OpenPGP-Key-Id: 0x14003F198FEFA3E77207EE8D2B58B8F83CCF1842 List-Id: Commit messages for the main branch of the src repository List-Archive: https://lists.freebsd.org/archives/dev-commits-src-main List-Help: List-Post: List-Subscribe: List-Unsubscribe: Sender: owner-dev-commits-src-main@freebsd.org X-BeenThere: dev-commits-src-main@freebsd.org MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII; format=flowed ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=freebsd.org; s=dkim; t=1662574570; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: in-reply-to:in-reply-to:references:references; bh=HVEm6vCVAIO4+zJ7072YvhNIyp5Fkpi3e2ZT3e3NvX0=; b=Ng7XH/f17sjhwoS8xoce3vwg4aNX5oebOexSwheF9BXyi+ShVJqahAu0cwVPGbZtvJ8mSo nVA38biBzfuKlDaOyr3R+bVIbo4NIHnU1Z7k+0MlpxcaK8d/4HTFgkL+RSBP+Tf/fkmhTY S4s85FPP1525ogUCKNzL8t8sTqKBNn1PlwQsFc1WwjTGmpJsIdyCWRhoVXpR6xg5tp0w9+ MZdrvxM0kMRnkoA0py7A7Q/Eub2Mes9DpzYREMorQaYqxEfwSziH9fWhpP8NKBhgkQY1/y BcyLTCrEQVZrcWO+dFaEH6nRGB7xPPR8Gt85QBJmSx+uyuORtSZF9xrulondfw== ARC-Seal: i=1; s=dkim; d=freebsd.org; t=1662574570; a=rsa-sha256; cv=none; b=sKni6RZ+Q/snUHhBGtsVm9jJhJFZjhRX3078+G3ngsSnI5pIg/USPP700l8getmOuZi/Y0 9NYRPPWRL6/GV2oOXnuqvsimGCmxeFfrMvimiDxQNGS0X8R7+w4/fjNxBXdfEXDw3+q6fK hPZx/gTvq6veGKDli/aQI/CFfSMH1eyt7/HZAvpgeXO7jSzqWDZtXM95pGckF3OyyueEQy UWXlNAPp40xWaad5INJADUQ0oTaKF1PTy3A9enJndqw5+JCUZSbcy9SznZPAtI31LbcRiT viXhbR/RA+oL+2pvw0NBddhKvxo6/sZFNxQSLfbIxPy91h4uDRWVUHGRZrj+Lw== ARC-Authentication-Results: i=1; mx1.freebsd.org; none X-ThisMailContainsUnwantedMimeParts: N On Wed, 7 Sep 2022, Gleb Smirnoff wrote: > On Wed, Sep 07, 2022 at 09:54:49AM +0000, Bjoern A. Zeeb wrote: > B> On Tue, 6 Sep 2022, Gleb Smirnoff wrote: > B> > B> > On Sun, Sep 04, 2022 at 06:39:53AM +0000, Bjoern A. Zeeb wrote: > B> > B> > B> > Author: Gleb Smirnoff > B> > B> > B> > AuthorDate: 2022-08-30 22:09:21 +0000 > B> > B> > B> > Commit: Gleb Smirnoff > B> > B> > B> > CommitDate: 2022-08-30 22:09:21 +0000 > B> > B> > B> > > B> > B> > B> > divert(4): make it compilable and working without INET > B> > B> > B> > > B> > B> > B> > Differential revision: https://reviews.freebsd.org/D36383 > B> > B> > B> > B> > B> > B> Well, almost at least.... > B> > B> > > B> > B> > I wonder why LINT-NOINET builds without this?? > B> > B> > B> > B> I wondered the same last night.. I would be good to find out? > B> > > B> > Ok, I think that assumption that sysctl parts of divert require > B> > INET were just wrong, that's why LINT-NOINET builds. > B> > B> Not entirely. It's because sys/netinet/in_proto.c uncodonitionally > B> currently provides _net_inet for the sysctl. > B> The reason for that is that a lot of AF independent sysctls were initially > B> put under inet but are equally used for inet6 or neither in the network stack. > B> > B> You will need at least #if defined(INET6) || defined(INET) around the sysctls > B> in divert.c then. > B> > B> A LINT-NOIP kernel will complain about the missing symbol if we keep compiling > B> divert without INET or INET6 in the kernel: > > My goal was not to make divert(4) supported on NOIP kernel. The goal was to make > it supported on NOINET, an IPv6 only kernel. I don't see any value for divert(4) > on a kernel without either stack. then we should stop compiling it in that case. -- Bjoern A. Zeeb r15:7