Re: [ovs-dev] [PATCH] vswitchd: Wait for a bridge exit before replying to exit unixctl.

2023-07-07 Thread Eelco Chaudron
On 7 Jul 2023, at 14:19, Ilya Maximets wrote: > On 7/7/23 14:12, Eelco Chaudron wrote: >> >> >> On 7 Jul 2023, at 12:58, Ilya Maximets wrote: >> >>> On 7/5/23 15:54, Eelco Chaudron wrote: On 4 Jul 2023, at 15:11, Ilya Maximets wrote: > Before the cleanup option, the

Re: [ovs-dev] [PATCH] vswitchd: Wait for a bridge exit before replying to exit unixctl.

2023-07-07 Thread Ilya Maximets
On 7/7/23 14:12, Eelco Chaudron wrote: > > > On 7 Jul 2023, at 12:58, Ilya Maximets wrote: > >> On 7/5/23 15:54, Eelco Chaudron wrote: >>> >>> >>> On 4 Jul 2023, at 15:11, Ilya Maximets wrote: >>> Before the cleanup option, the bridge_exit() call was fairly fast, because it didn't

Re: [ovs-dev] [PATCH] vswitchd: Wait for a bridge exit before replying to exit unixctl.

2023-07-07 Thread Eelco Chaudron
On 7 Jul 2023, at 12:58, Ilya Maximets wrote: > On 7/5/23 15:54, Eelco Chaudron wrote: >> >> >> On 4 Jul 2023, at 15:11, Ilya Maximets wrote: >> >>> Before the cleanup option, the bridge_exit() call was fairly fast, >>> because it didn't include any particularly long operations. However, >>>

Re: [ovs-dev] [PATCH] vswitchd: Wait for a bridge exit before replying to exit unixctl.

2023-07-07 Thread Ilya Maximets
On 7/5/23 15:54, Eelco Chaudron wrote: > > > On 4 Jul 2023, at 15:11, Ilya Maximets wrote: > >> Before the cleanup option, the bridge_exit() call was fairly fast, >> because it didn't include any particularly long operations. However, >> with the cleanup flag, this function destroys a lot of

Re: [ovs-dev] [PATCH] vswitchd: Wait for a bridge exit before replying to exit unixctl.

2023-07-05 Thread Eelco Chaudron
On 4 Jul 2023, at 15:11, Ilya Maximets wrote: > Before the cleanup option, the bridge_exit() call was fairly fast, > because it didn't include any particularly long operations. However, > with the cleanup flag, this function destroys a lot of datapath > resources freeing a lot of memory,

[ovs-dev] [PATCH] vswitchd: Wait for a bridge exit before replying to exit unixctl.

2023-07-04 Thread Ilya Maximets
Before the cleanup option, the bridge_exit() call was fairly fast, because it didn't include any particularly long operations. However, with the cleanup flag, this function destroys a lot of datapath resources freeing a lot of memory, waiting on RCU and talking to the kernel. That may take a