[dpdk-dev] test: fix bonded device name

Message ID 20170726154829.32736-1-tomaszx.kulasek@intel.com (mailing list archive)
State Accepted, archived
Delegated to: Ferruh Yigit
Headers

Checks

Context Check Description
ci/checkpatch success coding style OK
ci/Intel-compilation success Compilation OK

Commit Message

Tomasz Kulasek July 26, 2017, 3:48 p.m. UTC
  Bonding devices name must start with "net_bonding" prefix.

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

Signed-off-by: Tomasz Kulasek <tomaszx.kulasek@intel.com>
---
 test/test/test_link_bonding.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
  

Comments

Doherty, Declan July 26, 2017, 5:06 p.m. UTC | #1
On 26/07/2017 4:48 PM, Tomasz Kulasek wrote:
> Bonding devices name must start with "net_bonding" prefix.
>
> Fixes: 9bf4901d1a11 ("bus/vdev: remove probe with driver name option")
>
> Signed-off-by: Tomasz Kulasek <tomaszx.kulasek@intel.com>
> ---
>  test/test/test_link_bonding.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/test/test/test_link_bonding.c b/test/test/test_link_bonding.c
> index aa2a1a2..828f7f7 100644
> --- a/test/test/test_link_bonding.c
> +++ b/test/test/test_link_bonding.c
> @@ -83,7 +83,7 @@
>  #define MAX_PKT_BURST			(512)
>  #define DEF_PKT_BURST			(16)
>
> -#define BONDED_DEV_NAME			("unit_test_bond_dev")
> +#define BONDED_DEV_NAME			("net_bonding_ut")
>
>  #define INVALID_SOCKET_ID		(-1)
>  #define INVALID_PORT_ID			(-1)
> @@ -939,7 +939,7 @@ struct rte_fdir_conf fdir_conf = {
>  	/*
>  	 * 1. a - Create / configure  bonded / slave ethdevs
>  	 */
> -	bonded_port_id = rte_eth_bond_create("ethdev_bond_mac_ass_test",
> +	bonded_port_id = rte_eth_bond_create("net_bonding_mac_ass_test",
>  			BONDING_MODE_ACTIVE_BACKUP, rte_socket_id());
>  	TEST_ASSERT(bonded_port_id > 0, "failed to create bonded device");
>
>

Acked-by: Declan Doherty <declan.doherty@intel.com>
  
Ferruh Yigit July 31, 2017, 3:53 p.m. UTC | #2
On 7/26/2017 6:06 PM, Declan Doherty wrote:
> On 26/07/2017 4:48 PM, Tomasz Kulasek wrote:
>> Bonding devices name must start with "net_bonding" prefix.
>>
>> Fixes: 9bf4901d1a11 ("bus/vdev: remove probe with driver name option")
>>
>> Signed-off-by: Tomasz Kulasek <tomaszx.kulasek@intel.com>

> Acked-by: Declan Doherty <declan.doherty@intel.com>

Applied to dpdk-next-net/master, thanks.
  

Patch

diff --git a/test/test/test_link_bonding.c b/test/test/test_link_bonding.c
index aa2a1a2..828f7f7 100644
--- a/test/test/test_link_bonding.c
+++ b/test/test/test_link_bonding.c
@@ -83,7 +83,7 @@ 
 #define MAX_PKT_BURST			(512)
 #define DEF_PKT_BURST			(16)
 
-#define BONDED_DEV_NAME			("unit_test_bond_dev")
+#define BONDED_DEV_NAME			("net_bonding_ut")
 
 #define INVALID_SOCKET_ID		(-1)
 #define INVALID_PORT_ID			(-1)
@@ -939,7 +939,7 @@  struct rte_fdir_conf fdir_conf = {
 	/*
 	 * 1. a - Create / configure  bonded / slave ethdevs
 	 */
-	bonded_port_id = rte_eth_bond_create("ethdev_bond_mac_ass_test",
+	bonded_port_id = rte_eth_bond_create("net_bonding_mac_ass_test",
 			BONDING_MODE_ACTIVE_BACKUP, rte_socket_id());
 	TEST_ASSERT(bonded_port_id > 0, "failed to create bonded device");