From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga02.intel.com (mga02.intel.com [134.134.136.20]) by dpdk.org (Postfix) with ESMTP id 83A37FA40; Thu, 19 Jan 2017 17:37:23 +0100 (CET) Received: from fmsmga005.fm.intel.com ([10.253.24.32]) by orsmga101.jf.intel.com with ESMTP; 19 Jan 2017 08:37:01 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.33,254,1477983600"; d="scan'208";a="55221204" Received: from fyigit-mobl1.ger.corp.intel.com (HELO [10.237.220.38]) ([10.237.220.38]) by fmsmga005.fm.intel.com with ESMTP; 19 Jan 2017 08:36:59 -0800 To: Alejandro Lucero , "Tan, Jianfeng" References: <3355891.l3I590SjcV@polaris> <608e7dfd-5226-3e30-f43b-0fbe01aee16a@intel.com> <7605618f-dc86-3060-473e-aff5545cac72@intel.com> Cc: Gregory Etelson , dev , "users@dpdk.org" From: Ferruh Yigit Message-ID: Date: Thu, 19 Jan 2017 16:36:58 +0000 User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:45.0) Gecko/20100101 Thunderbird/45.6.0 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit Subject: Re: [dpdk-dev] [dpdk-users] IGB_UIO: PCI Resources Management 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: Thu, 19 Jan 2017 16:37:25 -0000 On 1/13/2017 11:10 AM, Alejandro Lucero wrote: > I completely misread the patch, and I wrongly thought that code was > linked to module removal, but I see this is not about that, but about > releasing the /dev/uio file calling release function, what is done by > the kernel when the process exits. > > So yes, the patch avoids the problem I talked about. > > However, calling that specific ixgbe driver function will break other > devices relying on igb_uio. What about implementing a notifier chain for > this? The igb_uio code would be agnostic and each interested driver, > like ixgbe or nfp_net, could execute the specific port close code when > the notifier chain triggers. Can you please elaborate, how can this be done? This is kernel code, and interested drivers are in userspace. This patch benefit from Linux kernel driver of same device. > > > On Fri, Jan 13, 2017 at 5:33 AM, Tan, Jianfeng > wrote: > > > > > -----Original Message----- > > From: Yigit, Ferruh > > Sent: Friday, January 13, 2017 10:05 AM > > To: Tan, Jianfeng; Alejandro Lucero > > Cc: Gregory Etelson; dev; users@dpdk.org > > Subject: Re: [dpdk-users] [dpdk-dev] IGB_UIO: PCI Resources Management > > > > On 1/13/2017 1:51 AM, Tan, Jianfeng wrote: > > > > > > > > >> -----Original Message----- > > >> From: users [mailto:users-bounces@dpdk.org > ] On Behalf Of Ferruh Yigit > > >> Sent: Thursday, January 12, 2017 8:22 PM > > >> To: Alejandro Lucero > > >> Cc: Gregory Etelson; dev; users@dpdk.org > > >> Subject: Re: [dpdk-users] [dpdk-dev] IGB_UIO: PCI Resources > > Management > > >> > > >> On 1/12/2017 12:12 PM, Alejandro Lucero wrote: > > >>> > > >>> > > >>> On Thu, Jan 12, 2017 at 11:55 AM, Ferruh Yigit > > > >>> >> wrote: > > >>> > > >>> On 12/9/2016 8:54 AM, Gregory Etelson wrote: > > >>> > Hello, > > >>> > > > >>> > IGB_UIO driver does not close port PCI activities after > DPDK process > > >> exits. > > >>> > DPDK API provides rte_eth_dev_close() to manage port PCI, > > >>> > but it can be skipped if process receives SIGKILL signal > > >>> > > >>> I guess I understand the problem. > > >>> > > >>> > > >>> This is a known problem, but it is not just a UIO problem, and > this > > >>> patch does not solve it, maybe it just solves part of it. > > >>> > > >>> In fact, a DPDK program crashing could imply the NIC DMAing > after that > > >>> and after that memory was assigned to another program. > > >> > > >> Yes. > > >> Can there be a way to stop NIC DMA, (or prevent it access to mem > > >> anymore) when app crashes? > > >> I think that is what this patch is looking for. > > > > > > If I understand it correctly, you are looking for this patch? > > > http://dpdk.org/dev/patchwork/patch/17495/ > > > > > > > > That is good, thanks Jianfeng, I will check it. > > > > btw, patch's current state is rejected, which is by mistake, it > seems I > > confused it with "iomem and ioport mapping" patch, sorry about it, I > > will update its status immediately. > > No problem at all. This patch is rejected as it's based on "iomem > and ioport mapping" patch. As "iomem and ioport mapping" patch has > backward compatibility issue, we need to figure out a way to > resubmit this patch without changing the original "iomem and ioport > mapping" in igb_uio. > > Thanks, > Jianfeng > >