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.

Sign up for GitHub

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

Jump to bottom

Refactor PoW implementation context #57

Closed
furuame opened this issue Aug 8, 2018 · 1 comment
Closed

Refactor PoW implementation context #57

furuame opened this issue Aug 8, 2018 · 1 comment
Assignees
@furuame
Labels
enhancement New feature or request

Comments

@furuame
Copy link
Member

furuame commented Aug 8, 2018

In the current dcurl_entry implementation, it can only be compiled with two implementations, which are CPU and GPU. With the introducing of the FPGA solution, there is a possibility that a platform consists of CPU, GPU, and FPGA. Hence, a more general dcurl_entry is needed.

The new dcurl_entry should achieve the following goals:

  • No more thread management inside the entry, which is hard to read.
  • Easy to register the PoW implementations according to the platform.
  • A priority mechanism for PoW implementations
@furuame furuame added the enhancement New feature or request label Aug 8, 2018
This was referenced Aug 12, 2018
Closed
Merged
@jserv jserv changed the title A more general dcurl_entry needed to build with more and more implementations Refactor PoW implementation context Aug 16, 2018
@jserv jserv mentioned this issue Aug 16, 2018
Closed
@furuame
Copy link
Member Author

furuame commented Aug 17, 2018

The PR #59 achieves the goal. Close.

@furuame furuame closed this as completed Aug 17, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@furuame furuame

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@furuame