* [dpdk-dev] [PATCH] app/testpmd:add bond type description
@ 2017-06-30 7:56 RongQiang Xie
2017-06-30 15:39 ` Declan Doherty
0 siblings, 1 reply; 6+ messages in thread
From: RongQiang Xie @ 2017-06-30 7:56 UTC (permalink / raw)
To: jingjing.wu; +Cc: dev, RongQiang Xie
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");
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);
+ }
}
--
1.8.3.1
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [dpdk-dev] [PATCH] app/testpmd:add bond type description
2017-06-30 7:56 [dpdk-dev] [PATCH] app/testpmd:add bond type description RongQiang Xie
@ 2017-06-30 15:39 ` Declan Doherty
2017-07-02 18:11 ` Thomas Monjalon
0 siblings, 1 reply; 6+ messages in thread
From: Declan Doherty @ 2017-06-30 15:39 UTC (permalink / raw)
To: RongQiang Xie, jingjing.wu; +Cc: dev
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);
> + }
>
> }
>
>
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [dpdk-dev] [PATCH] app/testpmd:add bond type description
2017-06-30 15:39 ` Declan Doherty
@ 2017-07-02 18:11 ` Thomas Monjalon
2017-08-16 2:31 ` [dpdk-dev] 答复: " xie.rongqiang
0 siblings, 1 reply; 6+ messages in thread
From: Thomas Monjalon @ 2017-07-02 18:11 UTC (permalink / raw)
To: Declan Doherty; +Cc: dev, RongQiang Xie, jingjing.wu
30/06/2017 17:39, Declan Doherty:
> 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");
I would say no.
Can we think how to implement this kind of things inside the bonding code?
^ permalink raw reply [flat|nested] 6+ messages in thread
* [dpdk-dev] 答复: Re: [PATCH] app/testpmd:add bond type description
2017-07-02 18:11 ` Thomas Monjalon
@ 2017-08-16 2:31 ` xie.rongqiang
2017-08-23 20:22 ` Thomas Monjalon
0 siblings, 1 reply; 6+ messages in thread
From: xie.rongqiang @ 2017-08-16 2:31 UTC (permalink / raw)
To: Thomas Monjalon; +Cc: Declan Doherty, dev, jingjing.wu
I am sorry to reply so late for some reason.
And i figure out two ways to implement this kind of things inside the
bonding code,
First,if can the function rte_eth_bond_mode_get() return string, so we can
print
the bond mode straight, but in this way, we need fix the other c source
where call the function.
Second, we add an interface return bond mode string, in this way, we just
call it in function
cmd_show_bonding_config_parsed().
Finally, which way do you agree more?
Looking forward to your early reply,Thank your.
Thomas Monjalon <thomas@monjalon.net> 2017/07/03 02:11:52:
> : Thomas Monjalon <thomas@monjalon.net>
> : Declan Doherty <declan.doherty@intel.com>,
> : dev@dpdk.org, RongQiang Xie <xie.rongqiang@zte.com.cn>,
> jingjing.wu@intel.com
> : 2017/07/03 02:12
> : Re: [dpdk-dev] [PATCH] app/testpmd:add bond type description
>
> 30/06/2017 17:39, Declan Doherty:
> > 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");
>
> I would say no.
> Can we think how to implement this kind of things inside the bonding
code?
>
>
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [dpdk-dev] 答复: Re: [PATCH] app/testpmd:add bond type description
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
0 siblings, 1 reply; 6+ messages in thread
From: Thomas Monjalon @ 2017-08-23 20:22 UTC (permalink / raw)
To: xie.rongqiang, Declan Doherty; +Cc: dev, jingjing.wu
16/08/2017 04:31, xie.rongqiang@zte.com.cn:
> I am sorry to reply so late for some reason.
>
> And i figure out two ways to implement this kind of things inside the
> bonding code,
>
> First,if can the function rte_eth_bond_mode_get() return string, so we can
> print
No it is better to use integers in API.
> the bond mode straight, but in this way, we need fix the other c source
> where call the function.
>
> Second, we add an interface return bond mode string, in this way, we just
> call it in function
Yes a new function to convert integer to string seems better.
At the end, Declan should approve/decide.
> cmd_show_bonding_config_parsed().
>
> Finally, which way do you agree more?
>
> Looking forward to your early reply,Thank your.
>
>
> Thomas Monjalon <thomas@monjalon.net> 2017/07/03 02:11:52:
>
> > : Thomas Monjalon <thomas@monjalon.net>
> > : Declan Doherty <declan.doherty@intel.com>,
> > : dev@dpdk.org, RongQiang Xie <xie.rongqiang@zte.com.cn>,
> > jingjing.wu@intel.com
> > : 2017/07/03 02:12
> > : Re: [dpdk-dev] [PATCH] app/testpmd:add bond type description
> >
> > 30/06/2017 17:39, Declan Doherty:
> > > 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");
> >
> > I would say no.
> > Can we think how to implement this kind of things inside the bonding
> code?
> >
> >
>
^ permalink raw reply [flat|nested] 6+ messages in thread
* [dpdk-dev] 答复: Re: 答复: Re: [PATCH] app/testpmd:add bond type description
2017-08-23 20:22 ` Thomas Monjalon
@ 2017-08-24 11:07 ` xie.rongqiang
0 siblings, 0 replies; 6+ messages in thread
From: xie.rongqiang @ 2017-08-24 11:07 UTC (permalink / raw)
To: Thomas Monjalon; +Cc: Declan Doherty, dev, jingjing.wu
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1: Type: text/plain; charset="GB2312", Size: 5291 bytes --]
Hi,
I make a new patch for this issue becase the previous patch has delete
when the version 17.08 release.
The website is http://www.dpdk.org/dev/patchwork/patch/27851/,Thank
you.
Thomas Monjalon <thomas@monjalon.net> дÓÚ 2017/08/24 04:22:17:
> ·¢¼þÈË: Thomas Monjalon <thomas@monjalon.net>
> ÊÕ¼þÈË: xie.rongqiang@zte.com.cn, Declan Doherty
> <declan.doherty@intel.com>,
> ³ËÍ: dev@dpdk.org, jingjing.wu@intel.com
> ÈÕÆÚ: 2017/08/24 04:23
> Ö÷Ìâ: Re: [dpdk-dev] ´ð¸´: Re: [PATCH] app/testpmd:add bond type
description
>
> 16/08/2017 04:31, xie.rongqiang@zte.com.cn:
> > I am sorry to reply so late for some reason.
> >
> > And i figure out two ways to implement this kind of things inside the
> > bonding code,
> >
> > First,if can the function rte_eth_bond_mode_get() return string, so we
can
> > print
>
> No it is better to use integers in API.
>
> > the bond mode straight, but in this way, we need fix the other c
source
> > where call the function.
> >
> > Second, we add an interface return bond mode string, in this way, we
just
> > call it in function
>
> Yes a new function to convert integer to string seems better.
>
> At the end, Declan should approve/decide.
>
> > cmd_show_bonding_config_parsed().
> >
> > Finally, which way do you agree more?
> >
> > Looking forward to your early reply,Thank your.
> >
> >
> > Thomas Monjalon <thomas@monjalon.net> 2017/07/03 02:11:52:
> >
> > > : Thomas Monjalon <thomas@monjalon.net>
> > > : Declan Doherty <declan.doherty@intel.com>,
> > > : dev@dpdk.org, RongQiang Xie <xie.rongqiang@zte.com.cn>,
> > > jingjing.wu@intel.com
> > > : 2017/07/03 02:12
> > > : Re: [dpdk-dev] [PATCH] app/testpmd:add bond type description
> > >
> > > 30/06/2017 17:39, Declan Doherty:
> > > > 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");
> > >
> > > I would say no.
> > > Can we think how to implement this kind of things inside the bonding
> > code?
> > >
> > >
> >
>
>
>
>
\x16º&¢¸¤jg¥•©èÅÊ&N^[€º\r¸×~wÛ´×´^¶êç=«a¢¸¤jg¥•©èÅÊ&Eç\x1eŠ÷~º&\x11DtÕQ5¡·¨ºÖèºwi®‹^rبž‹–Š$r‰¦j)o{W®¯MhŸM<ö‹n‹§všèµç-Š‰èºÙh¢G(›]8ã½|õ¼¥Ù(®\x03è²×âÇ\b„DŒLøóд\x13°öáú+uëÝ¥Ù(®\x04á»n\0º\r´×½wç}ºûM´Ð!\x12L2ˆ1( «në
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2017-08-24 11:07 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-06-30 7:56 [dpdk-dev] [PATCH] app/testpmd:add bond type description RongQiang Xie
2017-06-30 15:39 ` Declan Doherty
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
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).