On 08/01/2011 09:43 PM, liubo wrote: > On 08/02/2011 09:32 AM, liubo wrote: >> On 08/02/2011 12:11 AM, Josef Bacik wrote: >>> We always look for delalloc bytes in our io_tree so we can fill in delalloc. >>> This is fine in most cases, but if we're writing out the btree_inode this is >>> just a superfluous tree search on the io_tree, and if we have a lot of >>> metadata >>> dirty this could be an expensive check. So instead check to see if our >>> io_tree >>> has a ->fill_delalloc op, and if not don't even bother doing the lookup. >>> Thanks, >>> >>> Signed-off-by: Josef Bacik <jo...@redhat.com> >>> --- >> > > sorry, I mixed the patch with others... > > The patch is ok. >
Good because I was horribly confused for a moment :), Josef -- To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html