On Mon, Jul 24, 2017 at 02:37:05PM +0300, Timofey Titovets wrote:
> Based on kdave for-next
> As heuristic skeleton already merged
> Populate heuristic with basic code that:
> 1. Collect sample from input data
> 2. Calculate byte set for sample
>For detect easily compressible data
> 3. Calculate byte core set size
>For detect easily and not compressible data
>
> Timofey Titovets (3):
> Btrfs: heuristic add simple sampling logic
> Btrfs: heuristic add byte set calculation
> Btrfs: heuristic add byte core set calculation
>
> fs/btrfs/compression.c | 111
> -
> fs/btrfs/compression.h | 14 +++
> 2 files changed, 123 insertions(+), 2 deletions(-)
>
Overall these look fine, I'd like more verbose changelogs to describe in more
detail the heuristic you are adding. I had to go back and work out what you
were doing a few times because it wasn't clear what each patch was doing.
Thanks,
Josef
--
To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html