Received: by 2002:a05:6358:9144:b0:117:f937:c515 with SMTP id r4csp7206530rwr; Tue, 2 May 2023 10:59:14 -0700 (PDT) X-Google-Smtp-Source: ACHHUZ4Gx1pMK/I7JMMRdLevh/zA3JRv7GADfzoT++IJBliVmPgQPE4hyYRsaPWxtH+YwRAuc14r X-Received: by 2002:a17:902:c64b:b0:1ab:f74:a118 with SMTP id s11-20020a170902c64b00b001ab0f74a118mr1897191pls.19.1683050353799; Tue, 02 May 2023 10:59:13 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1683050353; cv=none; d=google.com; s=arc-20160816; b=uQwdatz1lVz1KoXpcbO1pYty7XsqWjHyikfmFgjAoIqenpAtrFp5NogLzTPwp02F2d /thIC5uMjWpkvIaVoyoxosTDzPcbBpyNnAmmxMJnCK5M6K9J0QzhZSD8ZcHSkWlBNznP yA3LRrTJxS7Rq1KuFcF/zk40gP7H5Knf/xhIRu/UWbtRcRhOaHJldZobK6OJ/QZwXBOK V+vv7QM74rjkeTtN1eAUDzatmcprVfTQjfXMVWkMGltrzpbyShMT9xmu4O0SHPBpZxEK ONZ9ZTnBqg3eRSb41ScyEGs63oBv6fwzlI9CY4OIA2rlyeEO+ublv+A8Q9O2368RKX7E M0kg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:mime-version :references:in-reply-to:message-id:subject:cc:to:from:date :dkim-signature; bh=tiE1DWJVu6SR5fUQri2EkqUC06NuFzPAjn+VDe/bbX0=; b=U8DpR4EOYPitJWzohjGMwaL8LYpOMR6s8Q60dxixHR4vqA5qdXq7NG6lFecJP71R6R eGZokRKPAzc7x+YIYoCOyOzTbd2nK3vJPc5R0sUgmEj4R/aoxcNzgAFh36hLPQ1KeyGM db846K7dgF6kyRmTQGneqbCM4pWtjLCBrAyV0HF0bBqvdIhJZPBcEYvu2xmMnHNErxFa PompsXrwv4QcEtsJGp9tw7OmvU4mNMHWEZEtKbJqmYG7LLJ8B0uLNYM+GN8RxSJ7DVtN AEfDj+t0vNYzE7epbCKWxEEjDBA52LmbrtYkW5cBezss8/hxwchlHWaKnWi4S5y+tO4d tMUg== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@networkplumber-org.20221208.gappssmtp.com header.s=20221208 header.b=a0zP5mMW; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=fail (p=QUARANTINE sp=QUARANTINE dis=NONE) header.from=networkplumber.org Return-Path: Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id j3-20020a170902da8300b001a6e98a5f21si32924884plx.586.2023.05.02.10.59.01; Tue, 02 May 2023 10:59:13 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20; Authentication-Results: mx.google.com; dkim=pass header.i=@networkplumber-org.20221208.gappssmtp.com header.s=20221208 header.b=a0zP5mMW; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=fail (p=QUARANTINE sp=QUARANTINE dis=NONE) header.from=networkplumber.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S233956AbjEBR6k (ORCPT + 99 others); Tue, 2 May 2023 13:58:40 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:45168 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S234564AbjEBR6Y (ORCPT ); Tue, 2 May 2023 13:58:24 -0400 Received: from mail-pf1-x42e.google.com (mail-pf1-x42e.google.com [IPv6:2607:f8b0:4864:20::42e]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 83A6183 for ; Tue, 2 May 2023 10:58:23 -0700 (PDT) Received: by mail-pf1-x42e.google.com with SMTP id d2e1a72fcca58-64115e652eeso36666459b3a.0 for ; Tue, 02 May 2023 10:58:23 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=networkplumber-org.20221208.gappssmtp.com; s=20221208; t=1683050303; x=1685642303; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:subject:cc:to:from:date:from:to:cc:subject:date :message-id:reply-to; bh=tiE1DWJVu6SR5fUQri2EkqUC06NuFzPAjn+VDe/bbX0=; b=a0zP5mMWN/KjUOhGaEfPf8s4GKxrE/biIQPS8edlBr0BnWsUKt+pKjQLHRdaP8j5PS VX1XEEGEg+HLO7JfJlt4fgOQ1fJk1MOLyVigx1xJSDwgvaBueNg889SV/9Qzf+8LuJ6q wO/tzIlmWSO/MjB35RBTm8IfWyJsq0GZAg0rpD053LEIVzybo4ES1xvQHznAI+XjtdXY Q7rT5Ovf8xYv5DjefiHXWQPZUjk7T0fMT7VpBU2nGTNTs+Cv1BD45hdyHmmgdd5gVNOM R1cHAMIn5BaDJdkMExyPI14x3Ab8tAjj283Bn3RgtSekq5QD5Bj1bsHgP2vbPxIyC3Pp CnrQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20221208; t=1683050303; x=1685642303; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:subject:cc:to:from:date:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=tiE1DWJVu6SR5fUQri2EkqUC06NuFzPAjn+VDe/bbX0=; b=GKDu9mPvkrFc0Q+eZ2MQKV9chxJN5/rpIZNDp6+HmQHmskRfq+UNhKKZX/ymhQV7ZB VttG+oOMFPogCu8IPHN3XLDYjCeubEY8E0jCC7kYKp6vojRifiBs6YeGXSrx/FzB89Tk Me2n+v8v3TKKd/kEz4D8LlDa4v3Ej6EKFzbuNL8W8uqx7i3JEkSA9Tvm6B48n/8Wa6PZ DcukHTcv/pnHpO+xG5omuHBUp4D/j8MdzO0ADtJO7iS9BqZK6xg4E9FRzBAMg5+SfIhl aT8kyhVuXzBM5VFXI+aBBliD5nwdmgCCIBiDnMrtGWi64EFf1X2Gw+U/nkgxtSeJGKjs fApA== X-Gm-Message-State: AC+VfDxNjvdlFExgIVXhTTFEmjSBicjOgkRUPkbS2xQrGD9U94LMQdGF kKywYn/dzdS+qel3chZ38gtZa9C6w6CSziJxP399Hw== X-Received: by 2002:a05:6a20:72a6:b0:f2:817c:2038 with SMTP id o38-20020a056a2072a600b000f2817c2038mr22302768pzk.18.1683050302978; Tue, 02 May 2023 10:58:22 -0700 (PDT) Received: from hermes.local (204-195-120-218.wavecable.com. [204.195.120.218]) by smtp.gmail.com with ESMTPSA id n12-20020a056a00212c00b0063f172b1c47sm20469544pfj.35.2023.05.02.10.58.22 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 02 May 2023 10:58:22 -0700 (PDT) Date: Tue, 2 May 2023 10:58:20 -0700 From: Stephen Hemminger To: Jakub Kicinski Cc: Ganesh Babu , "netdev@vger.kernel.org" , "linux-kernel@vger.kernel.org" Subject: Re: [PATCH] net: mroute6.h: change type of mif6c_pifi to __u32 Message-ID: <20230502105820.2c27630d@hermes.local> In-Reply-To: <20230502085718.0551a86d@kernel.org> References: <20230328191456.43d2222e@kernel.org> <20230502085718.0551a86d@kernel.org> MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit X-Spam-Status: No, score=-1.9 required=5.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,RCVD_IN_DNSWL_NONE,SPF_HELO_NONE,SPF_PASS, T_SCC_BODY_TEXT_LINE autolearn=unavailable autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, 2 May 2023 08:57:18 -0700 Jakub Kicinski wrote: > On Tue, 2 May 2023 08:07:10 +0000 Ganesh Babu wrote: > > Thank you for your response. Regarding the proposed change to > > the mif6ctl structure in mroute6.h, I would like to clarify, > > that changing the datatype of mif6c_pifi from __u16 to __u32 > > will not change the offset of the structure members, which > > means that the size of the structure remains the same and > > the ABI remains compatible. Furthermore, ifindex is treated > > as an integer in all the subsystems of the kernel and not > > as a 16-bit value. Therefore, changing the datatype of > > mif6c_pifi from __u16 to __u32 is a natural and expected > > change that aligns with the existing practice in the kernel. > > I understand that the mif6ctl structure is part of the uAPI > > and changing its geometry is not allowed. However, in this > > case, we are not changing the geometry of the structure, > > as the size of the structure remains the same and the offset > > of the structure members will not change. Thus, the proposed > > change will not affect the ABI or the user API. Instead, it > > will allow the kernel to handle 32-bit ifindex values without > > any issues, which is essential for the smooth functioning of > > the PIM6 protocol. I hope this explanation clarifies any > > concerns you may have had. Let me know if you have any further > > questions or need any more details. > > Please don't top post on the list. > > How does the hole look on big endian? Does it occupy the low or > the high bytes? > > There's also the problem of old user space possibly not initializing > the hole, and passing in garbage. Looks like multicast routing is one of the last places with no netlink API, and only ioctl. There is no API to modify multicast routes in iproute2.