btrfs: change btrfs_csum_final result param type to u8

csum member of struct btrfs_super_block has array type of u8. It makes
sense that function btrfs_csum_final should be also declared to accept
u8 *. I changed the declaration of method void btrfs_csum_final(u32 crc,
char *result); to void btrfs_csum_final(u32 crc, u8 *result);

Signed-off-by: Domagoj Tršan <domagoj.trsan@gmail.com>
[ changed cast to u8 at several call sites ]
Signed-off-by: David Sterba <dsterba@suse.com>

diff --git a/fs/btrfs/compression.c b/fs/btrfs/compression.c
index d4d8b7e36b2ffe7b84ddb78e4064b606aaee5e01..49108036dd4c5fe93d310fa660c13a7a2a53ed43 100644 (file)
--- a/fs/btrfs/compression.c
+++ b/fs/btrfs/compression.c
@@ -120,7 +120,7 @@ static int check_compressed_csum(struct inode *inode,
 
                kaddr = kmap_atomic(page);
                csum = btrfs_csum_data(kaddr, csum, PAGE_SIZE);
-               btrfs_csum_final(csum, (char *)&csum);
+               btrfs_csum_final(csum, (u8 *)&csum);
                kunmap_atomic(kaddr);
 
                if (csum != *cb_sum) {

diff --git a/fs/btrfs/disk-io.c b/fs/btrfs/disk-io.c
index 5d1da78f044b0167d8df36756903a33070e4535d..8677d29efade93c64b9da65233f9ceb0f4a199ae 100644 (file)
--- a/fs/btrfs/disk-io.c
+++ b/fs/btrfs/disk-io.c
@@ -271,7 +271,7 @@ u32 btrfs_csum_data(char *data, u32 seed, size_t len)
        return btrfs_crc32c(seed, data, len);
 }
 
-void btrfs_csum_final(u32 crc, char *result)
+void btrfs_csum_final(u32 crc, u8 *result)
 {
        put_unaligned_le32(~crc, result);
 }

diff --git a/fs/btrfs/disk-io.h b/fs/btrfs/disk-io.h
index 124e30c76626d32ebfd449b0f8a1bb8f9db12024..7295407014587cd8e6d6c6f5b15139e4fecafd88 100644 (file)
--- a/fs/btrfs/disk-io.h
+++ b/fs/btrfs/disk-io.h
@@ -119,7 +119,7 @@ int btrfs_buffer_uptodate(struct extent_buffer *buf, u64 parent_transid,
                          int atomic);
 int btrfs_read_buffer(struct extent_buffer *buf, u64 parent_transid);
 u32 btrfs_csum_data(char *data, u32 seed, size_t len);
-void btrfs_csum_final(u32 crc, char *result);
+void btrfs_csum_final(u32 crc, u8 *result);
 int btrfs_bio_wq_end_io(struct btrfs_fs_info *info, struct bio *bio,
                        enum btrfs_wq_endio_type metadata);
 int btrfs_wq_submit_bio(struct btrfs_fs_info *fs_info, struct inode *inode,

diff --git a/fs/btrfs/free-space-cache.c b/fs/btrfs/free-space-cache.c
index a754865b3cb1efcdf19a0d2eeab4508a9d7d7fb5..e690d386ee5ee59479f318c10beb78a1943f1bc4 100644 (file)
--- a/fs/btrfs/free-space-cache.c
+++ b/fs/btrfs/free-space-cache.c
@@ -476,7 +476,7 @@ static void io_ctl_set_crc(struct btrfs_io_ctl *io_ctl, int index)
 
        crc = btrfs_csum_data(io_ctl->orig + offset, crc,
                              PAGE_SIZE - offset);
-       btrfs_csum_final(crc, (char *)&crc);
+       btrfs_csum_final(crc, (u8 *)&crc);
        io_ctl_unmap_page(io_ctl);
        tmp = page_address(io_ctl->pages[0]);
        tmp += index;
@@ -504,7 +504,7 @@ static int io_ctl_check_crc(struct btrfs_io_ctl *io_ctl, int index)
        io_ctl_map_page(io_ctl, 0);
        crc = btrfs_csum_data(io_ctl->orig + offset, crc,
                              PAGE_SIZE - offset);
-       btrfs_csum_final(crc, (char *)&crc);
+       btrfs_csum_final(crc, (u8 *)&crc);
        if (val != crc) {
                btrfs_err_rl(io_ctl->root->fs_info,
                        "csum mismatch on free space cache");

diff --git a/fs/btrfs/inode.c b/fs/btrfs/inode.c
index 66c1d65bd4766ad841ea07bff7dea06a041e3dc9..e6300d00c0630cdc2e7cb2fd136e752e696a2eae 100644 (file)
--- a/fs/btrfs/inode.c
+++ b/fs/btrfs/inode.c
@@ -3102,7 +3102,7 @@ static int __readpage_endio_check(struct inode *inode,
 
        kaddr = kmap_atomic(page);
        csum = btrfs_csum_data(kaddr + pgoff, csum,  len);
-       btrfs_csum_final(csum, (char *)&csum);
+       btrfs_csum_final(csum, (u8 *)&csum);
        if (csum != csum_expected)
                goto zeroit;