Re: uao references & uao_swap_off() cleanup

2021-06-28 Thread Martin Pieuchot
On 23/06/21(Wed) 23:03, Jonathan Matthew wrote: > On Wed, Jun 23, 2021 at 09:37:10AM +0200, Martin Pieuchot wrote: > > On 16/06/21(Wed) 11:26, Martin Pieuchot wrote: > > > Diff below does two things: > > > > > > - Use atomic operations for incrementing/decrementing references of > > > anonymous

Re: uao references & uao_swap_off() cleanup

2021-06-23 Thread Jonathan Matthew
On Wed, Jun 23, 2021 at 09:37:10AM +0200, Martin Pieuchot wrote: > On 16/06/21(Wed) 11:26, Martin Pieuchot wrote: > > Diff below does two things: > > > > - Use atomic operations for incrementing/decrementing references of > > anonymous objects. This allows us to manipulate them without holding

Re: uao references & uao_swap_off() cleanup

2021-06-23 Thread Martin Pieuchot
On 16/06/21(Wed) 11:26, Martin Pieuchot wrote: > Diff below does two things: > > - Use atomic operations for incrementing/decrementing references of > anonymous objects. This allows us to manipulate them without holding > the KERNEL_LOCK(). > > - Rewrite the loop from uao_swap_off() to only

uao references & uao_swap_off() cleanup

2021-06-16 Thread Martin Pieuchot
Diff below does two things: - Use atomic operations for incrementing/decrementing references of anonymous objects. This allows us to manipulate them without holding the KERNEL_LOCK(). - Rewrite the loop from uao_swap_off() to only keep a reference to the next item in the list. This is