Date: Sat, 03 Feb 2001 16:00:11 -0800 From: Dima Dorfman <dima@unixfreak.org> To: Matt Dillon <dillon@earth.backplane.com> Cc: mi@aldan.algebra.com, deischen@FreeBSD.ORG, hackers@FreeBSD.ORG Subject: Re: mount_md (was: mdconfig config file (was: cvs commit: src/sys/i386/conf GENERIC)) Message-ID: <20010204000016.95DEE3E02@bazooka.unixfreak.org> In-Reply-To: Message from Matt Dillon <dillon@earth.backplane.com> of "Sat, 03 Feb 2001 11:36:04 PST." <200102031936.f13Ja4G11258@earth.backplane.com>
next in thread | previous in thread | raw e-mail | index | archive | help
> > :Some thoughts: > : > : - If you want softupdates, you need to specify the -S flag. Should > : softupdates be the default? > > That's a hard one. Considering that people who have mfs in their fstab > probably expect as little disk I/O as possible, softupdate sshould > probably be enabled by default. Okay. Now the -S flag is used to disable softupdates. You can find the updated code at the same place (for your convenience, it is: http://www.unixfreak.org/~dima/home/mount_md.c). Also, the only reason mount_md runs tunefs is to enable softupdates. Perhaps it would be benificial to enable it during the newfs run? This would save one fork/exec sequence; I don't know whether it's worth it. I wrote a small patch to newfs to add a -g flag which will enable softupdates; you can find the diff at http://www.unixfreak.org/~dima/home/newfs-softdep.diff. It seems like it might be a good convenience feature (for manual newfs use and otherwise). > > : - To suppress the output of disklabel/newfs/etc., it closes fds 0-2. Is > : this evil? Should I devise another method of doing this (something > : like opening /dev/null and dup2'ing that fd to 0-2)? > > Yes, that is evil. opening and dup2()ing /dev/null to 0, 1, and 2 > is the prefered method. Fixed. > > : - Should there be an option to make use of md's autounit feature? I > : don't think it will be very hard to implement, but I don't know how > : useful it would be. > > Yes, I think this should be the default. Also fixed. It now accepts '/dev/md' or 'md' as the md-device parameter (the first after the arguments) to mean 'use autounit'. Regards Dima Dorfman dima@unixfreak.org To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe freebsd-hackers" in the body of the message
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20010204000016.95DEE3E02>