Aneesh Kumar K.V wrote:
+ /* first, we need to know whether the block is allocated already
+ * XXX: when the filesystem has a lot of free blocks, we could
+ * reserve even allocated blocks to save this lookup */
+ ret = ext4_get_blocks_wrap(NULL, inode, iblock, 1, bh_result, 0,
0);
+ if (ret >= 0) {
I guess this should be (ret > 0)
well, no. it has to catch allocated and non-allocated blocks
(though can be written more clear, probably)
I was not able to find mpage_da_writepages()..
sorry, it's part of vfs delayed allocation patch, should be on the list now.
thanks, Alex
-
To unsubscribe from this list: send the line "unsubscribe linux-ext4" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
