Re: Fewer pool_get() in kqueue_register()

2020-08-19 Thread Visa Hankala
On Wed, Aug 19, 2020 at 12:10:12PM +0200, Martin Pieuchot wrote: > On 18/08/20(Tue) 15:30, Visa Hankala wrote: > > On Tue, Aug 18, 2020 at 11:04:47AM +0200, Martin Pieuchot wrote: > > > Diff below changes the order of operations in kqueue_register() to get > > > rid of an unnecessary pool_get().

Re: Fewer pool_get() in kqueue_register()

2020-08-19 Thread Martin Pieuchot
On 18/08/20(Tue) 15:30, Visa Hankala wrote: > On Tue, Aug 18, 2020 at 11:04:47AM +0200, Martin Pieuchot wrote: > > Diff below changes the order of operations in kqueue_register() to get > > rid of an unnecessary pool_get(). When an event is already present on > > the list try to acquire it first.

Re: Fewer pool_get() in kqueue_register()

2020-08-18 Thread Visa Hankala
On Tue, Aug 18, 2020 at 11:04:47AM +0200, Martin Pieuchot wrote: > Diff below changes the order of operations in kqueue_register() to get > rid of an unnecessary pool_get(). When an event is already present on > the list try to acquire it first. Note that knote_acquire() may sleep > in which

Re: Fewer pool_get() in kqueue_register()

2020-08-18 Thread Martin Pieuchot
On 18/08/20(Tue) 11:22, Mark Kettenis wrote: > > Date: Tue, 18 Aug 2020 11:04:47 +0200 > > From: Martin Pieuchot > > > > Diff below changes the order of operations in kqueue_register() to get > > rid of an unnecessary pool_get(). When an event is already present on > > the list try to acquire

Re: Fewer pool_get() in kqueue_register()

2020-08-18 Thread Mark Kettenis
> Date: Tue, 18 Aug 2020 11:04:47 +0200 > From: Martin Pieuchot > > Diff below changes the order of operations in kqueue_register() to get > rid of an unnecessary pool_get(). When an event is already present on > the list try to acquire it first. Note that knote_acquire() may sleep > in which

Fewer pool_get() in kqueue_register()

2020-08-18 Thread Martin Pieuchot
Diff below changes the order of operations in kqueue_register() to get rid of an unnecessary pool_get(). When an event is already present on the list try to acquire it first. Note that knote_acquire() may sleep in which case the list might have changed so the lookup has to always begin from the