mirror of
https://git.proxmox.com/git/mirror_zfs.git
synced 2024-12-27 11:29:36 +03:00
ee36c709c3
perf: 2.75x faster ddt_entry_compare() First 256bits of ddt_key_t is a block checksum, which are expected to be close to random data. Hence, on average, comparison only needs to look at first few bytes of the keys. To reduce number of conditional jump instructions, the result is computed as: sign(memcmp(k1, k2)). Sign of an integer 'a' can be obtained as: `(0 < a) - (a < 0)` := {-1, 0, 1} , which is computed efficiently. Synthetic performance evaluation of original and new algorithm over 1G random keys on 2.6GHz Intel(R) Xeon(R) CPU E5-2660 v3: old 6.85789 s new 2.49089 s perf: 2.8x faster vdev_queue_offset_compare() and vdev_queue_timestamp_compare() Compute the result directly instead of using conditionals perf: zfs_range_compare() Speedup between 1.1x - 2.5x, depending on compiler version and optimization level. perf: spa_error_entry_compare() `bcmp()` is not suitable for comparator use. Use `memcmp()` instead. perf: 2.8x faster metaslab_compare() and metaslab_rangesize_compare() perf: 2.8x faster zil_bp_compare() perf: 2.8x faster mze_compare() perf: faster dbuf_compare() perf: faster compares in spa_misc perf: 2.8x faster layout_hash_compare() perf: 2.8x faster space_reftree_compare() perf: libzfs: faster avl tree comparators perf: guid_compare() perf: dsl_deadlist_compare() perf: perm_set_compare() perf: 2x faster range_tree_seg_compare() perf: faster unique_compare() perf: faster vdev_cache _compare() perf: faster vdev_uberblock_compare() perf: faster fuid _compare() perf: faster zfs_znode_hold_compare() Signed-off-by: Gvozden Neskovic <neskovic@gmail.com> Signed-off-by: Richard Elling <richard.elling@gmail.com> Signed-off-by: Brian Behlendorf <behlendorf1@llnl.gov> Closes #5033
113 lines
2.5 KiB
C
113 lines
2.5 KiB
C
/*
|
|
* CDDL HEADER START
|
|
*
|
|
* The contents of this file are subject to the terms of the
|
|
* Common Development and Distribution License (the "License").
|
|
* You may not use this file except in compliance with the License.
|
|
*
|
|
* You can obtain a copy of the license at usr/src/OPENSOLARIS.LICENSE
|
|
* or http://www.opensolaris.org/os/licensing.
|
|
* See the License for the specific language governing permissions
|
|
* and limitations under the License.
|
|
*
|
|
* When distributing Covered Code, include this CDDL HEADER in each
|
|
* file and include the License file at usr/src/OPENSOLARIS.LICENSE.
|
|
* If applicable, add the following below this CDDL HEADER, with the
|
|
* fields enclosed by brackets "[]" replaced with your own identifying
|
|
* information: Portions Copyright [yyyy] [name of copyright owner]
|
|
*
|
|
* CDDL HEADER END
|
|
*/
|
|
/*
|
|
* Copyright 2007 Sun Microsystems, Inc. All rights reserved.
|
|
* Use is subject to license terms.
|
|
*/
|
|
|
|
|
|
|
|
#include <sys/zfs_context.h>
|
|
#include <sys/avl.h>
|
|
#include <sys/unique.h>
|
|
|
|
static avl_tree_t unique_avl;
|
|
static kmutex_t unique_mtx;
|
|
|
|
typedef struct unique {
|
|
avl_node_t un_link;
|
|
uint64_t un_value;
|
|
} unique_t;
|
|
|
|
#define UNIQUE_MASK ((1ULL << UNIQUE_BITS) - 1)
|
|
|
|
static int
|
|
unique_compare(const void *a, const void *b)
|
|
{
|
|
const unique_t *una = (const unique_t *)a;
|
|
const unique_t *unb = (const unique_t *)b;
|
|
|
|
return (AVL_CMP(una->un_value, unb->un_value));
|
|
}
|
|
|
|
void
|
|
unique_init(void)
|
|
{
|
|
avl_create(&unique_avl, unique_compare,
|
|
sizeof (unique_t), offsetof(unique_t, un_link));
|
|
mutex_init(&unique_mtx, NULL, MUTEX_DEFAULT, NULL);
|
|
}
|
|
|
|
void
|
|
unique_fini(void)
|
|
{
|
|
avl_destroy(&unique_avl);
|
|
mutex_destroy(&unique_mtx);
|
|
}
|
|
|
|
uint64_t
|
|
unique_create(void)
|
|
{
|
|
uint64_t value = unique_insert(0);
|
|
unique_remove(value);
|
|
return (value);
|
|
}
|
|
|
|
uint64_t
|
|
unique_insert(uint64_t value)
|
|
{
|
|
avl_index_t idx;
|
|
unique_t *un = kmem_alloc(sizeof (unique_t), KM_SLEEP);
|
|
|
|
un->un_value = value;
|
|
|
|
mutex_enter(&unique_mtx);
|
|
while (un->un_value == 0 || un->un_value & ~UNIQUE_MASK ||
|
|
avl_find(&unique_avl, un, &idx)) {
|
|
mutex_exit(&unique_mtx);
|
|
(void) random_get_pseudo_bytes((void*)&un->un_value,
|
|
sizeof (un->un_value));
|
|
un->un_value &= UNIQUE_MASK;
|
|
mutex_enter(&unique_mtx);
|
|
}
|
|
|
|
avl_insert(&unique_avl, un, idx);
|
|
mutex_exit(&unique_mtx);
|
|
|
|
return (un->un_value);
|
|
}
|
|
|
|
void
|
|
unique_remove(uint64_t value)
|
|
{
|
|
unique_t un_tofind;
|
|
unique_t *un;
|
|
|
|
un_tofind.un_value = value;
|
|
mutex_enter(&unique_mtx);
|
|
un = avl_find(&unique_avl, &un_tofind, NULL);
|
|
if (un != NULL) {
|
|
avl_remove(&unique_avl, un);
|
|
kmem_free(un, sizeof (unique_t));
|
|
}
|
|
mutex_exit(&unique_mtx);
|
|
}
|