From owner-freebsd-stable@FreeBSD.ORG Tue Jun 19 20:24:43 2012 Return-Path: Delivered-To: stable@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id C7EAA106564A for ; Tue, 19 Jun 2012 20:24:43 +0000 (UTC) (envelope-from dewayne.geraghty@heuristicsystems.com.au) Received: from nskntmtas01p.mx.bigpond.com (nskntmtas01p.mx.bigpond.com [61.9.168.137]) by mx1.freebsd.org (Postfix) with ESMTP id 51C788FC12 for ; Tue, 19 Jun 2012 20:24:42 +0000 (UTC) Received: from nskntcmgw09p ([61.9.169.169]) by nskntmtas01p.mx.bigpond.com with ESMTP id <20120619202435.PIR25972.nskntmtas01p.mx.bigpond.com@nskntcmgw09p>; Tue, 19 Jun 2012 20:24:35 +0000 Received: from hermes.heuristicsystems.com.au ([58.172.112.105]) by nskntcmgw09p with BigPond Outbound id QLQa1j00B2GVmci01LQaCY; Tue, 19 Jun 2012 20:24:35 +0000 X-Authority-Analysis: v=2.0 cv=Lam+G0ji c=1 sm=1 a=0GO/22z+lHYfckWJ4naYnw==:17 a=2xJL4Qrc9dMA:10 a=twTT4oUKOlYA:10 a=kj9zAlcOel0A:10 a=GHIR_BbyAAAA:8 a=4kJ3bFl4dHPlTN7XGVYA:9 a=CjuIK1q_8ugA:10 a=0GO/22z+lHYfckWJ4naYnw==:117 Received: from white (white.hs [10.0.5.2]) (authenticated bits=0) by hermes.heuristicsystems.com.au (8.14.5/8.13.6) with ESMTP id q5JKJtU4029211 (version=TLSv1/SSLv3 cipher=RC4-MD5 bits=128 verify=NOT); Wed, 20 Jun 2012 06:19:58 +1000 (EST) (envelope-from dewayne.geraghty@heuristicsystems.com.au) From: "Dewayne" To: Date: Wed, 20 Jun 2012 06:19:55 +1000 Organization: Heuristic Systems Pty Ltd Message-ID: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit X-Mailer: Microsoft Office Outlook 11 Thread-Index: Ac1OWOCsyiwTrODfT9udcWyWRuioGA== X-MimeOLE: Produced By Microsoft MimeOLE V6.00.2900.6157 Cc: brian@maven-group.org Subject: Diskless ignore remount - fix (Re: conf/158127: [patch] remount_optional option in rc.initdiskless doesn' t actually work) X-BeenThere: freebsd-stable@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: Production branch of FreeBSD source code List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 19 Jun 2012 20:24:43 -0000 I notice that PR 158127 remains outstanding, June, 2011. I've enclosed a patch for Stable 9.0, to correct diskless booting. When a mountpoint fails, the failure may be ignored as required/documented. --- /tmp/rc.initdiskless 2012-06-19 19:01:33.000000000 +1000 +++ /etc/rc.initdiskless 2012-06-19 19:02:16.000000000 +1000 @@ -166,7 +166,7 @@ chkerr() { lastitem () ( n=$(($# - 1)) ; shift $n ; echo $1 ) mountpoint="$(lastitem $2)" - [ -r $mountpoint/remount_optional ] && ( echo "$2 failed: ignoring due to remount_optional" ; return ) + [ -r $mountpoint/remount_optional ] && echo "$2 failed: ignoring due to remount_optional" && return case $1 in 0) The patch has been tested with/without /conf/default/etc/remount_optional & is functionally correct. It would be appreciated if someone would verify and commit. Regards, Dewayne. PS My apologies if Outlook has corrupted the patches appearance,we have to work with what we've got.