From owner-freebsd-current@FreeBSD.ORG Sun Aug 8 13:05:51 2010 Return-Path: Delivered-To: freebsd-current@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 1246C1065672 for ; Sun, 8 Aug 2010 13:05:51 +0000 (UTC) (envelope-from pjd@garage.freebsd.pl) Received: from mail.garage.freebsd.pl (chello089077043238.chello.pl [89.77.43.238]) by mx1.freebsd.org (Postfix) with ESMTP id 0D2268FC0A for ; Sun, 8 Aug 2010 13:05:39 +0000 (UTC) Received: by mail.garage.freebsd.pl (Postfix, from userid 65534) id 5873B45D8D; Sun, 8 Aug 2010 14:37:07 +0200 (CEST) Received: from localhost (chello089077043238.chello.pl [89.77.43.238]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (No client certificate requested) by mail.garage.freebsd.pl (Postfix) with ESMTP id D4C4A45C89; Sun, 8 Aug 2010 14:37:01 +0200 (CEST) Date: Sun, 8 Aug 2010 14:36:53 +0200 From: Pawel Jakub Dawidek To: Ivan Voras Message-ID: <20100808123653.GC2037@garage.freebsd.pl> References: <20100808103055.GA2037@garage.freebsd.pl> Mime-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="xo44VMWPx7vlQ2+2" Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.4.2.3i X-PGP-Key-URL: http://people.freebsd.org/~pjd/pjd.asc X-OS: FreeBSD 9.0-CURRENT amd64 X-Spam-Checker-Version: SpamAssassin 3.0.4 (2005-06-05) on mail.garage.freebsd.pl X-Spam-Level: X-Spam-Status: No, score=-0.6 required=4.5 tests=BAYES_00,RCVD_IN_SORBS_DUL autolearn=no version=3.0.4 Cc: freebsd-hackers@freebsd.org, freebsd-current@freebsd.org, freebsd-geom@freebsd.org Subject: Re: glabel "force sectorsize" patch X-BeenThere: freebsd-current@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: Discussions about the use of FreeBSD-current List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sun, 08 Aug 2010 13:05:51 -0000 --xo44VMWPx7vlQ2+2 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Sun, Aug 08, 2010 at 02:02:17PM +0200, Ivan Voras wrote: > On 8.8.2010 12:30, Pawel Jakub Dawidek wrote: > > So why do you want to obfuscate glabel with it? For people to start > > depend on it? Once we start supporting 4kB sectors what do we do with > > such a change? Remove it and decrease version number? What people will > > do with providers already labeled this way? > >=20 > > If its temporary, just allow to list providers you want to increase > > sector size in /boot/loader.conf. Once we start supporting it properly > > people might simply remove it from loader.conf and it should just work. > >=20 > > Glabel is not for that and I don't agree for such obfuscation. >=20 > Of course, there are good and bad sides to it. My take on it is that the > only bad side is that it really isn't glabel's primary function to > (optionally) fixup geometry, while the good sides are: It isn't its secondary function either. > * glabel is in GENERIC and judging by the mailing lists' traffic it is > one of the better used parts of the system so people are familiar with > it. It is also already used as a perfectly valid fixup for device > renaming, making both UFS and ZFS more stable for usage. That's an excellent argument. But you know what? The em(4) is also in GENERIC, why not to add it in there? > * You can't really "make people depend on glabel" both because it is in > GENERIC and because of it storing metadata in the last sector, making > the rest of the drive completely usable without it in the event native > 4k sector support is grown. I never said that. I do want people to depend on glabel, because it is free of such ugly hacks, so I know it won't bite them in the future. I don't want people to start depend on the fact that glabel supports changing sector sizes. Once we start supporting 4kB sectors properly people configuration will stop working, because glabel won't be able to read its metadata anymore. Your hack will break all configurations that started to depend on your hack. In what I proposed, GEOM provider will be presented to glabel (or any other GEOM class) as 4kB provider and everything will just work, also after adding proper support for 4kB sectors. > I'd like to hear comments from the wider audience. In respect with your > comment, I will compromise: as 4k sector drives have become available > over the counter more than 6 months ago and so far I think this is the > first effort to give some support for them, I will commit this patch > before 9.0 code freeze only if no other support gets developed. I'll repeat. You won't commit this patch, because it is totally wrong solution and can only do a lot of damage in the future. If you look forward, even temporary solutions can be done right. --=20 Pawel Jakub Dawidek http://www.wheelsystems.com pjd@FreeBSD.org http://www.FreeBSD.org FreeBSD committer Am I Evil? Yes, I Am! --xo44VMWPx7vlQ2+2 Content-Type: application/pgp-signature Content-Disposition: inline -----BEGIN PGP SIGNATURE----- Version: GnuPG v2.0.14 (FreeBSD) iEYEARECAAYFAkxepOUACgkQForvXbEpPzTsugCgzpbR4hcr74ZwU9NFoiUibiyx eF0An1jqSoN7hiDyIAL0r80ZtVVei9GG =KZuE -----END PGP SIGNATURE----- --xo44VMWPx7vlQ2+2--