From owner-freebsd-geom@FreeBSD.ORG Fri Apr 29 04:11:34 2011 Return-Path: Delivered-To: freebsd-geom@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 92704106564A; Fri, 29 Apr 2011 04:11:34 +0000 (UTC) (envelope-from jhellenthal@gmail.com) Received: from mail-iw0-f182.google.com (mail-iw0-f182.google.com [209.85.214.182]) by mx1.freebsd.org (Postfix) with ESMTP id 2BBBB8FC15; Fri, 29 Apr 2011 04:11:33 +0000 (UTC) Received: by iwn33 with SMTP id 33so3874748iwn.13 for ; Thu, 28 Apr 2011 21:11:33 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma; h=domainkey-signature:sender:date:from:to:cc:subject:message-id :references:mime-version:content-type:content-disposition :in-reply-to:x-openpgp-key-id:x-openpgp-key-fingerprint :x-openpgp-key-url; bh=nz8SxfrqzwOpyXEV/im6uDjcSKSOY8XXAflT3DmJ0/A=; b=n+50lxoI0vT8lLvJbT9kz3wqFlM2O1ZMArNmPj3t3cOZwqPphfug/NcjWgGxXBjAEn l7kVSjoDi8MGftwIJzOhPvPu2FvxCPfzPjMdf/GPEqsp4S1gYm0tgpzVbivWSsNc+2bp SclVlXLRoG7RlF7iy7BCt2iM1y0dhttzQJsj8= DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=sender:date:from:to:cc:subject:message-id:references:mime-version :content-type:content-disposition:in-reply-to:x-openpgp-key-id :x-openpgp-key-fingerprint:x-openpgp-key-url; b=Qc6WtsVT0IkykcY0t6HTcf9jIW2rR95dvo/NKgqC/mzsnBriPEI7MEI4i23dKid4Eh Ty/++zdU29kgFOjM7lyT74gzemm4adEQKu/rhFx/kW9YQjRRYeB2wapTQ+DQPDOfza+h H8pxfxr/bxTgZTmWKIzDvpzfHR714msCwngUU= Received: by 10.42.39.18 with SMTP id f18mr5551098ice.281.1304048904674; Thu, 28 Apr 2011 20:48:24 -0700 (PDT) Received: from DataIX.net (adsl-99-190-84-116.dsl.klmzmi.sbcglobal.net [99.190.84.116]) by mx.google.com with ESMTPS id d9sm966932ibb.53.2011.04.28.20.48.22 (version=TLSv1/SSLv3 cipher=OTHER); Thu, 28 Apr 2011 20:48:22 -0700 (PDT) Sender: "J. Hellenthal" Received: from DataIX.net (localhost [127.0.0.1]) by DataIX.net (8.14.4/8.14.4) with ESMTP id p3T3mJNi040467 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=NO); Thu, 28 Apr 2011 23:48:20 -0400 (EDT) (envelope-from jhell@DataIX.net) Received: (from jhell@localhost) by DataIX.net (8.14.4/8.14.4/Submit) id p3T3mJMD040466; Thu, 28 Apr 2011 23:48:19 -0400 (EDT) (envelope-from jhell@DataIX.net) Date: Thu, 28 Apr 2011 23:48:19 -0400 From: "Jason J. Hellenthal" To: "Andrey V. Elsukov" Message-ID: <20110429034819.GA38949@DataIX.net> References: <4DB54F40.8050608@FreeBSD.org> <4DB7C7B7.9020201@FreeBSD.org> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="ZPt4rx8FFjLCG7dd" Content-Disposition: inline In-Reply-To: <4DB7C7B7.9020201@FreeBSD.org> X-OpenPGP-Key-Id: 0x89D8547E X-OpenPGP-Key-Fingerprint: 85EF E26B 07BB 3777 76BE B12A 9057 8789 89D8 547E X-OpenPGP-Key-URL: http://bit.ly/0x89D8547E Cc: FreeBSD Current , Pawel Jakub Dawidek , Marcel Moolenaar , freebsd-geom@freebsd.org Subject: Re: A replacement for GEOM_LABEL's gpt/gptid X-BeenThere: freebsd-geom@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: GEOM-specific discussions and implementations List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 29 Apr 2011 04:11:34 -0000 --ZPt4rx8FFjLCG7dd Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable Andrey, Great work, thank you. I have a little destraught about this over the years since its inception with having just a label class for itself. Now that this has all be properly tested it seems like a good idea to me to split them up into their respective sections as you have done. Now with the following I have some insight and question of if these sysctls are really needed. >> The patch is here: >> http://people.freebsd.org/~ae/gpart_labels.diff > >I updated the patch, it is in the same location. >I turned off glabel's gpt/gpid support and added loader tunables: > >kern.geom.part_label.apm.enable >kern.geom.part_label.gpt.enable >kern.geom.part_label.gptid.enable >kern.geom.part_label.pc98.enable > As I see from the patch listed above, you have properly turned off the support of these in the label class to move them to the consumers that they are actually used in. With that said, I do not understand and cant seem to wrap my mind around why there needs to be a compatibility layer to provide both a sysctl for what you have done and the old sysctl. Why if this is only going to be used by the class where it is only enabled do we need two sysctl ? Can we not just keep them under the same OID they are under now ? It is just a label after all and I dont see a need for renaming the sysctl. >Also for compatibility glabel's tunables still here: > >kern.geom.label.gpt.enable >kern.geom.label.gptid.enable > >So, if you have them in your loader.conf and want to have gpt/gptid labels, >you should remove them from loader.conf. >Also now they are only loader tunables and they can not be changed in runt= ime. > >If there will no objections i am planning to commit patch in this weekend. > > Anyway, thank you for you time. --=20 Regards, (jhell) Jason Hellenthal --ZPt4rx8FFjLCG7dd Content-Type: application/pgp-signature -----BEGIN PGP SIGNATURE----- Version: GnuPG v2.0.17 (FreeBSD) Comment: http://bit.ly/0x89D8547E iQEcBAEBAgAGBQJNujUCAAoJEJBXh4mJ2FR+wPIIAJprZxl1Kwv0IJLyLPEnKqIj IaSj5887jCWU6sJSHiBEvuTqSI/BfnZg9fzF27kHn8+4BofkXvuappVNFSBD4bPD lzgdQVnbO/xsDmE5NNs2Zvms38gr2K3YQGVABxKPJnA7LXhVW0qkFtUXxONOPE/i mCMUNmWS5JJXZmgvWPpVWotUabBJ1gL0bYCoSPADIki860CbgCbFtEv6ZwTSUldk 6maevcM18FP4S7E3FIOOlUrtbPlHuQVPbOAASGA+6dDXKOo90TbNrprtI2glGKFe fJbEW9jJ0eNx45ZirMHCKCoHNBaHFBRcJmAHe0Z57VSY3Z7GJ8sEHaR16tV9Id0= =X5Ti -----END PGP SIGNATURE----- --ZPt4rx8FFjLCG7dd--