[047/111] xfs: split out a btree type from the btree ops geometry flags

Message ID	171634532410.2478931.10379864954761996361.stgit@frogsfrogsfrogs (mailing list archive)
State	Superseded, archived
Headers	show Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 1AA4B4C89 for <linux-xfs@vger.kernel.org>; Wed, 22 May 2024 03:01:02 +0000 (UTC) Date: Tue, 21 May 2024 20:01:02 -0700 Subject: [PATCH 047/111] xfs: split out a btree type from the btree ops geometry flags From: "Darrick J. Wong" <djwong@kernel.org> To: djwong@kernel.org, cem@kernel.org Cc: Christoph Hellwig <hch@lst.de>, linux-xfs@vger.kernel.org Message-ID: <171634532410.2478931.10379864954761996361.stgit@frogsfrogsfrogs> In-Reply-To: <171634531590.2478931.8474978645585392776.stgit@frogsfrogsfrogs> References: <171634531590.2478931.8474978645585392776.stgit@frogsfrogsfrogs> User-Agent: StGit/0.19 Precedence: bulk MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit
Series	[001/111] xfs: convert kmem_zalloc() to kzalloc() \| expand [001/111] xfs: convert kmem_zalloc() to kzalloc() [002/111] xfs: convert kmem_alloc() to kmalloc() [003/111] xfs: convert remaining kmem_free() to kfree() [004/111] xfs: use __GFP_NOLOCKDEP instead of GFP_NOFS [005/111] xfs: use GFP_KERNEL in pure transaction contexts [006/111] xfs: clean up remaining GFP_NOFS users [007/111] xfs: use xfs_defer_alloc a bit more [008/111] xfs: Replace xfs_isilocked with xfs_assert_ilocked [009/111] xfs: create a static name for the dot entry too [010/111] xfs: create a predicate to determine if two xfs_names are the same [011/111] xfs: create a macro for decoding ftypes in tracepoints [012/111] xfs: report the health of quota counts [013/111] xfs: implement live quotacheck inode scan [014/111] xfs: report health of inode link counts [015/111] xfs: teach scrub to check file nlinks [016/111] xfs: separate the marking of sick and checked metadata [017/111] xfs: report fs corruption errors to the health tracking system [018/111] xfs: report ag header corruption errors to the health tracking system [019/111] xfs: report block map corruption errors to the health tracking system [020/111] xfs: report btree block corruption errors to the health system [021/111] xfs: report dir/attr block corruption errors to the health system [022/111] xfs: report inode corruption errors to the health system [023/111] xfs: report realtime metadata corruption errors to the health system [024/111] xfs: report XFS_IS_CORRUPT errors to the health system [025/111] xfs: add secondary and indirect classes to the health tracking system [026/111] xfs: remember sick inodes that get inactivated [027/111] xfs: update health status if we get a clean bill of health [028/111] xfs: consolidate btree block freeing tracepoints [029/111] xfs: consolidate btree block allocation tracepoints [030/111] xfs: set the btree cursor bc_ops in xfs_btree_alloc_cursor [031/111] xfs: drop XFS_BTREE_CRC_BLOCKS [032/111] xfs: encode the btree geometry flags in the btree ops structure [033/111] xfs: remove bc_ino.flags [034/111] xfs: consolidate the xfs_alloc_lookup_* helpers [035/111] xfs: turn the allocbt cursor active field into a btree flag [036/111] xfs: extern some btree ops structures [037/111] xfs: initialize btree blocks using btree_ops structure [038/111] xfs: rename btree block/buffer init functions [039/111] xfs: btree convert xfs_btree_init_block to xfs_btree_init_buf calls [040/111] xfs: remove the unnecessary daddr paramter to _init_block [041/111] xfs: set btree block buffer ops in _init_buf [042/111] xfs: move lru refs to the btree ops structure [043/111] xfs: move the btree stats offset into struct btree_ops [044/111] xfs: factor out a xfs_btree_owner helper [045/111] xfs: factor out a btree block owner check [046/111] xfs: store the btree pointer length in struct xfs_btree_ops [047/111] xfs: split out a btree type from the btree ops geometry flags [048/111] xfs: split the per-btree union in struct xfs_btree_cur [049/111] xfs: create predicate to determine if cursor is at inode root level [050/111] xfs: move comment about two 2 keys per pointer in the rmap btree [051/111] xfs: add a xfs_btree_init_ptr_from_cur [052/111] xfs: don't override bc_ops for staging btrees [053/111] xfs: fold xfs_allocbt_init_common into xfs_allocbt_init_cursor [054/111] xfs: remove xfs_allocbt_stage_cursor [055/111] xfs: fold xfs_inobt_init_common into xfs_inobt_init_cursor [056/111] xfs: remove xfs_inobt_stage_cursor [057/111] xfs: fold xfs_refcountbt_init_common into xfs_refcountbt_init_cursor [058/111] xfs: remove xfs_refcountbt_stage_cursor [059/111] xfs: fold xfs_rmapbt_init_common into xfs_rmapbt_init_cursor [060/111] xfs: remove xfs_rmapbt_stage_cursor [061/111] xfs: make full use of xfs_btree_stage_ifakeroot in xfs_bmbt_stage_cursor [062/111] xfs: make staging file forks explicit [063/111] xfs: fold xfs_bmbt_init_common into xfs_bmbt_init_cursor [064/111] xfs: remove xfs_bmbt_stage_cursor [065/111] xfs: split the agf_roots and agf_levels arrays [066/111] xfs: add a name field to struct xfs_btree_ops [067/111] xfs: add a sick_mask to struct xfs_btree_ops [068/111] xfs: split xfs_allocbt_init_cursor [069/111] xfs: remove xfs_inobt_cur [070/111] xfs: remove the btnum argument to xfs_inobt_count_blocks [071/111] xfs: split xfs_inobt_insert_sprec [072/111] xfs: split xfs_inobt_init_cursor [073/111] xfs: pass a 'bool is_finobt' to xfs_inobt_insert [074/111] xfs: remove xfs_btnum_t [075/111] xfs: simplify xfs_btree_check_sblock_siblings [076/111] xfs: simplify xfs_btree_check_lblock_siblings [077/111] xfs: open code xfs_btree_check_lptr in xfs_bmap_btree_to_extents [078/111] xfs: consolidate btree ptr checking [079/111] xfs: misc cleanups for __xfs_btree_check_sblock [080/111] xfs: remove the crc variable in __xfs_btree_check_lblock [081/111] xfs: tighten up validation of root block in inode forks [082/111] xfs: consolidate btree block verification [083/111] xfs: rename btree helpers that depends on the block number representation [084/111] xfs: factor out a __xfs_btree_check_lblock_hdr helper [085/111] xfs: remove xfs_btree_reada_bufl [086/111] xfs: remove xfs_btree_reada_bufs [087/111] xfs: move and rename xfs_btree_read_bufl [088/111] libxfs: teach buftargs to maintain their own buffer hashtable [089/111] libxfs: add xfile support [090/111] libxfs: partition memfd files to avoid using too many fds [091/111] xfs: teach buftargs to maintain their own buffer hashtable [092/111] libxfs: support in-memory buffer cache targets [093/111] xfs: add a xfs_btree_ptrs_equal helper [094/111] xfs: support in-memory btrees [095/111] xfs: launder in-memory btree buffers before transaction commit [096/111] xfs: create a helper to decide if a file mapping targets the rt volume [097/111] xfs: repair the rmapbt [098/111] xfs: create a shadow rmap btree during rmap repair [099/111] xfs: hook live rmap operations during a repair operation [100/111] xfs: clean up bmap log intent item tracepoint callsites [101/111] xfs: move xfs_bmap_defer_add to xfs_bmap_item.c [102/111] xfs: fix xfs_bunmapi to allow unmapping of partial rt extents [103/111] xfs: add a realtime flag to the bmap update log redo items [104/111] xfs: support deferred bmap updates on the attr fork [105/111] xfs: xfs_bmap_finish_one should map unwritten extents properly [106/111] xfs: move xfs_symlink_remote.c declarations to xfs_symlink_remote.h [107/111] xfs: move remote symlink target read function to libxfs [108/111] xfs: move symlink target write function to libxfs [109/111] xfs: xfs_btree_bload_prep_block() should use __GFP_NOFAIL [110/111] xfs: shrink failure needs to hold AGI buffer [111/111] xfs: allow sunit mount option to repair bad primary sb stripe values

