From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 1454371470 for ; Mon, 18 Dec 2023 22:24:26 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="AYfW+woI" Received: by smtp.kernel.org (Postfix) with ESMTPSA id 5038CC433C7; Mon, 18 Dec 2023 22:24:26 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1702938266; bh=ANYkOI2y+y0ieA3uw4Cj7QxgqGQuSFlxWwFAdyD6ZSs=; h=Date:From:To:Cc:Subject:In-Reply-To:References:From; b=AYfW+woIPc25C8ojuCBOElQgB6PuY6b1QdygEMABSEZFyULnih7QisrEpdzl7frHO 4eBz+4qSWh5b0QnoVs8hvSZQH5Ma7hn/V7YyWqDY0lWKi94Twnvqp/Qj3og66LJheF TUjdnk0/jV/9XVLaktJkhXFt0yZSHpDVBZ70pwiT67xhWlNiOB9/eet2q4OargG4Ee tr7HHE/rTXYlKGKr/2ts5JkT20EmFN6A1yH9A2/TuDn9MkrvlbDo9UrWBfXsmBUgAs aGZOZZQ5C3ytWKra1micrCLzLK1062JPnb/Ixn4cZM4rVAKhDvO+4abt/lAjwXWSdx 21dDg64nT+uwQ== Date: Mon, 18 Dec 2023 14:24:25 -0800 From: Jakub Kicinski To: Hangbin Liu Cc: netdev@vger.kernel.org, Paolo Abeni , Davide Caratti Subject: Re: [PATCH net-next 3/3] netlink: specs: use exact-len for IPv6 addr Message-ID: <20231218142425.257a049f@kernel.org> In-Reply-To: References: <20231215035009.498049-1-liuhangbin@gmail.com> <20231215035009.498049-4-liuhangbin@gmail.com> <20231215180911.414b76d3@kernel.org> Precedence: bulk X-Mailing-List: netdev@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit On Sat, 16 Dec 2023 16:48:01 +0800 Hangbin Liu wrote: > On Fri, Dec 15, 2023 at 06:09:11PM -0800, Jakub Kicinski wrote: > > On Fri, 15 Dec 2023 11:50:09 +0800 Hangbin Liu wrote: > > > We should use the exact-len instead of min-len for IPv6 address. > > > > It does make sense, but these families historically used min-len.. > > Not sure if it's worth changing this now or we risk regressions. > > The addr6 in mptcp.yaml also use exact-len. I don't think the IPv6 address > could be larger than 16 bytes. So the min-len check looks incorrect. I understand, what I'm saying is that the nla_policy before we started using specs was buggy, so we kept the bugginess in the conversion. But it should be fine, so we can change it now and hope for the best..