From owner-svn-src-all@FreeBSD.ORG Sat Aug 7 20:13:00 2010 Return-Path: Delivered-To: svn-src-all@FreeBSD.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id B9BB8106564A; Sat, 7 Aug 2010 20:13:00 +0000 (UTC) (envelope-from jamie@FreeBSD.org) Received: from gritton.org (gritton.org [208.92.232.93]) by mx1.freebsd.org (Postfix) with ESMTP id 7DA538FC14; Sat, 7 Aug 2010 20:13:00 +0000 (UTC) Received: from glorfindel.gritton.org (c-67-177-8-107.hsd1.ut.comcast.net [67.177.8.107]) (authenticated bits=0) by gritton.org (8.14.3/8.14.3) with ESMTP id o77JwFRM016147; Sat, 7 Aug 2010 13:58:16 -0600 (MDT) (envelope-from jamie@FreeBSD.org) Message-ID: <4C5DBAD5.5080407@FreeBSD.org> Date: Sat, 07 Aug 2010 13:58:13 -0600 From: Jamie Gritton User-Agent: Mozilla/5.0 (X11; U; FreeBSD amd64; en-US; rv:1.9.1.10) Gecko/20100628 Thunderbird/3.0.5 MIME-Version: 1.0 To: src-committers@FreeBSD.org References: <201008062204.o76M4IvZ044635@svn.freebsd.org> <20100807170607.S48418@maildrop.int.zabbadoz.net> In-Reply-To: <20100807170607.S48418@maildrop.int.zabbadoz.net> Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Cc: svn-src-head@FreeBSD.org, svn-src-all@FreeBSD.org, "Bjoern A. Zeeb" Subject: Re: svn commit: r210974 - head/sys/kern X-BeenThere: svn-src-all@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: "SVN commit messages for the entire src tree \(except for " user" and " projects" \)" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sat, 07 Aug 2010 20:13:00 -0000 On 08/07/10 11:08, Bjoern A. Zeeb wrote: > On Fri, 6 Aug 2010, Jamie Gritton wrote: > >> Author: jamie >> Date: Fri Aug 6 22:04:18 2010 >> New Revision: 210974 >> URL: http://svn.freebsd.org/changeset/base/210974 >> >> Log: >> Implicitly make a new jail persistent if it's set not to attach. > > I am not sure this is a good idea, especially to implement it in the > kernel. This just means that if you accidentally mess up your command > line you are creating jails, possibly eating further resources most > likely without noticing. Lot's of foot shooting potential. > > What is the reason you need that? This error has been bugging me for a while, and I've been thinking it's more convenient not to need to explicitly specify "persist" for a jail that doesn't run any commands. So it's a matter of safety versus convenience. I suppose I should leave well enough alone and err on the side of safety. - Jamie