Re: [PATCH] Btrfs: skip looking for delalloc if we don't have -fill_delalloc

2011-08-02 Thread Josef Bacik
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

[PATCH] Btrfs: skip looking for delalloc if we don't have -fill_delalloc

2011-08-01 Thread Josef Bacik
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

Re: [PATCH] Btrfs: skip looking for delalloc if we don't have -fill_delalloc

2011-08-01 Thread liubo
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

Re: [PATCH] Btrfs: skip looking for delalloc if we don't have -fill_delalloc

2011-08-01 Thread liubo
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