On 11/11/24 10:41 PM, Miquel Raynal wrote:
Why not just enclose this list_del() within a mutex_lock(iflist_mtx) like the others? Would probably make more sense and prevent the use of yet another protection mechanism? Is there anything preventing the use of this mutex here?
Moreover, if we manage interfaces list with RCU and device status with an extra status bit, do we need 'iflist_mtx' at all? I've tried this in https://syzkaller.appspot.com/text?tag=Patch&x=17b1f4e8580000 and it looks good. Dmitry
