On Mon, Feb 8, 2021 at 9:50 AM Taehee Yoo <ap420...@gmail.com> wrote: > > This patchset changes context of MLD module. > Before this patch, MLD functions are atomic context so it couldn't use > sleepable functions and flags. > This patchset also contains other refactoring patches, which is to use > list macro, etc. > > 1~3 and 5~7 are refactoring patches and 4 and 8 patches actually > change context, which is the actual goal of this patchset. > MLD module has used timer API. The timer expiration function is > processed as the atomic context so in order to change context, > it should be replaced. > So, The fourth patch is to switch from timer to delayed_work. > And the eighth patch is to use the mutex instead of spinlock and > rwlock because A critical section of spinlock and rwlock is atomic.
Thanks for working on this. A quick question: are those cleanup or refactoring patches necessary for the main patch, that is patch 4? If not, please consider separating patch 4 and patch 8 out, as they fix a bug so they are different from the others. Thanks!
