http://bugzilla.kernel.org/show_bug.cgi?id=15018 --- Comment #6 from Aneesh Kumar K.V <aneesh.kumar@xxxxxxxxxxxxxxxxxx> 2010-01-27 09:46:39 --- On Tue, 26 Jan 2010 23:19:23 GMT, bugzilla-daemon@xxxxxxxxxxxxxxxxxxx wrote: > http://bugzilla.kernel.org/show_bug.cgi?id=15018 > > > > > > --- Comment #4 from Jan Kara <jack@xxxxxxx> 2010-01-26 23:19:18 --- > Hmm, I don't like the code in ext4_da_reserve_space even after your patch - the > retries are duplicated with the ones from ext4_da_write_begin and calling > write_inode_now from ext4_da_write_begin doesn't make too much sense either - > why not write all inodes with delayed allocation if we want to write something? > > Actually, instead of retrying delayed allocation, I think it makes more sense > to just fall back to normal allocation. Attached patch does that and it also > allows me to completely fill user's quota. > Switching to non delayed allocation may not guarantee that we succeed in block allocation. For quota we really need to flush this inode. Even if we flush other inodes of other user we may still get a quota error -aneesh -- Configure bugmail: http://bugzilla.kernel.org/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are watching the assignee of the bug. -- 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
