[dpdk-dev] test/bonding: fix namespace bonding mode4 unit tests

Message ID 20170705152747.55807-1-danielx.t.mrzyglod@intel.com (mailing list archive)
State Accepted, archived
Delegated to: Ferruh Yigit
Headers

Checks

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

Commit Message

Daniel Mrzyglod July 5, 2017, 3:27 p.m. UTC
  ring network driver is limited to 32 characters
the name of device was extened to net_ring_unit_test_mode4_slave_0 which is 33

Fixes: 5e41ab250dfa ("app/test: unit tests for bonding mode 4")

Signed-off-by: Daniel Mrzyglod <danielx.t.mrzyglod@intel.com>
---
 test/test/test_link_bonding_mode4.c | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)
  

Comments

Doherty, Declan July 5, 2017, 4:41 p.m. UTC | #1
On 05/07/2017 4:27 PM, Daniel Mrzyglod wrote:
> ring network driver is limited to 32 characters
> the name of device was extened to net_ring_unit_test_mode4_slave_0 which is 33
>
> Fixes: 5e41ab250dfa ("app/test: unit tests for bonding mode 4")
>
> Signed-off-by: Daniel Mrzyglod <danielx.t.mrzyglod@intel.com>
> ---
>  test/test/test_link_bonding_mode4.c | 8 ++++----
>  1 file changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/test/test/test_link_bonding_mode4.c b/test/test/test_link_bonding_mode4.c
> index c6745e8..8b64bbf 100644
> --- a/test/test/test_link_bonding_mode4.c
> +++ b/test/test/test_link_bonding_mode4.c
> @@ -73,11 +73,11 @@
>  #define MAX_PKT_BURST           (32)
>  #define DEF_PKT_BURST           (16)
>
> -#define BONDED_DEV_NAME         ("unit_test_mode4_bond_dev")
> +#define BONDED_DEV_NAME         ("ut_mode4_bond_dev")
>
> -#define SLAVE_DEV_NAME_FMT      ("unit_test_mode4_slave_%d")
> -#define SLAVE_RX_QUEUE_FMT      ("unit_test_mode4_slave_%d_rx")
> -#define SLAVE_TX_QUEUE_FMT      ("unit_test_mode4_slave_%d_tx")
> +#define SLAVE_DEV_NAME_FMT      ("ut_mode4_slave_%d")
> +#define SLAVE_RX_QUEUE_FMT      ("ut_mode4_slave_%d_rx")
> +#define SLAVE_TX_QUEUE_FMT      ("ut_mode4_slave_%d_tx")
>
>  #define INVALID_SOCKET_ID       (-1)
>  #define INVALID_PORT_ID         (0xFF)
>

Acked-by: Declan Doherty <declan.doherty@intel.com>
  
Ferruh Yigit July 5, 2017, 5:37 p.m. UTC | #2
On 7/5/2017 5:41 PM, Declan Doherty wrote:
> On 05/07/2017 4:27 PM, Daniel Mrzyglod wrote:
>> ring network driver is limited to 32 characters
>> the name of device was extened to net_ring_unit_test_mode4_slave_0 which is 33
>>
>> Fixes: 5e41ab250dfa ("app/test: unit tests for bonding mode 4")
>>
>> Signed-off-by: Daniel Mrzyglod <danielx.t.mrzyglod@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_mode4.c b/test/test/test_link_bonding_mode4.c
index c6745e8..8b64bbf 100644
--- a/test/test/test_link_bonding_mode4.c
+++ b/test/test/test_link_bonding_mode4.c
@@ -73,11 +73,11 @@ 
 #define MAX_PKT_BURST           (32)
 #define DEF_PKT_BURST           (16)
 
-#define BONDED_DEV_NAME         ("unit_test_mode4_bond_dev")
+#define BONDED_DEV_NAME         ("ut_mode4_bond_dev")
 
-#define SLAVE_DEV_NAME_FMT      ("unit_test_mode4_slave_%d")
-#define SLAVE_RX_QUEUE_FMT      ("unit_test_mode4_slave_%d_rx")
-#define SLAVE_TX_QUEUE_FMT      ("unit_test_mode4_slave_%d_tx")
+#define SLAVE_DEV_NAME_FMT      ("ut_mode4_slave_%d")
+#define SLAVE_RX_QUEUE_FMT      ("ut_mode4_slave_%d_rx")
+#define SLAVE_TX_QUEUE_FMT      ("ut_mode4_slave_%d_tx")
 
 #define INVALID_SOCKET_ID       (-1)
 #define INVALID_PORT_ID         (0xFF)