Bug #3179
closed
KKASSERT(bp->b_flags & B_VMIO);
0%
Description
This call
https://github.com/DragonFlyBSD/DragonFlyBSD/blob/master/sys/kern/vfs_bio.c#L1526
looks to be incorrect given
https://github.com/DragonFlyBSD/DragonFlyBSD/blob/master/sys/kern/vfs_bio.c#L3050
Not really sure how this "else" case happen though, as filesystem buffers are all B_VMIO (if not IO_VMIO).
Updated by dillon over 5 years ago
The buffer cache used to have B_MALLOC buffers, but I recently removed those so it should now generally only be able to have B_VMIO buffers. There are specialized buffers that are not B_VMIO, but they are not attached to vnodes and do not use the brelse() mechanism.
I'm not sure if I want to remove that else clause yet (line 1520-1531), but it does warrant clarification in the code comment so I will adjust the comment for now.
-Matt
Updated by