From owner-cvs-src@FreeBSD.ORG Tue Dec 7 10:18:13 2004 Return-Path: Delivered-To: cvs-src@freebsd.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id 6CBBC16A4CE for ; Tue, 7 Dec 2004 10:18:13 +0000 (GMT) Received: from c00l3r.networx.ch (c00l3r.networx.ch [62.48.2.2]) by mx1.FreeBSD.org (Postfix) with ESMTP id 5EB9C43D62 for ; Tue, 7 Dec 2004 10:18:12 +0000 (GMT) (envelope-from andre@freebsd.org) Received: (qmail 65878 invoked from network); 7 Dec 2004 10:08:21 -0000 Received: from unknown (HELO freebsd.org) ([62.48.0.53]) (envelope-sender ) by c00l3r.networx.ch (qmail-ldap-1.03) with SMTP for ; 7 Dec 2004 10:08:21 -0000 Message-ID: <41B58367.EB32CF1@freebsd.org> Date: Tue, 07 Dec 2004 11:18:15 +0100 From: Andre Oppermann X-Mailer: Mozilla 4.8 [en] (Windows NT 5.0; U) X-Accept-Language: en MIME-Version: 1.0 To: SUZUKI Shinsuke References: <200412061902.iB6J2hJ6000543@repoman.freebsd.org> Content-Type: text/plain; charset=us-ascii Content-Transfer-Encoding: 7bit cc: cvs-src@FreeBSD.org cc: glebius@FreeBSD.org cc: cvs-all@FreeBSD.org cc: src-committers@FreeBSD.org Subject: Re: cvs commit: src/sys/netinet in_gif.c X-BeenThere: cvs-src@freebsd.org X-Mailman-Version: 2.1.1 Precedence: list List-Id: CVS commit messages for the src tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 07 Dec 2004 10:18:13 -0000 SUZUKI Shinsuke wrote: > > >>>>> On Mon, 6 Dec 2004 19:02:43 +0000 (UTC) > >>>>> glebius@FreeBSD.org(Gleb Smirnoff) said: > > > - Make route cacheing optional, configurable via IFF_LINK0 flag. > > - Turn it off by default. > > Revision Changes Path > > 1.27 +6 -0 src/sys/netinet/in_gif.c > > Three comments, and one proposal. > > - please comment this change in share/man/man4/gif.4 > > - you need the same fix to sys/netinet6/in6_gif.c (for IPv4 over IPv6 > tunnel) > > - it seems like sys/net/if_stf.c also needs the similar fix > (I haven't confirmed it though, I'll check it later) > > - Here's a patch to update route cache periodically, to prevent > unexpected gif failure due to a stale route cache. Is it okay to > commit it together with your change? I hope to remove the ability to have pointers into the routing table before 6.0 release. -- Andre