From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jiri Benc Subject: Re: [PATCH net-next 0/5] rtnetlink: add IFA_IF_NETNSID for RTM_GETADDR Date: Sun, 2 Sep 2018 11:58:51 +0200 Message-ID: <20180902115851.2b224c23@redhat.com> References: <81379a4f-7149-10ff-2453-886314d0b0c4@virtuozzo.com> <20180830144544.tpross4jd6awou4u@gmail.com> <20180831.175847.339882322459287210.davem@davemloft.net> <20180901184704.lzynbw5zkyiqc6wb@gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Cc: David Miller , nicolas.dichtel@6wind.com, ktkhai@virtuozzo.com, netdev@vger.kernel.org, linux-kernel@vger.kernel.org, kuznet@ms2.inr.ac.ru, yoshfuji@linux-ipv6.org, pombredanne@nexb.com, kstewart@linuxfoundation.org, gregkh@linuxfoundation.org, dsahern@gmail.com, fw@strlen.de, lucien.xin@gmail.com, jakub.kicinski@netronome.com To: Christian Brauner Return-path: Received: from mx3-rdu2.redhat.com ([66.187.233.73]:36170 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1726003AbeIBOOM (ORCPT ); Sun, 2 Sep 2018 10:14:12 -0400 In-Reply-To: <20180901184704.lzynbw5zkyiqc6wb@gmail.com> Sender: netdev-owner@vger.kernel.org List-ID: On Sat, 1 Sep 2018 20:47:05 +0200, Christian Brauner wrote: > Sorry for the late reply. Yup, sounds good to me. > But maybe IFA_TARGET_NETNSID to indicate that we're talking network > namespaces here? Seems to me that NSID might give the wrong impression. > I'll send v1 soon. I expect tomorrow or sometime next week. On the other hand, we currently have IFLA_IF_NETNSID for the link operations. IFA_IF_NETNSID is more consistent with the existing attribute. It may be confusing to authors of user space programs to have attribute names doing the same thing constructed differently for different calls (IFLA_IF_NETNSID and IFA_TARGET_NETNSID). As for the patch set itself, it makes sense to me, whatever the final attribute name is. Thanks, Jiri