ping^2
On 11 December 2015 at 16:54, Stuart Haslam <stuart.has...@linaro.org> wrote:
> ping - needs review.
>
> On 28 October 2015 at 18:45, Stuart Haslam <stuart.has...@linaro.org> wrote:
>> The results from the initial test run are often worse than would
>> normally be expected due to there being no warm up phase. As a
>> simple way to warm up run first test stage twice, ignoring the
>> results of the first run.
>>
>> Signed-off-by: Stuart Haslam <stuart.has...@linaro.org>
>> ---
>> test/performance/odp_pktio_perf.c | 12 ++++++++++--
>> 1 file changed, 10 insertions(+), 2 deletions(-)
>>
>> diff --git a/test/performance/odp_pktio_perf.c
>> b/test/performance/odp_pktio_perf.c
>> index efd26dc..38d684a 100644
>> --- a/test/performance/odp_pktio_perf.c
>> +++ b/test/performance/odp_pktio_perf.c
>> @@ -134,6 +134,7 @@ typedef struct {
>> uint64_t pps_curr; /* Current attempted PPS */
>> uint64_t pps_pass; /* Highest passing PPS */
>> uint64_t pps_fail; /* Lowest failing PPS */
>> + int warmup; /* Warmup stage - ignore results */
>> } test_status_t;
>>
>> /* Thread specific arguments */
>> @@ -647,7 +648,10 @@ static int run_test_single(odp_cpumask_t *thd_mask_tx,
>> /* wait for receivers */
>> odph_linux_pthread_join(&thd_tbl[0], num_rx_workers);
>>
>> - return process_results(expected_tx_cnt, status);
>> + if (!status->warmup)
>> + return process_results(expected_tx_cnt, status);
>> +
>> + return 1;
>> }
>>
>> static int run_test(void)
>> @@ -659,6 +663,7 @@ static int run_test(void)
>> .pps_curr = gbl_args->args.pps,
>> .pps_pass = 0,
>> .pps_fail = 0,
>> + .warmup = 1,
>> };
>>
>> if (setup_txrx_masks(&txmask, &rxmask) != 0)
>> @@ -679,6 +684,10 @@ static int run_test(void)
>> printf("%s ", gbl_args->args.ifaces[i]);
>> printf("\n");
>>
>> + /* first time just run the test but throw away the results */
>> + run_test_single(&txmask, &rxmask, &status);
>> + status.warmup = 0;
>> +
>> while (ret > 0)
>> ret = run_test_single(&txmask, &rxmask, &status);
>>
>> @@ -957,7 +966,6 @@ static void parse_args(int argc, char *argv[],
>> test_args_t *args)
>> LOG_ABORT("Failed to alloc iface storage\n");
>>
>> strcpy(args->if_str, optarg);
>> -
>> for (token = strtok(args->if_str, ",");
>> token != NULL && args->num_ifaces <
>> MAX_NUM_IFACES;
>> token = strtok(NULL, ","))
>> --
>> 2.1.1
>>
_______________________________________________
lng-odp mailing list
lng-odp@lists.linaro.org
https://lists.linaro.org/mailman/listinfo/lng-odp
