Re: [PATCH] btrfs: make use of inode_need_compress()

2017-07-21 Thread David Sterba
On Thu, Jul 20, 2017 at 10:22:13AM +0800, Anand Jain wrote: > >> @@ -1189,11 +1189,10 @@ static int cow_file_range_async(struct inode > >> *inode, struct page *locked_page, > >>async_cow->locked_page = locked_page; > >>async_cow->start = start; > >> > >> - if

Re: [PATCH] btrfs: make use of inode_need_compress()

2017-07-19 Thread Anand Jain
On 07/18/2017 11:01 PM, David Sterba wrote: On Tue, Jul 18, 2017 at 05:37:47PM +0800, Anand Jain wrote: Its better to have the policy enforcement going through a function, so that we have better control and visibility of the decision logic. Signed-off-by: Anand Jain

Re: [PATCH] btrfs: make use of inode_need_compress()

2017-07-18 Thread David Sterba
On Tue, Jul 18, 2017 at 05:37:47PM +0800, Anand Jain wrote: > Its better to have the policy enforcement going through a function, > so that we have better control and visibility of the decision logic. > > Signed-off-by: Anand Jain > --- > fs/btrfs/inode.c | 7 +++ > 1

[PATCH] btrfs: make use of inode_need_compress()

2017-07-18 Thread Anand Jain
Its better to have the policy enforcement going through a function, so that we have better control and visibility of the decision logic. Signed-off-by: Anand Jain --- fs/btrfs/inode.c | 7 +++ 1 file changed, 3 insertions(+), 4 deletions(-) diff --git