app/test: rework command registration

[dpdk.git] / app / test / test_devargs.c

diff --git a/app/test/test_devargs.c b/app/test/test_devargs.c
index 08fb781..151ac2a 100644 (file)
--- a/app/test/test_devargs.c
+++ b/app/test/test_devargs.c
@@ -48,8 +48,7 @@ static void free_devargs_list(void)
        while (!TAILQ_EMPTY(&devargs_list)) {
                devargs = TAILQ_FIRST(&devargs_list);
                TAILQ_REMOVE(&devargs_list, devargs, next);
-               if (devargs->args)
-                       free(devargs->args);
+               free(devargs->args);
                free(devargs);
        }
 }
@@ -91,8 +90,8 @@ test_devargs(void)
        if (rte_eal_devargs_add(RTE_DEVTYPE_VIRTUAL, "eth_ring1,k1=val,k2=val2") < 0)
                goto fail;
        devargs = TAILQ_FIRST(&devargs_list);
-       if (strncmp(devargs->virtual.drv_name, "eth_ring1",
-                       sizeof(devargs->virtual.drv_name)) != 0)
+       if (strncmp(devargs->virt.drv_name, "eth_ring1",
+                       sizeof(devargs->virt.drv_name)) != 0)
                goto fail;
        if (!devargs->args || strcmp(devargs->args, "k1=val,k2=val2") != 0)
                goto fail;
@@ -107,7 +106,7 @@ test_devargs(void)
                devargs->pci.addr.devid != 0 ||
                devargs->pci.addr.function != 1)
                goto fail;
-       if (devargs->args)
+       if (!devargs->args || strcmp(devargs->args, "") != 0)
                goto fail;
        free_devargs_list();
 
@@ -132,8 +131,4 @@ test_devargs(void)
        return -1;
 }
 
-static struct test_command devargs_cmd = {
-       .command = "devargs_autotest",
-       .callback = test_devargs,
-};
-REGISTER_TEST_COMMAND(devargs_cmd);
+REGISTER_TEST_COMMAND(devargs_autotest, test_devargs);