app/testpmd: fix incorrect port number check
authorYong Liu <yong.liu@intel.com>
Wed, 11 Mar 2015 07:23:10 +0000 (15:23 +0800)
committerThomas Monjalon <thomas.monjalon@6wind.com>
Wed, 11 Mar 2015 14:59:56 +0000 (15:59 +0100)
testpmd parameter "nb-port" mean the number of forwarding port.
It's incorrect to use function port_id_is_invalid to check number of ports.

Fixes: edab33b1c01d ("app/testpmd: support port hotplug")

Signed-off-by: Yong Liu <yong.liu@intel.com>
Acked-by: Thomas Monjalon <thomas.monjalon@6wind.com>
app/test-pmd/parameters.c

index 11d9af0..f1daa6e 100644 (file)
@@ -628,8 +628,7 @@ launch_args_parse(int argc, char** argv)
 #endif
                        if (!strcmp(lgopts[opt_idx].name, "nb-ports")) {
                                n = atoi(optarg);
-                               if (n > 0 &&
-                                   !port_id_is_invalid(n, DISABLED_WARN))
+                               if (n > 0 && n <= nb_ports)
                                        nb_fwd_ports = (uint8_t) n;
                                else
                                        rte_exit(EXIT_FAILURE,