From owner-svn-src-all@freebsd.org  Wed Feb 10 20:08:26 2016
Return-Path: <owner-svn-src-all@freebsd.org>
Delivered-To: svn-src-all@mailman.ysv.freebsd.org
Received: from mx1.freebsd.org (mx1.freebsd.org
 [IPv6:2001:1900:2254:206a::19:1])
 by mailman.ysv.freebsd.org (Postfix) with ESMTP id 133A7AA3626;
 Wed, 10 Feb 2016 20:08:26 +0000 (UTC) (envelope-from devin@shxd.cx)
Received: from shxd.cx (mail.shxd.cx [64.201.244.140])
 (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits))
 (Client did not present a certificate)
 by mx1.freebsd.org (Postfix) with ESMTPS id 027CFAEB;
 Wed, 10 Feb 2016 20:08:26 +0000 (UTC) (envelope-from devin@shxd.cx)
Received: from 50-196-156-133-static.hfc.comcastbusiness.net
 ([50.196.156.133]:55485 helo=tinkerbell.pixel8networks.com)
 by shxd.cx with esmtpsa (TLSv1:AES256-SHA:256) (Exim 4.77 (FreeBSD))
 (envelope-from <devin@shxd.cx>)
 id 1aTVso-0005pa-P7; Wed, 10 Feb 2016 06:36:43 -0800
Mime-Version: 1.0 (Mac OS X Mail 8.2 \(2104\))
Subject: Re: svn commit: r295471 - in head/etc: defaults rc.d
From: Devin Teske <dteske@freebsd.org>
In-Reply-To: <0CAF4F00-7711-4F0F-8487-0B17381940F2@FreeBSD.org>
Date: Wed, 10 Feb 2016 12:08:25 -0800
Cc: Ronald Klop <ronald-lists@klop.ws>, src-committers@freebsd.org,
 svn-src-all@freebsd.org, svn-src-head@freebsd.org,
 Devin Teske <dteske@FreeBSD.org>, Matteo Riondato <matteo@freebsd.org>
Message-Id: <AC459150-C3C2-4F7C-B460-0CF830DC7F4B@freebsd.org>
References: <201602101613.u1AGDx0I003885@repo.freebsd.org>
 <op.ycmm2qq8kndu52@53555a16.cm-6-6b.dynamic.ziggo.nl>
 <0CAF4F00-7711-4F0F-8487-0B17381940F2@FreeBSD.org>
To: Mark Felder <feld@freebsd.org>
X-Mailer: Apple Mail (2.2104)
Sender: devin@shxd.cx
Content-Type: text/plain;
	charset=utf-8
Content-Transfer-Encoding: quoted-printable
X-Content-Filtered-By: Mailman/MimeDel 2.1.20
X-BeenThere: svn-src-all@freebsd.org
X-Mailman-Version: 2.1.20
Precedence: list
List-Id: "SVN commit messages for the entire src tree \(except for &quot;
 user&quot; and &quot; projects&quot; \)" <svn-src-all.freebsd.org>
List-Unsubscribe: <https://lists.freebsd.org/mailman/options/svn-src-all>,
 <mailto:svn-src-all-request@freebsd.org?subject=unsubscribe>
List-Archive: <http://lists.freebsd.org/pipermail/svn-src-all/>
List-Post: <mailto:svn-src-all@freebsd.org>
List-Help: <mailto:svn-src-all-request@freebsd.org?subject=help>
List-Subscribe: <https://lists.freebsd.org/mailman/listinfo/svn-src-all>,
 <mailto:svn-src-all-request@freebsd.org?subject=subscribe>
X-List-Received-Date: Wed, 10 Feb 2016 20:08:26 -0000


> On Feb 10, 2016, at 9:28 AM, Matteo Riondato <matteo@freebsd.org> =
wrote:
>=20
>>=20
>> On Feb 10, 2016, at 12:04 PM, Ronald Klop <ronald-lists@klop.ws> =
wrote:
>>=20
>> On Wed, 10 Feb 2016 17:13:59 +0100, Mark Felder <feld@freebsd.org> =
wrote:
>>=20
>>> Author: feld (ports committer)
>>> Date: Wed Feb 10 16:13:59 2016
>>> New Revision: 295471
>>> URL: https://svnweb.freebsd.org/changeset/base/295471
>>>=20
>>> Log:
>>> Add new rc.conf parameter "jail_reverse_stop"
>>> When a user defines "jail_list" in rc.conf the jails are started in =
the
>>> order defined. Currently the jails are not are stopped in reverse =
order
>>> which may break dependencies between jails/services and prevent a =
clean
>>> shutdown. The new parameter "jail_reverse_stop" will shutdown jails =
in
>>> "jail_list" in reverse order when set to "YES".
>>> Please note that this does not affect manual invocation of the jail =
rc
>>> script. If a user runs the command
>>>  # service jail stop jail1 jail2 jail3
>>> the jails will be stopped in exactly the order specified regardless =
of
>>> jail_reverse_stop being defined in rc.conf.
>>> PR:		196152
>>=20
>> Good feature. I know POLA is always very important in FreeBSD =
development, but having this setting on by default seems very reasonable =
to me.
>=20
> +1, perhaps with a note in UPDATING and, when MFC=E2=80=99ed, having =
the default to off.
>=20

+1 Great work, Mark!
--=20
Devin=