xfs
[Top] [All Lists]

Re: [PATCH 05/16] xfs: add CRC checks to the AGFL

To: Dave Chinner <david@xxxxxxxxxxxxx>
Subject: Re: [PATCH 05/16] xfs: add CRC checks to the AGFL
From: Ben Myers <bpm@xxxxxxx>
Date: Fri, 1 Mar 2013 17:56:41 -0600
Cc: xfs@xxxxxxxxxxx
Delivered-to: xfs@xxxxxxxxxxx
In-reply-to: <1361755901-12453-6-git-send-email-david@xxxxxxxxxxxxx>
References: <1361755901-12453-1-git-send-email-david@xxxxxxxxxxxxx> <1361755901-12453-6-git-send-email-david@xxxxxxxxxxxxx>
User-agent: Mutt/1.5.20 (2009-06-14)
Hi Dave,

On Mon, Feb 25, 2013 at 12:31:30PM +1100, Dave Chinner wrote:
> From: Christoph Hellwig <hch@xxxxxx>
> 
> Add CRC checks, location information and a magic number to the AGFL.
> Previously the AGFL was just a block containing nothing but the
> free block pointers.  The new AGFL has a real header with the usual
> boilerplate instead, so that we can verify it's not corrupted and
> written into the right place.
> 
> [dchinner@xxxxxxxxxx] Added LSN field, reworked significantly to fit
> into new verifier structure and growfs structure, enabled full
> verifier functionality now there is a header to verify and we can
> guarantee an initialised AGFL.
> 
> Signed-off-by: Christoph Hellwig <hch@xxxxxx>
> Signed-off-by: Dave Chinner <dchinner@xxxxxxxxxx>
> ---
>  fs/xfs/xfs_ag.h          |   25 +++++++++-
>  fs/xfs/xfs_alloc.c       |  119 
> ++++++++++++++++++++++++++++++----------------
>  fs/xfs/xfs_buf_item.h    |    4 +-
>  fs/xfs/xfs_fsops.c       |    5 ++
>  fs/xfs/xfs_log_recover.c |   10 ++++
>  5 files changed, 119 insertions(+), 44 deletions(-)
> 
> diff --git a/fs/xfs/xfs_ag.h b/fs/xfs/xfs_ag.h
> index 9f4815e..272ef09 100644
> --- a/fs/xfs/xfs_ag.h
> +++ b/fs/xfs/xfs_ag.h
> @@ -30,6 +30,7 @@ struct xfs_trans;
>  
>  #define      XFS_AGF_MAGIC   0x58414746      /* 'XAGF' */
>  #define      XFS_AGI_MAGIC   0x58414749      /* 'XAGI' */
> +#define      XFS_AGFL_MAGIC  0x5841464c      /* 'XAFL' */
>  #define      XFS_AGF_VERSION 1
>  #define      XFS_AGI_VERSION 1
>  
> @@ -190,11 +191,31 @@ extern const struct xfs_buf_ops xfs_agi_buf_ops;
>   */
>  #define XFS_AGFL_DADDR(mp)   ((xfs_daddr_t)(3 << (mp)->m_sectbb_log))
>  #define      XFS_AGFL_BLOCK(mp)      XFS_HDR_BLOCK(mp, XFS_AGFL_DADDR(mp))
> -#define XFS_AGFL_SIZE(mp)    ((mp)->m_sb.sb_sectsize / sizeof(xfs_agblock_t))
>  #define      XFS_BUF_TO_AGFL(bp)     ((xfs_agfl_t *)((bp)->b_addr))
>  
> +#define XFS_BUF_TO_AGFL_BNO(mp, bp) \
> +     (xfs_sb_version_hascrc(&((mp)->m_sb)) ? \
> +             &(XFS_BUF_TO_AGFL(bp)->agfl_bno[0]) : \
> +             (__be32 *)(bp)->b_addr)
> +
> +/*
> + * Size of the AGFL.  For CRC-enabled filesystes we steal a couple of
> + * slots in the beginning of the block for a proper header with the
> + * location information and CRC.
> + */
> +#define XFS_AGFL_SIZE(mp) \
> +     (((mp)->m_sb.sb_sectsize - \
> +      (xfs_sb_version_hascrc(&((mp)->m_sb)) ? \
> +             sizeof(struct xfs_agfl) : 0)) / \
> +       sizeof(xfs_agblock_t))
> +
>  typedef struct xfs_agfl {
> -     __be32          agfl_bno[1];    /* actually XFS_AGFL_SIZE(mp) */
> +     __be32          agfl_magicnum;

  +     __be32          agfl_versionnum;

Now would be a good time to add this, and it would be consistent with the AGF
and AGI structures.  There is no shortage of agfl slots according to my
calculation, and this might be helpful for xfsrepair later.

-Ben

> +     __be32          agfl_seqno;
> +     uuid_t          agfl_uuid;
> +     __be64          agfl_lsn;
> +     __be32          agfl_crc;
> +     __be32          agfl_bno[];     /* actually XFS_AGFL_SIZE(mp) */
>  } xfs_agfl_t;

<Prev in Thread] Current Thread [Next in Thread>
  • Re: [PATCH 05/16] xfs: add CRC checks to the AGFL, Ben Myers <=