diff options
| author | Romain Dolbeau <[email protected]> | 2021-04-26 21:42:42 +0200 |
|---|---|---|
| committer | GitHub <[email protected]> | 2021-04-26 12:42:42 -0700 |
| commit | 0375465536c9790a7fb3e1ac94412fef91076e3f (patch) | |
| tree | 86f621b18bd8becda02e5492450422763088790b /module/zcommon | |
| parent | f9bece92e24bbb679232085e41c4ddf5b74ccaa4 (diff) | |
Fix AVX512BW Fletcher code on AVX512-but-not-BW machines
Introduce a specific valid function for avx512f+avx512bw (instead
of checking only for avx512f).
Reviewed-by: Brian Behlendorf <[email protected]>
Reviewed-by: Adam Moss <[email protected]>
Signed-off-by: Romain Dolbeau <[email protected]>
Closes #11937
Closes #11938
Diffstat (limited to 'module/zcommon')
| -rw-r--r-- | module/zcommon/zfs_fletcher_avx512.c | 8 |
1 files changed, 7 insertions, 1 deletions
diff --git a/module/zcommon/zfs_fletcher_avx512.c b/module/zcommon/zfs_fletcher_avx512.c index 300ec4c1f..963f089b0 100644 --- a/module/zcommon/zfs_fletcher_avx512.c +++ b/module/zcommon/zfs_fletcher_avx512.c @@ -210,6 +210,12 @@ fletcher_4_avx512bw_byteswap(fletcher_4_ctx_t *ctx, const void *buf, } STACK_FRAME_NON_STANDARD(fletcher_4_avx512bw_byteswap); +static boolean_t +fletcher_4_avx512bw_valid(void) +{ + return (fletcher_4_avx512f_valid() && zfs_avx512bw_available()); +} + const fletcher_4_ops_t fletcher_4_avx512bw_ops = { .init_native = fletcher_4_avx512f_init, .fini_native = fletcher_4_avx512f_fini, @@ -217,7 +223,7 @@ const fletcher_4_ops_t fletcher_4_avx512bw_ops = { .init_byteswap = fletcher_4_avx512f_init, .fini_byteswap = fletcher_4_avx512f_fini, .compute_byteswap = fletcher_4_avx512bw_byteswap, - .valid = fletcher_4_avx512f_valid, + .valid = fletcher_4_avx512bw_valid, .name = "avx512bw" }; #endif |