diff options
| author | Alexander Motin <[email protected]> | 2021-06-22 19:35:23 -0400 |
|---|---|---|
| committer | Tony Hutter <[email protected]> | 2021-09-14 12:10:17 -0700 |
| commit | f3969ea78b5f54935474d49455cf9c4d6a1e107a (patch) | |
| tree | 38dbfbee2cd0bbd24bbc642e2023d3f20bd82b6f /module/zfs/space_map.c | |
| parent | 6fe6192796f6835163523db94130fb842022f6cf (diff) | |
Optimize small random numbers generation
In all places except two spa_get_random() is used for small values,
and the consumers do not require well seeded high quality values.
Switch those two exceptions directly to random_get_pseudo_bytes()
and optimize spa_get_random(), renaming it to random_in_range(),
since it is not related to SPA or ZFS in general.
On FreeBSD directly map random_in_range() to new prng32_bounded() KPI
added in FreeBSD 13. On Linux and in user-space just reduce the type
used to uint32_t to avoid more expensive 64bit division.
Reviewed-by: Ryan Moeller <[email protected]>
Reviewed-by: Brian Behlendorf <[email protected]>
Signed-off-by: Alexander Motin <[email protected]>
Sponsored-By: iXsystems, Inc.
Closes #12183
Diffstat (limited to 'module/zfs/space_map.c')
| -rw-r--r-- | module/zfs/space_map.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/module/zfs/space_map.c b/module/zfs/space_map.c index 138e6c75e..11d479892 100644 --- a/module/zfs/space_map.c +++ b/module/zfs/space_map.c @@ -726,7 +726,7 @@ space_map_write_impl(space_map_t *sm, range_tree_t *rt, maptype_t maptype, length > SM_RUN_MAX || vdev_id != SM_NO_VDEVID || (zfs_force_some_double_word_sm_entries && - spa_get_random(100) == 0))) + random_in_range(100) == 0))) words = 2; space_map_write_seg(sm, rs_get_start(rs, rt), rs_get_end(rs, |