From owner-freebsd-rc@FreeBSD.ORG Mon Sep 14 11:07:07 2009 Return-Path: Delivered-To: freebsd-rc@FreeBSD.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id DE5681065692 for ; Mon, 14 Sep 2009 11:07:07 +0000 (UTC) (envelope-from owner-bugmaster@FreeBSD.org) Received: from freefall.freebsd.org (freefall.freebsd.org [IPv6:2001:4f8:fff6::28]) by mx1.freebsd.org (Postfix) with ESMTP id CC5D18FC14 for ; Mon, 14 Sep 2009 11:07:07 +0000 (UTC) Received: from freefall.freebsd.org (localhost [127.0.0.1]) by freefall.freebsd.org (8.14.3/8.14.3) with ESMTP id n8EB77N3072466 for ; Mon, 14 Sep 2009 11:07:07 GMT (envelope-from owner-bugmaster@FreeBSD.org) Received: (from gnats@localhost) by freefall.freebsd.org (8.14.3/8.14.3/Submit) id n8EB77cV072462 for freebsd-rc@FreeBSD.org; Mon, 14 Sep 2009 11:07:07 GMT (envelope-from owner-bugmaster@FreeBSD.org) Date: Mon, 14 Sep 2009 11:07:07 GMT Message-Id: <200909141107.n8EB77cV072462@freefall.freebsd.org> X-Authentication-Warning: freefall.freebsd.org: gnats set sender to owner-bugmaster@FreeBSD.org using -f From: FreeBSD bugmaster To: freebsd-rc@FreeBSD.org Cc: Subject: Current problem reports assigned to freebsd-rc@FreeBSD.org X-BeenThere: freebsd-rc@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: "Discussion related to /etc/rc.d design and implementation." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 14 Sep 2009 11:07:07 -0000 Note: to view an individual PR, use: http://www.freebsd.org/cgi/query-pr.cgi?pr=(number). The following is a listing of current problems submitted by FreeBSD users. These represent problem reports covering all versions including experimental development code and obsolete releases. S Tracker Resp. Description -------------------------------------------------------------------------------- o conf/138686 rc [rc.conf] $named_conf in rc.conf is not used o conf/138460 rc [patch] start local rc scripts in background o conf/138208 rc [rc] [patch] Making rc.firewall (workstation) IPv6 awa o conf/137629 rc [rc] background_dhclient rc.conf option causing double o conf/137470 rc [PATCH] /etc/rc.d/mdconfig2 : prioritize cli parameter o conf/136875 rc [request] _flags appending o conf/136624 rc [rc.d] sysctl variables for ipnat are not applied on b o conf/134918 rc [patch] rc.subr fails to detect perl daemons o conf/134660 rc [patch] rc-script for initializing ng_netflow+ng_ipfw o conf/134333 rc PPP configuration problem in the rc.d scripts in combi o conf/134006 rc [patch] Unload console screensaver kernel modules if s o conf/133987 rc [rc.d] defaultroute broken with DHCP in some cases o conf/133890 rc [patch] sshd(8): add multiple profiles to the rc.d scr o conf/132766 rc wait_for_pids() in /etc/rc.subr is dull. o conf/132483 rc rc.subr(8) [patch] setfib(1) support for rc.subr o conf/132476 rc [rc.d] [patch] add support setfib(1) in rc.d/routing o conf/130414 rc [patch] rc services started with onestart are not stop o conf/128299 rc [patch] /etc/rc.d/geli does not mount partitions using o conf/127917 rc [patch] dumpon rejects on start with physmem>swap even o bin/126562 rc rcorder(8) fails to run unrelated startup scripts when o conf/126392 rc [patch] rc.conf ifconfig_xx keywords cannot be escaped o bin/126324 rc [patch] rc.d/tmp: Prevent mounting /tmp in second tim o conf/124747 rc [patch] savecore can't create dump from encrypted swap o conf/124248 rc [jail] [patch] add support for nice value for rc.d/jai o conf/123734 rc [patch] Chipset VIA CX700 requires extra initializatio o conf/123222 rc [patch] Add rtprio(1)/idprio(1) support to rc.subr(8). o conf/122477 rc [patch] /etc/rc.d/mdconfig and mdconfig2 are ignoring o conf/122170 rc [patch] [request] New feature: notify admin via page o o kern/121566 rc [nfs] [request] [patch] ethernet iface should be broug o conf/120431 rc [patch] devfs.rules are not initialized under certain o conf/120406 rc [devd] [patch] Handle newly attached pcm devices (eg. o conf/119874 rc [patch] "/etc/rc.d/pf reload" fails if there are macro o conf/119076 rc [patch] [rc.d] /etc/rc.d/netif tries to remove alias a o bin/118325 rc [patch] [request] new periodic script to test statuses o conf/118255 rc savecore never finding kernel core dumps (rcorder prob o conf/117935 rc [patch] ppp fails to start at boot because of missing o conf/113915 rc [patch] ndis wireless driver fails to associate when i o conf/109980 rc /etc/rc.d/netif restart doesn't destroy cloned_interfa o conf/109562 rc [rc.d] [patch] [request] Make rc.d/devfs usable from c o conf/108589 rc rtsol(8) fails due to default ipfw rules o conf/106009 rc [ppp] [patch] [request] Fix pppoed startup script to p o conf/105689 rc [ppp] [request] syslogd starts too late at boot o conf/105568 rc [patch] [request] Add more flexibility to rc.conf, to o conf/105145 rc [ppp] [patch] [request] add redial function to rc.d/pp o conf/104549 rc [patch] rc.d/nfsd needs special _find_processes functi o conf/102700 rc [geli] [patch] Add encrypted /tmp support to GELI/GBDE o conf/99721 rc [patch] /etc/rc.initdiskless problem copy dotfile in s o conf/99444 rc [patch] Enhancement: rc.subr could easily support star o conf/96343 rc [patch] rc.d order change to start inet6 before pf o conf/93815 rc [patch] Adds in the ability to save ipfw rules to rc.d o conf/92523 rc [patch] allow rc scripts to kill process after a timeo o conf/89870 rc [patch] [request] make netif verbose rc.conf toggle o conf/89061 rc [patch] IPv6 6to4 auto-configuration enhancement o conf/88913 rc [patch] wrapper support for rc.subr o conf/85819 rc [patch] script allowing multiuser mode in spite of fsc o kern/81006 rc ipnat not working with tunnel interfaces on startup o conf/77663 rc Suggestion: add /etc/rc.d/addnetswap after addcritremo o conf/73677 rc [patch] add support for powernow states to power_profi o conf/58939 rc [patch] dumb little hack for /etc/rc.firewall{,6} o conf/56934 rc [patch] rc.firewall rules for natd expect an interface o conf/45226 rc [patch] Fix for rc.network, ppp-user annoyance o conf/44170 rc [patch] Add ability to run multiple pppoed(8) on start 62 problems total. From owner-freebsd-rc@FreeBSD.ORG Mon Sep 14 22:13:48 2009 Return-Path: Delivered-To: freebsd-rc@hub.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id EF2C51065694; Mon, 14 Sep 2009 22:13:48 +0000 (UTC) (envelope-from dougb@FreeBSD.org) Received: from freefall.freebsd.org (freefall.freebsd.org [IPv6:2001:4f8:fff6::28]) by mx1.freebsd.org (Postfix) with ESMTP id C6FF98FC1D; Mon, 14 Sep 2009 22:13:48 +0000 (UTC) Received: from freefall.freebsd.org (dougb@localhost [127.0.0.1]) by freefall.freebsd.org (8.14.3/8.14.3) with ESMTP id n8EMDmYc046880; Mon, 14 Sep 2009 22:13:48 GMT (envelope-from dougb@freefall.freebsd.org) Received: (from dougb@localhost) by freefall.freebsd.org (8.14.3/8.14.3/Submit) id n8EMDms3046874; Mon, 14 Sep 2009 22:13:48 GMT (envelope-from dougb) Date: Mon, 14 Sep 2009 22:13:48 GMT Message-Id: <200909142213.n8EMDms3046874@freefall.freebsd.org> To: villa.alberto@gmail.com, dougb@FreeBSD.org, freebsd-rc@FreeBSD.org From: dougb@FreeBSD.org Cc: Subject: Re: conf/138460: [patch] start local rc scripts in background X-BeenThere: freebsd-rc@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: "Discussion related to /etc/rc.d design and implementation." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 14 Sep 2009 22:13:49 -0000 Synopsis: [patch] start local rc scripts in background State-Changed-From-To: open->closed State-Changed-By: dougb State-Changed-When: Mon Sep 14 22:13:08 UTC 2009 State-Changed-Why: No support for this idea has arisen on the list, so close the PR. http://www.freebsd.org/cgi/query-pr.cgi?pr=138460 From owner-freebsd-rc@FreeBSD.ORG Tue Sep 15 11:26:54 2009 Return-Path: Delivered-To: freebsd-rc@hub.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id A2816106566C; Tue, 15 Sep 2009 11:26:54 +0000 (UTC) (envelope-from dougb@FreeBSD.org) Received: from freefall.freebsd.org (freefall.freebsd.org [IPv6:2001:4f8:fff6::28]) by mx1.freebsd.org (Postfix) with ESMTP id 78D9E8FC0C; Tue, 15 Sep 2009 11:26:54 +0000 (UTC) Received: from freefall.freebsd.org (dougb@localhost [127.0.0.1]) by freefall.freebsd.org (8.14.3/8.14.3) with ESMTP id n8FBQsee081800; Tue, 15 Sep 2009 11:26:54 GMT (envelope-from dougb@freefall.freebsd.org) Received: (from dougb@localhost) by freefall.freebsd.org (8.14.3/8.14.3/Submit) id n8FBQr5d081796; Tue, 15 Sep 2009 11:26:53 GMT (envelope-from dougb) Date: Tue, 15 Sep 2009 11:26:53 GMT Message-Id: <200909151126.n8FBQr5d081796@freefall.freebsd.org> To: turutani@scphys.kyoto-u.ac.jp, dougb@FreeBSD.org, freebsd-rc@FreeBSD.org, dougb@FreeBSD.org From: dougb@FreeBSD.org Cc: Subject: Re: conf/138686: [rc.conf] $named_conf in rc.conf is not used X-BeenThere: freebsd-rc@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: "Discussion related to /etc/rc.d design and implementation." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 15 Sep 2009 11:26:54 -0000 Synopsis: [rc.conf] $named_conf in rc.conf is not used State-Changed-From-To: open->analyzed State-Changed-By: dougb State-Changed-When: Tue Sep 15 11:22:03 UTC 2009 State-Changed-Why: named does use /etc/namedb/named.conf for its configuration file by default, so this isn't really an error. I will look again at the possibility of tying the $named_conf variable into the script more tightly after 8.0 release. Responsible-Changed-From-To: freebsd-rc->dougb Responsible-Changed-By: dougb Responsible-Changed-When: Tue Sep 15 11:22:03 UTC 2009 Responsible-Changed-Why: This is my area. http://www.freebsd.org/cgi/query-pr.cgi?pr=138686 From owner-freebsd-rc@FreeBSD.ORG Wed Sep 16 21:28:15 2009 Return-Path: Delivered-To: freebsd-rc@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 6FCDC106566C for ; Wed, 16 Sep 2009 21:28:15 +0000 (UTC) (envelope-from dougb@FreeBSD.org) Received: from mail2.fluidhosting.com (mx21.fluidhosting.com [204.14.89.4]) by mx1.freebsd.org (Postfix) with ESMTP id 0385F8FC17 for ; Wed, 16 Sep 2009 21:28:14 +0000 (UTC) Received: (qmail 2071 invoked by uid 399); 16 Sep 2009 21:28:12 -0000 Received: from localhost (HELO foreign.dougb.net) (dougb@dougbarton.us@127.0.0.1) by localhost with ESMTPAM; 16 Sep 2009 21:28:12 -0000 X-Originating-IP: 127.0.0.1 X-Sender: dougb@dougbarton.us Message-ID: <4AB15865.9030406@FreeBSD.org> Date: Wed, 16 Sep 2009 14:28:05 -0700 From: Doug Barton Organization: http://www.FreeBSD.org/ User-Agent: Thunderbird 2.0.0.23 (X11/20090822) MIME-Version: 1.0 To: Ed Maste References: <200909141652.n8EGqdd1060471@svn.freebsd.org> In-Reply-To: <200909141652.n8EGqdd1060471@svn.freebsd.org> X-Enigmail-Version: 0.96.0 OpenPGP: id=D5B2F0FB Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit Cc: svn-src-head@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org, freebsd-rc@freebsd.org Subject: Re: svn commit: r197196 - head/etc/rc.d X-BeenThere: freebsd-rc@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: "Discussion related to /etc/rc.d design and implementation." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 16 Sep 2009 21:28:15 -0000 I personally would much prefer that people making changes in rc.d-land be more careful about what they change and thoroughly test their changes than to add more code like this that will both hide breakage and become stale over time. Doug Ed Maste wrote: > Author: emaste > Date: Mon Sep 14 16:52:38 2009 > New Revision: 197196 > URL: http://svn.freebsd.org/changeset/base/197196 > > Log: > Protect cross-script invocation by checking that the target script exists. > This allows pruning of rc.d scripts without getting too many ugly boottime > error messages. > > Inspired by phk's r128714 change to netif. > > Modified: > head/etc/rc.d/ppp > > Modified: head/etc/rc.d/ppp > ============================================================================== > --- head/etc/rc.d/ppp Mon Sep 14 16:16:07 2009 (r197195) > +++ head/etc/rc.d/ppp Mon Sep 14 16:52:38 2009 (r197196) > @@ -95,8 +95,12 @@ ppp_poststart() > { > # Re-Sync ipfilter and pf so they pick up any new network interfaces > # > - /etc/rc.d/ipfilter quietresync > - /etc/rc.d/pf quietresync > + if [ -f /etc/rc.d/ipfilter ]; then > + /etc/rc.d/ipfilter quietresync > + fi > + if [ -f /etc/rc.d/pf ]; then > + /etc/rc.d/pf quietresync > + fi > } > > ppp_stop_profile() { > -- This .signature sanitized for your protection From owner-freebsd-rc@FreeBSD.ORG Wed Sep 16 21:59:50 2009 Return-Path: Delivered-To: freebsd-rc@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 81A781065670 for ; Wed, 16 Sep 2009 21:59:50 +0000 (UTC) (envelope-from dougb@FreeBSD.org) Received: from mail2.fluidhosting.com (mx21.fluidhosting.com [204.14.89.4]) by mx1.freebsd.org (Postfix) with ESMTP id 3559D8FC1B for ; Wed, 16 Sep 2009 21:59:50 +0000 (UTC) Received: (qmail 21060 invoked by uid 399); 16 Sep 2009 21:59:49 -0000 Received: from localhost (HELO foreign.dougb.net) (dougb@dougbarton.us@127.0.0.1) by localhost with ESMTPAM; 16 Sep 2009 21:59:49 -0000 X-Originating-IP: 127.0.0.1 X-Sender: dougb@dougbarton.us Message-ID: <4AB15FCE.70505@FreeBSD.org> Date: Wed, 16 Sep 2009 14:59:42 -0700 From: Doug Barton Organization: http://www.FreeBSD.org/ User-Agent: Thunderbird 2.0.0.23 (X11/20090822) MIME-Version: 1.0 To: Hiroki Sato References: <200909122222.n8CMMV3d099311@svn.freebsd.org> In-Reply-To: <200909122222.n8CMMV3d099311@svn.freebsd.org> X-Enigmail-Version: 0.96.0 OpenPGP: id=D5B2F0FB Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit Cc: svn-src-head@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org, freebsd-rc@freebsd.org Subject: Re: svn commit: r197145 - in head: etc/defaults share/man/man5 X-BeenThere: freebsd-rc@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: "Discussion related to /etc/rc.d design and implementation." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 16 Sep 2009 21:59:50 -0000 [Please direct follow-ups to freebsd-rc@] Hiroki, I realize that you've posted your patches in the past, and I definitely had it in mind to review them in detail and give you feedback on them. However I got focused on my own projects for the pending release, and then since we were so close to the release I did not think you would be committing these changes until after it was done so I let review of these patches slip down my list of priorities. Therefore I ask you to accept my apologies for this "after the fact" review. Before I forget, you keep putting "mfc after 3 days" in your commit messages. You don't actually plan to MFC these changes to RELENG_8 prior to the 8.0-RELEASE do you? I would not be supportive of this given the sweeping nature of the changes and the (unfortunately) small percentage of our userbase that uses and tests IPv6. I think shaking this code out in HEAD for several weeks at least would be a good thing. That said I very much like the idea of integrating IPv6 support into the overall network code, and I am very supportive of that. I do have some small problems with the direction that you've taken in a few areas, which happen to be neatly summarized in this post. Hiroki Sato wrote: > Author: hrs > Date: Sat Sep 12 22:22:31 2009 > New Revision: 197145 > URL: http://svn.freebsd.org/changeset/base/197145 > > Log: > The following changes are added because of > network_ipv6->rc.d/netif integration: > > - $ipv6_enable is now obsolete. Instead, IPv6 is enabled by > default if the kernel supports it, and $ipv6_network_interfaces > is "none" by default. If you want to use IPv6, define > $ipv6_network_interfaces and $ifconfig_xxx_ipv6. In general I have a problem with the idea of drastically changing the semantics of the current code when I can't see any real value in doing so. I object to this change specifically because on my laptop I really like having the ability to easily disable IPv6 when I am not on my home network. My preferred scenario would be something similar to what we have now, which is that if ipv6_enable is set that it takes the same list of interfaces as ipv4 (defaults to AUTO) and that rtadv is enabled by default for each of those interfaces. My feeling is that this not only significantly reduces POLA it will also more precisely fit the way that the vast majority of our users will actually use IPv6. On a "marketing" note I really think it would be valuable to make it as easy as possible for the average user to get IPv6 working. We have IPv4 down to it more or less "just works," I think IPv6 should be the same way. (On a side note, I'd actually like to see DHCP be the default for IPv4 such that if you have DHCP available on a network you wouldn't have to do any configuration at all to get FreeBSD on line, but that's a whole other topic.) > An interface which is in $network_interfaces and not in > $ipv6_network_interfaces will be marked as "inet6 > -auto_linklocal ifdisabled" (see ifconfig(8)). I think that if a user specifies both lists, and that they differ, that this is a very good way to handle it. > - $ipv6_ifconfig_xxx is renamed to ifconfig_xxx_ipv6 for > consistency with other address families. The old variables > still work but can be removed in the future. Note that > ipv6_ifconfig_xxx="..." should be replaced with > ifconfig_xxx_ipv6="inet6 ...". I see that you are giving a warn'ing when the old format is detected, which is good. > - Receiving ICMPv6 Router Advertisement is not automatically > enabled even if there is no manual configuration of IPv6 in > rc.conf. If you want it, define > ifconfig_xxx_ipv6="inet6 ... accept_rtadv". What is the reason for this change? While I am definitely in favor of making it easier to disable rtadv, I still think it should be the default. > - The rc.d/ip6addrctl now chooses address selection policy based > on $ipv6_prefer, not $ipv6_enable. The default is > ipv6_prefer=NO. Once again, what is the reason for this change? My read of the IPv6-using community is that if they have it available they want to use it as a first choice. I know that is certainly my preference. Once again my apologies for the late review, but hopefully this is useful. Doug -- This .signature sanitized for your protection From owner-freebsd-rc@FreeBSD.ORG Thu Sep 17 00:25:10 2009 Return-Path: Delivered-To: freebsd-rc@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id CFF48106566C; Thu, 17 Sep 2009 00:25:10 +0000 (UTC) (envelope-from ady@ady.ro) Received: from ey-out-2122.google.com (ey-out-2122.google.com [74.125.78.24]) by mx1.freebsd.org (Postfix) with ESMTP id 23B9F8FC0C; Thu, 17 Sep 2009 00:25:09 +0000 (UTC) Received: by ey-out-2122.google.com with SMTP id 4so22144eyf.9 for ; Wed, 16 Sep 2009 17:25:08 -0700 (PDT) MIME-Version: 1.0 Sender: ady@ady.ro Received: by 10.211.145.8 with SMTP id x8mr3054026ebn.67.1253147108807; Wed, 16 Sep 2009 17:25:08 -0700 (PDT) In-Reply-To: <20090911154142.GF1673@garage.freebsd.pl> References: <200909091831.n89IVOS9065418@freefall.freebsd.org> <78cb3d3f0909110334g6757f08fh57a0ab97c02b2d9a@mail.gmail.com> <20090911122050.GC1673@garage.freebsd.pl> <78cb3d3f0909110819m35f2d5ceo64de128320091a76@mail.gmail.com> <20090911154142.GF1673@garage.freebsd.pl> From: Adrian Penisoara Date: Thu, 17 Sep 2009 02:24:48 +0200 X-Google-Sender-Auth: b34c0cd07341cf05 Message-ID: <78cb3d3f0909161724i4f4b7b3andcd9fed79ca32278@mail.gmail.com> To: Pawel Jakub Dawidek Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: quoted-printable Cc: guido@freebsd.org, freebsd-rc@freebsd.org Subject: Re: conf/120228: [zfs] [patch] Split ZFS volume startup / ease ZFS swap volumes management X-BeenThere: freebsd-rc@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: "Discussion related to /etc/rc.d design and implementation." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 17 Sep 2009 00:25:11 -0000 Hi, On Fri, Sep 11, 2009 at 5:41 PM, Pawel Jakub Dawidek wrot= e: > On Fri, Sep 11, 2009 at 05:19:41PM +0200, Adrian Penisoara wrote: >> =A0Actually using /etc/fstab for zvol swap is not quite working in the >> current setup since /etc/rc.d/swap1 does not automatically swapoff >> upon shutdown which makes 'zvol stop' attempt to close the (swap) >> volume(s) while still in use. >> =A0 I guess 'zfs volfini' should be triggered by a shutdown hook >> similarly to what geom(4) classes are using. >> >> =A0I see Guido committed a swapoff hook in changeset 189624 (similar to >> what I suggested in my initial PR) but immediately backed it out the >> next day. Funny thing is that this swapoff hook flip-flopped over >> time, see changesets 135389, 137255. >> >> =A0Is there a reason not to have the swap devices automatically >> unmounted from /etc/swap1 upon shutdown ? > > The answer is simple: we don't want swapoff to fail. > > There really no point in removing swap on shutdown (it could still be a > good idea to do it when administrator executes /etc/rc.d/swap1 stop). > We unmount file systems on shutdown, because we want to keep them > consistent. There is no need to keep swap consistent, as the data will > be useless after reboot anyway. > > All in all running swapoff at shutdown time increases reboot time and > introduce possibility of failure (I think the kernel can panic if there > is no enough memory to move everything out of the swap, but I might be > wrong here). Some (if not most) of the Linux distros do turn off swap at shutdown time and AFAIK they did not encounter issues. One more white ball to do the same. > > There are some tricky cases too. For example: > > =A0 =A0 =A0 =A0# truncate -s 1g /swap.img > =A0 =A0 =A0 =A0# mdconfig -a -f /swap.img > =A0 =A0 =A0 =A0# swapon /dev/md0 > > Now if we don't remove swap we won't be able to unmount file system in > non-forcible way. Hmm, I guess we still need to "mdconfig -d" to have a clean filesystem unmount anyway... > >> > system to 'legacy' and then you can mount it from to /etc/fstab too. >> > >> >> =A0Perhaps not the best idea: what if I export and then import a zpool >> from one system to another -- then I will have some swap volumes >> automatically mounted upon boot, breaking POLA. >> >> =A0You can argue that ZFS filesystems will also be automatically mounted >> too, but this is a consistent behavior across different OSes while the >> discussed swap volume automount is FreeBSD specific. >> >> =A0Sorry to insist, but I still think we should we keep it simple... >> Unless people are already using/depending on this feature. > > Please understand that I don't want to disallow swap configuration from > /etc/fstab. I'm just giving an option. Sure. > > I'll think about it a bit more and talk to some people what they think > about this and I'll get back to you. Maybe this property is indeed > redundant, but I don't want to make any rush decisions. > OK, waiting your feedback. Thanks, Adrian Penisoara EnterpriseBSD From owner-freebsd-rc@FreeBSD.ORG Thu Sep 17 04:01:07 2009 Return-Path: Delivered-To: freebsd-rc@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id CD749106566B; Thu, 17 Sep 2009 04:01:07 +0000 (UTC) (envelope-from jhay@meraka.csir.co.za) Received: from zibbi.meraka.csir.co.za (zibbi.meraka.csir.co.za [IPv6:2001:4200:7000:2::1]) by mx1.freebsd.org (Postfix) with ESMTP id 136178FC13; Thu, 17 Sep 2009 04:01:07 +0000 (UTC) Received: by zibbi.meraka.csir.co.za (Postfix, from userid 3973) id 74BC13982C; Thu, 17 Sep 2009 06:01:01 +0200 (SAST) Date: Thu, 17 Sep 2009 06:01:01 +0200 From: John Hay To: Doug Barton Message-ID: <20090917040101.GA69461@zibbi.meraka.csir.co.za> References: <200909122222.n8CMMV3d099311@svn.freebsd.org> <4AB15FCE.70505@FreeBSD.org> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <4AB15FCE.70505@FreeBSD.org> User-Agent: Mutt/1.4.2.3i Cc: Hiroki Sato , freebsd-rc@freebsd.org Subject: Re: svn commit: r197145 - in head: etc/defaults share/man/man5 X-BeenThere: freebsd-rc@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: "Discussion related to /etc/rc.d design and implementation." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 17 Sep 2009 04:01:07 -0000 On Wed, Sep 16, 2009 at 02:59:42PM -0700, Doug Barton wrote: ... > > Hiroki Sato wrote: > > Author: hrs > > Date: Sat Sep 12 22:22:31 2009 > > New Revision: 197145 > > URL: http://svn.freebsd.org/changeset/base/197145 > > > > Log: > > The following changes are added because of > > network_ipv6->rc.d/netif integration: > > > > - $ipv6_enable is now obsolete. Instead, IPv6 is enabled by > > default if the kernel supports it, and $ipv6_network_interfaces > > is "none" by default. If you want to use IPv6, define > > $ipv6_network_interfaces and $ifconfig_xxx_ipv6. > > In general I have a problem with the idea of drastically changing the > semantics of the current code when I can't see any real value in doing > so. I object to this change specifically because on my laptop I really > like having the ability to easily disable IPv6 when I am not on my > home network. > > My preferred scenario would be something similar to what we have now, > which is that if ipv6_enable is set that it takes the same list of > interfaces as ipv4 (defaults to AUTO) and that rtadv is enabled by > default for each of those interfaces. My feeling is that this not only > significantly reduces POLA it will also more precisely fit the way > that the vast majority of our users will actually use IPv6. Can we then have it default to ipv6_enable="YES" and let people who do not want it, put ipv6_enable="NO" in their rc.conf? Most of the rest of the world has changed that we and we were also there a few years ago. (Yes I know because we run ipv6 at work and I can see how many machines just work without people having to fiddle with it. On the FreeBSD boxes you still have to go and enable it. :-((( ) John -- John Hay -- jhay@meraka.csir.co.za / jhay@FreeBSD.org From owner-freebsd-rc@FreeBSD.ORG Thu Sep 17 08:48:26 2009 Return-Path: Delivered-To: freebsd-rc@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 097AE1065676 for ; Thu, 17 Sep 2009 08:48:26 +0000 (UTC) (envelope-from dougb@FreeBSD.org) Received: from mail2.fluidhosting.com (mx21.fluidhosting.com [204.14.89.4]) by mx1.freebsd.org (Postfix) with ESMTP id A9CF48FC17 for ; Thu, 17 Sep 2009 08:48:25 +0000 (UTC) Received: (qmail 25871 invoked by uid 399); 17 Sep 2009 08:48:22 -0000 Received: from localhost (HELO foreign.dougb.net) (dougb@dougbarton.us@127.0.0.1) by localhost with ESMTPAM; 17 Sep 2009 08:48:22 -0000 X-Originating-IP: 127.0.0.1 X-Sender: dougb@dougbarton.us Message-ID: <4AB1F7CF.3080905@FreeBSD.org> Date: Thu, 17 Sep 2009 01:48:15 -0700 From: Doug Barton Organization: http://www.FreeBSD.org/ User-Agent: Thunderbird 2.0.0.23 (X11/20090822) MIME-Version: 1.0 To: John Hay References: <200909122222.n8CMMV3d099311@svn.freebsd.org> <4AB15FCE.70505@FreeBSD.org> <20090917040101.GA69461@zibbi.meraka.csir.co.za> In-Reply-To: <20090917040101.GA69461@zibbi.meraka.csir.co.za> X-Enigmail-Version: 0.96.0 OpenPGP: id=D5B2F0FB Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Cc: Hiroki Sato , freebsd-rc@freebsd.org Subject: Re: svn commit: r197145 - in head: etc/defaults share/man/man5 X-BeenThere: freebsd-rc@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: "Discussion related to /etc/rc.d design and implementation." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 17 Sep 2009 08:48:26 -0000 John Hay wrote: > On Wed, Sep 16, 2009 at 02:59:42PM -0700, Doug Barton wrote: > ... >> Hiroki Sato wrote: >>> Author: hrs >>> Date: Sat Sep 12 22:22:31 2009 >>> New Revision: 197145 >>> URL: http://svn.freebsd.org/changeset/base/197145 >>> >>> Log: >>> The following changes are added because of >>> network_ipv6->rc.d/netif integration: >>> >>> - $ipv6_enable is now obsolete. Instead, IPv6 is enabled by >>> default if the kernel supports it, and $ipv6_network_interfaces >>> is "none" by default. If you want to use IPv6, define >>> $ipv6_network_interfaces and $ifconfig_xxx_ipv6. >> In general I have a problem with the idea of drastically changing the >> semantics of the current code when I can't see any real value in doing >> so. I object to this change specifically because on my laptop I really >> like having the ability to easily disable IPv6 when I am not on my >> home network. >> >> My preferred scenario would be something similar to what we have now, >> which is that if ipv6_enable is set that it takes the same list of >> interfaces as ipv4 (defaults to AUTO) and that rtadv is enabled by >> default for each of those interfaces. My feeling is that this not only >> significantly reduces POLA it will also more precisely fit the way >> that the vast majority of our users will actually use IPv6. > > Can we then have it default to ipv6_enable="YES" and let people who do > not want it, put ipv6_enable="NO" in their rc.conf? Most of the rest > of the world has changed that we and we were also there a few years > ago. (Yes I know because we run ipv6 at work and I can see how many > machines just work without people having to fiddle with it. On the > FreeBSD boxes you still have to go and enable it. :-((( ) Certainly, I agree that "enabled" should be the default. Sorry if I wasn't clear about that. Doug -- This .signature sanitized for your protection From owner-freebsd-rc@FreeBSD.ORG Thu Sep 17 10:42:42 2009 Return-Path: Delivered-To: freebsd-rc@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 6BF361065676 for ; Thu, 17 Sep 2009 10:42:42 +0000 (UTC) (envelope-from dougb@FreeBSD.org) Received: from mail2.fluidhosting.com (mx21.fluidhosting.com [204.14.89.4]) by mx1.freebsd.org (Postfix) with ESMTP id A994E8FC12 for ; Thu, 17 Sep 2009 10:42:40 +0000 (UTC) Received: (qmail 11105 invoked by uid 399); 17 Sep 2009 10:42:36 -0000 Received: from localhost (HELO foreign.dougb.net) (dougb@dougbarton.us@127.0.0.1) by localhost with ESMTPAM; 17 Sep 2009 10:42:36 -0000 X-Originating-IP: 127.0.0.1 X-Sender: dougb@dougbarton.us Message-ID: <4AB21295.5080103@FreeBSD.org> Date: Thu, 17 Sep 2009 03:42:29 -0700 From: Doug Barton Organization: http://www.FreeBSD.org/ User-Agent: Thunderbird 2.0.0.23 (X11/20090822) MIME-Version: 1.0 To: Ian Smith References: <4AAFED75.4020502@andric.com> <4AB099BB.9040603@therek.net> <4AB17CB9.3040704@FreeBSD.org> <20090917183525.O73400@sola.nimnet.asn.au> In-Reply-To: <20090917183525.O73400@sola.nimnet.asn.au> X-Enigmail-Version: 0.96.0 OpenPGP: id=D5B2F0FB Content-Type: multipart/mixed; boundary="------------090801000801060201050607" Cc: freebsd-rc@freebsd.org, Oliver Fromme , Dimitry Andric , freebsd-stable@freebsd.org, Cezary Morga Subject: Re: 8.0 rc.d scripts less verbose? X-BeenThere: freebsd-rc@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: "Discussion related to /etc/rc.d design and implementation." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 17 Sep 2009 10:42:42 -0000 This is a multi-part message in MIME format. --------------090801000801060201050607 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Please follow up to this post on the freebsd-rc@freebsd.org list, thanks. See below for my comments. Ian Smith wrote: > On Wed, 16 Sep 2009, Doug Barton wrote: >> Cezary Morga wrote: >>> Dimitry Andric pisze: >>>> Index: etc/rc.subr >>>> =================================================================== >>>> --- etc/rc.subr (revision 196888) +++ etc/rc.subr >>>> (working copy) @@ -680,7 +680,7 @@ run_rc_command() >>>> >>>> # setup the full command to run # - [ -z >>>> "${rc_quiet}" ] && echo "Starting ${name}." + echo >>>> "Starting ${name}." if [ -n "$_chroot" ]; then _doit="\ >>>> ${_nice:+nice -n $_nice }\ >>> >>> Wouldn't it be easier to set rc_quiet to zero-length value, >>> like rc_quiet=""? >> >> That actually would give off too much information because >> rc_quiet suppresses error messages about scripts that don't >> currently have _enable set to anything useful. That's arguably a >> bug, but it's a bug that we've come to depend on. >> >> The attached patch adds a knob you can set in rc.conf to see >> "Starting foo" messages again. If people find this useful I'll be >> glad to commit it. I'm also open to suggestions for a better >> name. > >> From your patch: > - [ -z "${rc_quiet}" ] && echo "Starting ${name}." > + [ -z "${rc_quiet}" -o -n "${rc_starting}" ] && echo "Starting ${name}." > > Presumably this variable would go in /etc/defaults/rc.conf .. seems > that all the other rc.conf binary choices are either set "YES" or > "NO" rather than something or empty, so perhaps more consistent > with that might be: > > [ -z "${rc_quiet}" -o "${rc_starting}" = "YES" ] && echo "Starting > ${name}." No, we don't test boolean flags for specific values. I've attached a proper patch to this message. > As for name, not fussed, but maybe rc_startmsgs maybe more > indicative? That sounds fine to me, if anyone else has a better idea follow up on the freebsd-rc list. > Also, I agree with Oliver; YES should probably be the default > (POLA) so folks annoyed by 'too much information' in console.log > can disable it. This late in the release cycle I'm comfortable adding a knob, but I'm not comfortable toggling the default. If there is consensus on the freebsd-rc list to make the default "yes" then I will go with that consensus. Dimitry Andric wrote: > On 2009-09-17 09:17, Oliver Fromme wrote: >> My feeling is that hiding all of the "starting" messages is a >> regression that needs to be fixed. I cannot think of a good >> reason why they should be hidden, but there are certainly good >> reasons to display them. > > The commit that introduced this, r179946, has this log message: > > ==== Move the diagnostic output when the rc.subr(8) glue > automatically starts a > service behind $rc_quiet. Instead, output a warning if the > pre-command routine or the command itself failed. Arguably, it's > more useful to know when > a command failed to start than it is to have an endless list of > "Starting ...." lines[1]. > > [1] - This change actually helped me to discover a bug in > rc.d/{lockd,statd} > (fixed in r179941) that used to fail silently before. ==== > > It is really a matter of taste, I guess. There is something to be > said for eliminating verbosity, in order to make real errors more > visible. > > On the other hand, if one of your startup rc scripts hangs (for > whatever reason), it is nice to have an approximate idea which one > it is. :) I tend to agree that the current default is a little too un-verbose. If there is support for adding this knob I'll follow up with re@ to see about getting it into 8.0-release, although I'm not sure how they'll feel about a change like this so late in the cycle. Doug -- This .signature sanitized for your protection --------------090801000801060201050607 Content-Type: text/plain; name="rc-startmsgs.diff" Content-Transfer-Encoding: 7bit Content-Disposition: inline; filename="rc-startmsgs.diff" Index: share/man/man5/rc.conf.5 =================================================================== --- share/man/man5/rc.conf.5 (revision 197259) +++ share/man/man5/rc.conf.5 (working copy) @@ -24,7 +24,7 @@ .\" .\" $FreeBSD$ .\" -.Dd September 2, 2009 +.Dd September 17, 2009 .Dt RC.CONF 5 .Os .Sh NAME @@ -95,6 +95,13 @@ Informational messages are displayed when a condition that is not serious enough to warrant a warning or an error occurs. +.It Va rc_startmsgs +.Pq Vt bool +If set to +.Dq Li YES , +show +.Dq Starting foo: +when faststart is used (e.g., at boot time). .It Va early_late_divider .Pq Vt str The name of the script that should be used as the Index: etc/defaults/rc.conf =================================================================== --- etc/defaults/rc.conf (revision 197259) +++ etc/defaults/rc.conf (working copy) @@ -23,6 +23,7 @@ rc_debug="NO" # Set to YES to enable debugging output from rc.d rc_info="NO" # Enables display of informational messages at boot. +rc_startmsgs="NO" # Show "Starting foo:" messages at boot rcshutdown_timeout="30" # Seconds to wait before terminating rc.shutdown early_late_divider="FILESYSTEMS" # Script that separates early/late # stages of the boot process. Make sure you know Index: etc/rc.subr =================================================================== --- etc/rc.subr (revision 197259) +++ etc/rc.subr (working copy) @@ -708,7 +708,13 @@ # setup the full command to run # - [ -z "${rc_quiet}" ] && echo "Starting ${name}." + _show_startmsgs=1 + if [ -n "${rc_quiet}" ]; then + if ! checkyesno rc_startmsgs; then + unset _show_startmsgs + fi + fi + [ -n "$_show_startmsgs" ] && echo "Starting ${name}." if [ -n "$_chroot" ]; then _doit="\ ${_nice:+nice -n $_nice }\ --------------090801000801060201050607--