From owner-freebsd-mobile  Tue Aug 21 12:54: 0 2001
Delivered-To: freebsd-mobile@freebsd.org
Received: from ns.yogotech.com (ns.yogotech.com [206.127.123.66])
	by hub.freebsd.org (Postfix) with ESMTP
	id 2ABBB37B41E; Tue, 21 Aug 2001 12:53:25 -0700 (PDT)
	(envelope-from nate@yogotech.com)
Received: from nomad.yogotech.com (nomad.yogotech.com [206.127.123.131])
	by ns.yogotech.com (8.9.3/8.9.3) with ESMTP id NAA19180;
	Tue, 21 Aug 2001 13:53:20 -0600 (MDT)
	(envelope-from nate@nomad.yogotech.com)
Received: (from nate@localhost)
	by nomad.yogotech.com (8.8.8/8.8.8) id NAA05676;
	Tue, 21 Aug 2001 13:53:20 -0600 (MDT)
	(envelope-from nate)
From: Nate Williams <nate@yogotech.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Transfer-Encoding: 7bit
Message-ID: <15234.48176.6699.983659@nomad.yogotech.com>
Date: Tue, 21 Aug 2001 13:53:20 -0600
To: Toshihiko ARAI <toshi@jp.FreeBSD.org>
Cc: nate@yogotech.com (Nate Williams), freebsd-mobile@FreeBSD.org,
	imp@FreeBSD.org
Subject: Re: Documentation for rc.conf's removable_interfaces?
In-Reply-To: <200108211348.f7LDmTh30188.toshi@jp.FreeBSD.org>
References: <15228.2988.181554.313833@nomad.yogotech.com>
	<200108200357.f7K3vMn18594.toshi@jp.FreeBSD.org>
	<15233.22136.524384.459698@nomad.yogotech.com>
	<200108211348.f7LDmTh30188.toshi@jp.FreeBSD.org>
X-Mailer: VM 6.95 under 21.1 (patch 12) "Channel Islands" XEmacs Lucid
Reply-To: nate@yogotech.com (Nate Williams)
Sender: owner-freebsd-mobile@FreeBSD.ORG
Precedence: bulk
List-ID: <freebsd-mobile.FreeBSD.ORG>
List-Archive: <http://docs.freebsd.org/mail/> (Web Archive)
List-Help: <mailto:majordomo@FreeBSD.ORG?subject=help> (List Instructions)
List-Subscribe: <mailto:majordomo@FreeBSD.ORG?subject=subscribe%20freebsd-mobile>
List-Unsubscribe: <mailto:majordomo@FreeBSD.ORG?subject=unsubscribe%20freebsd-mobile>
X-Loop: FreeBSD.org

> Sorry. There is not my English well.

No problem.  Your English is much better than my Japanese.

> + <15233.22136.524384.459698@nomad.yogotech.com>, Nate Williams wrote:
> 
> >> I thought that it is necessary for sysinstall at first, but
> >> i think that removable_interfaces variable is unnecessary now.
> >> Because I am busy currently, a patch for sysinstall is
> >> unfinished.
> 
> > You realize that FreeBSD 4.4-RELEASE is due to be released in a couple
> > of weeks, and that currently static routing is broken.
> 
> > Will you have time in the near future to work on this, or would you
> > suggest we instead back out these changes so that things work again for
> > the new release?
> 
> I agree to delete 'removable_*' variables.
> How about with the following patches?

This is good.  However, there is still one more problem with the file as
it exists.  If a default_router exists, it will not be used because of
the following lines.

                # Add default route into $static_routes
                case ${defaultrouter} in
                [Nn][Oo] | '')
                        ;;
                *)
                        static_routes="default ${static_routes}"
                        route_default="default ${defaultrouter}"
                        ;;
                esac

However, I'm not sure what the purpose of adding 'default' to
static_routes is, and unfortunately 'route_default' is not used anywhere
in the file.

Can you explain what is intended here?

Thanks!


Nate

ps. The below patch is good, although it may be necessary to do a route
flush in certain cases.

> 
> Index: src/etc/pccard_ether
> ===================================================================
> RCS file: /home/ncvs/src/etc/pccard_ether,v
> retrieving revision 1.25
> diff -u -r1.25 pccard_ether
> --- src/etc/pccard_ether	2001/05/09 20:30:55	1.25
> +++ src/etc/pccard_ether	2001/08/21 13:46:15
> @@ -48,7 +48,6 @@
>  
>  case ${pccard_ifconfig} in
>  [Nn][Oo] | '')
> -	expr "${removable_interfaces}" : ".*${interface}" > /dev/null || exit 0
>  	;;
>  *)
>  	# Backward compatible
> @@ -171,15 +170,5 @@
>  	# Remove the network interface and cleaning ARP table
>  	ifconfig ${interface} delete
>  	arp -d -a
> -
> -	# Clean the routing table
> -	case ${removable_route_flush} in
> -	[Nn][Oo])
> -	        ;;
> -	*)	
> -		# flush beforehand, just in case....
> -		route -n flush -inet
> -		;;
> -	esac
>  	;;
>  esac
> 
> --
> Toshihiko ARAI

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