From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ferruh Yigit Subject: Re: [PATCH] kni: error rollback with kni_dev_remove could cause a kernel crash Date: Thu, 25 Aug 2016 14:49:03 +0100 Message-ID: <57BEF74F.6040702@intel.com> References: <1472132490-27425-1-git-send-email-ferruh.yigit@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit Cc: ZhouYates To: dev@dpdk.org Return-path: Received: from mga03.intel.com (mga03.intel.com [134.134.136.65]) by dpdk.org (Postfix) with ESMTP id 003C5558A for ; Thu, 25 Aug 2016 15:49:05 +0200 (CEST) In-Reply-To: <1472132490-27425-1-git-send-email-ferruh.yigit@intel.com> List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" Hi ZhouYates, On 8/25/2016 2:41 PM, Ferruh Yigit wrote: > From: ZhouYates I have re-sent your exact patch which was sent as attachment. Thank you for the patch, can you please follow contribution guide for next version of the patch? More details on: http://dpdk.org/doc/guides/contributing/patches.html#sending-patches > > Signed-off-by: ZhouYates > --- > lib/librte_eal/linuxapp/kni/kni_misc.c | 4 +++- > 1 files changed, 3 insertions(+), 1 deletions(-) > > diff --git a/lib/librte_eal/linuxapp/kni/kni_misc.c b/lib/librte_eal/linuxapp/kni/kni_misc.c > index 59d15ca..7ef17f5 100644 > --- a/lib/librte_eal/linuxapp/kni/kni_misc.c > +++ b/lib/librte_eal/linuxapp/kni/kni_misc.c > @@ -448,7 +448,6 @@ kni_ioctl_create(struct net *net, > > kni = netdev_priv(net_dev); > > - kni->net_dev = net_dev; > kni->group_id = dev_info.group_id; > kni->core_id = dev_info.core_id; > strncpy(kni->name, dev_info.name, RTE_KNI_NAMESIZE); > @@ -561,6 +560,9 @@ kni_ioctl_create(struct net *net, > kni_dev_remove(kni); But this cause allocated net_dev not freed here ... > return -ENODEV; > } > + /*Set the kni->net_dev when the net_dev has registered success. > + * Avoid unregistering unregistered net_dev by kni_dev_remove.*/ Can you please use multi line comment syntax same as rest of the document. Or perhaps first line can be enough. > + kni->net_dev = net_dev; > > #ifdef RTE_KNI_VHOST > kni_vhost_init(kni); >