Re: Buffer state bits

[Jan Kara <jack@suse.cz>]

On Thu 27-08-09 18:03:18, Aneesh Kumar K.V wrote:
> > 
> > BH_Mapped
> > - Buffer has a physical block backing it stored in b_bdev + b_blocknr. This bit
> >   is set by filesystem's get_block() function (or by VFS itself for block device
> >   mappings).
> > 
> >   XXX: Some filesystems set BH_Mapped even for buffers that do no really
> >   have the backing block (like buffers for delayed allocation). I think
> >   we should get rid of it...
> > 
> 
> Also we don't want get_block to be called multiple times for the same file offset.
> __block_prepare_write does get_block looking at the BH_Mapped flag. ie one of reason
> delay and unwritten buffer_heads are also marked mapped.
  Unwritten buffers should be mapped as they really have the backing block.
So that's fine.
  Delayed buffers should not be mapped. We could change the check in
__block_prepare_write to "!buffer_mapped(bh) && !buffer_delay(bh)". But I'd
rather avoid it because what I'd like to do in ext3 is to delay-allocate
mmapped blocks and allocate normally blocks written via write(2). So I want
ext3_get_block() to be called in block_prepare_write() even for delay
buffers.
   If ext4 doesn't want to do anything for delay buffers in get_block() called
from block_prepare_write(), it can just return from the beginning of
ext4_get_block() when it finds the buffer is delay...

									Honza
-- 
Jan Kara <jack@suse.cz>
SUSE Labs, CR