On Mon, Feb 8, 2021 at 9:50 AM Taehee Yoo [off-list ref] 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!