diff options
| author | Alexander Motin <[email protected]> | 2021-07-01 11:32:31 -0400 |
|---|---|---|
| committer | GitHub <[email protected]> | 2021-07-01 09:32:31 -0600 |
| commit | b192a2c0a19133234972b491e9c06093b3d21358 (patch) | |
| tree | 8df9296060d668a224032aad4926d359be3eabcf /module/avl | |
| parent | 490c845efe3ca29eaa8aa6ea1e3f45eda72895fe (diff) | |
Remove avl_size field from struct avl_tree
This field is used only by illumos mdb. On other platforms it only
increases the struct size from 32 to 40 bytes. For struct vdev_queue
including 13 instances of avl_tree_t size means active cache lines.
Keep the padding in user-space for now to not break the ABI.
Reviewed-by: Brian Behlendorf <[email protected]>
Reviewed-by: Matthew Ahrens <[email protected]>
Signed-off-by: Alexander Motin <[email protected]>
Sponsored-By: iXsystems, Inc.
Closes #12290
Diffstat (limited to 'module/avl')
| -rw-r--r-- | module/avl/avl.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/module/avl/avl.c b/module/avl/avl.c index 1a95092bc..3d36d4c87 100644 --- a/module/avl/avl.c +++ b/module/avl/avl.c @@ -875,7 +875,6 @@ avl_swap(avl_tree_t *tree1, avl_tree_t *tree2) ASSERT3P(tree1->avl_compar, ==, tree2->avl_compar); ASSERT3U(tree1->avl_offset, ==, tree2->avl_offset); - ASSERT3U(tree1->avl_size, ==, tree2->avl_size); temp_node = tree1->avl_root; temp_numnodes = tree1->avl_numnodes; @@ -903,7 +902,6 @@ avl_create(avl_tree_t *tree, int (*compar) (const void *, const void *), tree->avl_compar = compar; tree->avl_root = NULL; tree->avl_numnodes = 0; - tree->avl_size = size; tree->avl_offset = offset; } |