[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[Xen-devel] [PATCH v2 3/6] xen: remove xen dependency in app, examples, test



Signed-off-by: Jianfeng Tan <jianfeng.tan@xxxxxxxxx>
Acked-by: Bruce Richardson <bruce.richardson@xxxxxxxxx>
---
 examples/ip_pipeline/app.h          |  4 --
 examples/ip_pipeline/config_parse.c | 19 ---------
 examples/ip_pipeline/init.c         |  5 ---
 examples/kni/main.c                 |  3 --
 test/test/process.h                 | 10 -----
 test/test/test.c                    |  4 --
 test/test/test_eal_flags.c          | 81 -------------------------------------
 7 files changed, 126 deletions(-)

diff --git a/examples/ip_pipeline/app.h b/examples/ip_pipeline/app.h
index e41290e..94e7a6d 100644
--- a/examples/ip_pipeline/app.h
+++ b/examples/ip_pipeline/app.h
@@ -428,10 +428,6 @@ struct app_eal_params {
        /* Interrupt mode for VFIO (legacy|msi|msix) */
        char *vfio_intr;
 
-       /* Support running on Xen dom0 without hugetlbfs */
-       uint32_t xen_dom0_present;
-       int xen_dom0;
-
        uint32_t parsed;
 };
 
diff --git a/examples/ip_pipeline/config_parse.c 
b/examples/ip_pipeline/config_parse.c
index 0b76134..3211c6a 100644
--- a/examples/ip_pipeline/config_parse.c
+++ b/examples/ip_pipeline/config_parse.c
@@ -809,21 +809,6 @@ parse_eal(struct app_params *app,
                        continue;
                }
 
-               /* xen_dom0 */
-               if (strcmp(entry->name, "xen_dom0") == 0) {
-                       int val;
-
-                       PARSE_ERROR_DUPLICATE((p->xen_dom0_present == 0),
-                               section_name,
-                               entry->name);
-                       p->xen_dom0_present = 1;
-
-                       val = parser_read_arg_bool(entry->value);
-                       PARSE_ERROR((val >= 0), section_name, entry->name);
-                       p->xen_dom0 = val;
-                       continue;
-               }
-
                /* unrecognized */
                PARSE_ERROR_INVALID(0, section_name, entry->name);
        }
@@ -2643,10 +2628,6 @@ save_eal_params(struct app_params *app, FILE *f)
        if (p->vfio_intr)
                fprintf(f, "%s = %s\n", "vfio_intr", p->vfio_intr);
 
-       if (p->xen_dom0_present)
-               fprintf(f, "%s = %s\n", "xen_dom0",
-                       (p->xen_dom0) ? "yes" : "no");
-
        fputc('\n', f);
 }
 
diff --git a/examples/ip_pipeline/init.c b/examples/ip_pipeline/init.c
index 7cde49a..034c238 100644
--- a/examples/ip_pipeline/init.c
+++ b/examples/ip_pipeline/init.c
@@ -296,11 +296,6 @@ app_init_eal(struct app_params *app)
                app->eal_argv[n_args++] = strdup(buffer);
        }
 
-       if ((p->xen_dom0_present) && (p->xen_dom0)) {
-               snprintf(buffer, sizeof(buffer), "--xen-dom0");
-               app->eal_argv[n_args++] = strdup(buffer);
-       }
-
        snprintf(buffer, sizeof(buffer), "--");
        app->eal_argv[n_args++] = strdup(buffer);
 
diff --git a/examples/kni/main.c b/examples/kni/main.c
index e3bc2fb..9f9d227 100644
--- a/examples/kni/main.c
+++ b/examples/kni/main.c
@@ -919,9 +919,6 @@ main(int argc, char** argv)
                        continue;
                kni_free_kni(port);
        }
