cvs commit: src/sys/dev/em if_em.c

Robert Watson rwatson at FreeBSD.org
Mon Jul 31 13:49:08 UTC 2006


On Mon, 31 Jul 2006, Gleb Smirnoff wrote:

> On Thu, Jul 20, 2006 at 03:57:58AM +0000, Pyun YongHyeon wrote:
> P> yongari     2006-07-20 03:57:58 UTC
> P>
> P>   FreeBSD src repository
> P>
> P>   Modified files:
> P>     sys/dev/em           if_em.c
> P>   Log:
> P>   Honor IFF_DRV_OACTIVE in em_start_locked().
>
> Isn't it better to shift this change to the em_start(), to avoid double 
> check of IFF_DRV_RUNNING? Or may be just merge em_start_locked() into the 
> em_start()?

Isn't em_start_locked() called from quite a few different places, including 
the interrupt handler, etc, where the mutex is already held?

Robert N M Watson
Computer Laboratory
University of Cambridge


More information about the cvs-all mailing list