From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-we0-f178.google.com (mail-we0-f178.google.com [74.125.82.178]) by dpdk.org (Postfix) with ESMTP id 4696B5917 for ; Thu, 15 May 2014 18:08:53 +0200 (CEST) Received: by mail-we0-f178.google.com with SMTP id u56so1285985wes.23 for ; Thu, 15 May 2014 09:09:01 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:from:to:subject:date:message-id:in-reply-to :references; bh=FmmbII4HqHZdy+bcGWgr4LFqBTaKAIdLpmpwcjoxp94=; b=TlZcq3d2XIrff3HYPOufq+tt4DgTfhHaf4yrHdGCcAkFRKDYPZO0qOaL2/zeH5Oq9G a1uL48Kr97WSVFjLGD4ZJQS8TESJRaDrDgvkGEIzKlnyE5AJ6pUe7x+20auU1rtHOnmB FzmlTpS/0RrtJszv/6PvbFJmZU7uCT446VH+VkFXUWbUsJECVYblYM/SeqbuNVIj2GUo D0B3vTySFb5/mvSBO54WZ/ONaFRhANK8/b5OQWHplKcyXRIe2WWY4pS2nY7d2tCmxblb EsfV8XLjPYOWaPql8Hkv2cPrSZ/fY/GePsJrB6dxv+k4wBvnlspNjBm9IS7aG43+95Sd QCcQ== X-Gm-Message-State: ALoCoQkRxaAea5f85j9VgSvDPO1+K9EpxwJpxgaQ1XqZg37Jnb0Oa+kWBEw954QAod2+pWxm20vn X-Received: by 10.180.8.66 with SMTP id p2mr32102872wia.37.1400170141206; Thu, 15 May 2014 09:09:01 -0700 (PDT) Received: from XPS13.dev.6wind.com (6wind.net2.nerim.net. [213.41.180.237]) by mx.google.com with ESMTPSA id lz11sm11802427wic.0.2014.05.15.09.08.59 for (version=TLSv1.2 cipher=ECDHE-RSA-AES128-SHA bits=128/128); Thu, 15 May 2014 09:09:00 -0700 (PDT) From: Thomas Monjalon To: dev@dpdk.org Date: Thu, 15 May 2014 18:08:41 +0200 Message-Id: <1400170121-2737-1-git-send-email-thomas.monjalon@6wind.com> X-Mailer: git-send-email 1.9.2 In-Reply-To: <1400146513-14283-1-git-send-email-thomas.monjalon@6wind.com> References: <1400146513-14283-1-git-send-email-thomas.monjalon@6wind.com> Subject: [dpdk-dev] [PATCH v3] app/testpmd: list forwarding engines X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 15 May 2014 16:08:53 -0000 Having a function to list forwarding engines helps to show them in cli help and in parameters usage witout duplicating code. Signed-off-by: Thomas Monjalon --- app/test-pmd/cmdline.c | 52 +++++++++++++++++++++++++++-------------------- app/test-pmd/config.c | 19 +++++++++++++++++ app/test-pmd/parameters.c | 3 ++- app/test-pmd/testpmd.h | 1 + 4 files changed, 52 insertions(+), 23 deletions(-) changes in v2: - factorize also contextual help changes in v3: - remove useless loop diff --git a/app/test-pmd/cmdline.c b/app/test-pmd/cmdline.c index 7becedc..c6a417a 100644 --- a/app/test-pmd/cmdline.c +++ b/app/test-pmd/cmdline.c @@ -294,17 +294,9 @@ static void cmd_help_long_parsed(void *parsed_result, " bit 3 - insert sctp checksum offload if set\n" " Please check the NIC datasheet for HW limits.\n\n" -#ifdef RTE_LIBRTE_IEEE1588 - "set fwd (io|mac|mac_retry|rxonly|txonly|csum|ieee1588)\n" - " Set IO, MAC, MAC_RETRY, RXONLY, CSUM or TXONLY or ieee1588" - " packet forwarding mode.\n\n" + "set fwd (%s)\n" + " Set packet forwarding mode.\n\n" -#else - "set fwd (io|mac|mac_retry|rxonly|txonly|csum)\n" - " Set IO, MAC, MAC_RETRY, RXONLY, CSUM or TXONLY packet" - " forwarding mode.\n\n" - -#endif "mac_addr add (port_id) (XX:XX:XX:XX:XX:XX)\n" " Add a MAC address on port_id.\n\n" @@ -398,6 +390,7 @@ static void cmd_help_long_parsed(void *parsed_result, " using the lowest port on the NIC.\n\n" #endif + , list_pkt_forwarding_modes() ); } @@ -2688,22 +2681,12 @@ cmdline_parse_token_string_t cmd_setfwd_fwd = TOKEN_STRING_INITIALIZER(struct cmd_set_fwd_mode_result, fwd, "fwd"); cmdline_parse_token_string_t cmd_setfwd_mode = TOKEN_STRING_INITIALIZER(struct cmd_set_fwd_mode_result, mode, -#ifdef RTE_LIBRTE_IEEE1588 - "io#mac#mac_retry#rxonly#txonly#csum#ieee1588"); -#else - "io#mac#mac_retry#rxonly#txonly#csum"); -#endif + "" /* defined at init */); cmdline_parse_inst_t cmd_set_fwd_mode = { .f = cmd_set_fwd_mode_parsed, .data = NULL, -#ifdef RTE_LIBRTE_IEEE1588 - .help_str = "set fwd io|mac|mac_retry|rxonly|txonly|csum|ieee1588 - set IO, MAC," - " MAC_RETRY, RXONLY, TXONLY, CSUM or IEEE1588 packet forwarding mode", -#else - .help_str = "set fwd io|mac|mac_retry|rxonly|txonly|csum - set IO, MAC," - " MAC_RETRY, RXONLY, CSUM or TXONLY packet forwarding mode", -#endif + .help_str = NULL, /* defined at init */ .tokens = { (void *)&cmd_setfwd_set, (void *)&cmd_setfwd_fwd, @@ -2712,6 +2695,28 @@ cmdline_parse_inst_t cmd_set_fwd_mode = { }, }; +static void cmd_set_fwd_mode_init(void) +{ + char *modes, *c; + static char token[128]; + static char help[256]; + cmdline_parse_token_string_t *token_struct; + + modes = list_pkt_forwarding_modes(); + rte_snprintf(help, sizeof help, "set fwd %s - " + "set packet forwarding mode", modes); + cmd_set_fwd_mode.help_str = help; + + /* string token separator is # */ + for (c = token; *modes != '\0'; modes++) + if (*modes == '|') + *c++ = '#'; + else + *c++ = *modes; + token_struct = (cmdline_parse_token_string_t*)cmd_set_fwd_mode.tokens[2]; + token_struct->string_data.str = token; +} + /* *** SET BURST TX DELAY TIME RETRY NUMBER *** */ struct cmd_set_burst_tx_retry_result { cmdline_fixed_string_t set; @@ -5198,6 +5203,9 @@ prompt(void) { struct cmdline *cl; + /* initialize non-constant commands */ + cmd_set_fwd_mode_init(); + cl = cmdline_stdin_new(main_ctx, "testpmd> "); if (cl == NULL) { return; diff --git a/app/test-pmd/config.c b/app/test-pmd/config.c index d0e21b7..41326fe 100644 --- a/app/test-pmd/config.c +++ b/app/test-pmd/config.c @@ -1231,6 +1231,25 @@ set_tx_pkt_segments(unsigned *seg_lengths, unsigned nb_segs) tx_pkt_nb_segs = (uint8_t) nb_segs; } +char* +list_pkt_forwarding_modes(void) +{ + static char fwd_modes[128] = ""; + const char *separator = "|"; + struct fwd_engine *fwd_eng; + unsigned i = 0; + + if (strlen (fwd_modes) == 0) { + while ((fwd_eng = fwd_engines[i++]) != NULL) { + strcat(fwd_modes, fwd_eng->fwd_mode_name); + strcat(fwd_modes, separator); + } + fwd_modes[strlen(fwd_modes) - strlen(separator)] = '\0'; + } + + return fwd_modes; +} + void set_pkt_forwarding_mode(const char *fwd_mode_name) { diff --git a/app/test-pmd/parameters.c b/app/test-pmd/parameters.c index 67081d7..8a3ce2c 100644 --- a/app/test-pmd/parameters.c +++ b/app/test-pmd/parameters.c @@ -158,7 +158,8 @@ usage(char* progname) printf(" --disable-rss: disable rss.\n"); printf(" --port-topology=N: set port topology (N: paired (default) or " "chained).\n"); - printf(" --forward-mode=N: set forwarding mode.\n"); + printf(" --forward-mode=N: set forwarding mode (N: %s).\n", + list_pkt_forwarding_modes()); printf(" --rss-ip: set RSS functions to IPv4/IPv6 only .\n"); printf(" --rss-udp: set RSS functions to IPv4/IPv6 + UDP.\n"); printf(" --rxq=N: set the number of RX queues per port to N.\n"); diff --git a/app/test-pmd/testpmd.h b/app/test-pmd/testpmd.h index 52d3543..0e4a35e 100644 --- a/app/test-pmd/testpmd.h +++ b/app/test-pmd/testpmd.h @@ -491,6 +491,7 @@ void tx_cksum_set(portid_t port_id, uint8_t cksum_mask); void set_verbose_level(uint16_t vb_level); void set_tx_pkt_segments(unsigned *seg_lengths, unsigned nb_segs); void set_nb_pkt_per_burst(uint16_t pkt_burst); +char *list_pkt_forwarding_modes(void); void set_pkt_forwarding_mode(const char *fwd_mode); void start_packet_forwarding(int with_tx_first); void stop_packet_forwarding(void); -- 1.9.2