From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ferruh Yigit Subject: Re: [PATCH 10/13] KNI: provided netif name's source is user-space Date: Wed, 14 Dec 2016 17:35:25 +0000 Message-ID: References: <817f957e-80c3-bfc0-e02b-55f397581708@intel.com> <20161214171911.nupex5gi5jxbbbhm@rere.qmqm.pl> Mime-Version: 1.0 Content-Type: text/plain; charset=iso-8859-2 Content-Transfer-Encoding: 8bit Cc: dev@dpdk.org To: =?UTF-8?B?TWljaGHFgiBNaXJvc8WCYXc=?= 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) In-Reply-To: <20161214171911.nupex5gi5jxbbbhm@rere.qmqm.pl> List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" 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 >