Re: [PATCH 2/2] blk-snap - snapshots and change-tracking for block devices

From: Sergei Shtepa
Date: Wed Oct 21 2020 - 07:15:35 EST


> And this is a 8600+ lines patch.
> Can you split this into manageable pieces ?

Yes, the module was quite large. But I think it's not good to show
the elephant in parts.
https://en.wikipedia.org/wiki/Blind_men_and_an_elephant

> I do not think anybody will review such a huge patch.

Yes, it will be a lot of work. But I hope that the code architecture
and splitting entities into separate files will help.

If someone can advise how to divide a module into a chain of patches,
I will be happy. I do not dare to divide it without losing meaning.

--
Sergei Shtepa
Veeam Software developer.