From owner-freebsd-rc@FreeBSD.ORG Wed Jun 20 06:10:11 2012 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 4B21B106564A for ; Wed, 20 Jun 2012 06:10:11 +0000 (UTC) (envelope-from gnats@FreeBSD.org) Received: from freefall.freebsd.org (freefall.freebsd.org [IPv6:2001:4f8:fff6::28]) by mx1.freebsd.org (Postfix) with ESMTP id 366AE8FC16 for ; Wed, 20 Jun 2012 06:10:11 +0000 (UTC) Received: from freefall.freebsd.org (localhost [127.0.0.1]) by freefall.freebsd.org (8.14.5/8.14.5) with ESMTP id q5K6ABrr018025 for ; Wed, 20 Jun 2012 06:10:11 GMT (envelope-from gnats@freefall.freebsd.org) Received: (from gnats@localhost) by freefall.freebsd.org (8.14.5/8.14.5/Submit) id q5K6ABG3018024; Wed, 20 Jun 2012 06:10:11 GMT (envelope-from gnats) Date: Wed, 20 Jun 2012 06:10:11 GMT Message-Id: <201206200610.q5K6ABG3018024@freefall.freebsd.org> To: freebsd-rc@FreeBSD.org From: Mark Linimon Cc: Subject: Re: conf/158127: [patch] remount_optional option in rc.initdiskless doesn't actually work X-BeenThere: freebsd-rc@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list Reply-To: Mark Linimon 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, 20 Jun 2012 06:10:11 -0000 The following reply was made to PR conf/158127; it has been noted by GNATS. From: Mark Linimon To: bug-followup@FreeBSD.org Cc: Subject: Re: conf/158127: [patch] remount_optional option in rc.initdiskless doesn't actually work Date: Wed, 20 Jun 2012 01:01:04 -0500 ----- Forwarded message from Dewayne ----- Date: Wed, 20 Jun 2012 06:19:55 +1000 From: Dewayne To: stable@freebsd.org Cc: brian@maven-group.org Subject: Diskless ignore remount - fix (Re: conf/158127: [patch] remount_optional option in rc.initdiskless doesn' t actually work) 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. ----- End forwarded message -----