Re: [lng-odp] [API-NEXT] validation: classification: added additional suite to test individual PMRs

2015-08-03 Thread Ivan Khoronzhuk
As a proposition, we had talk some time ago about, You moved packet creation to common classification file. Maybe it's time to assign correct src/dst MAC address, taken from pktio? In separate patch. -- Regards, Ivan Khoronzhuk ___ lng-odp mailing list

Re: [lng-odp] [API-NEXT] validation: classification: added additional suite to test individual PMRs

2015-08-03 Thread Bala Manoharan
On 3 August 2015 at 21:23, Ivan Khoronzhuk ivan.khoronz...@linaro.org wrote: As a proposition, we had talk some time ago about, You moved packet creation to common classification file. Maybe it's time to assign correct src/dst MAC address, taken from pktio? In separate patch. Yes. I agree

Re: [lng-odp] [API-NEXT] validation: classification: added additional suite to test individual PMRs

2015-08-03 Thread Ivan Khoronzhuk
One more issue On 30.07.15 18:20, Balasubramanian Manoharan wrote: Additional test suite is added to classification validation suite to test individual PMRs. This suite will test the defined PMRs by configuring pktio separately for every test case. Signed-off-by: Balasubramanian Manoharan

Re: [lng-odp] [API-NEXT] validation: classification: added additional suite to test individual PMRs

2015-08-03 Thread Bala Manoharan
On 3 August 2015 at 21:44, Ivan Khoronzhuk ivan.khoronz...@linaro.org wrote: One more issue On 30.07.15 18:20, Balasubramanian Manoharan wrote: Additional test suite is added to classification validation suite to test individual PMRs. This suite will test the defined PMRs by configuring

Re: [lng-odp] [API-NEXT] validation: classification: added additional suite to test individual PMRs

2015-08-03 Thread Mike Holmes
I don’t think this has to be API-NEXT, it does touch a helper but that is not the API from odp/include On 3 August 2015 at 12:24, Bala Manoharan bala.manoha...@linaro.org wrote: On 3 August 2015 at 21:44, Ivan Khoronzhuk ivan.khoronz...@linaro.org wrote: One more issue On 30.07.15 18:20,

Re: [lng-odp] [API-NEXT] validation: classification: added additional suite to test individual PMRs

2015-08-03 Thread Ivan Khoronzhuk
On 03.08.15 19:24, Bala Manoharan wrote: On 3 August 2015 at 21:44, Ivan Khoronzhuk ivan.khoronz...@linaro.org mailto:ivan.khoronz...@linaro.org wrote: One more issue On 30.07.15 18:20, Balasubramanian Manoharan wrote: Additional test suite is added to classification

Re: [lng-odp] [API-NEXT] validation: classification: added additional suite to test individual PMRs

2015-08-03 Thread Bala Manoharan
On 3 August 2015 at 22:01, Ivan Khoronzhuk ivan.khoronz...@linaro.org wrote: On 03.08.15 19:24, Bala Manoharan wrote: On 3 August 2015 at 21:44, Ivan Khoronzhuk ivan.khoronz...@linaro.org mailto:ivan.khoronz...@linaro.org wrote: One more issue On 30.07.15 18:20,

Re: [lng-odp] [API-NEXT] validation: classification: added additional suite to test individual PMRs

2015-08-03 Thread Ivan Khoronzhuk
Bala, ... what in case of TCP? Incorrect seq num? Looks like a coding error. TCP should be added using a boolean flag. Incorrect seq num is tested in the ASSERT function after receiving the packet. Seq num is used to check whether

Re: [lng-odp] [API-NEXT] validation: classification: added additional suite to test individual PMRs

2015-08-03 Thread Bala Manoharan
On 3 August 2015 at 22:16, Ivan Khoronzhuk ivan.khoronz...@linaro.org wrote: Bala, ... what in case of TCP? Incorrect seq num? Looks like a coding error. TCP should be added using a boolean flag. Incorrect seq num is tested in the ASSERT function after

Re: [lng-odp] [API-NEXT] validation: classification: added additional suite to test individual PMRs

2015-08-03 Thread Ivan Khoronzhuk
Hi, Bala On 30.07.15 18:20, Balasubramanian Manoharan wrote: Additional test suite is added to classification validation suite to test individual PMRs. This suite will test the defined PMRs by configuring pktio separately for every test case. Signed-off-by: Balasubramanian Manoharan

Re: [lng-odp] [API-NEXT] validation: classification: added additional suite to test individual PMRs

2015-08-03 Thread Bala Manoharan
Hi, On 3 August 2015 at 17:52, Ivan Khoronzhuk ivan.khoronz...@linaro.org wrote: Hi, Bala On 30.07.15 18:20, Balasubramanian Manoharan wrote: Additional test suite is added to classification validation suite to test individual PMRs. This suite will test the defined PMRs by configuring

Re: [lng-odp] [API-NEXT] validation: classification: added additional suite to test individual PMRs

2015-07-31 Thread Christophe Milard
I am sorry, as this is not well documented yet (I am currentely working on the documentation), but this patch does not respect the naming convention for test function, test-suite... Possibly you started your work on this before my modification went in... Anyway here follows the naming convention

Re: [lng-odp] [API-NEXT] validation: classification: added additional suite to test individual PMRs

2015-07-31 Thread Bala Manoharan
Hi Christophe, Thanks for pointing this out :) I had started this work before the naming conventions were mandated. I will follow the naming conventions followed here before my final patch is out. I will update these changes along with review comments I get ;) Regards, Bala On 31 July 2015 at

[lng-odp] [API-NEXT] validation: classification: added additional suite to test individual PMRs

2015-07-30 Thread Balasubramanian Manoharan
Additional test suite is added to classification validation suite to test individual PMRs. This suite will test the defined PMRs by configuring pktio separately for every test case. Signed-off-by: Balasubramanian Manoharan bala.manoha...@linaro.org --- helper/include/odp/helper/tcp.h