From: Declan Doherty <declan.doherty@intel.com>
To: RongQiang Xie <xie.rongqiang@zte.com.cn>, jingjing.wu@intel.com
Cc: dev@dpdk.org
Subject: Re: [dpdk-dev] [PATCH] app/testpmd:add bond type description
Date: Fri, 30 Jun 2017 16:39:08 +0100 [thread overview]
Message-ID: <eb9fbd68-e870-bc15-f954-8e602836f7fc@intel.com> (raw)
In-Reply-To: <201706300758.v5U7wu8t037841@mse01.zte.com.cn>
On 30/06/17 08:56, RongQiang Xie wrote:
> In function cmd_show_bonding_config_parsed() used number represent
> the bond type,in order more detailed,add bond type description
> otherwise we may confused about the number type.
> And also,the primary port just use in mode active backup and tlb,
> so,when the mode is active backup or tlb show the primary port info
> may be more appropriate.
>
> Signed-off-by: RongQiang Xie <xie.rongqiang@zte.com.cn>
> ---
> app/test-pmd/cmdline.c | 17 +++++++++++------
> 1 file changed, 11 insertions(+), 6 deletions(-)
>
> diff --git a/app/test-pmd/cmdline.c b/app/test-pmd/cmdline.c
> index ff8ffd2..45845a4 100644
> --- a/app/test-pmd/cmdline.c
> +++ b/app/test-pmd/cmdline.c
> @@ -4390,7 +4390,9 @@ static void cmd_show_bonding_config_parsed(void *parsed_result,
> printf("\tFailed to get bonding mode for port = %d\n", port_id);
> return;
> } else
> - printf("\tBonding mode: %d\n", bonding_mode);
> + printf("\tBonding mode: %d ", bonding_mode);
> + printf("[0:Round Robin, 1:Active Backup, 2:Balance, 3:Broadcast, ");
> + printf("\n\t\t\t4:802.3AD, 5:Adaptive TLB, 6:Adaptive Load Balancing]\n");
>
Good idea, but it would be clearer if we just returned the actual mode
string so the user doesn't need to parse it themselves, like below.
- } else
- printf("\tBonding mode: %d ", bonding_mode);
- printf("[0:Round Robin, 1:Active Backup, 2:Balance, 3:Broadcast, ");
- printf("\n\t\t\t4:802.3AD, 5:Adaptive TLB, 6:Adaptive Load
Balancing]\n");
+ }
+
+ printf("\tBonding mode: %d (", bonding_mode);
+ switch (bonding_mode) {
+ case BONDING_MODE_ROUND_ROBIN:
+ printf("round-robin");
+ break;
+ case BONDING_MODE_ACTIVE_BACKUP:
+ printf("active-backup");
+ break;
+ case BONDING_MODE_BALANCE:
+ printf("link-aggregation");
+ break;
+ case BONDING_MODE_BROADCAST:
+ printf("broadcast");
+ break;
+ case BONDING_MODE_8023AD:
+ printf("link-aggregation-802.3ad");
+ break;
+ case BONDING_MODE_TLB:
+ printf("transmit-load-balancing");
+ break;
+ case BONDING_MODE_ALB:
+ printf("adaptive-load-balancing");
+ break;
+ default:
+ printf("unknown-mode");
+ }
+ printf(")\n");
> if (bonding_mode == BONDING_MODE_BALANCE) {
> int balance_xmit_policy;
> @@ -4454,12 +4456,15 @@ static void cmd_show_bonding_config_parsed(void *parsed_result,
>
> }
>
> - primary_id = rte_eth_bond_primary_get(port_id);
> - if (primary_id < 0) {
> - printf("\tFailed to get primary slave for port = %d\n", port_id);
> - return;
> - } else
> + if (bonding_mode == BONDING_MODE_ACTIVE_BACKUP ||
> + bonding_mode == BONDING_MODE_TLB) {
> + primary_id = rte_eth_bond_primary_get(port_id);
> + if (primary_id < 0) {
> + printf("\tFailed to get primary slave for port = %d\n", port_id);
> + return;
> + }
> printf("\tPrimary: [%d]\n", primary_id);
> + }
>
> }
>
>
next prev parent reply other threads:[~2017-06-30 15:39 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-06-30 7:56 RongQiang Xie
2017-06-30 15:39 ` Declan Doherty [this message]
2017-07-02 18:11 ` Thomas Monjalon
2017-08-16 2:31 ` [dpdk-dev] 答复: " xie.rongqiang
2017-08-23 20:22 ` Thomas Monjalon
2017-08-24 11:07 ` [dpdk-dev] 答复: " xie.rongqiang
-- strict thread matches above, loose matches on Subject: below --
2017-08-25 2:49 [dpdk-dev] " Rongqiang XIE
2017-08-24 10:55 Rongqiang XIE
2017-06-30 7:35 RongQiang Xie
2017-06-30 2:33 RongQiang Xie
2017-06-30 1:52 RongQiang Xie
2017-06-30 13:47 ` Declan Doherty
2017-06-30 13:52 ` Declan Doherty
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=eb9fbd68-e870-bc15-f954-8e602836f7fc@intel.com \
--to=declan.doherty@intel.com \
--cc=dev@dpdk.org \
--cc=jingjing.wu@intel.com \
--cc=xie.rongqiang@zte.com.cn \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).