On 12/07/2016 01:32 AM, NeilBrown wrote: > > I would expect to see as description of what a PPL actually is and how > it works here... but there is none. > > The change-log for patch 06 has a tiny bit more information which is > just enough to be able to start trying to understand the code, but it > isn't much. > And none of this description gets into the code, or into the > Documentation/. This makes it hard to review and hard to maintain. > > Remember: if you want people to review you code, it is in your interest > to make it easy. That means give lots of details. Hi Neil, Thank you for taking the time to look at this and for your feedback. I didn't try to make it hard to review... Sometimes it's easy to forget how non-obvious things are after looking at them for too long :) I will improve the descriptions and address the issues that you found in the next version of the patches. Thanks, Artur -- To unsubscribe from this list: send the line "unsubscribe linux-raid" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
