Re: modular net drivers, take 2

[Andrew Morton <andrewm@xxxxxxxxxx>]

Andi Kleen wrote:
> 
> 
> Ok, there is still a small race with the actual module unload. I think
> the cleanest solution is to let open/close run in the big kernel lock.
> They are not performance critical anyways.
> 
> Comments ?

- devinet_ioctl() calls dev_change_flags() direct, thus
  neatly bypassing your lock_kernel() :(

- sys_ioctl() and sys_delete_module() both already claim
  the big lock, so where's the race anyway?  I feel I'm missing
  something..