From owner-svn-src-head@FreeBSD.ORG Fri Mar 5 21:18:03 2010 Return-Path: Delivered-To: svn-src-head@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 23E7D1065670 for ; Fri, 5 Mar 2010 21:18:03 +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 AF9C58FC22 for ; Fri, 5 Mar 2010 21:18:02 +0000 (UTC) Received: (qmail 10851 invoked by uid 399); 5 Mar 2010 21:18:01 -0000 Received: from localhost (HELO freefall.freebsd.org) (dougb@dougbarton.us@127.0.0.1) by localhost with ESMTPAM; 5 Mar 2010 21:18:01 -0000 X-Originating-IP: 127.0.0.1 X-Sender: dougb@dougbarton.us Date: Fri, 5 Mar 2010 21:18:00 +0000 (UTC) From: Doug Barton To: Alexander Leidinger In-Reply-To: <201003051434.o25EYXBR024375@svn.freebsd.org> Message-ID: References: <201003051434.o25EYXBR024375@svn.freebsd.org> User-Agent: Alpine 2.00 (BSF 1167 2008-08-23) X-message-flag: Outlook -- Not just for spreading viruses anymore! X-OpenPGP-Key-ID: 0xD5B2F0FB Organization: http://SupersetSolutions.com/ MIME-Version: 1.0 Content-Type: MULTIPART/MIXED; BOUNDARY="844893955-1427392154-1267823881=:98935" Cc: svn-src-head@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org Subject: Re: svn commit: r204759 - in head: etc/defaults etc/rc.d share/man/man5 X-BeenThere: svn-src-head@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: SVN commit messages for the src tree for head/-current List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 05 Mar 2010 21:18:03 -0000 This message is in MIME format. The first part should be readable text, while the remaining parts are likely unreadable without MIME-aware tools. --844893955-1427392154-1267823881=:98935 Content-Type: TEXT/PLAIN; charset=US-ASCII; format=flowed On Fri, 5 Mar 2010, Alexander Leidinger wrote: > Author: netchild > Date: Fri Mar 5 14:34:33 2010 > New Revision: 204759 > URL: http://svn.freebsd.org/changeset/base/204759 I've got no comments on the jail-related stuff given that my knowledge of jails is almost non-existent. However I wish you had run your diff past freebsd-rc@ since if you had I (or someone else) could have let you know that the attached patch is a much cleaner way of implementing the bit about conditionalizing "parallel" execution (which, to the extent I understand the problem I agree with your solution of only doing it at when starting, FWIW). In general we try to avoid having any code in rc.d scripts run unconditionally. In this case it's harmless (although every cpu cycle counts) but in other cases it can cause problems, which is why as a general rule it's safer to avoid it altogether. hth, Doug --844893955-1427392154-1267823881=:98935 Content-Type: TEXT/PLAIN; charset=US-ASCII; name=jail-rc.diff Content-Transfer-Encoding: BASE64 Content-ID: Content-Description: Content-Disposition: attachment; filename=jail-rc.diff SW5kZXg6IGphaWwNCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0NCi0tLSBqYWls CShyZXZpc2lvbiAyMDQ3NzApDQorKysgamFpbAkod29ya2luZyBjb3B5KQ0K QEAgLTE4LDYgKzE4LDggQEANCiANCiBuYW1lPSJqYWlsIg0KIHJjdmFyPWBz ZXRfcmN2YXJgDQorDQorc3RhcnRfcHJlY21kPSJqYWlsX3ByZXN0YXJ0Ig0K IHN0YXJ0X2NtZD0iamFpbF9zdGFydCINCiBzdG9wX2NtZD0iamFpbF9zdG9w Ig0KIA0KQEAgLTU0NSw2ICs1NDcsMTMgQEANCiAJZG9uZQ0KIH0NCiANCitq YWlsX3ByZXN0YXJ0KCkNCit7DQorCWlmIGNoZWNreWVzbm8gamFpbF9wYXJh bGxlbF9zdGFydDsgdGhlbg0KKwkJY29tbWFuZF9hcmdzPScmJw0KKwlmaQ0K K30NCisNCiBqYWlsX3N0YXJ0KCkNCiB7DQogCWVjaG8gLW4gJ0NvbmZpZ3Vy aW5nIGphaWxzOicNCkBAIC03MzAsMTggKzczOSw0IEBADQogCWphaWxfbGlz dD0iJCoiDQogZmkNCiANCi0jIE9ubHkgYWxsb3cgdGhlIHBhcmFsbGVsIHN0 YXJ0IG9mIGphaWxzLCBvdGhlciBjb21tYW5kcyBhcmUgbm90DQotIyBzYWZl IHRvIGV4ZWN1dGUgaW4gcGFyYWxsZWwuDQotY2FzZSAiJHtjbWR9IiBpbg0K LSpzdGFydCkNCi0JOzsNCi0qKQ0KLQlqYWlsX3BhcmFsbGVsX3N0YXJ0PU5P DQotZXNhYw0KLQ0KLWlmIGNoZWNreWVzbm8gamFpbF9wYXJhbGxlbF9zdGFy dDsgdGhlbg0KLQlydW5fcmNfY29tbWFuZCAiJHtjbWR9IiAmDQotZWxzZQ0K LQlydW5fcmNfY29tbWFuZCAiJHtjbWR9Ig0KLWZpDQotDQorcnVuX3JjX2Nv bW1hbmQgIiR7Y21kfSINCg== --844893955-1427392154-1267823881=:98935--