Re: [PATCH] net/bonding: make bonded device configure method re-entrant

2022-10-18 Thread Andrew Rybchenko
Ivan, could you rebase the patch, please, and send a new version. Thanks, Andrew. On 10/18/22 13:34, Chas Williams wrote: On 10/17/22 10:10, Andrew Rybchenko wrote: On 10/17/22 15:32, Chas Williams wrote: This appears to be correct. A minor comment inline. On 10/17/22 04:42, Andrew Rybche

Re: [PATCH] net/bonding: make bonded device configure method re-entrant

2022-10-18 Thread Chas Williams
On 10/17/22 10:10, Andrew Rybchenko wrote: On 10/17/22 15:32, Chas Williams wrote: This appears to be correct. A minor comment inline. On 10/17/22 04:42, Andrew Rybchenko wrote: Chas, Cornor, could you review the patch, please. Thanks, Andrew. On 9/11/22 15:24, Ivan Malov wrote: Accordin

Re: [PATCH] net/bonding: make bonded device configure method re-entrant

2022-10-17 Thread Andrew Rybchenko
On 10/17/22 15:32, Chas Williams wrote: This appears to be correct. A minor comment inline. On 10/17/22 04:42, Andrew Rybchenko wrote: Chas, Cornor, could you review the patch, please. Thanks, Andrew. On 9/11/22 15:24, Ivan Malov wrote: According to the documentation, rte_eth_dev_configure()

Re: [PATCH] net/bonding: make bonded device configure method re-entrant

2022-10-17 Thread Chas Williams
This appears to be correct. A minor comment inline. On 10/17/22 04:42, Andrew Rybchenko wrote: Chas, Cornor, could you review the patch, please. Thanks, Andrew. On 9/11/22 15:24, Ivan Malov wrote: According to the documentation, rte_eth_dev_configure() can be invoked repeatedly while in stopp

Re: [PATCH] net/bonding: make bonded device configure method re-entrant

2022-10-17 Thread Andrew Rybchenko
Chas, Cornor, could you review the patch, please. Thanks, Andrew. On 9/11/22 15:24, Ivan Malov wrote: According to the documentation, rte_eth_dev_configure() can be invoked repeatedly while in stopped state. The current implementation in the bonding driver allows for that (technically), but the

[PATCH] net/bonding: make bonded device configure method re-entrant

2022-09-11 Thread Ivan Malov
According to the documentation, rte_eth_dev_configure() can be invoked repeatedly while in stopped state. The current implementation in the bonding driver allows for that (technically), but the user sees warnings which say that back-end devices have already been harnessed. Re-factor the code to hav