[v3,0/4] btrfs: support xxhash64 checksums

Message

Johannes Thumshirn Aug. 26, 2019, 11:48 a.m. UTC

Now that Nikolay's XXHASH64 support for the Crypto API has landed and BTRFS is
prepared for an easy addition of new checksums, this patchset implements
XXHASH64 as a second, fast but not cryptographically secure checksum hash.

For changes since v2, please see the individual patches.

David Sterba (1):
  btrfs: sysfs: export supported checksums

Johannes Thumshirn (3):
  btrfs: turn checksum type define into a enum
  btrfs: create structure to encode checksum type and length
  btrfs: use xxhash64 for checksumming

 fs/btrfs/Kconfig                |  1 +
 fs/btrfs/ctree.h                | 14 +++++++++-----
 fs/btrfs/disk-io.c              |  1 +
 fs/btrfs/super.c                |  1 +
 fs/btrfs/sysfs.c                | 29 +++++++++++++++++++++++++++++
 include/uapi/linux/btrfs_tree.h |  5 ++++-
 6 files changed, 45 insertions(+), 6 deletions(-)

Comments

Nikolay Borisov Aug. 26, 2019, 3:39 p.m. UTC | #1

On 26.08.19 г. 14:48 ч., Johannes Thumshirn wrote:
> Now that Nikolay's XXHASH64 support for the Crypto API has landed and BTRFS is
> prepared for an easy addition of new checksums, this patchset implements
> XXHASH64 as a second, fast but not cryptographically secure checksum hash.
> 
> For changes since v2, please see the individual patches.
> 
> David Sterba (1):
>   btrfs: sysfs: export supported checksums
> 
> Johannes Thumshirn (3):
>   btrfs: turn checksum type define into a enum
>   btrfs: create structure to encode checksum type and length
>   btrfs: use xxhash64 for checksumming
> 
>  fs/btrfs/Kconfig                |  1 +
>  fs/btrfs/ctree.h                | 14 +++++++++-----
>  fs/btrfs/disk-io.c              |  1 +
>  fs/btrfs/super.c                |  1 +
>  fs/btrfs/sysfs.c                | 29 +++++++++++++++++++++++++++++
>  include/uapi/linux/btrfs_tree.h |  5 ++++-
>  6 files changed, 45 insertions(+), 6 deletions(-)

Short and sweet, apart from the minor nit on 3/4 you can add:

Reviewed-by: Nikolay Borisov <[e-mail]>

>