diff mbox

[PATCHv8] validation: pktio: add mac, promisc and mtu tests

Message ID 1419259936-4361-1-git-send-email-maxim.uvarov@linaro.org
State Accepted
Commit 798031917802eeba232a9a1cde7bc92e791b0e64
Headers show

Commit Message

Maxim Uvarov Dec. 22, 2014, 2:52 p.m. UTC
Signed-off-by: Maxim Uvarov <maxim.uvarov@linaro.org>
---
 v8: fixed Mike's comments

 test/validation/odp_pktio.c | 89 ++++++++++++++++++++++++++++++++++++++++-----
 1 file changed, 80 insertions(+), 9 deletions(-)

Comments

Mike Holmes Dec. 22, 2014, 3:10 p.m. UTC | #1
On 22 December 2014 at 09:52, Maxim Uvarov <maxim.uvarov@linaro.org> wrote:

> Signed-off-by: Maxim Uvarov <maxim.uvarov@linaro.org>
>

Reviewed-by: Mike Holmes <mike.holmes@linaro.org>

Note, the conversion to tabs was so that the lists matched the other tests.



> ---
>  v8: fixed Mike's comments
>
>  test/validation/odp_pktio.c | 89
> ++++++++++++++++++++++++++++++++++++++++-----
>  1 file changed, 80 insertions(+), 9 deletions(-)
>
> diff --git a/test/validation/odp_pktio.c b/test/validation/odp_pktio.c
> index f7dc80b..ea721a8 100644
> --- a/test/validation/odp_pktio.c
> +++ b/test/validation/odp_pktio.c
> @@ -403,6 +403,74 @@ static void test_odp_pktio_sched_multi(void)
>         pktio_test_txrx(ODP_QUEUE_TYPE_SCHED, 4);
>  }
>
> +static void test_odp_pktio_mtu(void)
> +{
> +       int ret;
> +       int mtu;
> +       odp_pktio_t pktio = create_pktio(iface_name[0]);
> +
> +       mtu = odp_pktio_mtu(pktio);
> +       CU_ASSERT(mtu > 0);
> +
> +       printf(" %d ",  mtu);
> +
> +       ret = odp_pktio_close(pktio);
> +       CU_ASSERT(ret == 0);
> +
> +       return;
> +}
> +
> +static void test_odp_pktio_promisc(void)
> +{
> +       int ret;
> +       odp_pktio_t pktio = create_pktio(iface_name[0]);
> +
> +       ret = odp_pktio_promisc_mode_set(pktio, 1);
> +       CU_ASSERT(0 == ret);
> +
> +       /* Verify that promisc mode set */
> +       ret = odp_pktio_promisc_mode(pktio);
> +       CU_ASSERT(1 == ret);
> +
> +       ret = odp_pktio_promisc_mode_set(pktio, 0);
> +       CU_ASSERT(0 == ret);
> +
> +       /* Verify that promisc mode is not set */
> +       ret = odp_pktio_promisc_mode(pktio);
> +       CU_ASSERT(0 == ret);
> +
> +       ret = odp_pktio_close(pktio);
> +       CU_ASSERT(ret == 0);
> +
> +       return;
> +}
> +
> +static void test_odp_pktio_mac(void)
> +{
> +       unsigned char mac_addr[ODPH_ETHADDR_LEN];
> +       size_t mac_len;
> +       int ret;
> +       odp_pktio_t pktio = create_pktio(iface_name[0]);
> +
> +       printf("testing mac for %s\n", iface_name[0]);
> +
> +       mac_len = odp_pktio_mac_addr(pktio, mac_addr, ODPH_ETHADDR_LEN);
> +       CU_ASSERT(ODPH_ETHADDR_LEN == mac_len);
> +
> +       printf(" %X:%X:%X:%X:%X:%X ",
> +              mac_addr[0], mac_addr[1], mac_addr[2],
> +              mac_addr[3], mac_addr[4], mac_addr[5]);
> +
> +       /* Fail case: wrong addr_size. Expected 0. */
> +       mac_len = odp_pktio_mac_addr(pktio, mac_addr, 2);
> +       CU_ASSERT(0 == mac_len);
> +
> +       ret = odp_pktio_close(pktio);
> +       CU_ASSERT(0 == ret);
> +
> +       return;
> +}
> +
>  static void test_odp_pktio_open(void)
>  {
>         odp_pktio_t pktio;
> @@ -483,19 +551,22 @@ static int term_pktio_suite(void)
>  }
>
>  CU_TestInfo pktio_tests[] = {
> -       {"pktio open",         test_odp_pktio_open},
> -       {"pktio close",        test_odp_pktio_close},
> -       {"pktio inq",          test_odp_pktio_inq},
> -       {"pktio outq",         test_odp_pktio_outq},
> -       {"pktio poll queues",  test_odp_pktio_poll_queue},
> -       {"pktio poll multi",   test_odp_pktio_poll_multi},
> -       {"pktio sched queues", test_odp_pktio_sched_queue},
> -       {"pktio sched multi",  test_odp_pktio_sched_multi},
> +       {"pktio open",          test_odp_pktio_open},
> +       {"pktio close",         test_odp_pktio_close},
> +       {"pktio inq",           test_odp_pktio_inq},
> +       {"pktio outq",          test_odp_pktio_outq},
> +       {"pktio poll queues",   test_odp_pktio_poll_queue},
> +       {"pktio poll multi",    test_odp_pktio_poll_multi},
> +       {"pktio sched queues",  test_odp_pktio_sched_queue},
> +       {"pktio sched multi",   test_odp_pktio_sched_multi},
> +       {"pktio mtu",           test_odp_pktio_mtu},
> +       {"pktio promisc mode",  test_odp_pktio_promisc},
> +       {"pktio mac",           test_odp_pktio_mac},
>         CU_TEST_INFO_NULL
>  };
>
>  CU_SuiteInfo odp_testsuites[] = {
> -       {"odp_pktio",
> +       {"Packet I/O",
>                 init_pktio_suite, term_pktio_suite, NULL, NULL,
> pktio_tests},
>         CU_SUITE_INFO_NULL
>  };
> --
> 1.8.5.1.163.gd7aced9
>
>
> _______________________________________________
> lng-odp mailing list
> lng-odp@lists.linaro.org
> http://lists.linaro.org/mailman/listinfo/lng-odp
>
diff mbox

Patch

diff --git a/test/validation/odp_pktio.c b/test/validation/odp_pktio.c
index f7dc80b..ea721a8 100644
--- a/test/validation/odp_pktio.c
+++ b/test/validation/odp_pktio.c
@@ -403,6 +403,74 @@  static void test_odp_pktio_sched_multi(void)
 	pktio_test_txrx(ODP_QUEUE_TYPE_SCHED, 4);
 }
 
