chandan
2013-Sep-23 18:17 UTC
[PATCH] btrfs-progs: cmd_find_new: Sync fs before searching for modified files.
The sync makes sure that ''very recently'' introduced delayed work is accounted for in the output of ''btrfs subvolume find-new'' command. Signed-off-by: chandan <chandan@linux.vnet.ibm.com> --- cmds-subvolume.c | 9 +++++++++ 1 file changed, 9 insertions(+) diff --git a/cmds-subvolume.c b/cmds-subvolume.c index de246ab..8832303 100644 --- a/cmds-subvolume.c +++ b/cmds-subvolume.c @@ -786,6 +786,15 @@ static int cmd_find_new(int argc, char **argv) fprintf(stderr, "ERROR: can''t access ''%s''\n", subvol); return 1; } + + ret = ioctl(fd, BTRFS_IOC_SYNC); + if (ret < 0) { + fprintf(stderr, "ERROR: unable to fs-syncing ''%s'' - %s\n", + subvol, strerror(errno)); + close_file_or_dir(fd, dirstream); + return 1; + } + ret = btrfs_list_find_updated_files(fd, 0, last_gen); close_file_or_dir(fd, dirstream); return !!ret; -- 1.8.3.1 -- To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
chandan
2013-Sep-23 18:19 UTC
Re: [PATCH] btrfs-progs: cmd_find_new: Sync fs before searching for modified files.
This patch was tested on a kernel with "[PATCH v4] Btrfs: fix sync fs to actually wait for all data to be persisted" applied. Thanks, chandan -- To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html