From bf2064df7f02f29bcf1575742f1e24c5506ea935 Mon Sep 17 00:00:00 2001 From: Jens Steube Date: Sun, 2 May 2021 08:19:16 +0000 Subject: [PATCH] Update module_unstable_warning() for -m 172xx and -m 200xx --- src/modules/module_17200.c | 12 +++++++++++- src/modules/module_17220.c | 12 +++++++++++- src/modules/module_17225.c | 13 ++++++++++++- src/modules/module_20011.c | 6 ------ src/modules/module_20012.c | 6 ------ src/modules/module_20013.c | 6 ------ 6 files changed, 34 insertions(+), 21 deletions(-) diff --git a/src/modules/module_17200.c b/src/modules/module_17200.c index 6d87da526..fa510ba27 100644 --- a/src/modules/module_17200.c +++ b/src/modules/module_17200.c @@ -165,7 +165,17 @@ const char *module_st_pass (MAYBE_UNUSED const hashconfig_t *hashconfig, bool module_unstable_warning (MAYBE_UNUSED const hashconfig_t *hashconfig, MAYBE_UNUSED const user_options_t *user_options, MAYBE_UNUSED const user_options_extra_t *user_options_extra, MAYBE_UNUSED const hc_device_param_t *device_param) { - if (device_param->opencl_platform_vendor_id == VENDOR_ID_APPLE) + // problem with this kernel is the huge amount of register pressure on u8 tmp[TMPSIZ]; + // some runtimes cant handle it by swapping it to global memory + // it leads to CL_KERNEL_WORK_GROUP_SIZE to return 0 and later we will divide with 0 + // workaround would be to rewrite kernel to use global memory + + if (device_param->opencl_device_vendor_id == VENDOR_ID_AMD) + { + return true; + } + + if (device_param->opencl_device_vendor_id == VENDOR_ID_INTEL_SDK) { return true; } diff --git a/src/modules/module_17220.c b/src/modules/module_17220.c index 9855a3ed6..8f1beaf1a 100644 --- a/src/modules/module_17220.c +++ b/src/modules/module_17220.c @@ -165,7 +165,17 @@ const char *module_st_pass (MAYBE_UNUSED const hashconfig_t *hashconfig, bool module_unstable_warning (MAYBE_UNUSED const hashconfig_t *hashconfig, MAYBE_UNUSED const user_options_t *user_options, MAYBE_UNUSED const user_options_extra_t *user_options_extra, MAYBE_UNUSED const hc_device_param_t *device_param) { - if (device_param->opencl_platform_vendor_id == VENDOR_ID_APPLE) + // problem with this kernel is the huge amount of register pressure on u8 tmp[TMPSIZ]; + // some runtimes cant handle it by swapping it to global memory + // it leads to CL_KERNEL_WORK_GROUP_SIZE to return 0 and later we will divide with 0 + // workaround would be to rewrite kernel to use global memory + + if (device_param->opencl_device_vendor_id == VENDOR_ID_AMD) + { + return true; + } + + if (device_param->opencl_device_vendor_id == VENDOR_ID_INTEL_SDK) { return true; } diff --git a/src/modules/module_17225.c b/src/modules/module_17225.c index 36394ac0b..3b3291d5f 100644 --- a/src/modules/module_17225.c +++ b/src/modules/module_17225.c @@ -165,7 +165,17 @@ const char *module_st_pass (MAYBE_UNUSED const hashconfig_t *hashconfig, bool module_unstable_warning (MAYBE_UNUSED const hashconfig_t *hashconfig, MAYBE_UNUSED const user_options_t *user_options, MAYBE_UNUSED const user_options_extra_t *user_options_extra, MAYBE_UNUSED const hc_device_param_t *device_param) { - if (device_param->opencl_platform_vendor_id == VENDOR_ID_APPLE) + // problem with this kernel is the huge amount of register pressure on u8 tmp[TMPSIZ]; + // some runtimes cant handle it by swapping it to global memory + // it leads to CL_KERNEL_WORK_GROUP_SIZE to return 0 and later we will divide with 0 + // workaround would be to rewrite kernel to use global memory + + if (device_param->opencl_device_vendor_id == VENDOR_ID_AMD) + { + return true; + } + + if (device_param->opencl_device_vendor_id == VENDOR_ID_INTEL_SDK) { return true; } @@ -173,6 +183,7 @@ bool module_unstable_warning (MAYBE_UNUSED const hashconfig_t *hashconfig, MAYBE return false; } + u64 module_esalt_size (MAYBE_UNUSED const hashconfig_t *hashconfig, MAYBE_UNUSED const user_options_t *user_options, MAYBE_UNUSED const user_options_extra_t *user_options_extra) { const u64 esalt_size = (const u64) sizeof (pkzip_t); diff --git a/src/modules/module_20011.c b/src/modules/module_20011.c index 80a0fab08..a48a1d0d5 100644 --- a/src/modules/module_20011.c +++ b/src/modules/module_20011.c @@ -74,12 +74,6 @@ bool module_unstable_warning (MAYBE_UNUSED const hashconfig_t *hashconfig, MAYBE } } - // amdgpu-pro-20.50-1234664-ubuntu-20.04 (rocr): self-test failed. - if ((device_param->opencl_device_vendor_id == VENDOR_ID_AMD) && (device_param->has_vperm == true)) - { - return true; - } - // amdgpu-pro-20.50-1234664-ubuntu-20.04 (legacy): unhandled return code 255 if ((device_param->opencl_device_vendor_id == VENDOR_ID_AMD) && (device_param->has_vperm == false)) { diff --git a/src/modules/module_20012.c b/src/modules/module_20012.c index a97a0bf57..6005028db 100644 --- a/src/modules/module_20012.c +++ b/src/modules/module_20012.c @@ -74,12 +74,6 @@ bool module_unstable_warning (MAYBE_UNUSED const hashconfig_t *hashconfig, MAYBE } } - // amdgpu-pro-20.50-1234664-ubuntu-20.04 (rocr): self-test failed. - if ((device_param->opencl_device_vendor_id == VENDOR_ID_AMD) && (device_param->has_vperm == true)) - { - return true; - } - // amdgpu-pro-20.50-1234664-ubuntu-20.04 (legacy): unhandled return code 255 if ((device_param->opencl_device_vendor_id == VENDOR_ID_AMD) && (device_param->has_vperm == false)) { diff --git a/src/modules/module_20013.c b/src/modules/module_20013.c index 3b72ebfcf..b6df1939b 100644 --- a/src/modules/module_20013.c +++ b/src/modules/module_20013.c @@ -74,12 +74,6 @@ bool module_unstable_warning (MAYBE_UNUSED const hashconfig_t *hashconfig, MAYBE } } - // amdgpu-pro-20.50-1234664-ubuntu-20.04 (rocr): self-test failed. - if ((device_param->opencl_device_vendor_id == VENDOR_ID_AMD) && (device_param->has_vperm == true)) - { - return true; - } - // amdgpu-pro-20.50-1234664-ubuntu-20.04 (legacy): unhandled return code 255 if ((device_param->opencl_device_vendor_id == VENDOR_ID_AMD) && (device_param->has_vperm == false)) {