+static void test_odp_pktio_mtu(void)
+{
+	int ret;
+	int mtu;
+	odp_pktio_t pktio = create_pktio(iface_name[0]);
+
+	mtu = odp_pktio_mtu(pktio);
+	CU_ASSERT(mtu > 0);
+
+	printf(" %d ",  mtu);
+
+	ret = odp_pktio_close(pktio);
+	CU_ASSERT(ret == 0);
+
+	return;
+}
+
+static void test_odp_pktio_promisc(void)
+{
+	int ret;
+	odp_pktio_t pktio = create_pktio(iface_name[0]);
+
+	ret = odp_pktio_promisc_mode_set(pktio, 1);
+	CU_ASSERT(0 == ret);
+
+	/* Verify that promisc mode set */
+	ret = odp_pktio_promisc_mode(pktio);
+	CU_ASSERT(1 == ret);
+
+	ret = odp_pktio_promisc_mode_set(pktio, 0);
+	CU_ASSERT(0 == ret);
+
+	/* Verify that promisc mode is not set */
+	ret = odp_pktio_promisc_mode(pktio);
+	CU_ASSERT(0 == ret);
+
+	ret = odp_pktio_close(pktio);
+	CU_ASSERT(ret == 0);
+
+	return;
+}
+
+static void test_odp_pktio_mac(void)
+{
+	unsigned char mac_addr[ODPH_ETHADDR_LEN];
+	size_t mac_len;
+	int ret;
+	odp_pktio_t pktio = create_pktio(iface_name[0]);
+
+	printf("testing mac for %s\n", iface_name[0]);
+
+	mac_len = odp_pktio_mac_addr(pktio, mac_addr, ODPH_ETHADDR_LEN);
+	CU_ASSERT(ODPH_ETHADDR_LEN == mac_len);
+
+	printf(" %X:%X:%X:%X:%X:%X ",
+	       mac_addr[0], mac_addr[1], mac_addr[2],
+	       mac_addr[3], mac_addr[4], mac_addr[5]);
+
+	/* Fail case: wrong addr_size. Expected 0. */
+	mac_len = odp_pktio_mac_addr(pktio, mac_addr, 2);
+	CU_ASSERT(0 == mac_len);
+
+	ret = odp_pktio_close(pktio);
+	CU_ASSERT(0 == ret);
+
+	return;
+}
+
 static void test_odp_pktio_open(void)
 {
 	odp_pktio_t pktio;
@@ -483,19 +551,22 @@  static int term_pktio_suite(void)
 }
 
 CU_TestInfo pktio_tests[] = {
-	{"pktio open",         test_odp_pktio_open},
-	{"pktio close",        test_odp_pktio_close},
-	{"pktio inq",          test_odp_pktio_inq},
-	{"pktio outq",         test_odp_pktio_outq},
-	{"pktio poll queues",  test_odp_pktio_poll_queue},
-	{"pktio poll multi",   test_odp_pktio_poll_multi},
-	{"pktio sched queues", test_odp_pktio_sched_queue},
-	{"pktio sched multi",  test_odp_pktio_sched_multi},
+	{"pktio open",		test_odp_pktio_open},
+	{"pktio close",		test_odp_pktio_close},
+	{"pktio inq",		test_odp_pktio_inq},
+	{"pktio outq",		test_odp_pktio_outq},
+	{"pktio poll queues",	test_odp_pktio_poll_queue},
+	{"pktio poll multi",	test_odp_pktio_poll_multi},
+	{"pktio sched queues",	test_odp_pktio_sched_queue},
+	{"pktio sched multi",	test_odp_pktio_sched_multi},
+	{"pktio mtu",		test_odp_pktio_mtu},
+	{"pktio promisc mode",	test_odp_pktio_promisc},
+	{"pktio mac",		test_odp_pktio_mac},
 	CU_TEST_INFO_NULL
 };
 
 CU_SuiteInfo odp_testsuites[] = {
-	{"odp_pktio",
+	{"Packet I/O",
 		init_pktio_suite, term_pktio_suite, NULL, NULL, pktio_tests},
 	CU_SUITE_INFO_NULL
 };