Commit 38dee376 authored by Darrick J. Wong's avatar Darrick J. Wong

xfs: always compile the btree inorder check functions

The btree record and key inorder check functions will be used by the
btree scrubber code, so make sure they're always built.
Signed-off-by: default avatarDarrick J. Wong <darrick.wong@oracle.com>
Reviewed-by: default avatarBrian Foster <bfoster@redhat.com>
parent c8ce540d
...@@ -395,7 +395,6 @@ const struct xfs_buf_ops xfs_allocbt_buf_ops = { ...@@ -395,7 +395,6 @@ const struct xfs_buf_ops xfs_allocbt_buf_ops = {
}; };
#if defined(DEBUG) || defined(XFS_WARN)
STATIC int STATIC int
xfs_bnobt_keys_inorder( xfs_bnobt_keys_inorder(
struct xfs_btree_cur *cur, struct xfs_btree_cur *cur,
...@@ -442,7 +441,6 @@ xfs_cntbt_recs_inorder( ...@@ -442,7 +441,6 @@ xfs_cntbt_recs_inorder(
be32_to_cpu(r1->alloc.ar_startblock) < be32_to_cpu(r1->alloc.ar_startblock) <
be32_to_cpu(r2->alloc.ar_startblock)); be32_to_cpu(r2->alloc.ar_startblock));
} }
#endif /* DEBUG */
static const struct xfs_btree_ops xfs_bnobt_ops = { static const struct xfs_btree_ops xfs_bnobt_ops = {
.rec_len = sizeof(xfs_alloc_rec_t), .rec_len = sizeof(xfs_alloc_rec_t),
...@@ -462,10 +460,8 @@ static const struct xfs_btree_ops xfs_bnobt_ops = { ...@@ -462,10 +460,8 @@ static const struct xfs_btree_ops xfs_bnobt_ops = {
.key_diff = xfs_bnobt_key_diff, .key_diff = xfs_bnobt_key_diff,
.buf_ops = &xfs_allocbt_buf_ops, .buf_ops = &xfs_allocbt_buf_ops,
.diff_two_keys = xfs_bnobt_diff_two_keys, .diff_two_keys = xfs_bnobt_diff_two_keys,
#if defined(DEBUG) || defined(XFS_WARN)
.keys_inorder = xfs_bnobt_keys_inorder, .keys_inorder = xfs_bnobt_keys_inorder,
.recs_inorder = xfs_bnobt_recs_inorder, .recs_inorder = xfs_bnobt_recs_inorder,
#endif
}; };
static const struct xfs_btree_ops xfs_cntbt_ops = { static const struct xfs_btree_ops xfs_cntbt_ops = {
...@@ -486,10 +482,8 @@ static const struct xfs_btree_ops xfs_cntbt_ops = { ...@@ -486,10 +482,8 @@ static const struct xfs_btree_ops xfs_cntbt_ops = {
.key_diff = xfs_cntbt_key_diff, .key_diff = xfs_cntbt_key_diff,
.buf_ops = &xfs_allocbt_buf_ops, .buf_ops = &xfs_allocbt_buf_ops,
.diff_two_keys = xfs_cntbt_diff_two_keys, .diff_two_keys = xfs_cntbt_diff_two_keys,
#if defined(DEBUG) || defined(XFS_WARN)
.keys_inorder = xfs_cntbt_keys_inorder, .keys_inorder = xfs_cntbt_keys_inorder,
.recs_inorder = xfs_cntbt_recs_inorder, .recs_inorder = xfs_cntbt_recs_inorder,
#endif
}; };
/* /*
......
...@@ -687,7 +687,6 @@ const struct xfs_buf_ops xfs_bmbt_buf_ops = { ...@@ -687,7 +687,6 @@ const struct xfs_buf_ops xfs_bmbt_buf_ops = {
}; };
#if defined(DEBUG) || defined(XFS_WARN)
STATIC int STATIC int
xfs_bmbt_keys_inorder( xfs_bmbt_keys_inorder(
struct xfs_btree_cur *cur, struct xfs_btree_cur *cur,
...@@ -708,7 +707,6 @@ xfs_bmbt_recs_inorder( ...@@ -708,7 +707,6 @@ xfs_bmbt_recs_inorder(
xfs_bmbt_disk_get_blockcount(&r1->bmbt) <= xfs_bmbt_disk_get_blockcount(&r1->bmbt) <=
xfs_bmbt_disk_get_startoff(&r2->bmbt); xfs_bmbt_disk_get_startoff(&r2->bmbt);
} }
#endif /* DEBUG */
static const struct xfs_btree_ops xfs_bmbt_ops = { static const struct xfs_btree_ops xfs_bmbt_ops = {
.rec_len = sizeof(xfs_bmbt_rec_t), .rec_len = sizeof(xfs_bmbt_rec_t),
...@@ -726,10 +724,8 @@ static const struct xfs_btree_ops xfs_bmbt_ops = { ...@@ -726,10 +724,8 @@ static const struct xfs_btree_ops xfs_bmbt_ops = {
.init_ptr_from_cur = xfs_bmbt_init_ptr_from_cur, .init_ptr_from_cur = xfs_bmbt_init_ptr_from_cur,
.key_diff = xfs_bmbt_key_diff, .key_diff = xfs_bmbt_key_diff,
.buf_ops = &xfs_bmbt_buf_ops, .buf_ops = &xfs_bmbt_buf_ops,
#if defined(DEBUG) || defined(XFS_WARN)
.keys_inorder = xfs_bmbt_keys_inorder, .keys_inorder = xfs_bmbt_keys_inorder,
.recs_inorder = xfs_bmbt_recs_inorder, .recs_inorder = xfs_bmbt_recs_inorder,
#endif
}; };
/* /*
......
...@@ -163,7 +163,6 @@ struct xfs_btree_ops { ...@@ -163,7 +163,6 @@ struct xfs_btree_ops {
const struct xfs_buf_ops *buf_ops; const struct xfs_buf_ops *buf_ops;
#if defined(DEBUG) || defined(XFS_WARN)
/* check that k1 is lower than k2 */ /* check that k1 is lower than k2 */
int (*keys_inorder)(struct xfs_btree_cur *cur, int (*keys_inorder)(struct xfs_btree_cur *cur,
union xfs_btree_key *k1, union xfs_btree_key *k1,
...@@ -173,7 +172,6 @@ struct xfs_btree_ops { ...@@ -173,7 +172,6 @@ struct xfs_btree_ops {
int (*recs_inorder)(struct xfs_btree_cur *cur, int (*recs_inorder)(struct xfs_btree_cur *cur,
union xfs_btree_rec *r1, union xfs_btree_rec *r1,
union xfs_btree_rec *r2); union xfs_btree_rec *r2);
#endif
}; };
/* /*
......
...@@ -302,7 +302,6 @@ const struct xfs_buf_ops xfs_inobt_buf_ops = { ...@@ -302,7 +302,6 @@ const struct xfs_buf_ops xfs_inobt_buf_ops = {
.verify_write = xfs_inobt_write_verify, .verify_write = xfs_inobt_write_verify,
}; };
#if defined(DEBUG) || defined(XFS_WARN)
STATIC int STATIC int
xfs_inobt_keys_inorder( xfs_inobt_keys_inorder(
struct xfs_btree_cur *cur, struct xfs_btree_cur *cur,
...@@ -322,7 +321,6 @@ xfs_inobt_recs_inorder( ...@@ -322,7 +321,6 @@ xfs_inobt_recs_inorder(
return be32_to_cpu(r1->inobt.ir_startino) + XFS_INODES_PER_CHUNK <= return be32_to_cpu(r1->inobt.ir_startino) + XFS_INODES_PER_CHUNK <=
be32_to_cpu(r2->inobt.ir_startino); be32_to_cpu(r2->inobt.ir_startino);
} }
#endif /* DEBUG */
static const struct xfs_btree_ops xfs_inobt_ops = { static const struct xfs_btree_ops xfs_inobt_ops = {
.rec_len = sizeof(xfs_inobt_rec_t), .rec_len = sizeof(xfs_inobt_rec_t),
...@@ -339,10 +337,8 @@ static const struct xfs_btree_ops xfs_inobt_ops = { ...@@ -339,10 +337,8 @@ static const struct xfs_btree_ops xfs_inobt_ops = {
.init_ptr_from_cur = xfs_inobt_init_ptr_from_cur, .init_ptr_from_cur = xfs_inobt_init_ptr_from_cur,
.key_diff = xfs_inobt_key_diff, .key_diff = xfs_inobt_key_diff,
.buf_ops = &xfs_inobt_buf_ops, .buf_ops = &xfs_inobt_buf_ops,
#if defined(DEBUG) || defined(XFS_WARN)
.keys_inorder = xfs_inobt_keys_inorder, .keys_inorder = xfs_inobt_keys_inorder,
.recs_inorder = xfs_inobt_recs_inorder, .recs_inorder = xfs_inobt_recs_inorder,
#endif
}; };
static const struct xfs_btree_ops xfs_finobt_ops = { static const struct xfs_btree_ops xfs_finobt_ops = {
...@@ -360,10 +356,8 @@ static const struct xfs_btree_ops xfs_finobt_ops = { ...@@ -360,10 +356,8 @@ static const struct xfs_btree_ops xfs_finobt_ops = {
.init_ptr_from_cur = xfs_finobt_init_ptr_from_cur, .init_ptr_from_cur = xfs_finobt_init_ptr_from_cur,
.key_diff = xfs_inobt_key_diff, .key_diff = xfs_inobt_key_diff,
.buf_ops = &xfs_inobt_buf_ops, .buf_ops = &xfs_inobt_buf_ops,
#if defined(DEBUG) || defined(XFS_WARN)
.keys_inorder = xfs_inobt_keys_inorder, .keys_inorder = xfs_inobt_keys_inorder,
.recs_inorder = xfs_inobt_recs_inorder, .recs_inorder = xfs_inobt_recs_inorder,
#endif
}; };
/* /*
......
...@@ -285,7 +285,6 @@ const struct xfs_buf_ops xfs_refcountbt_buf_ops = { ...@@ -285,7 +285,6 @@ const struct xfs_buf_ops xfs_refcountbt_buf_ops = {
.verify_write = xfs_refcountbt_write_verify, .verify_write = xfs_refcountbt_write_verify,
}; };
#if defined(DEBUG) || defined(XFS_WARN)
STATIC int STATIC int
xfs_refcountbt_keys_inorder( xfs_refcountbt_keys_inorder(
struct xfs_btree_cur *cur, struct xfs_btree_cur *cur,
...@@ -306,7 +305,6 @@ xfs_refcountbt_recs_inorder( ...@@ -306,7 +305,6 @@ xfs_refcountbt_recs_inorder(
be32_to_cpu(r1->refc.rc_blockcount) <= be32_to_cpu(r1->refc.rc_blockcount) <=
be32_to_cpu(r2->refc.rc_startblock); be32_to_cpu(r2->refc.rc_startblock);
} }
#endif
static const struct xfs_btree_ops xfs_refcountbt_ops = { static const struct xfs_btree_ops xfs_refcountbt_ops = {
.rec_len = sizeof(struct xfs_refcount_rec), .rec_len = sizeof(struct xfs_refcount_rec),
...@@ -325,10 +323,8 @@ static const struct xfs_btree_ops xfs_refcountbt_ops = { ...@@ -325,10 +323,8 @@ static const struct xfs_btree_ops xfs_refcountbt_ops = {
.key_diff = xfs_refcountbt_key_diff, .key_diff = xfs_refcountbt_key_diff,
.buf_ops = &xfs_refcountbt_buf_ops, .buf_ops = &xfs_refcountbt_buf_ops,
.diff_two_keys = xfs_refcountbt_diff_two_keys, .diff_two_keys = xfs_refcountbt_diff_two_keys,
#if defined(DEBUG) || defined(XFS_WARN)
.keys_inorder = xfs_refcountbt_keys_inorder, .keys_inorder = xfs_refcountbt_keys_inorder,
.recs_inorder = xfs_refcountbt_recs_inorder, .recs_inorder = xfs_refcountbt_recs_inorder,
#endif
}; };
/* /*
......
...@@ -377,7 +377,6 @@ const struct xfs_buf_ops xfs_rmapbt_buf_ops = { ...@@ -377,7 +377,6 @@ const struct xfs_buf_ops xfs_rmapbt_buf_ops = {
.verify_write = xfs_rmapbt_write_verify, .verify_write = xfs_rmapbt_write_verify,
}; };
#if defined(DEBUG) || defined(XFS_WARN)
STATIC int STATIC int
xfs_rmapbt_keys_inorder( xfs_rmapbt_keys_inorder(
struct xfs_btree_cur *cur, struct xfs_btree_cur *cur,
...@@ -437,7 +436,6 @@ xfs_rmapbt_recs_inorder( ...@@ -437,7 +436,6 @@ xfs_rmapbt_recs_inorder(
return 1; return 1;
return 0; return 0;
} }
#endif /* DEBUG */
static const struct xfs_btree_ops xfs_rmapbt_ops = { static const struct xfs_btree_ops xfs_rmapbt_ops = {
.rec_len = sizeof(struct xfs_rmap_rec), .rec_len = sizeof(struct xfs_rmap_rec),
...@@ -456,10 +454,8 @@ static const struct xfs_btree_ops xfs_rmapbt_ops = { ...@@ -456,10 +454,8 @@ static const struct xfs_btree_ops xfs_rmapbt_ops = {
.key_diff = xfs_rmapbt_key_diff, .key_diff = xfs_rmapbt_key_diff,
.buf_ops = &xfs_rmapbt_buf_ops, .buf_ops = &xfs_rmapbt_buf_ops,
.diff_two_keys = xfs_rmapbt_diff_two_keys, .diff_two_keys = xfs_rmapbt_diff_two_keys,
#if defined(DEBUG) || defined(XFS_WARN)
.keys_inorder = xfs_rmapbt_keys_inorder, .keys_inorder = xfs_rmapbt_keys_inorder,
.recs_inorder = xfs_rmapbt_recs_inorder, .recs_inorder = xfs_rmapbt_recs_inorder,
#endif
}; };
/* /*
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment