Hi, On Wed, 2005-01-19 at 15:32, Alex Tomas wrote:
> during truncate ext3 calls journal_forget() for freed blocks, but > before these blocks go to the transaction and jbd reserves space > in log for them (->t_outstanding_credits). also, journal_forget() > removes these blocks from the transaction, but doesn't correct > log space reservation. for example, removal of 500MB file reserves > 136 blocks, but only 10 blocks go to the log. a commit is expensive > and correct reservation allows us to avoid needless commits. here > is the patch. tested on UP. Looks like a good approach to me, but would it not be better to return the credits to the handle instead of to the transaction? A really large truncate will typically be getting a bunch of credits, using those up and then extending itself continually as it encounters more and more indirect blocks. With your patch, the extended credits that the handle obtained will be returned to the transaction, effectively shrinking the transaction again and forcing the handle to extend itself yet again as it continues. If you returned them to the handle directly, it would be slightly more efficient. ACK either way, though --- the patch you've got now does look correct as it stands. Cheers, Stephen - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
