Hello Dave, Thank you for your comments. About diffstat, actually I send it but the subject was not 0/67. Please read the thread whose subject is "[RFC n/2] AUFS: merging/stacking several filesystems" The last message in the thread which I sent just before the patch series has diffstat. Do you want me to re-send it after changing the subject line? About the number of patches, I have to agree it is large. I chose the way to make a patch per a new file, and in case of modifying the existing multiple files I made a single patch. I guess the latter is the case you wrote "can be folded." About the description, simple one for the new files because they are brand new... I wrote a little more in case of modifying the existing files (see the last patch). Of course a more detail description will be necessary in future modification. If you think the functionality is obscure, please read the aufs documents included in the patch set (the first patches). Junjiro Okajima Dave Quigley: > A couple of comments about this. First off you might want to reconsider > how you are breaking up the patches. 67 patches seems a bit much to me > and while the separation between patches is clear it seems that some of > them can be folded. Second I'm not sure if I haven't gotten it yet or if > it wasn't sent but I am missing an Email 0/67. Typically this contains a > description of the patch set, an index of the patches (possibly with > descriptions), and a diffstat for the entire patch set. The last thing > is that each patch should have a worth wile description attached to it. > All of your comments are two lines and that may make sense based on how > you have it broken up but if you start breaking up the patch set > differently you will need better descriptions (possibly even now for > some of the more obscure functionality). -- 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
