]> Zhao Yanbai Git Server - minix.git/commit
Fixed nasty bug that would cause FS to loop forever in rw_scattered (while
authorBen Gras <ben@minix3.org>
Sun, 19 Jun 2005 22:33:49 +0000 (22:33 +0000)
committerBen Gras <ben@minix3.org>
Sun, 19 Jun 2005 22:33:49 +0000 (22:33 +0000)
commitd844d2a547c09e5f38df0ac04b3940414c979cd7
treec4e61fff86e8348600fa670431c38c2d1e2affc9
parent34a9677a5bf570f7b65f463af64b6cfa4eeab492
Fixed nasty bug that would cause FS to loop forever in rw_scattered (while
syncing, for instance) if blocks are unwritable. This could happen if:

  . write goes beyond device boundary to a block device
  . write is done for a complete block or more; thus the
    block is not retrieved first (at which point it would be noticed
    it doesn't exist), but the buffer is simply allocated
  . at write time, the device i/o doesn't succeed, but rw_scattered
    doesn't understand this and loops forever trying to get the block
    written.

Currently, if no blocks can be written, the loop aborts, leaving all
buffers intact but potentially dirty. When invalidate() is called on the
device, the buffers will disappear (even if dirty). Same story for if
the buffer is removed due to rmed from lru chain. There's not much we
can do about this, however - we can't keep these blocks around, forever
occupying a buffer in the buffer cache.

The second part of the solution is not to let unwritable buffers be
created in the first place. How to do this, however, without doing a
wasteful read first?

It looks like this code was in 2.0.4 too.
servers/fs/cache.c