Skip site navigation (1)Skip section navigation (2)
Date:      Mon, 4 Jun 2001 09:45:03 +0400
From:      Yar Tikhiy <yar@FreeBSD.org>
To:        Greg Lehey <grog@FreeBSD.org>
Cc:        cvs-committers@FreeBSD.org, cvs-all@FreeBSD.org, freebsd-arch@FreeBSD.org
Subject:   Re: cvs commit: src/sys/net if_gif.c
Message-ID:  <20010604094503.A43248@comp.chem.msu.su>
In-Reply-To: <20010604094827.N85812@wantadilla.lemis.com>; from grog@FreeBSD.org on Mon, Jun 04, 2001 at 09:48:27AM %2B0930
References:  <200106031731.f53HVBA42280@freefall.freebsd.org> <20010604094827.N85812@wantadilla.lemis.com>

next in thread | previous in thread | raw e-mail | index | archive | help
[Added freebsd-arch to the Cc: list]

On Mon, Jun 04, 2001 at 09:48:27AM +0930, Greg Lehey wrote:
> On Sunday,  3 June 2001 at 10:31:11 -0700, Yar Tikhiy wrote:
> > yar         2001/06/03 10:31:11 PDT
> >
> >   Modified files:
> >     sys/net              if_gif.c
> >   Log:
> >   First, wrap the if_up() call into splimp()/splx() because
> >   if_up() must be called at splnet or higher.
> >   Second, set the IFF_RUNNING flag on an interface after its
> >   resources (i.e. tunnel source and destination addresses)
> >   have been set. Note that we don't set IFF_UP because it is
> >   if_up()'s job to do that.
> 
> Presumably this is in preparation for an MFC?  splimp() and splx() are
> nops in -CURRENT.

Thank you for pointing me out that. Indeed, I've missed the change.

BTW, could anyone tell me what is the modern state of affairs as
to locks in the networking code? The comments and man pages are
still talking about splnet in -current, so they need to be fixed,
but what should one write there instead? The IF_ENQUEUE case is
obvious: Forget about splnet() and the manual modification of
counters, just use IF_HANDOFF instead of IF_ENQUEUE. But what about
other places where splnet/splimp is used? There are quite a few of
them there...

-- 
Yar

To Unsubscribe: send mail to majordomo@FreeBSD.org
with "unsubscribe cvs-all" in the body of the message




Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20010604094503.A43248>