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 <josef@xxxxxxxxxx> >>> --- >> > > 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 majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
