diff mbox

[API-NEXT,v2,08/11] validation: pktio: add pktin timestamping test

Message ID D6FC4FAD05B5CC44927BD3216F244C2A03C14177@DEMUMBX011.nsn-intra.net
State New
Headers show

Commit Message

Elo, Matias (Nokia - FI/Espoo) April 20, 2016, 6:22 a.m. UTC
OK, I just sent rebased patch set.

-Matias

From: EXT Bill Fischofer [mailto:bill.fischofer@linaro.org]

Sent: Tuesday, April 19, 2016 11:18 PM
To: Elo, Matias (Nokia - FI/Espoo) <matias.elo@nokia.com>
Cc: LNG ODP Mailman List <lng-odp@lists.linaro.org>
Subject: Re: [lng-odp] [API-NEXT PATCH v2 08/11] validation: pktio: add pktin timestamping test

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<mailto: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<mailto:matias.elo@nokia.com>>

---
 test/validation/pktio/pktio.c | 123 ++++++++++++++++++++++++++++++++++++++++++
 test/validation/pktio/pktio.h |   2 +
 2 files changed, 125 insertions(+)

--
1.9.1

_______________________________________________
lng-odp mailing list
lng-odp@lists.linaro.org<mailto:lng-odp@lists.linaro.org>
https://lists.linaro.org/mailman/listinfo/lng-odp
diff mbox

Patch

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<http://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[];