Skip to content

Add amdapp-sdk, amdadl-sdk and cgminer#452

Closed
offlinehacker wants to merge 13 commits intoNixOS:masterfrom
xtruder:amdapp-sdk
Closed

Add amdapp-sdk, amdadl-sdk and cgminer#452
offlinehacker wants to merge 13 commits intoNixOS:masterfrom
xtruder:amdapp-sdk

Conversation

@offlinehacker
Copy link
Copy Markdown
Contributor

  • AMD Accelerated Parallel Processing (APP) SDK, with OpenCL 1.2 support.
  • API to access display driver functionality for ATI graphics cards
  • CPU/GPU miner in c for bitcoin

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

license and platforms come from stdenv.lib.platforms and stdenv.lib.licenses

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tnx, will fix that :)

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Licenses are now updated, but for platforms i can't use any entry from stdenv.lib.platforms.

@offlinehacker
Copy link
Copy Markdown
Contributor Author

Goal achived, cgminer now working :)

@domenkozar
Copy link
Copy Markdown
Member

Looks ok to me

@offlinehacker
Copy link
Copy Markdown
Contributor Author

Wait, do not merge yet, it worked on debian using nix, but have to fix it for nixos ;)

@offlinehacker
Copy link
Copy Markdown
Contributor Author

This is now ready to be pulled in. Still some things needs to be fixed in future:

  • Test x86 support, before we can do that, ati_unfree drivers must be x86 supported, currently are not
  • Fix reaper to compile code located in store, not from current folder(needs fixing reaper)
  • Test with nvidia, to do that i need nvidia graphic card

Note: Please first merge #462 this one takes ATI driver patches from there

@domenkozar
Copy link
Copy Markdown
Member

Could you rebase so this can be merged?

@offlinehacker
Copy link
Copy Markdown
Contributor Author

This should be it. Once again #462 is included here.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this needs to be removed?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tnx, fixed ;)

@domenkozar
Copy link
Copy Markdown
Member

How do I test it?

@offlinehacker
Copy link
Copy Markdown
Contributor Author

Hum for cgminer just nix-env -iA your_repo_name.cgminer, this will also compile amdappsdk and amdadlsdk. For ati drivers i don't know.

@offlinehacker
Copy link
Copy Markdown
Contributor Author

This works for me without any problems, for quite some time. @iElectric what's up with testing, any progress? If necessary i can test it from clean install once again, i have some ATI cards lying around.

@domenkozar
Copy link
Copy Markdown
Member

Still needs to be tested :/ I'll try asap

@offlinehacker
Copy link
Copy Markdown
Contributor Author

Okay, if you will try mining for real take a look at https://github.com/offlinehacker/miner.litecoin.si/blob/master/miner.nix (at the end of the file), because you have to export some environment variables.

More precisely:

system.activationScripts.drifix = ''
    # Create the required /usr/lib/dri/fglrx_dri.so;
    mkdir -p /usr/lib/dri
    ln -fs /run/opengl-driver/lib/fglrx_dri.so /usr/lib/dri/fglrx_dri.so
  '';

shellInit = ''
  export DISPLAY=:0
  export GPU_MAX_ALLOC_PERCENT=100
  export GPU_USE_SYNC_OBJECTS=1
'';

etc.opencl = {
  source = "${pkgs.amdappsdk}/etc/OpenCL";
  target = "OpenCL";
};

@offlinehacker
Copy link
Copy Markdown
Contributor Author

Just rebased once again

@garbas
Copy link
Copy Markdown
Member

garbas commented May 13, 2013

are .topdeps and .topmsg files really needed?

@offlinehacker
Copy link
Copy Markdown
Contributor Author

Well no, they are reminder from TopGit from MarcWeber commits and must be
removed by merge. I was not using topgit so it was not updated
automatically.
On May 13, 2013 3:58 PM, "Rok Garbas" notifications@github.com wrote:

are .topdeps and .topmsg files really needed?


Reply to this email directly or view it on GitHubhttps://github.com//pull/452#issuecomment-17813014
.

@domenkozar
Copy link
Copy Markdown
Member

Can you remove .topdeps and .topmsg and ati related stuff and force push new changes?

@offlinehacker
Copy link
Copy Markdown
Contributor Author

Okay, i have just done that.

@offlinehacker
Copy link
Copy Markdown
Contributor Author

I am closing this of honor of #638 pull request, which is updated and rewritten version of this one.

@evenbrenden evenbrenden mentioned this pull request Feb 9, 2021
10 tasks
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.

3 participants