Skip site navigation (1)Skip section navigation (2)
Date:      Fri, 17 Mar 2000 20:03:43 +0100
From:      Poul-Henning Kamp <phk@freebsd.org>
To:        current@freebsd.org
Subject:   patches for test / review
Message-ID:  <5701.953319823@critter.freebsd.dk>

next in thread | raw e-mail | index | archive | help

I have two patches up for test at http://phk.freebsd.dk/misc

I'm looking for reviews and tests, in particular vinum testing
would be nice since Grog is quasi-offline at the moment.

Poul-Henning

20000317 BWRITE-STRATEGY.patch

        This patch is machine generated except for the ccd.c and buf.h
        parts.

        Rename existing BUF_STRATEGY to DEV_STRATEGY
        substitute BUF_WRITE(foo) for VOP_BWRITE(foo->b_vp, foo);
        substitute BUF_STRATEGY(foo) for VOP_STRATEGY(foo->b_vp, foo);

        Please test & review.


20000317 b_iocmd.patch

        This patch removes B_READ, B_WRITE and B_FREEBUF and replaces
        them with a new field in struct buf: b_iocmd.

        B_WRITE was bogusly defined as zero giving rise to obvious
        coding mistakes and a lot of code implicitly knew this.

        This patch also eliminates the redundant flag B_CALL, it can
        just as efficiently be done by comparing b_iodone to NULL.

        Should you get a panic or drop into the debugger, complaining about
        "b_iocmd", don't continue, it is likely to write where it should
        have read.

        Please test & review.


--
Poul-Henning Kamp             FreeBSD coreteam member
phk@FreeBSD.ORG               "Real hackers run -current on their laptop."
FreeBSD -- It will take a long time before progress goes too far!


To Unsubscribe: send mail to majordomo@FreeBSD.org
with "unsubscribe freebsd-current" in the body of the message




Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?5701.953319823>