From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga09.intel.com (mga09.intel.com [134.134.136.24]) by dpdk.org (Postfix) with ESMTP id BD3992C4F for ; Wed, 14 Dec 2016 18:35:27 +0100 (CET) Received: from fmsmga002.fm.intel.com ([10.253.24.26]) by orsmga102.jf.intel.com with ESMTP; 14 Dec 2016 09:35:26 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.33,347,1477983600"; d="scan'208";a="1099147701" Received: from fyigit-mobl1.ger.corp.intel.com (HELO [10.237.220.29]) ([10.237.220.29]) by fmsmga002.fm.intel.com with ESMTP; 14 Dec 2016 09:35:26 -0800 To: =?UTF-8?B?TWljaGHFgiBNaXJvc8WCYXc=?= References: <817f957e-80c3-bfc0-e02b-55f397581708@intel.com> <20161214171911.nupex5gi5jxbbbhm@rere.qmqm.pl> Cc: dev@dpdk.org From: Ferruh Yigit Message-ID: Date: Wed, 14 Dec 2016 17:35:25 +0000 User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:45.0) Gecko/20100101 Thunderbird/45.5.1 MIME-Version: 1.0 In-Reply-To: <20161214171911.nupex5gi5jxbbbhm@rere.qmqm.pl> Content-Type: text/plain; charset=iso-8859-2 Content-Transfer-Encoding: 8bit Subject: Re: [dpdk-dev] [PATCH 10/13] KNI: provided netif name's source is user-space X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 14 Dec 2016 17:35:28 -0000 On 12/14/2016 5:19 PM, Michał Mirosław wrote: > On Wed, Dec 14, 2016 at 05:06:23PM +0000, Ferruh Yigit wrote: >> Hi Michal, >> >> On 12/13/2016 1:08 AM, Michał Mirosław wrote: >>> Signed-off-by: Michał Mirosław >>> --- >>> lib/librte_eal/linuxapp/kni/kni_misc.c | 4 ++-- >>> 1 file changed, 2 insertions(+), 2 deletions(-) >>> >>> diff --git a/lib/librte_eal/linuxapp/kni/kni_misc.c b/lib/librte_eal/linuxapp/kni/kni_misc.c >>> index 497db9b..f0247aa 100644 >>> --- a/lib/librte_eal/linuxapp/kni/kni_misc.c >>> +++ b/lib/librte_eal/linuxapp/kni/kni_misc.c >>> @@ -363,8 +363,8 @@ kni_ioctl_create(struct net *net, uint32_t ioctl_num, >>> up_read(&knet->kni_list_lock); >>> >>> net_dev = alloc_netdev(sizeof(struct kni_dev), dev_info.name, >>> -#ifdef NET_NAME_UNKNOWN >>> - NET_NAME_UNKNOWN, >>> +#ifdef NET_NAME_USER >>> + NET_NAME_USER, >> >> Probably NET_NAME_USER fits better to definition. >> But functionally, I wonder if you have a use case to use >> "name_assign_type" ? > > I just read what NET_NAME_UNKNOWN/NET_NAME_USER is supposed to do. > UNKNOWN is for "old" drivers that don't know the source of the name. > USER is for a name, that comes from a user and as such is not to be > renamed by udev. That is what I wonder if you are doing anything special in userspace related iface name. But it seems the patch is to make it more proper, I have no objection. > > Best Regards, > Michał Mirosław >