On Tue, Dec 01, 2009 at 10:17:02AM -0800, Curt Wohlgemuth wrote: > This fixes a leak of blocks in an inode prealloc list if device failures > cause ext4_mb_mark_diskspace_used() to fail. I was going to say that it didn't make sense for ext4_discard_inode_pa() should be its own function, but maybe we need function that returns blocks back to the either inode *or* the group preallocation list? Otherwise right now it looks like we would also be potentially leaking blocks from the group preallocation list in case of a device failure? - Ted -- 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