Re: A few potentially useful patches (from my FreeBSD source)

[Matthew Dillon <dillon@xxxxxxxxxxxxxxxxxxxx>]

:--- /NetBSD/obj/hacks/DragonFly-src/source-hacks/sys/vfs/ufs/ffs_alloc.c-DIST	Sun Jul 18 21:43:48 2004
:+++ /NetBSD/obj/hacks/DragonFly-src/source-hacks/sys/vfs/ufs/ffs_alloc.c	Thu Aug  5 10:20:49 2004
:@@ -641,8 +641,12 @@
: ffs_dirpref(struct inode *pip)
: {
: 	struct fs *fs;
:-	int cg, prefcg, dirsize, cgsize;
:-	int avgifree, avgbfree, avgndir, curdirsize;
:...

    I committed a somewhat modified version of this patch.  Also, I checked
    the NetBSD code and while they tried to make a similar change they fail
    to properly cast one of the arguments in the multiplication so NetBSD
    doesn't actually fix the bug.

:The following disables a check which more than a few FreeBSD users
:have discovered breaks their soundcards (there's a PR about this
:too).  Well, not breaks, but keeps from working.  Explained in the
:comment.  NetBSD does no such check and worked out-of-box with my
:card.

    Committed, except note that your #ifdef cannot have a '-' in it... it's
    actually two symbols instead of one.  I replaced the '-' with an underscore
    '_'.

:This patch to userland ppp significantly decreases the amount
:of CPU usage.  I believe something like this is present in FreeBSD
:with a MFC, and it made a huge difference for me.

    Committed.

:Another performance improver with my FreeBSD was to merge in
:the following based on -current, which in part, avoids fsync()ing
:the syslog file at every line during boot, which otherwise slows
:my boot by several seconds and makes my disk sound like a small
:jackhammer.

    Committed.  I just synchronized the whole program with FreeBSD-current.

:And finally, this patch to `mount' helps those of us who use
:tab-completion shells that append trailing slashes to directory
:names, so that one need not erase the trailing slash before
:entering, say, `mount -u /tmp/'
 
    Committed (the FreeBSD version).

						-Matt