| Message ID | 1460725470-4457-9-git-send-email-matias.elo@nokia.com |
|---|---|
| State | Superseded |
| Headers | show |
Rebase needed? Attempting to apply part 8 of the v2 patch yields:
bill@Ubuntu15:~/linaro/matias$ git am --reject ~/Mail/Incoming/Matias/8
Applying: validation: pktio: add pktin timestamping test
Checking patch test/validation/pktio/pktio.c...
error: while searching for:
#define TX_BATCH_LEN 4
#define MAX_QUEUES 128
#undef DEBUG_STATS
/** interface names used for testing */
error: patch failed: test/validation/pktio/pktio.c:26
error: while searching for:
CU_ASSERT_FATAL(ret == 0);
}
static int create_pool(const char *iface, int num)
{
char pool_name[ODP_POOL_NAME_LEN];
error: patch failed: test/validation/pktio/pktio.c:1646
error: while searching for:
ODP_TEST_INFO(pktio_test_recv_multi_event),
ODP_TEST_INFO_CONDITIONAL(pktio_test_statistics_counters,
pktio_check_statistics_counters),
ODP_TEST_INFO_NULL
};
error: patch failed: test/validation/pktio/pktio.c:1778
Checking patch test/validation/pktio/pktio.h...
error: while searching for:
void pktio_test_recv_multi_event(void);
int pktio_check_statistics_counters(void);
void pktio_test_statistics_counters(void);
/* test arrays: */
extern odp_testinfo_t pktio_suite[];
error: patch failed: test/validation/pktio/pktio.h:40
Applying patch test/validation/pktio/pktio.c with 3 rejects...
Rejected hunk #1.
Rejected hunk #2.
Rejected hunk #3.
Applying patch test/validation/pktio/pktio.h with 1 reject...
Rejected hunk #1.
Patch failed at 0001 validation: pktio: add pktin timestamping test
The copy of the patch that failed is found in:
/home/bill/linaro/matias/.git/rebase-apply/patch
When you have resolved this problem, run "git am --continue".
If you prefer to skip this patch, run "git am --skip" instead.
To restore the original branch and stop patching, run "git am --abort".
On Fri, Apr 15, 2016 at 8:04 AM, Matias Elo <matias.elo@nokia.com> wrote:
> Add validation test for packet input timestamping. Tests
> also odp_pktin_ts_res() and odp_pktin_ts_from_ns().
>
> Signed-off-by: Matias Elo <matias.elo@nokia.com>
> ---
> test/validation/pktio/pktio.c | 123
> ++++++++++++++++++++++++++++++++++++++++++
> test/validation/pktio/pktio.h | 2 +
> 2 files changed, 125 insertions(+)
>
> diff --git a/test/validation/pktio/pktio.c b/test/validation/pktio/pktio.c
> index 162156a..39b825c 100644
> --- a/test/validation/pktio/pktio.c
> +++ b/test/validation/pktio/pktio.c
> @@ -26,6 +26,11 @@
> #define TX_BATCH_LEN 4
> #define MAX_QUEUES 128
>
> +#define PKTIN_TS_INTERVAL (50 * ODP_TIME_MSEC_IN_NS)
> +#define PKTIN_TS_MIN_RES 1000
> +#define PKTIN_TS_MAX_RES 10000000000
> +#define PKTIN_TS_CMP_RES 1
> +
> #undef DEBUG_STATS
>
> /** interface names used for testing */
> @@ -1646,6 +1651,122 @@ void pktio_test_send_on_ronly(void)
> CU_ASSERT_FATAL(ret == 0);
> }
>
> +int pktio_check_pktin_ts(void)
> +{
> + odp_pktio_t pktio;
> + odp_pktio_capability_t capa;
> + odp_pktio_param_t pktio_param;
> + int ret;
> +
> + odp_pktio_param_init(&pktio_param);
> + pktio_param.in_mode = ODP_PKTIN_MODE_DIRECT;
> +
> + pktio = odp_pktio_open(iface_name[0], pool[0], &pktio_param);
> + if (pktio == ODP_PKTIO_INVALID)
> + return ODP_TEST_INACTIVE;
> +
> + ret = odp_pktio_capability(pktio, &capa);
> + (void)odp_pktio_close(pktio);
> +
> + if (ret < 0 || !capa.config.pktin.bit.ts_all)
> + return ODP_TEST_INACTIVE;
> +
> + return ODP_TEST_ACTIVE;
> +}
> +
> +void pktio_test_pktin_ts(void)
> +{
> + odp_pktio_t pktio_tx, pktio_rx;
> + odp_pktio_t pktio[MAX_NUM_IFACES];
> + pktio_info_t pktio_rx_info;
> + odp_pktio_capability_t capa;
> + odp_pktio_config_t config;
> + odp_pktout_queue_t pktout_queue;
> + odp_packet_t pkt_tbl[TX_BATCH_LEN];
> + uint32_t pkt_seq[TX_BATCH_LEN];
> + uint64_t ns1, ns2;
> + uint64_t res;
> + odp_time_t ts_prev;
> + odp_time_t ts;
> + int num_rx = 0;
> + int ret;
> + int i;
> +
> + CU_ASSERT_FATAL(num_ifaces >= 1);
> +
> + /* Open and configure interfaces */
> + for (i = 0; i < num_ifaces; ++i) {
> + pktio[i] = create_pktio(i, ODP_PKTIN_MODE_DIRECT,
> + ODP_PKTOUT_MODE_DIRECT);
> + CU_ASSERT_FATAL(pktio[i] != ODP_PKTIO_INVALID);
> +
> + CU_ASSERT_FATAL(odp_pktio_capability(pktio[i], &capa) ==
> 0);
> + CU_ASSERT_FATAL(capa.config.pktin.bit.ts_all);
> +
> + odp_pktio_config_init(&config);
> + config.pktin.bit.ts_all = 1;
> + CU_ASSERT_FATAL(odp_pktio_config(pktio[i], &config) == 0);
> +
> + CU_ASSERT_FATAL(odp_pktio_start(pktio[i]) == 0);
> + }
> +
> + for (i = 0; i < num_ifaces; i++)
> + _pktio_wait_linkup(pktio[i]);
> +
> + pktio_tx = pktio[0];
> + pktio_rx = (num_ifaces > 1) ? pktio[1] : pktio_tx;
> + pktio_rx_info.id = pktio_rx;
> + pktio_rx_info.inq = ODP_QUEUE_INVALID;
> + pktio_rx_info.in_mode = ODP_PKTIN_MODE_DIRECT;
> +
> + /* Test odp_pktin_ts_res() and odp_pktin_ts_from_ns() */
> + res = odp_pktin_ts_res(pktio_tx);
> + CU_ASSERT(res > PKTIN_TS_MIN_RES);
> + CU_ASSERT(res < PKTIN_TS_MAX_RES);
> + ns1 = 100;
> + ts = odp_pktin_ts_from_ns(pktio_tx, ns1);
> + ns2 = odp_time_to_ns(ts);
> + /* Allow some arithmetic tolerance */
> + CU_ASSERT((ns2 <= (ns1 + PKTIN_TS_CMP_RES)) &&
> + (ns2 >= (ns1 - PKTIN_TS_CMP_RES)));
> +
> + ret = create_packets(pkt_tbl, pkt_seq, TX_BATCH_LEN, pktio_tx,
> + pktio_rx);
> + CU_ASSERT_FATAL(ret == TX_BATCH_LEN);
> +
> + ret = odp_pktout_queue(pktio_tx, &pktout_queue, 1);
> + CU_ASSERT_FATAL(ret > 0);
> +
> + /* Send packets one at a time and add delay between the packets */
> + for (i = 0; i < TX_BATCH_LEN; i++) {
> + CU_ASSERT_FATAL(odp_pktout_send(pktout_queue,
> + &pkt_tbl[i], 1) == 1);
> + ret = wait_for_packets(&pktio_rx_info, &pkt_tbl[i],
> &pkt_seq[i],
> + 1, ODP_PKTIN_MODE_DIRECT,
> + ODP_TIME_SEC_IN_NS);
> + if (ret != 1)
> + break;
> + odp_time_wait_ns(PKTIN_TS_INTERVAL);
> + }
> + num_rx = i;
> + CU_ASSERT(num_rx == TX_BATCH_LEN);
> +
> + ts_prev = ODP_TIME_NULL;
> + for (i = 0; i < num_rx; i++) {
> + ts = odp_packet_ts(pkt_tbl[i]);
> +
> + CU_ASSERT(odp_time_cmp(ts, ts_prev) > 0);
> +
> + ts_prev = ts;
> + odp_packet_free(pkt_tbl[i]);
> + }
> +
> + for (i = 0; i < num_ifaces; i++) {
> + CU_ASSERT_FATAL(odp_pktio_stop(pktio[i]) == 0);
> + CU_ASSERT_FATAL(odp_pktio_close(pktio[i]) == 0);
> + }
> +}
> +
> static int create_pool(const char *iface, int num)
> {
> char pool_name[ODP_POOL_NAME_LEN];
> @@ -1778,6 +1899,8 @@ odp_testinfo_t pktio_suite_unsegmented[] = {
> ODP_TEST_INFO(pktio_test_recv_multi_event),
> ODP_TEST_INFO_CONDITIONAL(pktio_test_statistics_counters,
> pktio_check_statistics_counters),
> + ODP_TEST_INFO_CONDITIONAL(pktio_test_pktin_ts,
> + pktio_check_pktin_ts),
> ODP_TEST_INFO_NULL
> };
>
> diff --git a/test/validation/pktio/pktio.h b/test/validation/pktio/pktio.h
> index 4bc2980..33f8649 100644
> --- a/test/validation/pktio/pktio.h
> +++ b/test/validation/pktio/pktio.h
> @@ -40,6 +40,8 @@ void pktio_test_sched_multi_event(void);
> void pktio_test_recv_multi_event(void);
> int pktio_check_statistics_counters(void);
> void pktio_test_statistics_counters(void);
> +int pktio_check_pktin_ts(void);
> +void pktio_test_pktin_ts(void);
>
> /* test arrays: */
> extern odp_testinfo_t pktio_suite[];
> --
> 1.9.1
>
> _______________________________________________
> lng-odp mailing list
> lng-odp@lists.linaro.org
> https://lists.linaro.org/mailman/listinfo/lng-odp
>
diff --git a/test/validation/pktio/pktio.c b/test/validation/pktio/pktio.c index 162156a..39b825c 100644 --- a/test/validation/pktio/pktio.c +++ b/test/validation/pktio/pktio.c @@ -26,6 +26,11 @@ #define TX_BATCH_LEN 4 #define MAX_QUEUES 128 +#define PKTIN_TS_INTERVAL (50 * ODP_TIME_MSEC_IN_NS) +#define PKTIN_TS_MIN_RES 1000 +#define PKTIN_TS_MAX_RES 10000000000 +#define PKTIN_TS_CMP_RES 1 + #undef DEBUG_STATS /** interface names used for testing */ @@ -1646,6 +1651,122 @@ void pktio_test_send_on_ronly(void) CU_ASSERT_FATAL(ret == 0); } +int pktio_check_pktin_ts(void) +{ + odp_pktio_t pktio; + odp_pktio_capability_t capa; + odp_pktio_param_t pktio_param; + int ret; + + odp_pktio_param_init(&pktio_param); + pktio_param.in_mode = ODP_PKTIN_MODE_DIRECT; + + pktio = odp_pktio_open(iface_name[0], pool[0], &pktio_param); + if (pktio == ODP_PKTIO_INVALID) + return ODP_TEST_INACTIVE; + + ret = odp_pktio_capability(pktio, &capa); + (void)odp_pktio_close(pktio); + + if (ret < 0 || !capa.config.pktin.bit.ts_all) + return ODP_TEST_INACTIVE; + + return ODP_TEST_ACTIVE; +} + +void pktio_test_pktin_ts(void) +{ + odp_pktio_t pktio_tx, pktio_rx; + odp_pktio_t pktio[MAX_NUM_IFACES]; + pktio_info_t pktio_rx_info; + odp_pktio_capability_t capa; + odp_pktio_config_t config; + odp_pktout_queue_t pktout_queue; + odp_packet_t pkt_tbl[TX_BATCH_LEN]; + uint32_t pkt_seq[TX_BATCH_LEN]; + uint64_t ns1, ns2; + uint64_t res; + odp_time_t ts_prev; + odp_time_t ts; + int num_rx = 0; + int ret; + int i; + + CU_ASSERT_FATAL(num_ifaces >= 1); + + /* Open and configure interfaces */ + for (i = 0; i < num_ifaces; ++i) { + pktio[i] = create_pktio(i, ODP_PKTIN_MODE_DIRECT, + ODP_PKTOUT_MODE_DIRECT); + CU_ASSERT_FATAL(pktio[i] != ODP_PKTIO_INVALID); + + CU_ASSERT_FATAL(odp_pktio_capability(pktio[i], &capa) == 0); + CU_ASSERT_FATAL(capa.config.pktin.bit.ts_all); + + odp_pktio_config_init(&config); + config.pktin.bit.ts_all = 1; + CU_ASSERT_FATAL(odp_pktio_config(pktio[i], &config) == 0); + + CU_ASSERT_FATAL(odp_pktio_start(pktio[i]) == 0); + } + + for (i = 0; i < num_ifaces; i++) + _pktio_wait_linkup(pktio[i]); + + pktio_tx = pktio[0]; + pktio_rx = (num_ifaces > 1) ? pktio[1] : pktio_tx; + pktio_rx_info.id = pktio_rx; + pktio_rx_info.inq = ODP_QUEUE_INVALID; + pktio_rx_info.in_mode = ODP_PKTIN_MODE_DIRECT; + + /* Test odp_pktin_ts_res() and odp_pktin_ts_from_ns() */ + res = odp_pktin_ts_res(pktio_tx); + CU_ASSERT(res > PKTIN_TS_MIN_RES); + CU_ASSERT(res < PKTIN_TS_MAX_RES); + ns1 = 100; + ts = odp_pktin_ts_from_ns(pktio_tx, ns1); + ns2 = odp_time_to_ns(ts); + /* Allow some arithmetic tolerance */ + CU_ASSERT((ns2 <= (ns1 + PKTIN_TS_CMP_RES)) && + (ns2 >= (ns1 - PKTIN_TS_CMP_RES))); + + ret = create_packets(pkt_tbl, pkt_seq, TX_BATCH_LEN, pktio_tx, + pktio_rx); + CU_ASSERT_FATAL(ret == TX_BATCH_LEN); + + ret = odp_pktout_queue(pktio_tx, &pktout_queue, 1); + CU_ASSERT_FATAL(ret > 0); + + /* Send packets one at a time and add delay between the packets */ + for (i = 0; i < TX_BATCH_LEN; i++) { + CU_ASSERT_FATAL(odp_pktout_send(pktout_queue, + &pkt_tbl[i], 1) == 1); + ret = wait_for_packets(&pktio_rx_info, &pkt_tbl[i], &pkt_seq[i], + 1, ODP_PKTIN_MODE_DIRECT, + ODP_TIME_SEC_IN_NS); + if (ret != 1) + break; + odp_time_wait_ns(PKTIN_TS_INTERVAL); + } + num_rx = i; + CU_ASSERT(num_rx == TX_BATCH_LEN); + + ts_prev = ODP_TIME_NULL; + for (i = 0; i < num_rx; i++) { + ts = odp_packet_ts(pkt_tbl[i]); + + CU_ASSERT(odp_time_cmp(ts, ts_prev) > 0); + + ts_prev = ts; + odp_packet_free(pkt_tbl[i]); + } + + for (i = 0; i < num_ifaces; i++) { + CU_ASSERT_FATAL(odp_pktio_stop(pktio[i]) == 0); + CU_ASSERT_FATAL(odp_pktio_close(pktio[i]) == 0); + } +} + static int create_pool(const char *iface, int num) { char pool_name[ODP_POOL_NAME_LEN]; @@ -1778,6 +1899,8 @@ odp_testinfo_t pktio_suite_unsegmented[] = { ODP_TEST_INFO(pktio_test_recv_multi_event), ODP_TEST_INFO_CONDITIONAL(pktio_test_statistics_counters, pktio_check_statistics_counters), + ODP_TEST_INFO_CONDITIONAL(pktio_test_pktin_ts, + pktio_check_pktin_ts), ODP_TEST_INFO_NULL }; diff --git a/test/validation/pktio/pktio.h b/test/validation/pktio/pktio.h index 4bc2980..33f8649 100644 --- a/test/validation/pktio/pktio.h +++ b/test/validation/pktio/pktio.h @@ -40,6 +40,8 @@ void pktio_test_sched_multi_event(void); void pktio_test_recv_multi_event(void); int pktio_check_statistics_counters(void); void pktio_test_statistics_counters(void); +int pktio_check_pktin_ts(void); +void pktio_test_pktin_ts(void); /* test arrays: */ extern odp_testinfo_t pktio_suite[];
Add validation test for packet input timestamping. Tests also odp_pktin_ts_res() and odp_pktin_ts_from_ns(). Signed-off-by: Matias Elo <matias.elo@nokia.com> --- test/validation/pktio/pktio.c | 123 ++++++++++++++++++++++++++++++++++++++++++ test/validation/pktio/pktio.h | 2 + 2 files changed, 125 insertions(+)