diff options
author | Poul-Henning Kamp <phk@FreeBSD.org> | 2000-03-20 10:44:49 +0000 |
---|---|---|
committer | Poul-Henning Kamp <phk@FreeBSD.org> | 2000-03-20 10:44:49 +0000 |
commit | 21144e3bf1f416a758f6546bfabfa8e4e8cba507 (patch) | |
tree | c87da548b12fd12c48f1e5c89d087ddfa088ee78 /sys/dev/amr | |
parent | 44bdcfa638d9ca31180090798a88116220f7f625 (diff) | |
download | src-21144e3bf1f416a758f6546bfabfa8e4e8cba507.tar.gz src-21144e3bf1f416a758f6546bfabfa8e4e8cba507.zip |
Remove B_READ, B_WRITE and B_FREEBUF and replace them with a new
field in struct buf: b_iocmd. The b_iocmd is enforced to have
exactly one bit set.
B_WRITE was bogusly defined as zero giving rise to obvious coding
mistakes.
Also eliminate the redundant struct buf 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 on your disk
where it should have been reading.
This change is a step in the direction towards a stackable BIO capability.
A lot of this patch were machine generated (Thanks to style(9) compliance!)
Vinum users: Greg has not had time to test this yet, be careful.
Notes
Notes:
svn path=/head/; revision=58345
Diffstat (limited to 'sys/dev/amr')
-rw-r--r-- | sys/dev/amr/amr.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/sys/dev/amr/amr.c b/sys/dev/amr/amr.c index 14dfec49e008..6d7524dc3679 100644 --- a/sys/dev/amr/amr.c +++ b/sys/dev/amr/amr.c @@ -853,7 +853,7 @@ amr_startio(struct amr_softc *sc) ac->ac_private = bp; ac->ac_data = bp->b_data; ac->ac_length = bp->b_bcount; - if (bp->b_flags & B_READ) { + if (bp->b_iocmd == BIO_READ) { ac->ac_flags |= AMR_CMD_DATAIN; cmd = AMR_CMD_LREAD; } else { |