Hello, On Fri, May 26, 2017 at 04:54:57PM +0200, Martin Pieuchot wrote: > The global list of softc is *not* accessed in the input path, so it > doesn't need splnet(). > > ioctl(2) handlers are already executed with the NET_LOCK() held, so > splnet() is superfluous.
changes look good to me, but still have a question: what is the plan for splnet()/splx() in trunk_init() and trunk_stop() functions. It seems to me they are executed through ioctl(2) only, so we can also let them go. or you want to leave it for another day? thanks and regards sasha