From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id E4B0BA0547 for ; Sat, 13 Feb 2021 00:41:01 +0100 (CET) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id C7DDF4068F; Sat, 13 Feb 2021 00:41:01 +0100 (CET) Received: from mail-wr1-f50.google.com (mail-wr1-f50.google.com [209.85.221.50]) by mails.dpdk.org (Postfix) with ESMTP id 0DA994068F for ; Sat, 13 Feb 2021 00:40:59 +0100 (CET) Received: by mail-wr1-f50.google.com with SMTP id v15so1309026wrx.4 for ; Fri, 12 Feb 2021 15:40:59 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=from:to:cc:subject:date:message-id:in-reply-to:references :mime-version:content-transfer-encoding; bh=tyN0D2apu93efXCC6CMLd2Jn/N2jUVsXNKtgs88ubtY=; b=Gkc8/R5p6w00NFbj8dL+L+MYQuiK8NKnCZh4q7ERBWpVWKKAdpxEN4/muYqIr4nDCi +9+qrZ93P345dPrwAqD7Lk0+0hKqrZbqecGuLKVpKf4absfpWc/36LtGc+bIHSnXcH/b 691V0TW9YP68R3NGZjDBYIpTqSBMWOUfp+sNfPdQaIzNIdR/6mMvaPJcR+ObhQMAdcul H/wEXTHGYFBdGexiemLPy41PKntVRPwGmAcAEzcExZHlVUct2RnPHJoD3yKGfCt5ICA/ b2t+zRDyNsn4bUrl5/kXfuYQmnGwA7/erR9wGHvSLd7aBgI88RJvn7dMAFu8uICRUm/f 1iYg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references:mime-version:content-transfer-encoding; bh=tyN0D2apu93efXCC6CMLd2Jn/N2jUVsXNKtgs88ubtY=; b=hQYTiuS+EMwaX4Rt+Y+3R76F4NSLw2agELYyZYEE38zOveh0KrGErTCEquEAik39CB VZhrExDFnOeouWxWEiL7AIQlfYWUKgZVfvExJHw82DsAnf3ykMfZboN8Ln53p9cYkk7J 5/WVuixLzlD1BaFE455U1mzJcDNbKNglgpaoeDuiEHkzUwjs3s0MYEuLCwx6snERI+dx sHWFc25TX7riu+lwz4RVa5I/EIi9DEb0B2bJ0JxzqX1ZQAUuXxhIrl7Gqhb4HgzLhQf8 CjHXyFEoKWJfEN3j5F2Pj+xbUP/3K5oJ/wFW3xthii8LHxTaXCe5SFioezaq5yU2WNpM jFtA== X-Gm-Message-State: AOAM531AhR41E7n/O9SxZstnQIEjNBt+0Et6/xhWsCTdTKh2f5I2TXZP MJdgzgJHkxskYYRb1IwXFvhPIGRWM9TRdA== X-Google-Smtp-Source: ABdhPJzkvYUvO57mfTf1QTEXh64cz2bXWL84/xxSbM4J8/h5Cb7HZzWRW8POQMtenXvQE2MbQPuQ2w== X-Received: by 2002:a05:6000:10c5:: with SMTP id b5mr6173689wrx.284.1613173258862; Fri, 12 Feb 2021 15:40:58 -0800 (PST) Received: from localhost ([88.98.246.218]) by smtp.gmail.com with ESMTPSA id d5sm4036198wrp.39.2021.02.12.15.40.58 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 12 Feb 2021 15:40:58 -0800 (PST) From: luca.boccassi@gmail.com To: Ferruh Yigit Cc: Jasvinder Singh , Xiaoyun Li , dpdk stable Date: Fri, 12 Feb 2021 23:40:49 +0000 Message-Id: <20210212234050.255787-5-luca.boccassi@gmail.com> X-Mailer: git-send-email 2.29.2 In-Reply-To: <20210212234050.255787-1-luca.boccassi@gmail.com> References: <20210205111920.1272063-1-luca.boccassi@gmail.com> <20210212234050.255787-1-luca.boccassi@gmail.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Subject: [dpdk-stable] patch 'app/testpmd: fix help of metering commands' has been queued to stable release 20.11.1 X-BeenThere: stable@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: patches for DPDK stable branches List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: stable-bounces@dpdk.org Sender: "stable" Hi, FYI, your patch has been queued to stable release 20.11.1 Note it hasn't been pushed to http://dpdk.org/browse/dpdk-stable yet. It will be pushed if I get no objections before 02/14/21. So please shout if anyone has objections. Also note that after the patch there's a diff of the upstream commit vs the patch applied to the branch. This will indicate if there was any rebasing needed to apply to the stable branch. If there were code changes for rebasing (ie: not only metadata diffs), please double check that the rebase was correctly done. Queued patches are on a temporary branch at: https://github.com/bluca/dpdk-stable This queued commit can be viewed at: https://github.com/bluca/dpdk-stable/commit/ef01e05a7ce8e394d6ec7021d0eace2a37db9163 Thanks. Luca Boccassi --- >From ef01e05a7ce8e394d6ec7021d0eace2a37db9163 Mon Sep 17 00:00:00 2001 From: Ferruh Yigit Date: Tue, 9 Feb 2021 14:15:06 +0000 Subject: [PATCH] app/testpmd: fix help of metering commands [ upstream commit 618f0f4ac689f9255eecbef9202e77e35a9a7ceb ] Helps strings syntax is "command : description", the 'command' part was missing, updated command help strings. Fixes: 281eeb8afc55 ("app/testpmd: add commands for metering and policing") Fixes: 30ffb4e67ee3 ("app/testpmd: add commands traffic metering and policing") Fixes: e63b50162aa3 ("app/testpmd: clean metering and policing commands") Signed-off-by: Ferruh Yigit Acked-by: Jasvinder Singh Acked-by: Xiaoyun Li --- app/test-pmd/cmdline_mtr.c | 33 +++++++++++++++++++-------------- 1 file changed, 19 insertions(+), 14 deletions(-) diff --git a/app/test-pmd/cmdline_mtr.c b/app/test-pmd/cmdline_mtr.c index 399ee56e07..3982787d20 100644 --- a/app/test-pmd/cmdline_mtr.c +++ b/app/test-pmd/cmdline_mtr.c @@ -312,7 +312,7 @@ static void cmd_show_port_meter_cap_parsed(void *parsed_result, cmdline_parse_inst_t cmd_show_port_meter_cap = { .f = cmd_show_port_meter_cap_parsed, .data = NULL, - .help_str = "Show port meter cap", + .help_str = "show port meter cap ", .tokens = { (void *)&cmd_show_port_meter_cap_show, (void *)&cmd_show_port_meter_cap_port, @@ -408,7 +408,7 @@ static void cmd_add_port_meter_profile_srtcm_parsed(void *parsed_result, cmdline_parse_inst_t cmd_add_port_meter_profile_srtcm = { .f = cmd_add_port_meter_profile_srtcm_parsed, .data = NULL, - .help_str = "Add port meter profile srtcm (rfc2697)", + .help_str = "add port meter profile srtcm_rfc2697 ", .tokens = { (void *)&cmd_add_port_meter_profile_srtcm_add, (void *)&cmd_add_port_meter_profile_srtcm_port, @@ -515,7 +515,7 @@ static void cmd_add_port_meter_profile_trtcm_parsed(void *parsed_result, cmdline_parse_inst_t cmd_add_port_meter_profile_trtcm = { .f = cmd_add_port_meter_profile_trtcm_parsed, .data = NULL, - .help_str = "Add port meter profile trtcm (rfc2698)", + .help_str = "add port meter profile trtcm_rfc2698 ", .tokens = { (void *)&cmd_add_port_meter_profile_trtcm_add, (void *)&cmd_add_port_meter_profile_trtcm_port, @@ -627,7 +627,7 @@ static void cmd_add_port_meter_profile_trtcm_rfc4115_parsed( cmdline_parse_inst_t cmd_add_port_meter_profile_trtcm_rfc4115 = { .f = cmd_add_port_meter_profile_trtcm_rfc4115_parsed, .data = NULL, - .help_str = "Add port meter profile trtcm (rfc4115)", + .help_str = "add port meter profile trtcm_rfc4115 ", .tokens = { (void *)&cmd_add_port_meter_profile_trtcm_rfc4115_add, (void *)&cmd_add_port_meter_profile_trtcm_rfc4115_port, @@ -702,7 +702,7 @@ static void cmd_del_port_meter_profile_parsed(void *parsed_result, cmdline_parse_inst_t cmd_del_port_meter_profile = { .f = cmd_del_port_meter_profile_parsed, .data = NULL, - .help_str = "Delete port meter profile", + .help_str = "del port meter profile ", .tokens = { (void *)&cmd_del_port_meter_profile_del, (void *)&cmd_del_port_meter_profile_port, @@ -827,7 +827,10 @@ static void cmd_create_port_meter_parsed(void *parsed_result, cmdline_parse_inst_t cmd_create_port_meter = { .f = cmd_create_port_meter_parsed, .data = NULL, - .help_str = "Create port meter", + .help_str = "create port meter (yes|no) " + "(R|Y|G|D) (R|Y|G|D) (R|Y|G|D) " + " " + "[ ...]", .tokens = { (void *)&cmd_create_port_meter_create, (void *)&cmd_create_port_meter_port, @@ -896,7 +899,7 @@ static void cmd_enable_port_meter_parsed(void *parsed_result, cmdline_parse_inst_t cmd_enable_port_meter = { .f = cmd_enable_port_meter_parsed, .data = NULL, - .help_str = "Enable port meter", + .help_str = "enable port meter ", .tokens = { (void *)&cmd_enable_port_meter_enable, (void *)&cmd_enable_port_meter_port, @@ -957,7 +960,7 @@ static void cmd_disable_port_meter_parsed(void *parsed_result, cmdline_parse_inst_t cmd_disable_port_meter = { .f = cmd_disable_port_meter_parsed, .data = NULL, - .help_str = "Disable port meter", + .help_str = "disable port meter ", .tokens = { (void *)&cmd_disable_port_meter_disable, (void *)&cmd_disable_port_meter_port, @@ -1018,7 +1021,7 @@ static void cmd_del_port_meter_parsed(void *parsed_result, cmdline_parse_inst_t cmd_del_port_meter = { .f = cmd_del_port_meter_parsed, .data = NULL, - .help_str = "Delete port meter", + .help_str = "del port meter ", .tokens = { (void *)&cmd_del_port_meter_del, (void *)&cmd_del_port_meter_port, @@ -1092,7 +1095,7 @@ static void cmd_set_port_meter_profile_parsed(void *parsed_result, cmdline_parse_inst_t cmd_set_port_meter_profile = { .f = cmd_set_port_meter_profile_parsed, .data = NULL, - .help_str = "Set port meter profile", + .help_str = "set port meter profile ", .tokens = { (void *)&cmd_set_port_meter_profile_set, (void *)&cmd_set_port_meter_profile_port, @@ -1166,7 +1169,8 @@ free_table: cmdline_parse_inst_t cmd_set_port_meter_dscp_table = { .f = cmd_set_port_meter_dscp_table_parsed, .data = NULL, - .help_str = "Update port meter dscp table", + .help_str = "set port meter dscp table " + "[ ... ]", .tokens = { (void *)&cmd_set_port_meter_dscp_table_set, (void *)&cmd_set_port_meter_dscp_table_port, @@ -1276,7 +1280,8 @@ static void cmd_set_port_meter_policer_action_parsed(void *parsed_result, cmdline_parse_inst_t cmd_set_port_meter_policer_action = { .f = cmd_set_port_meter_policer_action_parsed, .data = NULL, - .help_str = "Set port meter policer action", + .help_str = "set port meter policer action " + " [ ]", .tokens = { (void *)&cmd_set_port_meter_policer_action_set, (void *)&cmd_set_port_meter_policer_action_port, @@ -1355,7 +1360,7 @@ static void cmd_set_port_meter_stats_mask_parsed(void *parsed_result, cmdline_parse_inst_t cmd_set_port_meter_stats_mask = { .f = cmd_set_port_meter_stats_mask_parsed, .data = NULL, - .help_str = "Set port meter stats mask", + .help_str = "set port meter stats mask ", .tokens = { (void *)&cmd_set_port_meter_stats_mask_set, (void *)&cmd_set_port_meter_stats_mask_port, @@ -1459,7 +1464,7 @@ static void cmd_show_port_meter_stats_parsed(void *parsed_result, cmdline_parse_inst_t cmd_show_port_meter_stats = { .f = cmd_show_port_meter_stats_parsed, .data = NULL, - .help_str = "Show port meter stats", + .help_str = "show port meter stats (yes|no)", .tokens = { (void *)&cmd_show_port_meter_stats_show, (void *)&cmd_show_port_meter_stats_port, -- 2.29.2 --- Diff of the applied patch vs upstream commit (please double-check if non-empty: --- --- - 2021-02-12 23:40:10.053599217 +0000 +++ 0005-app-testpmd-fix-help-of-metering-commands.patch 2021-02-12 23:40:09.879090655 +0000 @@ -1 +1 @@ -From 618f0f4ac689f9255eecbef9202e77e35a9a7ceb Mon Sep 17 00:00:00 2001 +From ef01e05a7ce8e394d6ec7021d0eace2a37db9163 Mon Sep 17 00:00:00 2001 @@ -5,0 +6,2 @@ +[ upstream commit 618f0f4ac689f9255eecbef9202e77e35a9a7ceb ] + @@ -12 +13,0 @@ -Cc: stable@dpdk.org