Hi all, Here is the second version of original fsck.overlay. Changes split to each patch from first version "overlay: implement fsck utility" for readability. I have already handled most of comments from the first iteration and add/fix some infrastructure, no big features, move tests to xfstests (already tested). I will push this "incubator" version to github after review and fix. Any comments is helpful, thanks! Changes since v1: - Add "-n -p -y" options. (Comment from Amir and Darrick) - Move test cases to xfstests. (Amir, Eryu and Ted) - * Check lowers use base fd + relative path to speed up iterations. (Amir) - Handle missing case of redirect xattr check. (Amir) - Correct copyright and License. (Amir) - Remove duplicate redirect xattr in 'yes' mode. - Add objects counter. - Not enforce fs offline in 'no' mode. - Fix some code mistakes. *) This change will cost a lot of 'fd' (up to 500) and will not work if sysctl_nr_open is lower than lowerdir number (special case, default is 1024*1024). I think expand sysctl_nr_open temporary may have some side effect, so just return failure. Thanks! zhangyi (F) (18): overlay: implement fsck utility fsck.overlay: fix uninitialized variable fsck.overlay: add -n -p and -y options fsck.overlay: add path package and split helper fsck.overlay: convert path parse to use helper function fsck.overlay: open lowerdirs in advance fsck.overlay: check lowers use relative path fsck.overlay: fix spelling mistakes fsck.overlay: add counter of checked objects fsck.overlay: fix verbose flag fsck.overlay: add ovl_ask_invalid helper fsck.overlay: remove duplicate redirect xattr in yes mode fsck.overlay: handle missing case of redirecte directory fsck.overlay: correct copyright and License fsck.overlay: fix word mistake fsck.overlay: remove test cases fsck.overlay: not enforce overlayfs is offline in 'no changes' mode fsck.overlay: use relative path when checking lowers -- To unsubscribe from this list: send the line "unsubscribe linux-unionfs" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
