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

lyra2re algo #403

Closed
wants to merge 9 commits into from
Closed

lyra2re algo #403

wants to merge 9 commits into from

Conversation

djm34
Copy link

@djm34 djm34 commented Dec 12, 2014

Hi,

here is the code for the lyra2 algo.
Some modification were necessary (findnonce and opencl-driver) to support pre-calculation of the first blake256 round. I used the same logic as in cgminer (used for blakecoin).

lyra2/skein256/keccak256/groestl256/blake256 (+ logic for precalulation)
missing file
@mrbrdo
Copy link
Contributor

mrbrdo commented Dec 12, 2014

can you please squash these commits together into a single commit and force push

@mrbrdo
Copy link
Contributor

mrbrdo commented Dec 12, 2014

It looks ok for the most part but:

@mrbrdo
Copy link
Contributor

mrbrdo commented Dec 12, 2014

To remove opt_lyra you can access current algorithm from thr_info *thr (which is a parameter to opencl_prepare_work) like so:

thr->cgpu->algorithm

So you could add a bool with a name something like prepare_blake256 to algorithm_t. Since we do not want to compare string every time in opencl_prepare_work.

When you are done with the changes please squash and push a single commit.

@kenshirothefist
Copy link

@ystarnaud @djm34 @mrbrdo @badman74 so, anybody willing to push this through? badman74 also did some more cleanups: https://github.com/badman74/sgminer

@mrbrdo
Copy link
Contributor

mrbrdo commented Dec 22, 2014

@djm34 could you do the proposed changes? I cannot develop currently as I don't have hardware to test it. The necessary changes should be pretty simple and shouldn't take long.

@ystarnaud
Copy link
Contributor

I will take a look.

@troky troky closed this Mar 18, 2015
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.

5 participants