diff options
| author | Christoph Hellwig <hch@lst.de> | 2023-01-21 07:50:08 +0100 |
|---|---|---|
| committer | David Sterba <dsterba@suse.com> | 2023-02-15 19:38:51 +0100 |
| commit | 7ab0fdfc810209fc493f2f67bdd41d5aafbc3a05 (patch) | |
| tree | 4d96c43f929fded88076da7b61c6c0a11ce6af5a /fs/btrfs/bio.c | |
| parent | btrfs: handle checksum validation and repair at the storage layer (diff) | |
| download | linux-7ab0fdfc810209fc493f2f67bdd41d5aafbc3a05.tar.xz linux-7ab0fdfc810209fc493f2f67bdd41d5aafbc3a05.zip | |
btrfs: open code btrfs_bio_free_csum
btrfs_bio_free_csum has only one caller left, and that caller is always
for an data inode and doesn't need zeroing of the csum pointer as that
pointer will never be touched again. Just open code the conditional
kfree there.
Reviewed-by: Johannes Thumshirn <johannes.thumshirn@wdc.com>
Signed-off-by: Christoph Hellwig <hch@lst.de>
Reviewed-by: David Sterba <dsterba@suse.com>
Signed-off-by: David Sterba <dsterba@suse.com>
Diffstat (limited to 'fs/btrfs/bio.c')
| -rw-r--r-- | fs/btrfs/bio.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/fs/btrfs/bio.c b/fs/btrfs/bio.c index 9bfdb90bd897..6ef953234068 100644 --- a/fs/btrfs/bio.c +++ b/fs/btrfs/bio.c @@ -223,7 +223,8 @@ static void btrfs_check_read_bio(struct btrfs_bio *bbio, struct btrfs_device *de offset += sectorsize; } - btrfs_bio_free_csum(bbio); + if (bbio->csum != bbio->csum_inline) + kfree(bbio->csum); if (fbio) btrfs_repair_done(fbio); |