diff --git a/libxfs/xfs_alloc_btree.c b/libxfs/xfs_alloc_btree.c index e1637580c..b18ac7045 100644 --- a/libxfs/xfs_alloc_btree.c +++ b/libxfs/xfs_alloc_btree.c @@ -453,6 +453,8 @@ xfs_allocbt_keys_contiguous( } const struct xfs_btree_ops xfs_bnobt_ops = { + .type = XFS_BTREE_TYPE_AG, + .rec_len = sizeof(xfs_alloc_rec_t), .key_len = sizeof(xfs_alloc_key_t), .ptr_len = XFS_BTREE_SHORT_PTR_LEN, @@ -480,6 +482,7 @@ const struct xfs_btree_ops xfs_bnobt_ops = { }; const struct xfs_btree_ops xfs_cntbt_ops = { + .type = XFS_BTREE_TYPE_AG, .geom_flags = XFS_BTGEO_LASTREC_UPDATE, .rec_len = sizeof(xfs_alloc_rec_t), diff --git a/libxfs/xfs_bmap_btree.c b/libxfs/xfs_bmap_btree.c index d2399ea42..54020dea2 100644 --- a/libxfs/xfs_bmap_btree.c +++ b/libxfs/xfs_bmap_btree.c @@ -524,7 +524,7 @@ xfs_bmbt_keys_contiguous( } const struct xfs_btree_ops xfs_bmbt_ops = { - .geom_flags = XFS_BTGEO_ROOT_IN_INODE, + .type = XFS_BTREE_TYPE_INODE, .rec_len = sizeof(xfs_bmbt_rec_t), .key_len = sizeof(xfs_bmbt_key_t), diff --git a/libxfs/xfs_btree.c b/libxfs/xfs_btree.c index 2bce8ebbd..f8c348e49 100644 --- a/libxfs/xfs_btree.c +++ b/libxfs/xfs_btree.c @@ -444,10 +444,19 @@ xfs_btree_del_cursor( */ ASSERT(cur->bc_btnum != XFS_BTNUM_BMAP || cur->bc_ino.allocated == 0 || xfs_is_shutdown(cur->bc_mp) || error != 0); + + switch (cur->bc_ops->type) { + case XFS_BTREE_TYPE_AG: + if (cur->bc_ag.pag) + xfs_perag_put(cur->bc_ag.pag); + break; + case XFS_BTREE_TYPE_INODE: + /* nothing to do */ + break; + } + if (unlikely(cur->bc_flags & XFS_BTREE_STAGING)) kfree(cur->bc_ops); - if (!(cur->bc_ops->ptr_len == XFS_BTREE_LONG_PTR_LEN) && cur->bc_ag.pag) - xfs_perag_put(cur->bc_ag.pag); kmem_cache_free(cur->bc_cache, cur); } @@ -705,7 +714,7 @@ struct xfs_ifork * xfs_btree_ifork_ptr( struct xfs_btree_cur *cur) { - ASSERT(cur->bc_ops->geom_flags & XFS_BTGEO_ROOT_IN_INODE); + ASSERT(cur->bc_ops->type == XFS_BTREE_TYPE_INODE); if (cur->bc_flags & XFS_BTREE_STAGING) return cur->bc_ino.ifake->if_fork; @@ -737,8 +746,8 @@ xfs_btree_get_block( int level, /* level in btree */ struct xfs_buf **bpp) /* buffer containing the block */ { - if ((cur->bc_ops->geom_flags & XFS_BTGEO_ROOT_IN_INODE) && - (level == cur->bc_nlevels - 1)) { + if (cur->bc_ops->type == XFS_BTREE_TYPE_INODE && + level == cur->bc_nlevels - 1) { *bpp = NULL; return xfs_btree_get_iroot(cur); } @@ -980,8 +989,8 @@ xfs_btree_readahead( * No readahead needed if we are at the root level and the * btree root is stored in the inode. */ - if ((cur->bc_ops->geom_flags & XFS_BTGEO_ROOT_IN_INODE) && - (lev == cur->bc_nlevels - 1)) + if (cur->bc_ops->type == XFS_BTREE_TYPE_INODE && + lev == cur->bc_nlevels - 1) return 0; if ((cur->bc_levels[lev].ra | lr) == cur->bc_levels[lev].ra) @@ -1169,14 +1178,12 @@ __xfs_btree_init_block( buf->bb_u.l.bb_lsn = 0; } } else { - /* owner is a 32 bit value on short blocks */ - __u32 __owner = (__u32)owner; - buf->bb_u.s.bb_leftsib = cpu_to_be32(NULLAGBLOCK); buf->bb_u.s.bb_rightsib = cpu_to_be32(NULLAGBLOCK); if (crc) { buf->bb_u.s.bb_blkno = cpu_to_be64(blkno); - buf->bb_u.s.bb_owner = cpu_to_be32(__owner); + /* owner is a 32 bit value on short blocks */ + buf->bb_u.s.bb_owner = cpu_to_be32((__u32)owner); uuid_copy(&buf->bb_u.s.bb_uuid, &mp->m_sb.sb_meta_uuid); buf->bb_u.s.bb_lsn = 0; } @@ -1214,7 +1221,7 @@ static inline __u64 xfs_btree_owner( struct xfs_btree_cur *cur) { - if (cur->bc_ops->geom_flags & XFS_BTGEO_ROOT_IN_INODE) + if (cur->bc_ops->type == XFS_BTREE_TYPE_INODE) return cur->bc_ino.ip->i_ino; return cur->bc_ag.pag->pag_agno; } @@ -1635,7 +1642,7 @@ xfs_btree_increment( * confused or have the tree root in an inode. */ if (lev == cur->bc_nlevels) { - if (cur->bc_ops->geom_flags & XFS_BTGEO_ROOT_IN_INODE) + if (cur->bc_ops->type == XFS_BTREE_TYPE_INODE) goto out0; ASSERT(0); xfs_btree_mark_sick(cur); @@ -1729,7 +1736,7 @@ xfs_btree_decrement( * or the root of the tree is in an inode. */ if (lev == cur->bc_nlevels) { - if (cur->bc_ops->geom_flags & XFS_BTGEO_ROOT_IN_INODE) + if (cur->bc_ops->type == XFS_BTREE_TYPE_INODE) goto out0; ASSERT(0); xfs_btree_mark_sick(cur); @@ -1804,8 +1811,8 @@ xfs_btree_lookup_get_block( int error = 0; /* special case the root block if in an inode */ - if ((cur->bc_ops->geom_flags & XFS_BTGEO_ROOT_IN_INODE) && - (level == cur->bc_nlevels - 1)) { + if (cur->bc_ops->type == XFS_BTREE_TYPE_INODE && + level == cur->bc_nlevels - 1) { *blkp = xfs_btree_get_iroot(cur); return 0; } @@ -2340,7 +2347,7 @@ xfs_btree_lshift( int error; /* error return value */ int i; - if ((cur->bc_ops->geom_flags & XFS_BTGEO_ROOT_IN_INODE) && + if ((cur->bc_ops->type == XFS_BTREE_TYPE_INODE) && level == cur->bc_nlevels - 1) goto out0; @@ -2536,8 +2543,8 @@ xfs_btree_rshift( int error; /* error return value */ int i; /* loop counter */ - if ((cur->bc_ops->geom_flags & XFS_BTGEO_ROOT_IN_INODE) && - (level == cur->bc_nlevels - 1)) + if (cur->bc_ops->type == XFS_BTREE_TYPE_INODE && + level == cur->bc_nlevels - 1) goto out0; /* Set up variables for this block as "left". */ @@ -2987,7 +2994,6 @@ xfs_btree_split( #define xfs_btree_split __xfs_btree_split #endif /* __KERNEL__ */ - /* * Copy the old inode root contents into a real block and make the * broot point to it. @@ -3012,7 +3018,7 @@ xfs_btree_new_iroot( XFS_BTREE_STATS_INC(cur, newroot); - ASSERT(cur->bc_ops->geom_flags & XFS_BTGEO_ROOT_IN_INODE); + ASSERT(cur->bc_ops->type == XFS_BTREE_TYPE_INODE); level = cur->bc_nlevels - 1; @@ -3237,7 +3243,7 @@ xfs_btree_make_block_unfull( { int error = 0; - if ((cur->bc_ops->geom_flags & XFS_BTGEO_ROOT_IN_INODE) && + if (cur->bc_ops->type == XFS_BTREE_TYPE_INODE && level == cur->bc_nlevels - 1) { struct xfs_inode *ip = cur->bc_ino.ip; @@ -3323,8 +3329,8 @@ xfs_btree_insrec( * If we have an external root pointer, and we've made it to the * root level, allocate a new root block and we're done. */ - if (!(cur->bc_ops->geom_flags & XFS_BTGEO_ROOT_IN_INODE) && - (level >= cur->bc_nlevels)) { + if (cur->bc_ops->type != XFS_BTREE_TYPE_INODE && + level >= cur->bc_nlevels) { error = xfs_btree_new_root(cur, stat); xfs_btree_set_ptr_null(cur, ptrp); @@ -3611,7 +3617,7 @@ xfs_btree_kill_iroot( #endif int i; - ASSERT(cur->bc_ops->geom_flags & XFS_BTGEO_ROOT_IN_INODE); + ASSERT(cur->bc_ops->type == XFS_BTREE_TYPE_INODE); ASSERT(cur->bc_nlevels > 1); /* @@ -3848,7 +3854,7 @@ xfs_btree_delrec( * nothing left to do. */ if (level == cur->bc_nlevels - 1) { - if (cur->bc_ops->geom_flags & XFS_BTGEO_ROOT_IN_INODE) { + if (cur->bc_ops->type == XFS_BTREE_TYPE_INODE) { xfs_iroot_realloc(cur->bc_ino.ip, -1, cur->bc_ino.whichfork); @@ -3916,7 +3922,7 @@ xfs_btree_delrec( xfs_btree_get_sibling(cur, block, &rptr, XFS_BB_RIGHTSIB); xfs_btree_get_sibling(cur, block, &lptr, XFS_BB_LEFTSIB); - if (cur->bc_ops->geom_flags & XFS_BTGEO_ROOT_IN_INODE) { + if (cur->bc_ops->type == XFS_BTREE_TYPE_INODE) { /* * One child of root, need to get a chance to copy its contents * into the root and delete it. Can't go up to next level, @@ -4233,8 +4239,8 @@ xfs_btree_delrec( * If we joined with the right neighbor and there's a level above * us, increment the cursor at that level. */ - else if ((cur->bc_ops->geom_flags & XFS_BTGEO_ROOT_IN_INODE) || - (level + 1 < cur->bc_nlevels)) { + else if (cur->bc_ops->type == XFS_BTREE_TYPE_INODE || + level + 1 < cur->bc_nlevels) { error = xfs_btree_increment(cur, level + 1, &i); if (error) goto error0; @@ -4525,7 +4531,7 @@ xfs_btree_block_change_owner( * though, so everything is consistent in memory. */ if (!bp) { - ASSERT(cur->bc_ops->geom_flags & XFS_BTGEO_ROOT_IN_INODE); + ASSERT(cur->bc_ops->type == XFS_BTREE_TYPE_INODE); ASSERT(level == cur->bc_nlevels - 1); return 0; } diff --git a/libxfs/xfs_btree.h b/libxfs/xfs_btree.h index 559066e3a..5f2b5ef85 100644 --- a/libxfs/xfs_btree.h +++ b/libxfs/xfs_btree.h @@ -117,7 +117,15 @@ static inline enum xbtree_key_contig xbtree_key_contig(uint64_t x, uint64_t y) #define XFS_BTREE_LONG_PTR_LEN (sizeof(__be64)) #define XFS_BTREE_SHORT_PTR_LEN (sizeof(__be32)) +enum xfs_btree_type { + XFS_BTREE_TYPE_AG, + XFS_BTREE_TYPE_INODE, +}; + struct xfs_btree_ops { + /* Type of btree - AG-rooted or inode-rooted */ + enum xfs_btree_type type; + /* XFS_BTGEO_* flags that determine the geometry of the btree */ unsigned int geom_flags; @@ -216,9 +224,8 @@ struct xfs_btree_ops { }; /* btree geometry flags */ -#define XFS_BTGEO_ROOT_IN_INODE (1U << 0) /* root may be variable size */ -#define XFS_BTGEO_LASTREC_UPDATE (1U << 1) /* track last rec externally */ -#define XFS_BTGEO_OVERLAPPING (1U << 2) /* overlapping intervals */ +#define XFS_BTGEO_LASTREC_UPDATE (1U << 0) /* track last rec externally */ +#define XFS_BTGEO_OVERLAPPING (1U << 1) /* overlapping intervals */ /* * Reasons for the update_lastrec method to be called. @@ -292,7 +299,7 @@ struct xfs_btree_cur /* * Short btree pointers need an agno to be able to turn the pointers * into physical addresses for IO, so the btree cursor switches between - * bc_ino and bc_ag based on whether XFS_BTGEO_ROOT_IN_INODE is set for + * bc_ino and bc_ag based on bc_ops->type. * the cursor. */ union { diff --git a/libxfs/xfs_btree_staging.c b/libxfs/xfs_btree_staging.c index 39e95a771..e1fd57dee 100644 --- a/libxfs/xfs_btree_staging.c +++ b/libxfs/xfs_btree_staging.c @@ -136,7 +136,7 @@ xfs_btree_stage_afakeroot( struct xfs_btree_ops *nops; ASSERT(!(cur->bc_flags & XFS_BTREE_STAGING)); - ASSERT(!(cur->bc_ops->geom_flags & XFS_BTGEO_ROOT_IN_INODE)); + ASSERT(cur->bc_ops->type != XFS_BTREE_TYPE_INODE); ASSERT(cur->bc_tp == NULL); nops = kmalloc(sizeof(struct xfs_btree_ops), GFP_KERNEL | __GFP_NOFAIL); @@ -217,7 +217,7 @@ xfs_btree_stage_ifakeroot( struct xfs_btree_ops *nops; ASSERT(!(cur->bc_flags & XFS_BTREE_STAGING)); - ASSERT(cur->bc_ops->geom_flags & XFS_BTGEO_ROOT_IN_INODE); + ASSERT(cur->bc_ops->type == XFS_BTREE_TYPE_INODE); ASSERT(cur->bc_tp == NULL); nops = kmalloc(sizeof(struct xfs_btree_ops), GFP_KERNEL | __GFP_NOFAIL); @@ -397,7 +397,7 @@ xfs_btree_bload_prep_block( struct xfs_btree_block *new_block; int ret; - if ((cur->bc_ops->geom_flags & XFS_BTGEO_ROOT_IN_INODE) && + if (cur->bc_ops->type == XFS_BTREE_TYPE_INODE && level == cur->bc_nlevels - 1) { struct xfs_ifork *ifp = xfs_btree_ifork_ptr(cur); size_t new_size; @@ -702,7 +702,7 @@ xfs_btree_bload_compute_geometry( xfs_btree_bload_level_geometry(cur, bbl, level, nr_this_level, &avg_per_block, &level_blocks, &dontcare64); - if (cur->bc_ops->geom_flags & XFS_BTGEO_ROOT_IN_INODE) { + if (cur->bc_ops->type == XFS_BTREE_TYPE_INODE) { /* * If all the items we want to store at this level * would fit in the inode root block, then we have our @@ -761,7 +761,7 @@ xfs_btree_bload_compute_geometry( return -EOVERFLOW; bbl->btree_height = cur->bc_nlevels; - if (cur->bc_ops->geom_flags & XFS_BTGEO_ROOT_IN_INODE) + if (cur->bc_ops->type == XFS_BTREE_TYPE_INODE) bbl->nr_blocks = nr_blocks - 1; else bbl->nr_blocks = nr_blocks; @@ -888,7 +888,7 @@ xfs_btree_bload( } /* Initialize the new root. */ - if (cur->bc_ops->geom_flags & XFS_BTGEO_ROOT_IN_INODE) { + if (cur->bc_ops->type == XFS_BTREE_TYPE_INODE) { ASSERT(xfs_btree_ptr_is_null(cur, &ptr)); cur->bc_ino.ifake->if_levels = cur->bc_nlevels; cur->bc_ino.ifake->if_blocks = total_blocks - 1; diff --git a/libxfs/xfs_btree_staging.h b/libxfs/xfs_btree_staging.h index 9624ae06c..8e29cd3cc 100644 --- a/libxfs/xfs_btree_staging.h +++ b/libxfs/xfs_btree_staging.h @@ -76,8 +76,7 @@ struct xfs_btree_bload { /* * This function should return the size of the in-core btree root - * block. It is only necessary for XFS_BTGEO_ROOT_IN_INODE btree - * types. + * block. It is only necessary for XFS_BTREE_TYPE_INODE btrees. */ xfs_btree_bload_iroot_size_fn iroot_size; diff --git a/libxfs/xfs_ialloc_btree.c b/libxfs/xfs_ialloc_btree.c index a9b2a48a3..79ab04684 100644 --- a/libxfs/xfs_ialloc_btree.c +++ b/libxfs/xfs_ialloc_btree.c @@ -398,6 +398,8 @@ xfs_inobt_keys_contiguous( } const struct xfs_btree_ops xfs_inobt_ops = { + .type = XFS_BTREE_TYPE_AG, + .rec_len = sizeof(xfs_inobt_rec_t), .key_len = sizeof(xfs_inobt_key_t), .ptr_len = XFS_BTREE_SHORT_PTR_LEN, @@ -424,6 +426,8 @@ const struct xfs_btree_ops xfs_inobt_ops = { }; const struct xfs_btree_ops xfs_finobt_ops = { + .type = XFS_BTREE_TYPE_AG, + .rec_len = sizeof(xfs_inobt_rec_t), .key_len = sizeof(xfs_inobt_key_t), .ptr_len = XFS_BTREE_SHORT_PTR_LEN, diff --git a/libxfs/xfs_refcount_btree.c b/libxfs/xfs_refcount_btree.c index 4918c8bae..3d61eeaca 100644 --- a/libxfs/xfs_refcount_btree.c +++ b/libxfs/xfs_refcount_btree.c @@ -317,6 +317,8 @@ xfs_refcountbt_keys_contiguous( } const struct xfs_btree_ops xfs_refcountbt_ops = { + .type = XFS_BTREE_TYPE_AG, + .rec_len = sizeof(struct xfs_refcount_rec), .key_len = sizeof(struct xfs_refcount_key), .ptr_len = XFS_BTREE_SHORT_PTR_LEN, diff --git a/libxfs/xfs_rmap_btree.c b/libxfs/xfs_rmap_btree.c index b1d25d99d..f87e34a1d 100644 --- a/libxfs/xfs_rmap_btree.c +++ b/libxfs/xfs_rmap_btree.c @@ -471,6 +471,7 @@ xfs_rmapbt_keys_contiguous( } const struct xfs_btree_ops xfs_rmapbt_ops = { + .type = XFS_BTREE_TYPE_AG, .geom_flags = XFS_BTGEO_OVERLAPPING, .rec_len = sizeof(struct xfs_rmap_rec),

[047/111] xfs: split out a btree type from the btree ops geometry flags

Commit Message

Patch