From mboxrd@z Thu Jan 1 00:00:00 1970 From: Thomas Monjalon Subject: [PATCH] app/pdump: remove unused socket path options Date: Wed, 25 Apr 2018 15:03:39 +0200 Message-ID: <20180425130339.4021-1-thomas@monjalon.net> Cc: anatoly.burakov@intel.com, reshma.pattan@intel.com, jianfeng.tan@intel.com To: dev@dpdk.org Return-path: Received: from out2-smtp.messagingengine.com (out2-smtp.messagingengine.com [66.111.4.26]) by dpdk.org (Postfix) with ESMTP id E7A706CC7 for ; Wed, 25 Apr 2018 15:03:54 +0200 (CEST) List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" The options --server-socket-path and --client-socket-path were said to be deprecated and will be removed soon. No need to wait for removing application options which have no effect, and can confuse the user. Fixes: 660098d61f57 ("pdump: use generic multi-process channel") Cc: jianfeng.tan@intel.com Signed-off-by: Thomas Monjalon --- app/pdump/main.c | 29 +---------------------------- 1 file changed, 1 insertion(+), 28 deletions(-) diff --git a/app/pdump/main.c b/app/pdump/main.c index aa0f3a34d..6bcf8c498 100644 --- a/app/pdump/main.c +++ b/app/pdump/main.c @@ -37,8 +37,6 @@ #define PDUMP_RING_SIZE_ARG "ring-size" #define PDUMP_MSIZE_ARG "mbuf-size" #define PDUMP_NUM_MBUFS_ARG "total-num-mbufs" -#define CMD_LINE_OPT_SER_SOCK_PATH "server-socket-path" -#define CMD_LINE_OPT_CLI_SOCK_PATH "client-socket-path" #define VDEV_PCAP "net_pcap_%s_%d,tx_pcap=%s" #define VDEV_IFACE "net_pcap_%s_%d,tx_iface=%s" @@ -140,8 +138,6 @@ struct parse_val { int num_tuples; static struct rte_eth_conf port_conf_default; volatile uint8_t quit_signal; -static char server_socket_path[PATH_MAX]; -static char client_socket_path[PATH_MAX]; /**< display usage */ static void @@ -154,13 +150,7 @@ pdump_usage(const char *prgname) " tx-dev=," "[ring-size=default:16384]," "[mbuf-size=default:2176]," - "[total-num-mbufs=default:65535]'\n" - "[--server-socket-path=" - " which is deprecated and will be removed soon," - " default:/var/run/.dpdk/ (or) ~/.dpdk/]\n" - "[--client-socket-path=" - " which is deprecated and will be removed soon," - " default:/var/run/.dpdk/ (or) ~/.dpdk/]\n", + "[total-num-mbufs=default:65535]'\n", prgname); } @@ -385,8 +375,6 @@ launch_args_parse(int argc, char **argv, char *prgname) int option_index; static struct option long_option[] = { {"pdump", 1, 0, 0}, - {"server-socket-path", 1, 0, 0}, - {"client-socket-path", 1, 0, 0}, {NULL, 0, 0, 0} }; @@ -407,21 +395,6 @@ launch_args_parse(int argc, char **argv, char *prgname) return -1; } } - - if (!strncmp(long_option[option_index].name, - CMD_LINE_OPT_SER_SOCK_PATH, - sizeof(CMD_LINE_OPT_SER_SOCK_PATH))) { - strlcpy(server_socket_path, optarg, - sizeof(server_socket_path)); - } - - if (!strncmp(long_option[option_index].name, - CMD_LINE_OPT_CLI_SOCK_PATH, - sizeof(CMD_LINE_OPT_CLI_SOCK_PATH))) { - strlcpy(client_socket_path, optarg, - sizeof(client_socket_path)); - } - break; default: pdump_usage(prgname); -- 2.16.2