On 4/29/2019 5:45 PM, Chas Williams wrote: > Seems fine. > > Acked-by: Chas Williams <[email protected]>
Applied to dpdk-next-net/master, thanks. > > On 4/29/19 10:56 AM, Ferruh Yigit wrote: >> On 4/26/2019 11:30 PM, [email protected] wrote: >>> From: Krzysztof Kanas <[email protected]> >>> >>> 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: [email protected] >>> >>> Signed-off-by: Krzysztof Kanas <[email protected]> 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.
