From: "Tan, Jianfeng" <jianfeng.tan@intel.com>
To: "Yigit, Ferruh" <ferruh.yigit@intel.com>,
Alejandro Lucero <alejandro.lucero@netronome.com>
Cc: Gregory Etelson <gregory@weka.io>, dev <dev@dpdk.org>,
"users@dpdk.org" <users@dpdk.org>
Subject: Re: [dpdk-users] [dpdk-dev] IGB_UIO: PCI Resources Management
Date: Fri, 13 Jan 2017 05:33:18 +0000 [thread overview]
Message-ID: <ED26CBA2FAD1BF48A8719AEF02201E3651111E1E@SHSMSX103.ccr.corp.intel.com> (raw)
In-Reply-To: <7605618f-dc86-3060-473e-aff5545cac72@intel.com>
> -----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 <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.
> >
> > 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
next prev parent reply other threads:[~2017-01-13 5:33 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-12-09 8:54 [dpdk-users] " Gregory Etelson
2017-01-12 11:55 ` Ferruh Yigit
2017-01-12 12:12 ` [dpdk-users] [dpdk-dev] " Alejandro Lucero
2017-01-12 12:22 ` Ferruh Yigit
2017-01-12 12:58 ` Alejandro Lucero
2017-01-13 1:51 ` Tan, Jianfeng
2017-01-13 2:04 ` Ferruh Yigit
2017-01-13 5:33 ` Tan, Jianfeng [this message]
2017-01-13 11:10 ` Alejandro Lucero
2017-01-19 16:36 ` Ferruh Yigit
2017-01-19 20:33 ` Alejandro Lucero
2017-01-19 15:59 ` Ferruh Yigit
2017-01-19 16:09 ` George Prekas
2017-01-20 5:09 ` Tan, Jianfeng
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=ED26CBA2FAD1BF48A8719AEF02201E3651111E1E@SHSMSX103.ccr.corp.intel.com \
--to=jianfeng.tan@intel.com \
--cc=alejandro.lucero@netronome.com \
--cc=dev@dpdk.org \
--cc=ferruh.yigit@intel.com \
--cc=gregory@weka.io \
--cc=users@dpdk.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).