The patchset looks good to me.

Reviewed-by: Chao Yu <yuch...@huawei.com>

Thanks,

On 2017/4/22 17:22, Hou Pengyang wrote:
> For IPU, there is no need to read dnode, these three patches are to fix this
> issue.
> 
> patch 1): Reconstruct code in do_write_data_page, code logic stays unchanged,
> patch 2): Under high-probability IPU scenario, we check extent_tree first
> patch 3): Unlock dnode before IPU
> 
> v1->v2:
> 1) fix 
>               if(f2fs_encrypt_page())
>                       XXX
>       to 
>               err = f2fs_encrypt_page() 
>               if (err)
>                       XXX
> 
>       otherwise do_write_data_page would return a wrong return value.
>               
> 2) extract need_in_place_block(fio, page) for IPU checking.
> 
> Hou Pengyang (3):
>   f2fs: reconstruct code to write a data page
>   f2fs: lookup extent cache first under IPU scenario
>   f2fs: unlock dnode before IPU
> 
>  fs/f2fs/data.c | 109 
> ++++++++++++++++++++++++++++++++++++++++++++-------------
>  1 file changed, 84 insertions(+), 25 deletions(-)
> 


------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
_______________________________________________
Linux-f2fs-devel mailing list
Linux-f2fs-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/linux-f2fs-devel

Reply via email to