On 08/25/2014 05:43 PM, Christoph Hellwig wrote: > On Mon, Aug 25, 2014 at 10:36:09AM -0400, Anna Schumaker wrote: >> Whoa, I like the amount of code you manage to remove in this patch! However, there is a lot going on here. Any way this patch could be split into incremental changes? (it's okay to say "no"!) > > The answer really is no - it's a wholesale replacement of three tracking > structures with an integrated one that is managed very differently. The only > thing I had split out ealier where various trivial bug fixes like sector > truncations in code that gets ripped out entirely later, so I decided to > fold those again, which didn't increase the patch size by more than 10 lines. > Right, same here. Anna it is best to just go read the code and don't dare look at the patch itself. The patch will just complicate what is more simple at the code level. I have seen this patch and it is on my todo to review the new code wholesale after this patch, to learn the nice structures Christoph has introduced here. Thanks Boaz -- To unsubscribe from this list: send the line "unsubscribe linux-nfs" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
