From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <ferruh.yigit@intel.com>
Received: from mga04.intel.com (mga04.intel.com [192.55.52.120])
 by dpdk.org (Postfix) with ESMTP id D087569FC;
 Thu, 12 Jan 2017 13:22:12 +0100 (CET)
Received: from orsmga003.jf.intel.com ([10.7.209.27])
 by fmsmga104.fm.intel.com with ESMTP; 12 Jan 2017 04:22:11 -0800
X-ExtLoop1: 1
X-IronPort-AV: E=Sophos;i="5.33,349,1477983600"; d="scan'208";a="921710157"
Received: from fyigit-mobl1.ger.corp.intel.com (HELO [10.237.220.38])
 ([10.237.220.38])
 by orsmga003.jf.intel.com with ESMTP; 12 Jan 2017 04:22:10 -0800
To: Alejandro Lucero <alejandro.lucero@netronome.com>
References: <3355891.l3I590SjcV@polaris>
 <608e7dfd-5226-3e30-f43b-0fbe01aee16a@intel.com>
 <CAD+H991Zvaa56O6s_KareaH7mxYxJrn1pW6gG4cLDDKdDA=b0A@mail.gmail.com>
Cc: Gregory Etelson <gregory@weka.io>, dev <dev@dpdk.org>, users@dpdk.org
From: Ferruh Yigit <ferruh.yigit@intel.com>
Message-ID: <e1507eab-1604-b300-8f86-5d5ac2c98ee5@intel.com>
Date: Thu, 12 Jan 2017 12:22:09 +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: <CAD+H991Zvaa56O6s_KareaH7mxYxJrn1pW6gG4cLDDKdDA=b0A@mail.gmail.com>
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 8bit
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 <dev.dpdk.org>
List-Unsubscribe: <http://dpdk.org/ml/options/dev>,
 <mailto:dev-request@dpdk.org?subject=unsubscribe>
List-Archive: <http://dpdk.org/ml/archives/dev/>
List-Post: <mailto:dev@dpdk.org>
List-Help: <mailto:dev-request@dpdk.org?subject=help>
List-Subscribe: <http://dpdk.org/ml/listinfo/dev>,
 <mailto:dev-request@dpdk.org?subject=subscribe>
X-List-Received-Date: Thu, 12 Jan 2017 12:22:13 -0000

On 1/12/2017 12:12 PM, Alejandro Lucero wrote:
> 
> 
> On Thu, Jan 12, 2017 at 11:55 AM, Ferruh Yigit <ferruh.yigit@intel.com
> <mailto:ferruh.yigit@intel.com>> 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.

> 
>  
> 
> 
>     > The patches below provide IGB_UIO release callback and IXGBEVF release function
> 
>     But adding ixgbe specific code into igb_uio may not be good idea.
>     Can be anything done one upper layer, pci layer, generic to all drivers?
> 
> 
> This  module is not just being used for Intel cards, so this addition
> will break, at least, the NFP PMD support.
> 
> I was told to use igb_uio instead of adding a new NFP uio driver, so I
> guess that implies this igb_uio driver should be considered not only a
> igb driver.

No it is generic, I think names has igb_ just for historical reasons.

>  
> 
>     > With the patches, each time DPDK process terminates,
>     > UIO release callback will trigger port PCI close.
>     > On the down side, patched IGB_UIO can be bound to a single adapter type
>     >
>     > Regards,
>     > Gregory
> 
>     <...>
> 
>