-#ifdef RTE_LIBRTE_XEN_DOM0
-       rte_kni_close();
-#endif
        for (i = 0; i < RTE_MAX_ETHPORTS; i++)
                if (kni_port_params_array[i]) {
                        rte_free(kni_port_params_array[i]);
diff --git a/test/test/process.h b/test/test/process.h
index 4f8d121..51ced12 100644
--- a/test/test/process.h
+++ b/test/test/process.h
@@ -52,11 +52,7 @@ static inline int
 process_dup(const char *const argv[], int numargs, const char *env_value)
 {
        int num;
-#ifdef RTE_LIBRTE_XEN_DOM0
-       char *argv_cpy[numargs + 2];
-#else
        char *argv_cpy[numargs + 1];
-#endif
        int i, fd, status;
        char path[32];
 
@@ -67,14 +63,8 @@ process_dup(const char *const argv[], int numargs, const 
char *env_value)
                /* make a copy of the arguments to be passed to exec */
                for (i = 0; i < numargs; i++)
                        argv_cpy[i] = strdup(argv[i]);
-#ifdef RTE_LIBRTE_XEN_DOM0
-               argv_cpy[i] = strdup("--xen-dom0");
-               argv_cpy[i + 1] = NULL;
-               num = numargs + 1;
-#else
                argv_cpy[i] = NULL;
                num = numargs;
-#endif
 
                /* close all open file descriptors, check /proc/self/fd to only
                 * call close on open fds. Exclude fds 0, 1 and 2*/
diff --git a/test/test/test.c b/test/test/test.c
index c561eb5..9accbd1 100644
--- a/test/test/test.c
+++ b/test/test/test.c
@@ -87,11 +87,7 @@ do_recursive_call(void)
                        { "test_invalid_b_flag", no_action },
                        { "test_invalid_vdev_flag", no_action },
                        { "test_invalid_r_flag", no_action },
-#ifdef RTE_LIBRTE_XEN_DOM0
-                       { "test_dom0_misc_flags", no_action },
-#else
                        { "test_misc_flags", no_action },
-#endif
                        { "test_memory_flags", no_action },
                        { "test_file_prefix", no_action },
                        { "test_no_huge_flag", no_action },
diff --git a/test/test/test_eal_flags.c b/test/test/test_eal_flags.c
index 594d79d..310109e 100644
--- a/test/test/test_eal_flags.c
+++ b/test/test/test_eal_flags.c
@@ -51,11 +51,7 @@
 
 #include "process.h"
 
-#ifdef RTE_LIBRTE_XEN_DOM0
-#define DEFAULT_MEM_SIZE "30"
-#else
 #define DEFAULT_MEM_SIZE "18"
-#endif
 #define mp_flag "--proc-type=secondary"
 #define no_hpet "--no-hpet"
 #define no_huge "--no-huge"
@@ -809,72 +805,6 @@ test_no_huge_flag(void)
        return 0;
 }
 
-#ifdef RTE_LIBRTE_XEN_DOM0
-static int
-test_dom0_misc_flags(void)
-{
-       char prefix[PATH_MAX], tmp[PATH_MAX];
-
-       if (get_current_prefix(tmp, sizeof(tmp)) == NULL) {
-               printf("Error - unable to get current prefix!\n");
-               return -1;
-       }
-       snprintf(prefix, sizeof(prefix), "--file-prefix=%s", tmp);
-
-       /* check that some general flags don't prevent things from working.
-        * All cases, apart from the first, app should run.
-        * No further testing of output done.
-        */
-       /* sanity check - failure with invalid option */
-       const char *argv0[] = {prgname, prefix, mp_flag, "-c", "1", 
"--invalid-opt"};
-
-       /* With --no-pci */
-       const char *argv1[] = {prgname, prefix, mp_flag, "-c", "1", "--no-pci"};
-       /* With -v */
-       const char *argv2[] = {prgname, prefix, mp_flag, "-c", "1", "-v"};
-       /* With valid --syslog */
-       const char *argv3[] = {prgname, prefix, mp_flag, "-c", "1",
-                       "--syslog", "syslog"};
-       /* With empty --syslog (should fail) */
-       const char *argv4[] = {prgname, prefix, mp_flag, "-c", "1", "--syslog"};
-       /* With invalid --syslog */
-       const char *argv5[] = {prgname, prefix, mp_flag, "-c", "1", "--syslog", 
"error"};
-       /* With no-sh-conf */
-       const char *argv6[] = {prgname, "-c", "1", "-n", "2", "-m", "20",
-                       "--no-shconf", "--file-prefix=noshconf" };
-
-       if (launch_proc(argv0) == 0) {
-               printf("Error - process ran ok with invalid flag\n");
-               return -1;
-       }
-       if (launch_proc(argv1) != 0) {
-               printf("Error - process did not run ok with --no-pci flag\n");
-               return -1;
-       }
-       if (launch_proc(argv2) != 0) {
-               printf("Error - process did not run ok with -v flag\n");
-               return -1;
-       }
-       if (launch_proc(argv3) != 0) {
-               printf("Error - process did not run ok with --syslog flag\n");
-               return -1;
-       }
-       if (launch_proc(argv4) == 0) {
-               printf("Error - process run ok with empty --syslog flag\n");
-               return -1;
-       }
-       if (launch_proc(argv5) == 0) {
-               printf("Error - process run ok with invalid --syslog flag\n");
-               return -1;
-       }
-       if (launch_proc(argv6) != 0) {
-               printf("Error - process did not run ok with --no-shconf 
flag\n");
-               return -1;
-       }
-
-       return 0;
-}
-#else
 static int
 test_misc_flags(void)
 {
@@ -1061,7 +991,6 @@ test_misc_flags(void)
        }
        return 0;
 }
-#endif
 
 static int
 test_file_prefix(void)
@@ -1098,9 +1027,6 @@ test_file_prefix(void)
                printf("Error - unable to get current prefix!\n");
                return -1;
        }
-#ifdef RTE_LIBRTE_XEN_DOM0
-       return 0;
-#endif
 
        /* check if files for current prefix are present */
        if (process_hugefiles(prefix, HUGEPAGE_CHECK_EXISTS) != 1) {
@@ -1299,9 +1225,6 @@ test_memory_flags(void)
                printf("Error - process failed with valid -m flag!\n");
                return -1;
        }
-#ifdef RTE_LIBRTE_XEN_DOM0
-       return 0;
-#endif
        if (launch_proc(argv2) == 0) {
                printf("Error - process run ok with invalid (zero) 
--socket-mem!\n");
                return -1;
@@ -1427,11 +1350,7 @@ test_eal_flags(void)
                return ret;
        }
 
-#ifdef RTE_LIBRTE_XEN_DOM0
-       ret = test_dom0_misc_flags();
-#else
        ret = test_misc_flags();
-#endif
        if (ret < 0) {
                printf("Error in test_misc_flags()");
                return ret;
-- 
2.7.4


_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
https://lists.xen.org/xen-devel

 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.