Hi,
On Mon, Dec 08, 2014 at 03:29:42PM +0900, Changman Lee wrote:
> In case of checkpoint is called at fsync, we should clear track info.
> Otherwise, remained track info may influence to inode flag next
> eviction.
In do_checkpoint, release_dirty_inode() already removed UPDATE|APPEND_INO.
So, let's just drop the inode flags.
Thanks,
>
> Signed-off-by: Changman Lee <cm224.lee@xxxxxxxxxxx>
> ---
> fs/f2fs/file.c | 12 ++++++++----
> 1 file changed, 8 insertions(+), 4 deletions(-)
>
> diff --git a/fs/f2fs/file.c b/fs/f2fs/file.c
> index 3c27e0e..3860fb8 100644
> --- a/fs/f2fs/file.c
> +++ b/fs/f2fs/file.c
> @@ -177,6 +177,7 @@ int f2fs_sync_file(struct file *file, loff_t start, loff_t end, int datasync)
> nid_t ino = inode->i_ino;
> int ret = 0;
> bool need_cp = false;
> + bool flush = true;
> struct writeback_control wbc = {
> .sync_mode = WB_SYNC_ALL,
> .nr_to_write = LONG_MAX,
> @@ -238,10 +239,11 @@ go_write:
>
> /*
> * We've secured consistency through sync_fs. Following pino
> - * will be used only for fsynced inodes after checkpoint.
> + * will be used only for inodes to be fsynced after checkpoint.
> */
> try_to_fix_pino(inode);
> - goto out;
> + flush = false;
> + goto flush_cleanup;
> }
> sync_nodes:
> sync_node_pages(sbi, ino, &wbc);
> @@ -256,13 +258,15 @@ sync_nodes:
> if (ret)
> goto out;
>
> - /* once recovery info is written, don't need to tack this */
> + /* once recovery info is written, don't need to track this */
> +flush_cleanup:
> remove_dirty_inode(sbi, ino, APPEND_INO);
> clear_inode_flag(fi, FI_APPEND_WRITE);
> flush_out:
> remove_dirty_inode(sbi, ino, UPDATE_INO);
> clear_inode_flag(fi, FI_UPDATE_WRITE);
> - ret = f2fs_issue_flush(sbi);
> + if (flush)
> + ret = f2fs_issue_flush(sbi);
> out:
> trace_f2fs_sync_file_exit(inode, need_cp, datasync, ret);
> return ret;
> --
> 1.9.1
>
>
> ------------------------------------------------------------------------------
> Download BIRT iHub F-Type - The Free Enterprise-Grade BIRT Server
> from Actuate! Instantly Supercharge Your Business Reports and Dashboards
> with Interactivity, Sharing, Native Excel Exports, App Integration & more
> Get technology previously reserved for billion-dollar corporations, FREE
> http://pubads.g.doubleclick.net/gampad/clk?id=164703151&iu=/4140/ostg.clktrk
> _______________________________________________
> Linux-f2fs-devel mailing list
> Linux-f2fs-devel@xxxxxxxxxxxxxxxxxxxxx
> https://lists.sourceforge.net/lists/listinfo/linux-f2fs-devel
--
To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
