[dpdk-dev] [PATCH] net/bonding: fix test bonding MAC assignment

Ferruh Yigit ferruh.yigit at intel.com
Tue Apr 30 13:25:54 CEST 2019


On 4/29/2019 5:45 PM, Chas Williams wrote:
> Seems fine.
> 
> Acked-by: Chas Williams <chas3 at att.com>

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

> 
> On 4/29/19 10:56 AM, Ferruh Yigit wrote:
>> On 4/26/2019 11:30 PM, kkanas at marvell.com wrote:
>>> From: Krzysztof Kanas <kkanas at marvell.com>
>>>
>>> Fix test_set_bonded_port_initialization_mac_assignment so that it works
>>> after 're run' test_link_bonding.
>>>
>>> Fixes: f2ef6f21ee2e ("bond: fix mac assignment to slaves")
>>> Cc: declan.doherty at intel.com
>>>
>>> Signed-off-by: Krzysztof Kanas <kkanas at marvell.com>

Changed patch title as: "app/test: fix test bonding MAC assignment"

<...>

>>>   	/*
>>> -	 * 2. Add slave ethdevs to bonded device
>>> -	 */
>>> +	* 2. Add slave ethdevs to bonded device
>>> +	*/
>>>   	for (i = 0; i < BONDED_INIT_MAC_ASSIGNMENT_SLAVE_COUNT; i++) {
>>>   		TEST_ASSERT_SUCCESS(rte_eth_bond_slave_add(bonded_port_id,
>>>   				slave_port_ids[i]),
>>>   				"Failed to add slave (%d) to bonded port (%d).",
>>>   				slave_port_ids[i], bonded_port_id);
>>>   	}
>>> -
>>>   	slave_count = rte_eth_bond_slaves_get(bonded_port_id, slaves,
>>>   			RTE_MAX_ETHPORTS);

Removed these unrelated whitespace changes and did a few syntax update while
merging.


More information about the dev mailing list