Re: [dpdk-dev] [PATCH v4 0/4] Fixes on IOVA mode selection

2019-07-23 Thread Burakov, Anatoly
On 23-Jul-19 2:56 PM, Burakov, Anatoly wrote: On 23-Jul-19 11:25 AM, Thomas Monjalon wrote: 23/07/2019 11:57, Burakov, Anatoly: A machine without an IOMMU shouldn't have picked IOVA as VA in the first place. Perhaps this is something we could fix? I'm not sure how to detected that condition tho

Re: [dpdk-dev] [PATCH v4 0/4] Fixes on IOVA mode selection

2019-07-23 Thread Burakov, Anatoly
On 23-Jul-19 11:25 AM, Thomas Monjalon wrote: 23/07/2019 11:57, Burakov, Anatoly: A machine without an IOMMU shouldn't have picked IOVA as VA in the first place. Perhaps this is something we could fix? I'm not sure how to detected that condition though, i don't think there's a mechanism to know

Re: [dpdk-dev] [PATCH v4 0/4] Fixes on IOVA mode selection

2019-07-23 Thread Thomas Monjalon
23/07/2019 11:57, Burakov, Anatoly: > A machine without an IOMMU shouldn't have picked IOVA as VA in the first > place. Perhaps this is something we could fix? I'm not sure how to > detected that condition though, i don't think there's a mechanism to > know that for sure. Some kernels create a "

Re: [dpdk-dev] [PATCH v4 0/4] Fixes on IOVA mode selection

2019-07-23 Thread Burakov, Anatoly
; Jerin Jacob Kollanukkaran Cc: dev@dpdk.org Subject: [EXT] RE: [dpdk-dev] [PATCH v4 0/4] Fixes on IOVA mode selection This introduces a regression where uio-bound devies are attached to a DPDK app at runtime. Just to understand the requirements; # Is this requirement for SPDK? # Is brand new

Re: [dpdk-dev] [PATCH v4 0/4] Fixes on IOVA mode selection

2019-07-23 Thread Thomas Monjalon
23/07/2019 07:27, Jerin Jacob Kollanukkaran: > From: Stojaczyk, Dariusz > > From: Jerin Jacob Kollanukkaran [mailto:jer...@marvell.com] > > > From: Stojaczyk, Dariusz > > > > > > > > This introduces a regression where uio-bound devies are attached to > > > > a DPDK app at runtime. Yes it is a re

Re: [dpdk-dev] [PATCH v4 0/4] Fixes on IOVA mode selection

2019-07-22 Thread Jerin Jacob Kollanukkaran
> -Original Message- > From: Stojaczyk, Dariusz > Sent: Tuesday, July 23, 2019 10:24 AM > To: Jerin Jacob Kollanukkaran ; Thomas Monjalon > ; David Marchand ; > Burakov, Anatoly > Cc: dev@dpdk.org > Subject: [EXT] RE: [dpdk-dev] [PATCH v4 0/4] Fixe

Re: [dpdk-dev] [PATCH v4 0/4] Fixes on IOVA mode selection

2019-07-22 Thread Stojaczyk, Dariusz
avid Marchand > > ; Burakov, Anatoly > > ; Jerin Jacob Kollanukkaran > > > > Cc: dev@dpdk.org > > Subject: [EXT] RE: [dpdk-dev] [PATCH v4 0/4] Fixes on IOVA mode selection > > > > This introduces a regression where uio-bound devies are attached to a &g

Re: [dpdk-dev] [PATCH v4 0/4] Fixes on IOVA mode selection

2019-07-22 Thread Jerin Jacob Kollanukkaran
> -Original Message- > From: Stojaczyk, Dariusz > Sent: Tuesday, July 23, 2019 9:06 AM > To: Thomas Monjalon ; David Marchand > ; Burakov, Anatoly > ; Jerin Jacob Kollanukkaran > > Cc: dev@dpdk.org > Subject: [EXT] RE: [dpdk-dev] [PATCH v4 0/4] Fixes on IO

Re: [dpdk-dev] [PATCH v4 0/4] Fixes on IOVA mode selection

2019-07-22 Thread Stojaczyk, Dariusz
patch? D. > -Original Message- > From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Thomas Monjalon > Sent: Monday, July 22, 2019 5:53 PM > To: David Marchand ; Burakov, Anatoly > ; jer...@marvell.com > Cc: dev@dpdk.org > Subject: Re: [dpdk-dev] [PATCH v4 0/4] Fi

Re: [dpdk-dev] [PATCH v4 0/4] Fixes on IOVA mode selection

2019-07-22 Thread Thomas Monjalon
22/07/2019 14:56, David Marchand: > Following the issues reported by Jerin and the discussion that emerged > from it, here are fixes to restore and document the behavior of the EAL > and the pci bus driver. > > I pondered all the arguments and tried to have the less changes > possible. > I can't f

[dpdk-dev] [PATCH v4 0/4] Fixes on IOVA mode selection

2019-07-22 Thread David Marchand
Following the issues reported by Jerin and the discussion that emerged from it, here are fixes to restore and document the behavior of the EAL and the pci bus driver. I pondered all the arguments and tried to have the less changes possible. I can't find a need for a flag to just announce support o