QPQ-AG/GajuMine
11
0
Fork 0
Code Issues 2 Pull Requests 1 Actions Packages Projects Releases Wiki Activity

Disable AVX2 for now #4

Closed
spivee wants to merge 2 commits from spivee/disable-avx2 into master
pull from: spivee/disable-avx2
merge into: :master
Conversation 2 Commits 2 Files Changed 1
+2 -8
spivee commented 2025-06-23 22:20:11 +09:00
Member
Copy Link
Copy Source

AVX2 is much less available than one would expect, is hard for e.g. me to test, and because of the way gmhive_worker currently works, the illegal-instruction exit codes go undetected, leaving users confused as their balance stays at zero, despite the app saying they are getting solutions.

The branching on the OS is now kind of silly, but the comments documenting each OS are important.

AVX2 is much less available than one would expect, is hard for e.g. me to test, and because of the way gmhive_worker currently works, the illegal-instruction exit codes go undetected, leaving users confused as their balance stays at zero, despite the app saying they are getting solutions. The branching on the OS is now kind of silly, but the comments documenting each OS are important.
spivee added 1 commit 2025-06-23 22:20:11 +09:00
Disable AVX2 for now 92d061f572 
AVX2 is much less available than one would expect, is hard for e.g. me to test, and because of the way gmhive_worker currently works, the illegal-instruction exit codes go undetected, leaving users confused as their balance stays at zero, despite the app saying they are getting solutions.
spivee requested review from uwiger 2025-06-23 22:20:33 +09:00
spivee requested review from zxq9 2025-06-23 22:20:34 +09:00
uwiger approved these changes 2025-06-23 22:51:42 +09:00
spivee added 1 commit 2025-06-24 06:06:30 +09:00
Fix semicolon 8d3d30425b
spivee commented 2025-08-12 16:23:31 +09:00
Author
Member
Copy Link
Copy Source

We are sort of doing this now, since other fixes didn't work, so this PR is outdated now.

We are sort of doing this now, since other fixes didn't work, so this PR is outdated now.
spivee closed this pull request 2025-08-12 16:23:31 +09:00

Pull request closed

Please reopen this pull request to perform a merge.
Sign in to join this conversation.
Reviewers
No Reviewers
zxq9
uwiger
Labels
Clear labels
bug
Something is not working
 duplicate
This issue or pull request already exists
 enhancement
New feature
 help wanted
Need some help
 invalid
Something is wrong
 pig lipstick
Muggle-facing enhancements
 question
More information is needed
 wontfix
This won't be fixed
No labels
Milestone
No items
No Milestone
Projects
Clear projects
No projects
Assignees
Clear assignees
Dagadunn Pickmick apex132 dimitar.p.ivanov flambard (Markus Flambard) gchewy hans_sv pharpend spivee (Jarvis Carroll) thescientress (Cecille de Jesus) uwiger (Ulf Wiger) zxq9 (Craig Everett)
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: QPQ-AG/GajuMine#4