Skip site navigation (1)Skip section navigation (2)
Date:      Mon, 31 Aug 2009 11:17:35 +0100 (BST)
From:      Robert Watson <rwatson@FreeBSD.org>
To:        Qing Li <qingli@FreeBSD.org>
Cc:        svn-src-stable@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org, svn-src-stable-8@freebsd.org
Subject:   Re: svn commit: r196679 - in stable/8/sys: . amd64/include/xen cddl/contrib/opensolaris contrib/dev/acpica contrib/pf dev/xen/xenpci net
Message-ID:  <alpine.BSF.2.00.0908311116300.48497@fledge.watson.org>
In-Reply-To: <200908310018.n7V0IIC1010643@svn.freebsd.org>
References:  <200908310018.n7V0IIC1010643@svn.freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help
University of Cambridge

On Mon, 31 Aug 2009, Qing Li wrote:

>  As part of r196609, a call to  "rtalloc" did not take the fib into
>  account. So call the appropriate "rtalloc_ign_fib()" instead of
>  calling "rtalloc_ign()".
>
>  Reviewed by:	pointed out by bz
>  Approved by:	re

I don't have this in my list of re-approved merges.  Are you sure this change 
was approved by re?

Robert N M Watson
Computer Laboratory
University of Cambridge

>
> Modified:
>  stable/8/sys/   (props changed)
>  stable/8/sys/amd64/include/xen/   (props changed)
>  stable/8/sys/cddl/contrib/opensolaris/   (props changed)
>  stable/8/sys/contrib/dev/acpica/   (props changed)
>  stable/8/sys/contrib/pf/   (props changed)
>  stable/8/sys/dev/xen/xenpci/   (props changed)
>  stable/8/sys/net/rtsock.c
>
> Modified: stable/8/sys/net/rtsock.c
> ==============================================================================
> --- stable/8/sys/net/rtsock.c	Mon Aug 31 00:14:37 2009	(r196678)
> +++ stable/8/sys/net/rtsock.c	Mon Aug 31 00:18:17 2009	(r196679)
> @@ -527,7 +527,7 @@ route_output(struct mbuf *m, struct sock
>
> 		bzero(&gw_ro, sizeof(gw_ro));
> 		gw_ro.ro_dst = *info.rti_info[RTAX_GATEWAY];
> -		rtalloc_ign(&gw_ro, 0);
> +		rtalloc_ign_fib(&gw_ro, 0, so->so_fibnum);
> 		/*
> 		 * A host route through the loopback interface is
> 		 * installed for each interface adddress. In pre 8.0
>



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