Re: [ovs-dev] [PATCH v1 6/9] conntrack: Do not schedule zero ms timers

2021-02-24 Thread William Tu
On Tue, Feb 23, 2021 at 4:35 PM Gaëtan Rivet wrote: > > On Tue, Feb 23, 2021, at 22:56, William Tu wrote: > > On Wed, Feb 17, 2021 at 8:34 AM Gaetan Rivet wrote: > > > > > > When ct_sweep() is far behind on its work, the 'next_wake' returned can > > > be before the moment it started. When it

Re: [ovs-dev] [PATCH v1 6/9] conntrack: Do not schedule zero ms timers

2021-02-23 Thread Gaëtan Rivet
On Tue, Feb 23, 2021, at 22:56, William Tu wrote: > On Wed, Feb 17, 2021 at 8:34 AM Gaetan Rivet wrote: > > > > When ct_sweep() is far behind on its work, the 'next_wake' returned can > > be before the moment it started. When it happens, the thread schedules a > > zero ms timer that is logged as

Re: [ovs-dev] [PATCH v1 6/9] conntrack: Do not schedule zero ms timers

2021-02-23 Thread William Tu
On Wed, Feb 17, 2021 at 8:34 AM Gaetan Rivet wrote: > > When ct_sweep() is far behind on its work, the 'next_wake' returned can > be before the moment it started. When it happens, the thread schedules a > zero ms timer that is logged as an error. > > Instead, mark the thread for immediate wake in

[ovs-dev] [PATCH v1 6/9] conntrack: Do not schedule zero ms timers

2021-02-17 Thread Gaetan Rivet
When ct_sweep() is far behind on its work, the 'next_wake' returned can be before the moment it started. When it happens, the thread schedules a zero ms timer that is logged as an error. Instead, mark the thread for immediate wake in the next poll_block(). Signed-off-by: Gaetan Rivet