From owner-svn-src-all@FreeBSD.ORG Thu Dec 19 19:32:04 2013 Return-Path: Delivered-To: svn-src-all@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:1900:2254:206a::19:1]) (using TLSv1 with cipher ADH-AES256-SHA (256/256 bits)) (No client certificate requested) by hub.freebsd.org (Postfix) with ESMTPS id 0CC3BFD1; Thu, 19 Dec 2013 19:32:04 +0000 (UTC) Received: from mx1.fisglobal.com (mx1.fisglobal.com [199.200.24.190]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (No client certificate requested) by mx1.freebsd.org (Postfix) with ESMTPS id 133CA1167; Thu, 19 Dec 2013 19:32:02 +0000 (UTC) Received: from smtp.fisglobal.com ([10.132.206.16]) by ltcfislmsgpa03.fnfis.com (8.14.5/8.14.5) with ESMTP id rBJJW1kL009643 (version=TLSv1/SSLv3 cipher=AES128-SHA bits=128 verify=NOT); Thu, 19 Dec 2013 13:32:01 -0600 Received: from LTCFISWMSGMB21.FNFIS.com ([169.254.1.7]) by LTCFISWMSGHT05.FNFIS.com ([10.132.206.16]) with mapi id 14.03.0158.001; Thu, 19 Dec 2013 13:32:00 -0600 From: "Teske, Devin" To: Nathan Whitehorn Subject: Re: svn commit: r259479 - head/usr.sbin/bsdinstall/scripts Thread-Topic: svn commit: r259479 - head/usr.sbin/bsdinstall/scripts Thread-Index: AQHO+qdwM2JIT2GF3kyccjUlXq6l7JpcUT6A Date: Thu, 19 Dec 2013 19:31:59 +0000 Message-ID: References: <201312161947.rBGJl4Jh087762@svn.freebsd.org> <52AF6FF6.1030909@freebsd.org> <7BF56E5D-CDE7-4E57-BA47-3565CD07236C@fisglobal.com> <52AF75C1.9040502@freebsd.org> <52AF78B6.20706@freebsd.org> <34980C34-1D4F-458E-8931-8080E0395D8E@fisglobal.com> In-Reply-To: <34980C34-1D4F-458E-8931-8080E0395D8E@fisglobal.com> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-originating-ip: [10.132.253.120] Content-Type: text/plain; charset="iso-8859-1" Content-ID: <7F811FCEB0BA29448D3CC115A9058813@fisglobal.com> Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 X-Proofpoint-Virus-Version: vendor=fsecure engine=2.50.10432:5.11.87, 1.0.14, 0.0.0000 definitions=2013-12-19_07:2013-12-19,2013-12-19,1970-01-01 signatures=0 Cc: "svn-src-head@freebsd.org" , Devin Teske , "svn-src-all@freebsd.org" , "src-committers@freebsd.org" , "Teske, Devin" X-BeenThere: svn-src-all@freebsd.org X-Mailman-Version: 2.1.17 Precedence: list Reply-To: Devin Teske 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: Thu, 19 Dec 2013 19:32:04 -0000 On Dec 18, 2013, at 10:08 PM, Teske, Devin wrote: >=20 > On Dec 16, 2013, at 2:03 PM, Nathan Whitehorn wrote: >=20 >> On 12/16/13 15:55, Teske, Devin wrote: >>> On Dec 16, 2013, at 1:50 PM, Nathan Whitehorn wrote: >>>=20 >>>> On 12/16/13 15:48, Teske, Devin wrote: >>>>> On Dec 16, 2013, at 1:40 PM, Teske, Devin wrote: >>>>>=20 >>>>>> On Dec 16, 2013, at 1:26 PM, Nathan Whitehorn wrote: >>>>>>=20 >>>>>>> On 12/16/13 13:47, Devin Teske wrote: >>>>>>>> Author: dteske >>>>>>>> Date: Mon Dec 16 19:47:04 2013 >>>>>>>> New Revision: 259479 >>>>>>>> URL: http://svnweb.freebsd.org/changeset/base/259479 >>>>>>>>=20 >>>>>>>> Log: >>>>>>>> Add kern.geom.label.disk_ident.enable=3D"0" to loader.conf(5). >>>>>>>> Discussed on: -current, -stable >>>>>>>> MFC after: 3 days >>>>>>>>=20 >>>>>>>> Modified: >>>>>>>> head/usr.sbin/bsdinstall/scripts/zfsboot >>>>>>>>=20 >>>>>>>> Modified: head/usr.sbin/bsdinstall/scripts/zfsboot >>>>>>>> =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D >>>>>>>> --- head/usr.sbin/bsdinstall/scripts/zfsboot Mon Dec 16 19:44:45 2= 013 (r259478) >>>>>>>> +++ head/usr.sbin/bsdinstall/scripts/zfsboot Mon Dec 16 19:47:04 2= 013 (r259479) >>>>>>>> @@ -1159,6 +1159,9 @@ zfs_create_boot() >>>>>>>> $BSDINSTALL_TMPETC/rc.conf.zfs || return $FAILURE >>>>>>>> f_eval_catch $funcname echo "$ECHO_APPEND" 'zfs_load=3D\"YES\"' \ >>>>>>>> $BSDINSTALL_TMPBOOT/loader.conf.zfs || return $FAILU= RE >>>>>>>> + f_eval_catch $funcname echo "$ECHO_APPEND" \ >>>>>>>> + 'kern.geom.label.disk_ident.enable=3D\"0\"' \ >>>>>>>> + $BSDINSTALL_TMPBOOT/loader.conf.zfs || return $FAIL= URE >>>>>>>> # We're all done unless we should go on for boot pool >>>>>>>> [ "$ZFSBOOT_BOOT_POOL" ] || return $SUCCESS >>>>>>> Uh -- what is all of this? Why are we disabling kernel functions de= pending on what the root filesystem is? Please don't MFC this. >>>>>> https://urldefense.proofpoint.com/v1/url?u=3Dhttp://lists.freebsd.or= g/pipermail/freebsd-stable/2013-December/076365.html&k=3D%2FbkpAUdJWZuiTILC= q%2FFnQg%3D%3D%0A&r=3DLTzUWWrRnz2iN3PtHDubWRSAh9itVJ%2BMUcNBCQ4tyeo%3D%0A&m= =3DWCmXzB4036KuOzNScbJsBQLKdo%2BAo15QWLYq4A7DKis%3D%0A&s=3D4f16f0d6399e3a3c= 5e105a7869c580884327a8721c2f44c1711b319212a23db7 >>>>>> https://urldefense.proofpoint.com/v1/url?u=3Dhttp://lists.freebsd.or= g/pipermail/freebsd-stable/2013-December/076471.html&k=3D%2FbkpAUdJWZuiTILC= q%2FFnQg%3D%3D%0A&r=3DLTzUWWrRnz2iN3PtHDubWRSAh9itVJ%2BMUcNBCQ4tyeo%3D%0A&m= =3DWCmXzB4036KuOzNScbJsBQLKdo%2BAo15QWLYq4A7DKis%3D%0A&s=3D17882f97e3633c1e= 3ebd45f332e62d2212dc53d1f0577acc4ae15d8234d09c7f >>>>>>=20 >>>>>> NB: Happy to rip it out... but want something in-reply to those thre= ads (pretty please). >>>>> Basically... the logic is... >>>>>=20 >>>>> The ZFS pool is built on vdevs of a specific name. The names that are= used >>>>> should remain the same. Adding this to the loader.conf ensures that t= he names >>>>> that the pool(s) was/were built upon do not change. >>>>>=20 >>>>> This goes beyond just a swap partition I imagine. For example... copy= ing the >>>>> data to a new drive using a duplicator. I'm sure there are other case= s too. >>>> Thanks for the explanation! I wonder if we should just turn off the di= sk ident stuff by default globally -- it was causing problems for me as wel= l without ZFS root. >>> As I was making the commit to zfsboot... the very thought had occurred = to me. >>>=20 >>> I'm happy to rip this out in favor of a new global default. The end-res= ult is that >>> what Johan experienced won't be repeated. >>>=20 >>> I think there's an urgency to get something to solve this into 10. >>=20 >> Yeah, I can see that. Let me bring this up on -CURRENT, with a short tim= eout, and see what the options are. >=20 > Any updates on this? As requested, I have held-back SVN r259479 from the merge process. However, as I'm sure RC3 is approaching, wanted to know if we should continue to look into changing the default (otherwise, we have r259479 available). --=20 Devin _____________ The information contained in this message is proprietary and/or confidentia= l. If you are not the intended recipient, please: (i) delete the message an= d all copies; (ii) do not disclose, distribute or use the message in any ma= nner; and (iii) notify the sender immediately. In addition, please be aware= that any message addressed to our domain is subject to archiving and revie= w by persons other than the intended recipient. Thank you.