Re: CVS commit: src/sys/net

[Roy Marples <roy%marples.name@localhost>]

On 15/02/2016 08:08, Ryota Ozaki wrote:
> Module Name:	src
> Committed By:	ozaki-r
> Date:		Mon Feb 15 08:08:04 UTC 2016
> 
> Modified Files:
> 	src/sys/net: if.c if.h
> 
> Log Message:
> Run if_link_state_change in softint
> 
> if_link_state_change can execute the network stack that is expected to
> not run in hardware interrupt (at least now), however network drivers
> may call it in hardware interrupt. Avoid that by introducing a new
> softint for if_link_state_change.

I don't know anything about softints, so this could be a silly question,
but this change looks racey to me. Is there a guarantee anywhere that
if_link_state_change() won't be called again before
if_link_state_change_si() has been called?

Roy