Correction for benchmarks not running on Arch

[westonmyers]

L.,

I went to run hardinfo on Arch and was unable to complete any benchmarks. High CPU usage was observed and the application would stay at the benchmarking dialog for as long as you'd let it.

Decided to check if others were having issues as well and stumbled into people in the Arch User Repository commenting along with others here in the upstream. (#640, etc)

straced it down to a module dependency issue; as seen below.

I just wanted to push in at least a quick modification for Arch peeps and open dialog here. AUR is running a patch of the below for now as the maintainer there was able to confirm the ability to run benchmarks when patched.

I'll see about testing other distros, etc. Ideally checks would be added for this module issue. There were also some aberrations seen while testing that need to be debugged as well. Starting with something is better than nothing though!

Thanks for your time, Weston

[lpereira]

Thanks! I think that module dependencies might be broken ("-a" switch to autoload dependencies) if your patch is any indication. I'm far away from my computer but I'll take a look next week.

On Fri, Apr 29, 2022, at 12:40 PM, Weston Myers wrote:

L.,

I went to run hardinfo on Arch and was unable to complete any benchmarks. High CPU usage was observed and the application would stay at the benchmarking dialog for as long as you'd let it.

Decided to check if others were having issues as well and stumbled into people in the Arch User Repository https://aur.archlinux.org/packages/hardinfo-git commenting along with others here in the upstream. (#640 https://github.com/lpereira/hardinfo/issues/640, etc)

straced it down to a module dependency issue; as seen below.

I just wanted to push in at least a quick modification for Arch peeps and open dialog here. AUR is running a patch of the below for now as the maintainer there was able to confirm the ability to run benchmarks when patched.

I'll see about testing other distros, etc. Ideally checks would be added for this module issue. There were also some aberrations seen while testing that need to be debugged as well. Starting with something is better than nothing though!

Thanks for your time, Weston

You can view, comment on, or merge this pull request online at:

https://github.com/lpereira/hardinfo/pull/646

Commit Summary

• ec85aad https://github.com/lpereira/hardinfo/pull/646/commits/ec85aad2bdcf76bebb2ebfd1ef6f8fb55f73df10 Quick and dirty fix to re-enable benchmarking. Patch currently in Arch User Repository: https://aur.archlinux.org/packages/hardinfo-git File Changes

(1 file https://github.com/lpereira/hardinfo/pull/646/files)

• M modules/benchmark.c https://github.com/lpereira/hardinfo/pull/646/files#diff-3454ef879ab0d49212d9af68b6ac521e12909ed3df0bcf3afdfd2937d60453df (4) Patch Links:

• https://github.com/lpereira/hardinfo/pull/646.patch

• https://github.com/lpereira/hardinfo/pull/646.diff

— Reply to this email directly, view it on GitHub https://github.com/lpereira/hardinfo/pull/646, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAADVGITOBEC2MIQINVE62TVHQUJ7ANCNFSM5UW5AMGA. You are receiving this because you are subscribed to this thread.Message ID: @.***>