lkml.org 
[lkml]   [2018]   [Feb]   [28]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Date
    Subject[PATCH 3.2 005/140] btrfs: clear space cache inode generation always
    3.2.100-rc1 review patch.  If anyone has any objections, please let me know.

    ------------------

    From: Josef Bacik <jbacik@fb.com>

    commit 8e138e0d92c6c9d3d481674fb14e3439b495be37 upstream.

    We discovered a box that had double allocations, and suspected the space
    cache may be to blame. While auditing the write out path I noticed that
    if we've already setup the space cache we will just carry on. This
    means that any error we hit after cache_save_setup before we go to
    actually write the cache out we won't reset the inode generation, so
    whatever was already written will be considered correct, except it'll be
    stale. Fix this by _always_ resetting the generation on the block group
    inode, this way we only ever have valid or invalid cache.

    With this patch I was no longer able to reproduce cache corruption with
    dm-log-writes and my bpf error injection tool.

    Signed-off-by: Josef Bacik <jbacik@fb.com>
    Signed-off-by: David Sterba <dsterba@suse.com>
    Signed-off-by: Ben Hutchings <ben@decadent.org.uk>
    ---
    fs/btrfs/extent-tree.c | 14 +++++++-------
    1 file changed, 7 insertions(+), 7 deletions(-)

    --- a/fs/btrfs/extent-tree.c
    +++ b/fs/btrfs/extent-tree.c
    @@ -2759,13 +2759,6 @@ again:
    goto again;
    }

    - /* We've already setup this transaction, go ahead and exit */
    - if (block_group->cache_generation == trans->transid &&
    - i_size_read(inode)) {
    - dcs = BTRFS_DC_SETUP;
    - goto out_put;
    - }
    -
    /*
    * We want to set the generation to 0, that way if anything goes wrong
    * from here on out we know not to trust this cache when we load up next
    @@ -2775,6 +2768,13 @@ again:
    ret = btrfs_update_inode(trans, root, inode);
    WARN_ON(ret);

    + /* We've already setup this transaction, go ahead and exit */
    + if (block_group->cache_generation == trans->transid &&
    + i_size_read(inode)) {
    + dcs = BTRFS_DC_SETUP;
    + goto out_put;
    + }
    +
    if (i_size_read(inode) > 0) {
    ret = btrfs_truncate_free_space_cache(root, trans, path,
    inode);
    \
     
     \ /
      Last update: 2018-02-28 18:47    [W:2.330 / U:0.320 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site