Re: [RESEND][PATCH] btrfs: Align EOF length to block in extent_same

2015-06-12 Thread Chris Mason
On 06/10/2015 06:13 PM, Mark Fasheh wrote: Hi, Can this patch please be considered for upstream inclusion? I reviewed it myself and have tested it on a few files with an updated duperemove branch: https://github.com/markfasheh/duperemove/tree/alignment_fix as described below, this

Re: [RESEND][PATCH] btrfs: Align EOF length to block in extent_same

2015-06-12 Thread Mark Fasheh
On Fri, Jun 12, 2015 at 04:18:25PM -0400, Chris Mason wrote: Hi Mark, This one conflicts with your btrfs: Handle unaligned length in extent_same, which is in integration now. If we need both, can you please rework it? They both fix the same bug, so we just need one of them. You can keep

Re: [RESEND][PATCH] btrfs: Align EOF length to block in extent_same

2015-06-11 Thread Liu Bo
On Wed, Jun 10, 2015 at 03:13:55PM -0700, Mark Fasheh wrote: Hi, Can this patch please be considered for upstream inclusion? I reviewed it myself and have tested it on a few files with an updated duperemove branch: https://github.com/markfasheh/duperemove/tree/alignment_fix as

[RESEND][PATCH] btrfs: Align EOF length to block in extent_same

2015-06-10 Thread Mark Fasheh
Hi, Can this patch please be considered for upstream inclusion? I reviewed it myself and have tested it on a few files with an updated duperemove branch: https://github.com/markfasheh/duperemove/tree/alignment_fix as described below, this allows us to dedupe the tail extents of files whose size