DPDK patches and discussions
 help / color / mirror / Atom feed
* [dpdk-dev] [PATCH] app/testpmd: fix bonding initialization due to unproper name
@ 2017-07-13  9:48 Michal Jastrzebski
  2017-07-21  2:29 ` Wu, Jingjing
  0 siblings, 1 reply; 3+ messages in thread
From: Michal Jastrzebski @ 2017-07-13  9:48 UTC (permalink / raw)
  To: dev; +Cc: deepak.k.jain, declan.doherty, Michal Jastrzebski

when creating a bond device in testpmd, a name for a device must meet
the correct convention described in the documentation:
The device name must start with the net_bonding prefix
followed by numbers or letters.
Change for ALB mempool allocation - mem_name was too long.

Fixes: 9bf4901d1a11 ("bus/vdev: remove probe with driver name option")

Signed-off-by: Michal Jastrzebski <michalx.k.jastrzebski@intel.com>
---
 app/test-pmd/cmdline.c                 |    2 +-
 drivers/net/bonding/rte_eth_bond_alb.c |    2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/app/test-pmd/cmdline.c b/app/test-pmd/cmdline.c
index d4ff608..23361bc 100644
--- a/app/test-pmd/cmdline.c
+++ b/app/test-pmd/cmdline.c
@@ -4900,7 +4900,7 @@ static void cmd_create_bonded_device_parsed(void *parsed_result,
 		return;
 	}
 
-	snprintf(ethdev_name, RTE_ETH_NAME_MAX_LEN, "net_bond_testpmd_%d",
+	snprintf(ethdev_name, RTE_ETH_NAME_MAX_LEN, "net_bonding_testpmd_%d",
 			bond_dev_num++);
 
 	/* Create a new bonded device. */
diff --git a/drivers/net/bonding/rte_eth_bond_alb.c b/drivers/net/bonding/rte_eth_bond_alb.c
index 7f6884d..d9d3749 100644
--- a/drivers/net/bonding/rte_eth_bond_alb.c
+++ b/drivers/net/bonding/rte_eth_bond_alb.c
@@ -80,7 +80,7 @@
 		 * The value is chosen to be cache aligned.
 		 */
 		data_size = 256 + RTE_PKTMBUF_HEADROOM;
-		snprintf(mem_name, sizeof(mem_name), "%s_MODE6",
+		snprintf(mem_name, sizeof(mem_name), "%s_ALB",
 				bond_dev->device->name);
 		internals->mode6.mempool = rte_pktmbuf_pool_create(mem_name,
 			512 * RTE_MAX_ETHPORTS,
-- 
1.7.9.5

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [dpdk-dev] [PATCH] app/testpmd: fix bonding initialization due to unproper name
  2017-07-13  9:48 [dpdk-dev] [PATCH] app/testpmd: fix bonding initialization due to unproper name Michal Jastrzebski
@ 2017-07-21  2:29 ` Wu, Jingjing
  2017-07-21  6:37   ` Thomas Monjalon
  0 siblings, 1 reply; 3+ messages in thread
From: Wu, Jingjing @ 2017-07-21  2:29 UTC (permalink / raw)
  To: Jastrzebski, MichalX K, dev
  Cc: Jain, Deepak K, Doherty, Declan, Jastrzebski, MichalX K



> -----Original Message-----
> From: dev [mailto:dev-bounces@dpdk.org] On Behalf Of Michal Jastrzebski
> Sent: Thursday, July 13, 2017 5:49 PM
> To: dev@dpdk.org
> Cc: Jain, Deepak K <deepak.k.jain@intel.com>; Doherty, Declan
> <declan.doherty@intel.com>; Jastrzebski, MichalX K
> <michalx.k.jastrzebski@intel.com>
> Subject: [dpdk-dev] [PATCH] app/testpmd: fix bonding initialization due to
> unproper name
> 
> when creating a bond device in testpmd, a name for a device must meet the
> correct convention described in the documentation:
> The device name must start with the net_bonding prefix followed by numbers
> or letters.
> Change for ALB mempool allocation - mem_name was too long.
> 
> Fixes: 9bf4901d1a11 ("bus/vdev: remove probe with driver name option")
> 
> Signed-off-by: Michal Jastrzebski <michalx.k.jastrzebski@intel.com>

Reviewed-by: Jingjing Wu <jingjing.wu@intel.com>

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [dpdk-dev] [PATCH] app/testpmd: fix bonding initialization due to unproper name
  2017-07-21  2:29 ` Wu, Jingjing
@ 2017-07-21  6:37   ` Thomas Monjalon
  0 siblings, 0 replies; 3+ messages in thread
From: Thomas Monjalon @ 2017-07-21  6:37 UTC (permalink / raw)
  To: Jastrzebski, MichalX K; +Cc: dev, Wu, Jingjing, Jain, Deepak K, Doherty, Declan

21/07/2017 05:29, Wu, Jingjing:
> 
> > when creating a bond device in testpmd, a name for a device must meet the
> > correct convention described in the documentation:
> > The device name must start with the net_bonding prefix followed by numbers
> > or letters.
> > Change for ALB mempool allocation - mem_name was too long.
> > 
> > Fixes: 9bf4901d1a11 ("bus/vdev: remove probe with driver name option")

Cc: stable@dpdk.org

> > Signed-off-by: Michal Jastrzebski <michalx.k.jastrzebski@intel.com>
> 
> Reviewed-by: Jingjing Wu <jingjing.wu@intel.com>

Applied, thanks

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2017-07-21  6:37 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-07-13  9:48 [dpdk-dev] [PATCH] app/testpmd: fix bonding initialization due to unproper name Michal Jastrzebski
2017-07-21  2:29 ` Wu, Jingjing
2017-07-21  6:37   ` Thomas Monjalon

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).