[f2fs-dev] [PATCH] common/attr: adbjust acl_max of f2fs

2022-02-07 Thread Sun Ke via Linux-f2fs-devel
f2fs has set inline_xattr as a default option, and introduced a new option named 'noinline_xattr' for disabling default inline_xattr option. So in _acl_get_max we need to check 'noinline_xattr' string in fs option, otherwise we may select the wrong max acl number since we always found the string 'i

Re: [f2fs-dev] [PATCH] f2fs-tools: add atomic write related options to f2fs_io write command

2022-02-07 Thread Chao Yu via Linux-f2fs-devel
On 2022/2/3 4:17, Daeho Jeong wrote: From: Daeho Jeong Added two option to f2fs_io "write" command. - atomic_commit : atomic write & commit - atomic_abort : atomic write & abort Also, added an optional option {delay}, which is in ms unit and optional only for atomic_commit and atomic_abort

Re: [f2fs-dev] [PATCH] mkfs.f2fs: set project quota by default for -g android

2022-02-07 Thread Chao Yu
On 2022/2/4 1:44, Jaegeuk Kim wrote: With this patch, "-g android" enables project quota only. Signed-off-by: Jaegeuk Kim Reviewed-by: Chao Yu Thanks, ___ Linux-f2fs-devel mailing list [email protected] https://lists.sourcef

Re: [f2fs-dev] [PATCH v3] f2fs: add a way to limit roll forward recovery time

2022-02-07 Thread Chao Yu
On 2022/2/8 3:01, Jaegeuk Kim wrote: This adds a sysfs entry to call checkpoint during fsync() in order to avoid long elapsed time to run roll-forward recovery when booting the device. Default value doesn't enforce the limitation which is same as before. Signed-off-by: Jaegeuk Kim Reviewed-by

Re: [f2fs-dev] [PATCH v2] f2fs: skip f2fs_preallocate_blocks() for overwrite case

2022-02-07 Thread Chao Yu
On 2022/2/8 3:16, Jaegeuk Kim wrote: On 02/04, Chao Yu wrote: There is potential hangtask happened during swapfile's writeback: - loop_kthread_worker_fn- do_checkpoint - kthread_worker_fn - loop_queue_work - lo_rw_aio - f2fs_file_write_iter - f2fs_preall

Re: [f2fs-dev] [PATCH] f2fs: avoid unneeded preflush during checkpoint()

2022-02-07 Thread Chao Yu
On 2022/2/8 3:14, Jaegeuk Kim wrote: On 02/07, Chao Yu wrote: During checkpoint, we have committed preflush command via f2fs_flush_device_cache() 787 int f2fs_flush_device_cache(struct f2fs_sb_info *sbi) 788 { 789 int ret = 0, i; 790 791 if (!f2fs_is_multi_device(sbi

Re: [f2fs-dev] [PATCH v2] f2fs: skip f2fs_preallocate_blocks() for overwrite case

2022-02-07 Thread Jaegeuk Kim
On 02/04, Chao Yu wrote: > There is potential hangtask happened during swapfile's writeback: > > - loop_kthread_worker_fn - do_checkpoint > - kthread_worker_fn >- loop_queue_work > - lo_rw_aio > - f2fs_file_write_iter > - f2fs_preallocate_blocks >- f2fs_ma

Re: [f2fs-dev] [PATCH] f2fs: avoid unneeded preflush during checkpoint()

2022-02-07 Thread Jaegeuk Kim
On 02/07, Chao Yu wrote: > During checkpoint, we have committed preflush command via > f2fs_flush_device_cache() 787 int f2fs_flush_device_cache(struct f2fs_sb_info *sbi) 788 { 789 int ret = 0, i; 790 791 if (!f2fs_is_multi_device(sbi)) 792 return 0; Seems

Re: [f2fs-dev] [PATCH v3] f2fs: add a way to limit roll forward recovery time

2022-02-07 Thread Jaegeuk Kim
This adds a sysfs entry to call checkpoint during fsync() in order to avoid long elapsed time to run roll-forward recovery when booting the device. Default value doesn't enforce the limitation which is same as before. Signed-off-by: Jaegeuk Kim --- v3 from v2: - add missing percpu init - per

[f2fs-dev] [PATCH] f2fs: avoid unneeded preflush during checkpoint()

2022-02-07 Thread Chao Yu
During checkpoint, we have committed preflush command via f2fs_flush_device_cache() to persist all metadata pages except last #2 CP pack page, so we don't need to commit another preflush command in commit_checkpoint(), remove it to avoid unneeded write cache overhead. Signed-off-by: Chao Yu ---