From: Christoph Hellwig <hch@infradead.org>
To: xfs@oss.sgi.com
Subject: [PATCH 1/3] xfstests: fix 122 for recent xfsprogs
Date: Mon, 10 Oct 2011 14:21:56 -0400 [thread overview]
Message-ID: <20111010182156.GA1323@infradead.org> (raw)
- filter out xfs_alloctype_t, this was an internal emum that got removed
- filter out xfs_bmbt_rec_32_t, this is a variant of the xfs_bmbt_rec_t
that had almost no users and was removed
- filter out xfs_dinode_core_t, the separate dinode core is gone, and just
checking the size of the full dinode is enough
- accept xfs_bmbt_rec_t as the new canonical name for xfs_bmbt_rec_64_t,
and replace the old name with the new one in the output stream.
Signed-off-by: Christoph Hellwig <hch@lst.de>
Index: xfstests-dev/122
===================================================================
--- xfstests-dev.orig/122 2011-10-10 16:49:06.000000000 +0000
+++ xfstests-dev/122 2011-10-10 17:04:07.000000000 +0000
@@ -57,6 +57,12 @@ s/sizeof\( xfs_agf_t \) = 64/sizeof( xfs
fi
}
+# filter out known changes to xfs type names
+_type_name_filter()
+{
+ sed -e 's/xfs_bmbt_rec_64_t/xfs_bmbt_rec_t/'
+}
+
_attribute_filter()
{
sed -e 's/__attribute__ *[(][(]packed[)][)]//' \
@@ -73,7 +79,9 @@ cat >$cprog <<EOF
EOF
cat >$tmp.ignore <<EOF
+xfs_alloctype_t
xfs_buf_cancel_t
+xfs_bmbt_rec_32_t
xfs_swapext_t
xfs_dirops_t
xfs_efi_log_item_t
@@ -160,6 +168,7 @@ xfs_log_item_chunk_t
xfs_trans_t
xfs_dirent_t
xfs_fsop_getparents_handlereq_t
+xfs_dinode_core_t
EOF
echo 'int main(int argc, char *argv[]) {' >>$cprog
@@ -193,7 +202,7 @@ echo 'return 0; }' >>$cprog
# create and run program
cc -o $oprog $cprog >> $seq.full 2>&1 || \
_notrun "Could not compile test program (see end of $seq.full)"
-$oprog | _type_size_filter > $progout
+$oprog | _type_size_filter | _type_name_filter > $progout
#
# add addition sizes and xfs_sb_t fields that don't exist in the version
Index: xfstests-dev/122.out
===================================================================
--- xfstests-dev.orig/122.out 2011-10-10 16:53:08.000000000 +0000
+++ xfstests-dev/122.out 2011-10-10 16:56:59.000000000 +0000
@@ -50,7 +50,6 @@ sizeof( xfs_agfl_t ) = 4
sizeof( xfs_agi_t ) = 296
sizeof( xfs_alloc_rec_incore_t ) = 8
sizeof( xfs_alloc_rec_t ) = 8
-sizeof( xfs_alloctype_t ) = 4
sizeof( xfs_attr_inactive_list_t ) = 8
sizeof( xfs_attr_leaf_entry_t ) = 8
sizeof( xfs_attr_leaf_hdr_t ) = 32
@@ -61,14 +60,12 @@ sizeof( xfs_attr_leafblock_t ) = 56
sizeof( xfs_attr_shortform_t ) = 8
sizeof( xfs_attrlist_cursor_t ) = 16
sizeof( xfs_bmbt_key_t ) = 8
-sizeof( xfs_bmbt_rec_32_t ) = 16
-sizeof( xfs_bmbt_rec_64_t ) = 16
+sizeof( xfs_bmbt_rec_t ) = 16
sizeof( xfs_bmdr_block_t ) = 4
sizeof( xfs_btnum_t ) = 4
sizeof( xfs_buf_log_format_t ) = 24
sizeof( xfs_da_blkinfo_t ) = 12
sizeof( xfs_da_intnode_t ) = 24
-sizeof( xfs_dinode_core_t ) = 96
sizeof( xfs_dinode_fmt_t ) = 4
sizeof( xfs_dir2_data_free_t ) = 4
sizeof( xfs_dir2_data_hdr_t ) = 16
_______________________________________________
xfs mailing list
xfs@oss.sgi.com
http://oss.sgi.com/mailman/listinfo/xfs
next reply other threads:[~2011-10-10 18:22 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-10-10 18:21 Christoph Hellwig [this message]
2011-10-10 18:22 ` [PATCH 2/2] xfstests: split mapped writes vs direct I/O tests from 091 Christoph Hellwig
2011-10-10 18:22 ` [PATCH 3/3] xfstests: allow 091 for non-xfs filesystems Christoph Hellwig
2011-10-11 2:11 ` Dave Chinner
2011-10-11 12:46 ` Christoph Hellwig
2011-10-11 14:18 ` [PATCH 3/3 v2] " Christoph Hellwig
2011-10-11 22:40 ` Dave Chinner
2011-10-13 11:06 ` Alex Elder
2011-10-11 22:40 ` [PATCH 2/2] xfstests: split mapped writes vs direct I/O tests from 091 Dave Chinner
2011-10-12 7:32 ` Christoph Hellwig
2011-10-13 11:06 ` Alex Elder
2011-10-13 11:06 ` Alex Elder
2011-10-11 22:36 ` [PATCH 1/3] xfstests: fix 122 for recent xfsprogs Dave Chinner
2011-10-13 11:06 ` Alex Elder
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=20111010182156.GA1323@infradead.org \
--to=hch@infradead.org \
--cc=xfs@oss.sgi.com \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox