Re: [PATCH] ext2fs and large stores (> 1.5G)

[Neal H. Walfield]

> Here is the solution[1] to the "record_global_poke" problem: the
> .dirty field is replaced by .dirty_count field.  Now a block in the
> cache is never reused while "use_count || dirty_count".  This forces
> us to use the same amount of disk_cache_clear(block) calls (old name
> was disk_image_clear) as disk_cache_release(block,1).  One problem
> arises: when indirect block is made many times dirty, this will
> increment dirty_count many times, but _pokel_exec will decrement
> dirty_count only once.  To solve it, I made pokel_add to return
> boolean value that indicates whether the passed memory region is
> already in the pokel.  If it is, then disk_cache_clear is called.

I think it makes the most sense to clear the dirty bit in
disk_pager_write_page.  This is when you are really flushing it to
backing store.