Re: [dpdk-dev] [PATCH v1] doc: update mlx4 flow limitations

2018-02-12 Thread Thomas Monjalon
12/02/2018 15:19, Adrien Mazarguil: > On Mon, Feb 12, 2018 at 02:58:31PM +0100, Thomas Monjalon wrote: > > 12/02/2018 12:23, Adrien Mazarguil: > > > On Fri, Feb 09, 2018 at 05:39:50PM +0100, Thomas Monjalon wrote: > > > > I think it's better to have some random thoughts than nothing. > > > > All th

Re: [dpdk-dev] [PATCH v1] doc: update mlx4 flow limitations

2018-02-12 Thread Adrien Mazarguil
On Mon, Feb 12, 2018 at 02:58:31PM +0100, Thomas Monjalon wrote: > 12/02/2018 12:23, Adrien Mazarguil: > > On Fri, Feb 09, 2018 at 05:39:50PM +0100, Thomas Monjalon wrote: > > > I think it's better to have some random thoughts than nothing. > > > All the comments you gave in this thread deserve to

Re: [dpdk-dev] [PATCH v1] doc: update mlx4 flow limitations

2018-02-12 Thread Thomas Monjalon
12/02/2018 12:23, Adrien Mazarguil: > On Fri, Feb 09, 2018 at 05:39:50PM +0100, Thomas Monjalon wrote: > > I think it's better to have some random thoughts than nothing. > > All the comments you gave in this thread deserve to be written in > > the documentation as soon as possible. > > Right, but

Re: [dpdk-dev] [PATCH v1] doc: update mlx4 flow limitations

2018-02-12 Thread Shahaf Shuler
Monday, February 12, 2018 2:46 PM, Marcelo Ricardo Leitner: > On Mon, Feb 12, 2018 at 07:41:03AM +, Shahaf Shuler wrote: > The need to have eth dst specified. This is not correct. the device can support such rules. Are you running on VF? Can you paste the rule you try to create and the error

Re: [dpdk-dev] [PATCH v1] doc: update mlx4 flow limitations

2018-02-12 Thread Marcelo Ricardo Leitner
On Mon, Feb 12, 2018 at 07:41:03AM +, Shahaf Shuler wrote: > Hi Marcelo, > > Sunday, February 11, 2018 9:31 PM, Marcelo Ricardo: > > On Thu, Feb 08, 2018 at 06:55:54AM +, Ophir Munk wrote: > > > From: Moti Haimovsky > > > > > > This patch updates mlx4 documentation with flow configuration

Re: [dpdk-dev] [PATCH v1] doc: update mlx4 flow limitations

2018-02-12 Thread Adrien Mazarguil
On Fri, Feb 09, 2018 at 05:39:50PM +0100, Thomas Monjalon wrote: > 09/02/2018 17:21, Adrien Mazarguil: > > This section is titled "Limitations" but contains a mix of features, > > limitations and quirks, more like "Random thoughts regarding rte_flow > > support". I think this is not what users migh

Re: [dpdk-dev] [PATCH v1] doc: update mlx4 flow limitations

2018-02-11 Thread Shahaf Shuler
Hi Marcelo, Sunday, February 11, 2018 9:31 PM, Marcelo Ricardo: > On Thu, Feb 08, 2018 at 06:55:54AM +, Ophir Munk wrote: > > From: Moti Haimovsky > > > > This patch updates mlx4 documentation with flow configuration > > limitations imposed by NIC hardware and PMD implementation > > > > Signe

Re: [dpdk-dev] [PATCH v1] doc: update mlx4 flow limitations

2018-02-11 Thread Marcelo Ricardo Leitner
On Thu, Feb 08, 2018 at 06:55:54AM +, Ophir Munk wrote: > From: Moti Haimovsky > > This patch updates mlx4 documentation with flow > configuration limitations imposed by NIC hardware and > PMD implementation > > Signed-off-by: Ophir Munk > --- > doc/guides/nics/mlx4.rst | 77 > +++

Re: [dpdk-dev] [PATCH v1] doc: update mlx4 flow limitations

2018-02-09 Thread Thomas Monjalon
09/02/2018 17:21, Adrien Mazarguil: > This section is titled "Limitations" but contains a mix of features, > limitations and quirks, more like "Random thoughts regarding rte_flow > support". I think this is not what users might expect from such a > documentation which must be exhaustive and consist

Re: [dpdk-dev] [PATCH v1] doc: update mlx4 flow limitations

2018-02-09 Thread Adrien Mazarguil
Hi Ophir, On Thu, Feb 08, 2018 at 06:55:54AM +, Ophir Munk wrote: > From: Moti Haimovsky Relatively minor, patch author differs from the only sign off below, I don't think it's on purpose. > This patch updates mlx4 documentation with flow > configuration limitations imposed by NIC hardware

Re: [dpdk-dev] [PATCH v1] doc: update mlx4 flow limitations

2018-02-08 Thread Mcnamara, John
bject: [dpdk-dev] [PATCH v1] doc: update mlx4 flow limitations > > From: Moti Haimovsky > > This patch updates mlx4 documentation with flow configuration limitations > imposed by NIC hardware and PMD implementation > > + For example using testpmd command - src mask must b

[dpdk-dev] [PATCH v1] doc: update mlx4 flow limitations

2018-02-07 Thread Ophir Munk
From: Moti Haimovsky This patch updates mlx4 documentation with flow configuration limitations imposed by NIC hardware and PMD implementation Signed-off-by: Ophir Munk --- doc/guides/nics/mlx4.rst | 77 1 file changed, 77 insertions(+) diff --g