diff mbox

test: performance: lower the MAX_PKT_SIZE to 1518

Message ID 1490362838-21800-1-git-send-email-balakrishna.garapati@linaro.org
State New
Headers show

Commit Message

Balakrishna Garapati March 24, 2017, 1:40 p.m. UTC
"bench_packet_tailroom" test fails on odp-dpdk with the pkt size 2048
leaving no space for tailroom.

Signed-off-by: Balakrishna Garapati <balakrishna.garapati@linaro.org>

---
 test/common_plat/performance/odp_bench_packet.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

-- 
1.9.1

Comments

Balakrishna Garapati March 30, 2017, 7:13 a.m. UTC | #1
ping

On 24 March 2017 at 14:40, Balakrishna Garapati <
balakrishna.garapati@linaro.org> wrote:

> "bench_packet_tailroom" test fails on odp-dpdk with the pkt size 2048

> leaving no space for tailroom.

>

> Signed-off-by: Balakrishna Garapati <balakrishna.garapati@linaro.org>

> ---

>  test/common_plat/performance/odp_bench_packet.c | 4 ++--

>  1 file changed, 2 insertions(+), 2 deletions(-)

>

> diff --git a/test/common_plat/performance/odp_bench_packet.c

> b/test/common_plat/performance/odp_bench_packet.c

> index 7a3a004..6b73423 100644

> --- a/test/common_plat/performance/odp_bench_packet.c

> +++ b/test/common_plat/performance/odp_bench_packet.c

> @@ -35,7 +35,7 @@

>  #define TEST_MIN_PKT_SIZE 64

>

>  /** Maximum test packet size */

> -#define TEST_MAX_PKT_SIZE 2048

> +#define TEST_MAX_PKT_SIZE 1518

>

>  /** Number of test runs per individual benchmark */

>  #define TEST_REPEAT_COUNT 1000

> @@ -78,7 +78,7 @@ ODP_STATIC_ASSERT((TEST_ALIGN_OFFSET + TEST_ALIGN_LEN)

> <= TEST_MIN_PKT_SIZE,

>

>  /** Test packet sizes */

>  const uint32_t test_packet_len[] = {WARM_UP, TEST_MIN_PKT_SIZE, 128, 256,

> 512,

> -                                   1024, 1518, TEST_MAX_PKT_SIZE};

> +                                   1024, TEST_MAX_PKT_SIZE};

>

>  /**

>   * Parsed command line arguments

> --

> 1.9.1

>

>
Elo, Matias (Nokia - FI/Espoo) March 30, 2017, 7:55 a.m. UTC | #2
Hi,

The patch I just submitted "test: bench_packet: fix headroom/tailroom test" should fix this problem.

-Matias


> On 30 Mar 2017, at 10:13, Krishna Garapati <balakrishna.garapati@linaro.org> wrote:

> 

> ping

> 

> On 24 March 2017 at 14:40, Balakrishna Garapati <

> balakrishna.garapati@linaro.org> wrote:

> 

>> "bench_packet_tailroom" test fails on odp-dpdk with the pkt size 2048

>> leaving no space for tailroom.

>> 

>> Signed-off-by: Balakrishna Garapati <balakrishna.garapati@linaro.org>

>> ---

>> test/common_plat/performance/odp_bench_packet.c | 4 ++--

>> 1 file changed, 2 insertions(+), 2 deletions(-)

>> 

>> diff --git a/test/common_plat/performance/odp_bench_packet.c

>> b/test/common_plat/performance/odp_bench_packet.c

>> index 7a3a004..6b73423 100644

>> --- a/test/common_plat/performance/odp_bench_packet.c

>> +++ b/test/common_plat/performance/odp_bench_packet.c

>> @@ -35,7 +35,7 @@

>> #define TEST_MIN_PKT_SIZE 64

>> 

>> /** Maximum test packet size */

>> -#define TEST_MAX_PKT_SIZE 2048

>> +#define TEST_MAX_PKT_SIZE 1518

>> 

>> /** Number of test runs per individual benchmark */

>> #define TEST_REPEAT_COUNT 1000

>> @@ -78,7 +78,7 @@ ODP_STATIC_ASSERT((TEST_ALIGN_OFFSET + TEST_ALIGN_LEN)

>> <= TEST_MIN_PKT_SIZE,

>> 

>> /** Test packet sizes */

>> const uint32_t test_packet_len[] = {WARM_UP, TEST_MIN_PKT_SIZE, 128, 256,

>> 512,

>> -                                   1024, 1518, TEST_MAX_PKT_SIZE};

>> +                                   1024, TEST_MAX_PKT_SIZE};

>> 

>> /**

>>  * Parsed command line arguments

>> --

>> 1.9.1

>> 

>>
diff mbox

Patch

diff --git a/test/common_plat/performance/odp_bench_packet.c b/test/common_plat/performance/odp_bench_packet.c
index 7a3a004..6b73423 100644
--- a/test/common_plat/performance/odp_bench_packet.c
+++ b/test/common_plat/performance/odp_bench_packet.c
@@ -35,7 +35,7 @@ 
 #define TEST_MIN_PKT_SIZE 64
 
 /** Maximum test packet size */
-#define TEST_MAX_PKT_SIZE 2048
+#define TEST_MAX_PKT_SIZE 1518
 
 /** Number of test runs per individual benchmark */
 #define TEST_REPEAT_COUNT 1000
@@ -78,7 +78,7 @@  ODP_STATIC_ASSERT((TEST_ALIGN_OFFSET + TEST_ALIGN_LEN) <= TEST_MIN_PKT_SIZE,
 
 /** Test packet sizes */
 const uint32_t test_packet_len[] = {WARM_UP, TEST_MIN_PKT_SIZE, 128, 256, 512,
-				    1024, 1518, TEST_MAX_PKT_SIZE};
+				    1024, TEST_MAX_PKT_SIZE};
 
 /**
  * Parsed command line arguments