lkml.org 
[lkml]   [2020]   [Oct]   [27]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH 5.9 419/757] xfs: fix finobt btree block recovery ordering
    Date
    From: Dave Chinner <dchinner@redhat.com>

    [ Upstream commit 671459676ab0e1d371c8d6b184ad1faa05b6941e ]

    Nathan popped up on #xfs and pointed out that we fail to handle
    finobt btree blocks in xlog_recover_get_buf_lsn(). This means they
    always fall through the entire magic number matching code to "recover
    immediately". Whilst most of the time this is the correct behaviour,
    occasionally it will be incorrect and could potentially overwrite
    more recent metadata because we don't check the LSN in the on disk
    metadata at all.

    This bug has been present since the finobt was first introduced, and
    is a potential cause of the occasional xfs_iget_check_free_state()
    failures we see that indicate that the inode btree state does not
    match the on disk inode state.

    Fixes: aafc3c246529 ("xfs: support the XFS_BTNUM_FINOBT free inode btree type")
    Reported-by: Nathan Scott <nathans@redhat.com>
    Signed-off-by: Dave Chinner <dchinner@redhat.com>
    Reviewed-by: Darrick J. Wong <darrick.wong@oracle.com>
    Signed-off-by: Darrick J. Wong <darrick.wong@oracle.com>
    Reviewed-by: Brian Foster <bfoster@redhat.com>
    Signed-off-by: Sasha Levin <sashal@kernel.org>
    ---
    fs/xfs/xfs_buf_item_recover.c | 2 ++
    1 file changed, 2 insertions(+)

    diff --git a/fs/xfs/xfs_buf_item_recover.c b/fs/xfs/xfs_buf_item_recover.c
    index 8f0457d67d779..de2772394de21 100644
    --- a/fs/xfs/xfs_buf_item_recover.c
    +++ b/fs/xfs/xfs_buf_item_recover.c
    @@ -719,6 +719,8 @@ xlog_recover_get_buf_lsn(
    case XFS_ABTC_MAGIC:
    case XFS_RMAP_CRC_MAGIC:
    case XFS_REFC_CRC_MAGIC:
    + case XFS_FIBT_CRC_MAGIC:
    + case XFS_FIBT_MAGIC:
    case XFS_IBT_CRC_MAGIC:
    case XFS_IBT_MAGIC: {
    struct xfs_btree_block *btb = blk;
    --
    2.25.1


    \
     
     \ /
      Last update: 2020-10-27 17:08    [W:2.313 / U:0.100 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site