All of lore.kernel.org
 help / color / mirror / Atom feed
From: Dave Chinner <david@fromorbit.com>
To: "Darrick J. Wong" <darrick.wong@oracle.com>
Cc: eguan@redhat.com, linux-xfs@vger.kernel.org, fstests@vger.kernel.org
Subject: Re: [PATCH 1/3] xfs/122: decrease the log update done item sizes
Date: Thu, 29 Sep 2016 16:26:16 +1000	[thread overview]
Message-ID: <20160929062616.GK9806@dastard> (raw)
In-Reply-To: <147511610789.12284.10323409415801065382.stgit@birch.djwong.org>

On Wed, Sep 28, 2016 at 07:28:27PM -0700, Darrick J. Wong wrote:
> Redundant fields were removed from the rmap/refcount/bmap update done
> log items, so fix the size tests.
> 
> Signed-off-by: Darrick J. Wong <darrick.wong@oracle.com>

works for me.

Reviewed-by: Dave Chinner <dchinner@redhat.com>

> ---
>  tests/xfs/122.out |    6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
> 
> 
> diff --git a/tests/xfs/122.out b/tests/xfs/122.out
> index ae84283..67f7698 100644
> --- a/tests/xfs/122.out
> +++ b/tests/xfs/122.out
> @@ -64,10 +64,10 @@ sizeof(struct xfs_btree_block) = 72
>  sizeof(struct xfs_btree_block_lhdr) = 64
>  sizeof(struct xfs_btree_block_shdr) = 48
>  sizeof(struct xfs_bud_log_format) = 16
> -sizeof(struct xfs_bui_log_format) = 48
> +sizeof(struct xfs_bui_log_format) = 16
>  sizeof(struct xfs_clone_args) = 32
>  sizeof(struct xfs_cud_log_format) = 16
> -sizeof(struct xfs_cui_log_format) = 32
> +sizeof(struct xfs_cui_log_format) = 16
>  sizeof(struct xfs_da3_blkinfo) = 56
>  sizeof(struct xfs_da3_intnode) = 64
>  sizeof(struct xfs_da3_node_hdr) = 64
> @@ -94,7 +94,7 @@ sizeof(struct xfs_rtrmap_key) = 24
>  sizeof(struct xfs_rtrmap_rec) = 32
>  sizeof(struct xfs_rtrmap_root) = 4
>  sizeof(struct xfs_rud_log_format) = 16
> -sizeof(struct xfs_rui_log_format) = 48
> +sizeof(struct xfs_rui_log_format) = 16
>  sizeof(struct xfs_scrub_metadata) = 64
>  sizeof(xfs_agf_t) = 224
>  sizeof(xfs_agfl_t) = 36
> 
> 

-- 
Dave Chinner
david@fromorbit.com

      parent reply	other threads:[~2016-09-29  6:26 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-09-29  2:28 [PATCH 1/3] xfs/122: decrease the log update done item sizes Darrick J. Wong
2016-09-29  2:28 ` [PATCH 2/3] xfs/130: don't modify files after failed mount Darrick J. Wong
2016-09-29  6:27   ` Dave Chinner
2016-09-29  2:28 ` [PATCH 3/3] xfs/{207,327}: fsxattr no longer has the reflink flag Darrick J. Wong
2016-09-29  6:27   ` Dave Chinner
2016-09-29  6:26 ` Dave Chinner [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20160929062616.GK9806@dastard \
    --to=david@fromorbit.com \
    --cc=darrick.wong@oracle.com \
    --cc=eguan@redhat.com \
    --cc=fstests@vger.kernel.org \
    --cc=linux-xfs@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.