Re: [PATCH] btrfs: Use iocb to derive pos instead of passing a separate parameter

2018-06-25 Thread Misono Tomohiro
On 2018/06/26 1:20, David Sterba wrote: > On Mon, Jun 25, 2018 at 01:58:58PM +0900, Misono Tomohiro wrote: >> So, this is the updated version of >> https://patchwork.kernel.org/patch/10063039/ >> >> This time xfstest is ok and >> Reviewed-by: Misono Tomohiro > > Your comment about

Re: [PATCH] btrfs: Use iocb to derive pos instead of passing a separate parameter

2018-06-25 Thread Goldwyn Rodrigues
On 06-25 18:20, David Sterba wrote: > On Mon, Jun 25, 2018 at 01:58:58PM +0900, Misono Tomohiro wrote: > > So, this is the updated version of > > https://patchwork.kernel.org/patch/10063039/ > > > > This time xfstest is ok and > > Reviewed-by: Misono Tomohiro Yes, thats right. > > Your

Re: [PATCH] btrfs: Use iocb to derive pos instead of passing a separate parameter

2018-06-25 Thread David Sterba
On Mon, Jun 25, 2018 at 01:58:58PM +0900, Misono Tomohiro wrote: > So, this is the updated version of > https://patchwork.kernel.org/patch/10063039/ > > This time xfstest is ok and > Reviewed-by: Misono Tomohiro Your comment about invalidate_mapping_pages is also ok, right? As

Re: [PATCH] btrfs: Use iocb to derive pos instead of passing a separate parameter

2018-06-24 Thread Misono Tomohiro
So, this is the updated version of https://patchwork.kernel.org/patch/10063039/ This time xfstest is ok and Reviewed-by: Misono Tomohiro On 2018/06/18 2:39, Goldwyn Rodrigues wrote: > From: Goldwyn Rodrigues > > struct kiocb carries the ki_pos, so there is no need to pass it as > a separate

[PATCH] btrfs: Use iocb to derive pos instead of passing a separate parameter

2018-06-17 Thread Goldwyn Rodrigues
From: Goldwyn Rodrigues struct kiocb carries the ki_pos, so there is no need to pass it as a separate function parameter. generic_file_direct_write() increments ki_pos, so we now assign pos after the function. Signed-off-by: Goldwyn Rodrigues --- fs/btrfs/file.c | 15 --- 1 file