From owner-cvs-src@FreeBSD.ORG  Sat Jul 26 01:54:56 2003
Return-Path: <owner-cvs-src@FreeBSD.ORG>
Delivered-To: cvs-src@freebsd.org
Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125])
	by hub.freebsd.org (Postfix) with ESMTP
	id 8C3C137B401; Sat, 26 Jul 2003 01:54:56 -0700 (PDT)
Received: from mailman.zeta.org.au (mailman.zeta.org.au [203.26.10.16])
	by mx1.FreeBSD.org (Postfix) with ESMTP
	id 1BD5243FB1; Sat, 26 Jul 2003 01:54:54 -0700 (PDT)
	(envelope-from bde@zeta.org.au)
Received: from gamplex.bde.org (katana.zip.com.au [61.8.7.246])
	by mailman.zeta.org.au (8.9.3p2/8.8.7) with ESMTP id SAA16012;
	Sat, 26 Jul 2003 18:54:50 +1000
Date: Sat, 26 Jul 2003 18:54:48 +1000 (EST)
From: Bruce Evans <bde@zeta.org.au>
X-X-Sender: bde@gamplex.bde.org
To: Ruslan Ermilov <ru@freebsd.org>
In-Reply-To: <20030725082357.GA53826@sunbay.com>
Message-ID: <20030726184513.D5126@gamplex.bde.org>
References: <200307250010.h6P0AYKW061085@repoman.freebsd.org>
 <20030725053806.GA36982@sunbay.com> <3F20DF2D.8030209@freebsd.org>
 <20030725082357.GA53826@sunbay.com>
MIME-Version: 1.0
Content-Type: TEXT/PLAIN; charset=US-ASCII
cc: cvs-src@freebsd.org
cc: src-committers@freebsd.org
cc: Scott Long <scottl@freebsd.org>
cc: Bruce Evans <bde@freebsd.org>
cc: cvs-all@freebsd.org
Subject: Re: cvs commit: src/release/i386 drivers.conf
X-BeenThere: cvs-src@freebsd.org
X-Mailman-Version: 2.1.1
Precedence: list
List-Id: CVS commit messages for the src tree <cvs-src.freebsd.org>
List-Unsubscribe: <http://lists.freebsd.org/mailman/listinfo/cvs-src>,
	<mailto:cvs-src-request@freebsd.org?subject=unsubscribe>
List-Archive: <http://lists.freebsd.org/pipermail/cvs-src>
List-Post: <mailto:cvs-src@freebsd.org>
List-Help: <mailto:cvs-src-request@freebsd.org?subject=help>
List-Subscribe: <http://lists.freebsd.org/mailman/listinfo/cvs-src>,
	<mailto:cvs-src-request@freebsd.org?subject=subscribe>
X-List-Received-Date: Sat, 26 Jul 2003 08:54:57 -0000

On Fri, 25 Jul 2003, Ruslan Ermilov wrote:

> On Fri, Jul 25, 2003 at 01:41:33AM -0600, Scott Long wrote:
> > Ruslan Ermilov wrote:
> > >Which reminds me.  The floppy wasn't overflowing when building
> > >5.x snapshot on 4.x, due to the differences in file system
> > >layout between 4.x and 5.x.  Please see the attached if you're
> > >interested.
> > ...
> > Now that -O2 is the default for newfs in 5-current, does the 'Oflag ==
> > 2' trick still work?
> >
> Yes it does:
>
> 1.  doFS.sh creates file systems with -O1
> 2.  The patch affects the -O1 case:
>
> > Even if it does, it feels like a hack to me.  I'd
> > rather that newfs have an option to directly override the number of
> > c/g's, rather than rely on indirect calculations and magic constants.
> >
> *Shrug*, an option to override a minimum value?
>
> > Please don't commit this until we have discussed it more.

I think the 'Oflag == 2' trick is a hack to avoid touching the -O2 case,
since the -O2 implementors haven't responded and others don't have enough
experience with -O2 to know what is right.  OTOH, with -O1 it is well
known that you can't have too few cylinder groups except possibly for
unuseful disk sizes like 40MB, so removing the limit just fixes a bug.
Anyway, there was no limit in until recently, so we have lots of practical
experience with using 1 cylinder group.

Bruce