[dm-devel] [PATCH RFC 00/10] dm-dedup: device-mapper deduplication target
tarasov at vasily.name
Fri Jul 18 11:59:55 UTC 2014
Thanks for your initial reviewing steps! Here is what we plan to do:
1) Address your current comments
2) Few people mentioned that our documentation need to be formatted
better and be more detailed. We'll address this.
3) There are several things in dm-dedup that we would like to improve.
We will work on that to.
4) We will then post dm-dedup v2 patchset and prepare a fresh git
branch. It will definitely happen before you're available in
5) If Joe and/or Mikulas can take a look at the code meanwhile - that
would be great!
On Thu, Jul 17, 2014 at 10:43 PM, Mike Snitzer <snitzer at redhat.com> wrote:
> On Tue, May 06 2014 at 9:43am -0400,
> Vasily Tarasov <tarasov at vasily.name> wrote:
>> Interestingly, I can see 4, 5, and 7 in dm-devel's archive:
>> In any case, you can pull the patches from:
>> Branch: rfc-v1.1
>> Thanks for looking into this.
> I haven't been able to get to _really_ reviewing dm-dedup. It isn't
> anything against you guys.. I've just been quite busy with other tasks.
> I did start in on dm-dedup a month or so ago by staging a baseline of
> your work in a branch here:
> I found a few things that didn't look right, but they are more
> DM-specific mechanics and not anything to do with your approach for
> accomplishing dedup, see the FIXMEs I added to the documentation file in
> this commit:
> The reconstruct= param isn't needed. In both dm-thinp and dm-cache we
> use __superblock_all_zeroes to checks if the metadata device's
> superblock is all zeros. Ideally dm-dedup would do something
> I'm going to be on paternity leave until Sept. 8. It'd be great if Joe
> and/or Mikulas took some time to review dm-dedup but I'm not sure if
> they'll be able to. I do hope to be around to respond to emails
> periodically but my availability is TBD at this point.
> When I get back from leave I'll definitely make dm-dedup a priority if
> others don't beat me to it.
More information about the dm-devel