[dpdk-dev] [PATCH] doc: add limitation for L3fwd-power app

2016-11-07 Thread Thomas Monjalon
> > L3fwd-power app needs vector mode to be disabled in order to work
> > properly. The app used to work previously, because it was using Rx scalar
> > function, but now it uses vector function.
> > 
> > Vector mode needs to be disabled to make the app works, which has been
> > documented in release notes.
> > 
> > Signed-off-by: Pablo de Lara 
> 
> Acked-by: John McNamara 

Applied, thanks


[dpdk-dev] [PATCH] doc: add limitation for L3fwd-power app

2016-11-04 Thread De Lara Guarch, Pablo
Hi Thomas,

> -Original Message-
> From: Thomas Monjalon [mailto:thomas.monjalon at 6wind.com]
> Sent: Friday, November 04, 2016 5:00 PM
> To: De Lara Guarch, Pablo
> Cc: dev at dpdk.org
> Subject: Re: [dpdk-dev] [PATCH] doc: add limitation for L3fwd-power app
> 
> 2016-11-04 12:48, Pablo de Lara:
> > L3fwd-power requires the mbuf to have correct packet type,
> > but in some drivers, the vector mode must be disabled for this.
> 
> This is a severe bug.
> We must really fix packet type handling to have a constant support
> whatever the driver or options.

This is already documented in previous release (2.2).
Actually, the main problem for L3fwd-power app is not the packet type,
but the descriptor_done for the Rx vector function, that works differently
compared to the scalar function.

Thanks,
Pablo


[dpdk-dev] [PATCH] doc: add limitation for L3fwd-power app

2016-11-04 Thread Thomas Monjalon
2016-11-04 12:48, Pablo de Lara:
> L3fwd-power requires the mbuf to have correct packet type,
> but in some drivers, the vector mode must be disabled for this.

This is a severe bug.
We must really fix packet type handling to have a constant support
whatever the driver or options.


[dpdk-dev] [PATCH] doc: add limitation for L3fwd-power app

2016-11-04 Thread Mcnamara, John


> -Original Message-
> From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Pablo de Lara
> Sent: Friday, November 4, 2016 12:48 PM
> To: dev at dpdk.org
> Cc: De Lara Guarch, Pablo 
> Subject: [dpdk-dev] [PATCH] doc: add limitation for L3fwd-power app
> 
> L3fwd-power app needs vector mode to be disabled in order to work
> properly. The app used to work previously, because it was using Rx scalar
> function, but now it uses vector function.
> 
> Vector mode needs to be disabled to make the app works, which has been
> documented in release notes.
> 
> Signed-off-by: Pablo de Lara 

Acked-by: John McNamara 


[dpdk-dev] [PATCH] doc: add limitation for L3fwd-power app

2016-11-04 Thread Pablo de Lara
L3fwd-power app needs vector mode to be disabled in order to work
properly. The app used to work previously, because it was using
Rx scalar function, but now it uses vector function.

Vector mode needs to be disabled to make the app works,
which has been documented in release notes.

Signed-off-by: Pablo de Lara 
---
 doc/guides/rel_notes/release_16_11.rst | 11 +++
 1 file changed, 11 insertions(+)

diff --git a/doc/guides/rel_notes/release_16_11.rst 
b/doc/guides/rel_notes/release_16_11.rst
index aa0c09a..cb4ec97 100644
--- a/doc/guides/rel_notes/release_16_11.rst
+++ b/doc/guides/rel_notes/release_16_11.rst
@@ -178,6 +178,17 @@ Known Issues

This section is a comment. Make sure to start the actual text at the margin.

+* **L3fwd-power app does not work properly when Rx vector is enabled.**
+
+  Using some drivers with vector enabled, makes L3fwd-power app not work
+  properly, since the queue monitoring works differently when using
+  scalar compared to vector, making the frequency scaling not to work 
correctly.
+  In addition, L3fwd-power requires the mbuf to have correct packet type,
+  but in some drivers, the vector mode must be disabled for this.
+
+  Therefore, in order to use L3fwd-power, vector mode should be disabled
+  from the config file.
+

 API Changes
 ---
-- 
2.7.4