Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

changes are changes #137

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

sunnyhellious
Copy link
Contributor

@sunnyhellious sunnyhellious commented Jan 5, 2022

Hi!

I have added some basic optimisations for 5% additional goal.

Actually now kernel calculates several hashes per one flight. This parameter at the moment is defined in sha256.cpp. HPF define is currently setted to 3. Hpf - hashes per flight.

And can you fix this lines for old GPU compatibility in OpenCL kernel. Atomic add operation breaks kernel work on hd4890 for instance:

// if (atomic_add(&result->found, 1) == 0)
result->found++;

@DelphiR
Copy link

DelphiR commented Jan 5, 2022

The HFP is the main and fundamental feature of this PR optimisation
It can be between 3 and 100
The most omptimum quantity of it depends on the power of GPU card
For example for GTX1080Ti the HPF might be the 100 and increase the total power of algo up to 10% for example. Which is very cool!

@DelphiR
Copy link

DelphiR commented Jan 5, 2022

During the AMD test session we see more effect in terms of more accel from this commit

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants