diff options
author | Zhihui Zhang <zzhsuny@gmail.com> | 2018-07-02 20:00:54 -0400 |
---|---|---|
committer | David Sterba <dsterba@suse.com> | 2018-08-06 13:12:42 +0200 |
commit | 8f6c72a9e08c2a74d2159d2fc986fa9b29ceb851 (patch) | |
tree | 0b8a786d4e82e9dff40eb16ca25781ce1215d29a /fs/btrfs/free-space-cache.c | |
parent | 694c51fb2eb6b6d7c9552af78ba646023a843366 (diff) | |
download | linux-8f6c72a9e08c2a74d2159d2fc986fa9b29ceb851.tar.bz2 |
Btrfs: free space cache: make sure there is always room for generation number
io_ctl_set_generation() assumes that the generation number shares
the same page with inline CRCs. Let's make sure this is always true.
Signed-off-by: Zhihui Zhang <zzhsuny@gmail.com>
Reviewed-by: David Sterba <dsterba@suse.com>
Signed-off-by: David Sterba <dsterba@suse.com>
Diffstat (limited to 'fs/btrfs/free-space-cache.c')
-rw-r--r-- | fs/btrfs/free-space-cache.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/fs/btrfs/free-space-cache.c b/fs/btrfs/free-space-cache.c index d5f80cb300be..d2c0bdbd79ec 100644 --- a/fs/btrfs/free-space-cache.c +++ b/fs/btrfs/free-space-cache.c @@ -300,9 +300,9 @@ static int io_ctl_init(struct btrfs_io_ctl *io_ctl, struct inode *inode, if (btrfs_ino(BTRFS_I(inode)) != BTRFS_FREE_INO_OBJECTID) check_crcs = 1; - /* Make sure we can fit our crcs into the first page */ + /* Make sure we can fit our crcs and generation into the first page */ if (write && check_crcs && - (num_pages * sizeof(u32)) >= PAGE_SIZE) + (num_pages * sizeof(u32) + sizeof(u64)) > PAGE_SIZE) return -ENOSPC; memset(io_ctl, 0, sizeof(struct btrfs_io_ctl)); |